Actions
Forum Thread Details
  • Replies 15 replies
  • Subscribers 387 subscribers
  • Views 2823 views
  • Users 0 members are here
Related

Coding with else{}...what are the cost benefits analysis?

colporteur 

if (digitalRead(ButtonPin) == LOW) {  //}

OR

if (digitalRead(ButtonPin) == HIGH) {  //
  return HIGH;
} else {  //}

I came across these two methods of testing a button. They both accomplish the same thing. 

Why would a coder use one over the other? What are advantages/disadvantages to using either?

I am a code resurrectionist not a programmer or coder. I usually cut and paste code parts to develop a working project. I have code that use both methods. I'm curious what falls into best practice? Does one provide better performance?

Parents

  • Hi Sean,

    There's not much difference, one might be a fraction faster or slower but it's negligible. It also depends on if the push-button is wired to ground or to the supply.

    Everyone does it in different ways, personally I just bury that detail in the #define section, approximately like this:

    //****** Definitions ******
    #define BUTTON_PIN 10
    #define BUTTON_PRESSED (digitalRead(BUTTON_PIN)==LOW)
    #define BUTTON_UNPRESSED (digitalRead(BUTTON_PIN)!=LOW)

    //****** Code ********
    if (BUTTON_PRESSED) {
    // do whatever
    }

    (Cant see the Editor toolbar, otherwise I'd format it as code!). 

Reply

  • Hi Sean,

    There's not much difference, one might be a fraction faster or slower but it's negligible. It also depends on if the push-button is wired to ground or to the supply.

    Everyone does it in different ways, personally I just bury that detail in the #define section, approximately like this:

    //****** Definitions ******
    #define BUTTON_PIN 10
    #define BUTTON_PRESSED (digitalRead(BUTTON_PIN)==LOW)
    #define BUTTON_UNPRESSED (digitalRead(BUTTON_PIN)!=LOW)

    //****** Code ********
    if (BUTTON_PRESSED) {
    // do whatever
    }

    (Cant see the Editor toolbar, otherwise I'd format it as code!). 

Children
No Data