Actions
Forum Thread Details
  • State Suggested Answer
  • Replies 6 replies
  • Answers 4 answers
  • Subscribers 393 subscribers
  • Views 622 views
  • Users 0 members are here
Related

Please help me to debug this programme

richipearl

I wrote the following code to make a seven digit segment display number from 1 to 9 continuously but while compiling the arduino ide write error, too many argument  to function turnOn

 

 

byte ledPin[7]={2,3,4,5,6,8,9};

#define buttonPin 11

int no = 0;

int pushButton = 0;

void setup() {

  for(int a = 0;a < 7;a++){

  pinMode(ledPin[a],OUTPUT);

  }

  pinMode(buttonPin,INPUT_PULLUP);

  turnOn();

}

  // put your setup code here, to run once:

 

void turnOn()

{ 

  pushButton = digitalRead(buttonPin);

 

  {

    if (pushButton == LOW)

    no++;

    delay(200);

  }

{

switch(no)

  {

    case 1:

      digitalWrite(ledPin[0],LOW);

      digitalWrite(ledPin[1],HIGH);

      digitalWrite(ledPin[2],HIGH);

      digitalWrite(ledPin[3],LOW);

      digitalWrite(ledPin[4],LOW);

      digitalWrite(ledPin[5],LOW);

      digitalWrite(ledPin[6],LOW);

      break;

    case 2:

     digitalWrite(ledPin[0],HIGH);

     digitalWrite(ledPin[1],LOW);

     digitalWrite(ledPin[2],HIGH);

     digitalWrite(ledPin[3],HIGH);

     digitalWrite(ledPin[4],HIGH);

     digitalWrite(ledPin[5],HIGH);

     digitalWrite(ledPin[6],LOW);

     break;

    case 3:

      digitalWrite(ledPin[0],HIGH);

      digitalWrite(ledPin[1],HIGH);

      digitalWrite(ledPin[2],HIGH);

      digitalWrite(ledPin[3],HIGH);

      digitalWrite(ledPin[4],LOW);

      digitalWrite(ledPin[5],HIGH);

      digitalWrite(ledPin[6],LOW);

      break;

    case 4:

    digitalWrite(ledPin[0],LOW);

    digitalWrite(ledPin[1],HIGH);

    digitalWrite(ledPin[2],HIGH);

    digitalWrite(ledPin[3],LOW);

    digitalWrite(ledPin[4],LOW);

    digitalWrite(ledPin[5],HIGH);

    digitalWrite(ledPin[6],HIGH);

    break;

   case 5:

   digitalWrite(ledPin[0],HIGH);

   digitalWrite(ledPin[1],HIGH);

   digitalWrite(ledPin[2],LOW);

   digitalWrite(ledPin[3],HIGH);

   digitalWrite(ledPin[4],LOW);

   digitalWrite(ledPin[5],HIGH);

   digitalWrite(ledPin[6],LOW);

   break;

  case 6:

    digitalWrite(ledPin[0],HIGH);

    digitalWrite(ledPin[1],HIGH);

    digitalWrite(ledPin[2],LOW);

    digitalWrite(ledPin[3],HIGH);

    digitalWrite(ledPin[4],HIGH);

    digitalWrite(ledPin[5],HIGH);

    digitalWrite(ledPin[6],HIGH);

    break;

  case 7:

    digitalWrite(ledPin[0],HIGH);

    digitalWrite(ledPin[1],HIGH);

    digitalWrite(ledPin[2],HIGH);

    digitalWrite(ledPin[3],LOW);

    digitalWrite(ledPin[4],LOW);

    digitalWrite(ledPin[5],LOW);

    digitalWrite(ledPin[6],LOW);

    break;

  case 8:

      digitalWrite(ledPin[0],HIGH);

      digitalWrite(ledPin[1],HIGH);

      digitalWrite(ledPin[2],HIGH);

      digitalWrite(ledPin[3],HIGH);

      digitalWrite(ledPin[4],HIGH);

      digitalWrite(ledPin[5],HIGH);

      digitalWrite(ledPin[6],HIGH);

      break;

  case 9:

    digitalWrite(ledPin[0],HIGH);

    digitalWrite(ledPin[1],HIGH);

    digitalWrite(ledPin[2],HIGH);

    digitalWrite(ledPin[3],HIGH);

    digitalWrite(ledPin[4],LOW);

    digitalWrite(ledPin[5],HIGH);

    digitalWrite(ledPin[6],HIGH);

    break;

    }

    }

}

void loop()

