Actions
Forum Thread Details
  • Replies 4 replies
  • Subscribers 402 subscribers
  • Views 375 views
  • Users 0 members are here
Related

my second switch statement.

Former Member

I just wrote my first switch statemen and now Im working on the second but I cannot figure out why I get this error.  'states' was not declared in this case.  I am confused because it is just about the same as my first statement so why do I have this?

 

int state=8;

int mi=9;

int whichone;

int buttonst;

int buttonmi;

 

void setup()

{

Serial.begin(9600);

pinMode(state, INPUT);

pinMode(mi, INPUT);

buttonst=digitalRead(8);

buttonmi=digitalRead(9);

}

 

 

void loop()

{

switch(whichone)

  {

  case 0:

  buttonst=HIGH;

  buttonmi=LOW;

  states();

  break;

 

  case 1:

  buttonst=LOW&&buttonmi=HIGH;

  mis();

  break

 

  }

 

void states()

{

int states;

Serial.println("state");

delay (500);

 

}

 

 

void mis()

{

int mis;

Serial.println("mi");

delay (500);

 

}

 

}

  • Appears to be illegal overloading of name "states"

    I'll bet it was flagged in the bottom window of the 'Duino IDE.

     

    This is what I do when I get those hard to read error messages in the bottom window:

    1. Left click mouse to select message of interest.

    2. Right click mouse to 'copy' the highlighted text.

    3. Open Notepad or any other word processor.

    4. Paste the text into Notepad and make the font size large enough to easily see.

    5. Use my brain power to syntactically analyze the error message.   Informally this is called parsing.  image

    6. If there is some phrase I have no idea about; then it helps to look that part up on the WWW.

     

    -=SyntaxMatters=-

  • It looks like you have the function states() defined inside the function loop().

     

    That last curly brace needs to be moved up above where you defined void states()

     

    good luck.

     

    Mike

  • in reply to mconners

    BTW, this is an error as well, buttonst=LOW&&buttonmi=HIGH;

     

    Not sure what you are trying to do there

  • in reply to mconners

    The following will compile, not sure it does what you want though.

     

    int state=8;

    int mi=9;

    int whichone;

    int buttonst;

    int buttonmi;

     

     

    void setup()

    {

      Serial.begin(9600);

      pinMode(state, INPUT);

      pinMode(mi, INPUT);

      buttonst=digitalRead(8);

      buttonmi=digitalRead(9);

    }

     

     

    void loop()

    {

      switch(whichone)

      {

      case 0:

        buttonst=HIGH;

        buttonmi=LOW;

        states();

        break;

     

     

      case 1:

        buttonst=LOW;

        buttonmi=HIGH;

        mis();

        break;

      }

    }

     

     

    void states()

    {

      int states;

      Serial.println("state");

      delay (500);

    }

     

     

    void mis()

    {

      int mis;

      Serial.println("mi");

      delay (500);

    }