2017-01-23 15 views
0

私は現在、運動のためにラズベリーパイと協力しています。Arduino-uno project

私はブレッドボード上で以下で働いている:

  • 5私の次のコードは、次のように動作するようになっているLEDの
  • 2スイッチ

をして最初のボタンで次のLEDを選択し、2番目のボタンでは選択したLEDをオン/オフすることができます。

//const variables 
const int leds[] = {3, 5, 6, 9, 11}; 
const int buttons[] = {12, 13}; 

//variables that will change 
int buttonState[] = {false, false}; 
int lastButtonState[] = {false, false}; 
int values[] = {false, false, false, false}; 

void setup() { 
    //init LEDs 
    for(int i = 0; i < sizeof(leds); i++){ 
     pinMode(leds[i], OUTPUT); 
    } 

    //init buttons 
    for(int i = 0; i < sizeof(buttons); i++){ 
     pinMode(buttons[i], INPUT); 
    } 
} 

void loop() { 
    //fade when game starts 
    fade(); 

    //start game 
    start(); 

    //output of game 
    output(); 
} 

void output(){ 
    bool t1 = !values[0]; 
    bool t2 = t1 && values[1]; 
    bool t3 = values[2] || values[3]; 
    bool Q = !(t2 || t3); 
    if(!Q){ 
     digitalWrite(leds[4], true); 
    }else{ 
     digitalWrite(leds[4], false); 
    } 
} 

void start(){ 
    //total of leds 
    int j = 0; 
    //check if user is not at 5th led 
    while(j < 4){ 
     //loop through buttons 
     for(int i = 0; i < 2; i++){ 
      // Read button 
      buttonState[i] = digitalRead(buttons[i]); 
      // check button state 
      if (buttonState[i] != lastButtonState[i]) { 
       // if the state has changed 
       if (buttonState[i] == HIGH) { 
        //check if button 1 
        if(i == 0){ 
         //select next LED 
         j++; 
        } 
        //else button 2 
        else{ 
         // if the current state of the 2nd button is HIGH 
         while(i == 1){ 
          //if current value of led is false, put it true 
          if(values[j] == false){ 
           //put led on 
           digitalWrite(leds[j], true); 
           values[j] = true; 
           delay(50); 
          }else{ 
           //put led off 
           digitalWrite(leds[j], false); 
           delay(50); 
           values[j] = false; 
          } 
          //go back to button 1? 
          i = 0; 
         } 
        } 
        //go back to button 1? 
        i = 0; 
       } 
      } 
      // save the current state as the last state, 
      // for next time through the loop 
      lastButtonState[i] = buttonState[i]; 

      // wait a little 
      delay(50); 
     } 
    } 
} 

void fade(){ 
    //put every led on half-on 
    for(int i = 0; i < sizeof(leds); i++){ 
     analogWrite(leds[i], 100); 
    } 
} 
+2

確かにこれは警告なしでは構築されませんか?どうして問題は言わないの? 'pinMode(leds、OUTPUT);'ステートメントは間違っていなければならず、結局ループのポイントであった(そして入力と同じ) 'pinMode(leds [i]、OUTPUT);'でなければなりません。 – unwind

+0

私はそれを検証したところ、エラーは出ませんでしたが、今更新しました。それはもう問題ではありません。 現在、LEDのオン/オフを設定するはずの2番目のボタンは、LEDのみをオフにし、オンにしません。 – DiceOfDoom

+0

あなたの質問が純粋にソフトウェアに関するものである限り、ここでは話題になっています。しかし、記録のためだけに、ソフトウェアまたはハードウェアに関連する疑問は、http://electronics.stackexchange.com/(コードに加えていくつかの回路図が必要な場合)に問い合わせる必要があります。 Ardunioについての質問は、http://arduino.stackexchange.com/で確認してください。 http://raspberrypi.stackexchange.com/もあります。 – Lundin

答えて

1

に:私は私のコードで何が間違って表示されていない

Circuits :あなたが最後にいるとき、それは次の出力で(オンまたはオフ)真または偽与えLED CとC++の場合、2つの値を比較して等しいかどうかを比較する場合は、=(代入演算子)の代わりに==(比較演算子)を使用する必要があります。

while(i = 1){ 

そして、ここで:

if(values[j] = false){ 

変更==にそれらのあなたはaccidentilyここで間違った演算子を使用しています。

+0

私はそれを更新して言及をありがとうが、悲しいことに私の問題を解決しない – DiceOfDoom