element14 Community
element14 Community
    Register Log In
  • Site
  • Search
  • Log In Register
  • About Us
  • Community Hub
    Community Hub
    • What's New on element14
    • Feedback and Support
    • Benefits of Membership
    • Personal Blogs
    • Members Area
    • Achievement Levels
  • Learn
    Learn
    • Ask an Expert
    • eBooks
    • element14 presents
    • Learning Center
    • Tech Spotlight
    • STEM Academy
    • Webinars, Training and Events
    • Learning Groups
  • Technologies
    Technologies
    • 3D Printing
    • FPGA
    • Industrial Automation
    • Internet of Things
    • Power & Energy
    • Sensors
    • Technology Groups
  • Challenges & Projects
    Challenges & Projects
    • Design Challenges
    • element14 presents Projects
    • Project14
    • Arduino Projects
    • Raspberry Pi Projects
    • Project Groups
  • Products
    Products
    • Arduino
    • Avnet Boards Community
    • Dev Tools
    • Manufacturers
    • Multicomp Pro
    • Product Groups
    • Raspberry Pi
    • RoadTests & Reviews
  • Store
    Store
    • Visit Your Store
    • Choose another store...
      • Europe
      •  Austria (German)
      •  Belgium (Dutch, French)
      •  Bulgaria (Bulgarian)
      •  Czech Republic (Czech)
      •  Denmark (Danish)
      •  Estonia (Estonian)
      •  Finland (Finnish)
      •  France (French)
      •  Germany (German)
      •  Hungary (Hungarian)
      •  Ireland
      •  Israel
      •  Italy (Italian)
      •  Latvia (Latvian)
      •  
      •  Lithuania (Lithuanian)
      •  Netherlands (Dutch)
      •  Norway (Norwegian)
      •  Poland (Polish)
      •  Portugal (Portuguese)
      •  Romania (Romanian)
      •  Russia (Russian)
      •  Slovakia (Slovak)
      •  Slovenia (Slovenian)
      •  Spain (Spanish)
      •  Sweden (Swedish)
      •  Switzerland(German, French)
      •  Turkey (Turkish)
      •  United Kingdom
      • Asia Pacific
      •  Australia
      •  China
      •  Hong Kong
      •  India
      •  Korea (Korean)
      •  Malaysia
      •  New Zealand
      •  Philippines
      •  Singapore
      •  Taiwan
      •  Thailand (Thai)
      • Americas
      •  Brazil (Portuguese)
      •  Canada
      •  Mexico (Spanish)
      •  United States
      Can't find the country/region you're looking for? Visit our export site or find a local distributor.
  • Translate
  • Profile
  • Settings
Arduino
  • Products
  • More
Arduino
Arduino Forum Please help me to debug this programme
  • Blog
  • Forum
  • Documents
  • Quiz
  • Events
  • Polls
  • Files
  • Members
  • Mentions
  • Sub-Groups
  • Tags
  • More
  • Cancel
  • New
Join Arduino to participate - click to join for free!
Actions
  • Share
  • More
  • Cancel
Forum Thread Details
  • State Suggested Answer
  • Replies 6 replies
  • Answers 4 answers
  • Subscribers 393 subscribers
  • Views 620 views
  • Users 0 members are here
Related

Please help me to debug this programme

richipearl
richipearl over 6 years ago

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

  • Sign in to reply
  • Cancel

Top Replies

  • Fred27
    Fred27 over 6 years ago in reply to richipearl +5 suggested
    There's not a huge amount the explain. Only one line needs to change. turnOn(b); need to become turnOn(); void loop() { for (int b = 0;b < 10;b++) { turnOn(); delay(500); } } Of course, once it compiles…
  • Jan Cumps
    Jan Cumps over 6 years ago +4 suggested
    your turnOn() function expects 0 parameters, but you pass it 1 parameter: turnOn(b)
  • afmishaq
    afmishaq over 6 years ago in reply to richipearl +2 suggested
    Rachel, You have defined your function as: void turnOn() The initial void says that it will not return anything which is fine because your function will just turn on some LED segments and does not need…
  • Jan Cumps
    0 Jan Cumps over 6 years ago

    your turnOn() function expects 0 parameters, but you pass it 1 parameter: turnOn(b)

    • Cancel
    • Vote Up +4 Vote Down
    • Sign in to reply
    • Verify Answer
    • Reject Answer
    • Cancel
  • richipearl
    0 richipearl over 6 years ago in reply to Jan Cumps

    Thanks for your contribution but can you please explain more

    • Cancel
    • Vote Up +1 Vote Down
    • Sign in to reply
    • Verify Answer
    • Cancel
  • Fred27
    0 Fred27 over 6 years ago in reply to richipearl

    There's not a huge amount the explain. Only one line needs to change. turnOn(b); need to become turnOn();

     

    void loop()
    {
      for (int b = 0;b < 10;b++)
      {
        turnOn();
        delay(500);   
      }
    }

     

    Of course, once it compiles then it may not do exactly what you were expecting. It's best to tackle this one step at a time and work through the bugs. That's the only way you'll get an understanding of what you're trying to do.

    • Cancel
    • Vote Up +5 Vote Down
    • Sign in to reply
    • Verify Answer
    • Reject Answer
    • Cancel
  • afmishaq
    0 afmishaq over 6 years ago in reply to richipearl

    Rachel,

     

    You have defined your function as: void turnOn()

     

    The initial void says that it will not return anything which is fine because your function will just turn on  some LED segments and does not need to return any value.

     

    However, the empty brackets say that it does not any arguments. This is not correct because you need to pass a number to turn on the desired segments. So, you have to put something in the brackets to indicate tthat you will be passing an argument. In your case, you are passing an integer value, so you may write 'int b' there.

     

    Hope this helps,

    Faiz

    • Cancel
    • Vote Up +2 Vote Down
    • Sign in to reply
    • Verify Answer
    • Reject Answer
    • Cancel
  • beacon_dave
    0 beacon_dave over 6 years ago

    Looks like you have an opening brace in the wrong place as well:

     

    void turnOn()
    {  
      pushButton = digitalRead(buttonPin);
      
      {    
        if (pushButton == LOW)
        no++;
        delay(200);
      }

     

     

    perhaps should be:

     

    void turnOn()
    {  
      pushButton = digitalRead(buttonPin);
       
      if (pushButton == LOW)
      {
        no++;
        delay(200);
      }

    • Cancel
    • Vote Up +2 Vote Down
    • Sign in to reply
    • Verify Answer
    • Reject Answer
    • Cancel
  • richipearl
    0 richipearl over 6 years ago

    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

    • Cancel
    • Vote Up +2 Vote Down
    • Sign in to reply
    • Verify Answer
    • Cancel
element14 Community

element14 is the first online community specifically for engineers. Connect with your peers and get expert answers to your questions.

  • Members
  • Learn
  • Technologies
  • Challenges & Projects
  • Products
  • Store
  • About Us
  • Feedback & Support
  • FAQs
  • Terms of Use
  • Privacy Policy
  • Legal and Copyright Notices
  • Sitemap
  • Cookies

An Avnet Company © 2025 Premier Farnell Limited. All Rights Reserved.

Premier Farnell Ltd, registered in England and Wales (no 00876412), registered office: Farnell House, Forge Lane, Leeds LS12 2NE.

ICP 备案号 10220084.

Follow element14

  • X
  • Facebook
  • linkedin
  • YouTube