{

  for (int b = 0;b < 10;b++)

  {

    turnOn(b);

    delay(500);   

  }

 

  }

 

 

// put your main code here, to run repeated

Parents
  • 0

    Thanks so much to everyone's contribution

    byte ledPin[7]={2,3,4,5,6,8,9};// declare pins to led

    #define buttonPin 11 // set buttonpin

    int no = 0;

    int pushButton = 0;

     

     

    void setup() {

      for(int a = 0;a < 7;a++){

      pinMode(ledPin[a],OUTPUT);

      }               // set ledPin as output

      pinMode(buttonPin,INPUT);  // set buttonPin to input

      pinMode(buttonPin,INPUT_PULLUP);

    }

      // put your setup code here, to run once:

      void loop()

      {

        turnOn();   // declare turnOn as function

        if(no > 9)

          no = 0; 

       

      }

      void turnOn()

    { 

      pushButton = digitalRead(buttonPin);// set the pushbutton to read into from the button

      delay(500);

        if (pushButton == LOW) // when buttonPress is detected display number

        {

        no++;

    switch(no)

      {

        case 1:

          digitalWrite(ledPin[0],LOW);

          digitalWrite(ledPin[1],HIGH);

          digitalWrite(ledPin[2],HIGH);

          digitalWrite(ledPin[3],LOW);

          digitalWrite(ledPin[4],LOW);

          digitalWrite(ledPin[5],LOW);

          digitalWrite(ledPin[6],LOW);

          break;

        case 2:

         digitalWrite(ledPin[0],HIGH);

         digitalWrite(ledPin[1],HIGH);

         digitalWrite(ledPin[2],LOW);

         digitalWrite(ledPin[3],HIGH);

         digitalWrite(ledPin[4],HIGH);

         digitalWrite(ledPin[5],LOW);

         digitalWrite(ledPin[6],HIGH);

         break;

        case 3:

          digitalWrite(ledPin[0],HIGH);

          digitalWrite(ledPin[1],HIGH);

          digitalWrite(ledPin[2],HIGH);

          digitalWrite(ledPin[3],HIGH);

          digitalWrite(ledPin[4],LOW);

          digitalWrite(ledPin[5],LOW);

          digitalWrite(ledPin[6],HIGH);

          break;

        case 4:

        digitalWrite(ledPin[0],LOW);

        digitalWrite(ledPin[1],HIGH);

        digitalWrite(ledPin[2],HIGH);

        digitalWrite(ledPin[3],LOW);

        digitalWrite(ledPin[4],LOW);

        digitalWrite(ledPin[5],HIGH);

        digitalWrite(ledPin[6],HIGH);

        break;

       case 5:

       digitalWrite(ledPin[0],HIGH);

       digitalWrite(ledPin[1],LOW);

       digitalWrite(ledPin[2],HIGH);

       digitalWrite(ledPin[3],HIGH);

       digitalWrite(ledPin[4],LOW);

       digitalWrite(ledPin[5],HIGH);

       digitalWrite(ledPin[6],HIGH);

       break;

      case 6:

        digitalWrite(ledPin[0],HIGH);

        digitalWrite(ledPin[1],LOW);

        digitalWrite(ledPin[2],HIGH);

        digitalWrite(ledPin[3],HIGH);

        digitalWrite(ledPin[4],HIGH);

        digitalWrite(ledPin[5],HIGH);

        digitalWrite(ledPin[6],HIGH);

        break;

      case 7:

        digitalWrite(ledPin[0],HIGH);

        digitalWrite(ledPin[1],HIGH);

        digitalWrite(ledPin[2],HIGH);

        digitalWrite(ledPin[3],LOW);

        digitalWrite(ledPin[4],LOW);

        digitalWrite(ledPin[5],LOW);

        digitalWrite(ledPin[6],LOW);

        break;

      case 8:

          digitalWrite(ledPin[0],HIGH);

          digitalWrite(ledPin[1],HIGH);

          digitalWrite(ledPin[2],HIGH);

          digitalWrite(ledPin[3],HIGH);

          digitalWrite(ledPin[4],HIGH);

          digitalWrite(ledPin[5],HIGH);

          digitalWrite(ledPin[6],HIGH);

          break;

      case 9:

        digitalWrite(ledPin[0],HIGH);

        digitalWrite(ledPin[1],HIGH);

        digitalWrite(ledPin[2],HIGH);

        digitalWrite(ledPin[3],HIGH);

        digitalWrite(ledPin[4],LOW);

        digitalWrite(ledPin[5],HIGH);

        digitalWrite(ledPin[6],HIGH);

        break;

        }

        }

    }

    my code is now working

Reply
  • 0

    Thanks so much to everyone's contribution

    byte ledPin[7]={2,3,4,5,6,8,9};// declare pins to led

    #define buttonPin 11 // set buttonpin

    int no = 0;

    int pushButton = 0;

     

     

    void setup() {

      for(int a = 0;a < 7;a++){

      pinMode(ledPin[a],OUTPUT);

      }               // set ledPin as output

      pinMode(buttonPin,INPUT);  // set buttonPin to input

      pinMode(buttonPin,INPUT_PULLUP);

    }

      // put your setup code here, to run once:

      void loop()

      {

        turnOn();   // declare turnOn as function

        if(no > 9)

          no = 0; 

       

      }

      void turnOn()

    { 

      pushButton = digitalRead(buttonPin);// set the pushbutton to read into from the button

      delay(500);

        if (pushButton == LOW) // when buttonPress is detected display number

        {

        no++;

    switch(no)

      {

        case 1:

          digitalWrite(ledPin[0],LOW);

          digitalWrite(ledPin[1],HIGH);

          digitalWrite(ledPin[2],HIGH);

          digitalWrite(ledPin[3],LOW);

          digitalWrite(ledPin[4],LOW);

          digitalWrite(ledPin[5],LOW);

          digitalWrite(ledPin[6],LOW);

          break;

        case 2:

         digitalWrite(ledPin[0],HIGH);

         digitalWrite(ledPin[1],HIGH);

         digitalWrite(ledPin[2],LOW);

         digitalWrite(ledPin[3],HIGH);

         digitalWrite(ledPin[4],HIGH);

         digitalWrite(ledPin[5],LOW);

         digitalWrite(ledPin[6],HIGH);

         break;

        case 3:

          digitalWrite(ledPin[0],HIGH);

          digitalWrite(ledPin[1],HIGH);

          digitalWrite(ledPin[2],HIGH);

          digitalWrite(ledPin[3],HIGH);

          digitalWrite(ledPin[4],LOW);

          digitalWrite(ledPin[5],LOW);

          digitalWrite(ledPin[6],HIGH);

          break;

        case 4:

        digitalWrite(ledPin[0],LOW);

        digitalWrite(ledPin[1],HIGH);

        digitalWrite(ledPin[2],HIGH);

        digitalWrite(ledPin[3],LOW);

        digitalWrite(ledPin[4],LOW);

        digitalWrite(ledPin[5],HIGH);

        digitalWrite(ledPin[6],HIGH);

        break;

       case 5:

       digitalWrite(ledPin[0],HIGH);

       digitalWrite(ledPin[1],LOW);

       digitalWrite(ledPin[2],HIGH);

       digitalWrite(ledPin[3],HIGH);

       digitalWrite(ledPin[4],LOW);

       digitalWrite(ledPin[5],HIGH);

       digitalWrite(ledPin[6],HIGH);

       break;

      case 6:

        digitalWrite(ledPin[0],HIGH);

        digitalWrite(ledPin[1],LOW);

        digitalWrite(ledPin[2],HIGH);

        digitalWrite(ledPin[3],HIGH);

        digitalWrite(ledPin[4],HIGH);

        digitalWrite(ledPin[5],HIGH);

        digitalWrite(ledPin[6],HIGH);

        break;

      case 7:

        digitalWrite(ledPin[0],HIGH);

        digitalWrite(ledPin[1],HIGH);

        digitalWrite(ledPin[2],HIGH);

        digitalWrite(ledPin[3],LOW);

        digitalWrite(ledPin[4],LOW);

        digitalWrite(ledPin[5],LOW);

        digitalWrite(ledPin[6],LOW);

        break;

      case 8:

          digitalWrite(ledPin[0],HIGH);

          digitalWrite(ledPin[1],HIGH);

          digitalWrite(ledPin[2],HIGH);

          digitalWrite(ledPin[3],HIGH);

          digitalWrite(ledPin[4],HIGH);

          digitalWrite(ledPin[5],HIGH);

          digitalWrite(ledPin[6],HIGH);

          break;

      case 9:

        digitalWrite(ledPin[0],HIGH);

        digitalWrite(ledPin[1],HIGH);

        digitalWrite(ledPin[2],HIGH);

        digitalWrite(ledPin[3],HIGH);

        digitalWrite(ledPin[4],LOW);

        digitalWrite(ledPin[5],HIGH);

        digitalWrite(ledPin[6],HIGH);

        break;

        }

        }

    }

    my code is now working

Children
No Data