arduino if statement giving error

Question

I am working on some arduino code and my program keeps on giving me this error,

ISO C++ forbids comparison between pointer and integer [-fpermissive]

I've tried searching on the internet to solve this issue but, either the solution is incorrect, or irrelevant. here is where the arduino software is saying the problem is,

if((millis - incLastDebounce) > debounceDelay) {

and if you need the rest of the code here it is,

#include <LiquidCrystal.h>

int freq = 0;
int change = 0;
const int incPin = 3;
const int setPin = 2;
int incButtonState;
int setButtonState;
int incPreviousState;
int setPreviousState;
int incLastDebounce;
int setLastDebounce;
const int debounceDelay = 50;


LiquidCrystal lcd(12, 11, 5, 4, 3, 2);
void setup() {
  // put your setup code here, to run once:
lcd.begin(16, 2);
pinMode(setPin, INPUT);
pinMode(incPin, INPUT);
lcd.print("Frequency: " + freq);
}

void loop() {
  // put your main code here, to run repeatedly:
int incReading = digitalRead(incPin);
int setReading = digitalRead(setPin);

if(setReading != setPreviousState) {
  setLastDebounce = millis();
}
if(incReading != incPreviousState) {
  incLastDebounce = millis();
}

if((millis - setLastDebounce) > debounceDelay) {

  if(setReading != setButtonState) {
    setButtonState = setReading;
  }
  if(setButtonState == HIGH) {
    //Okay so here you will do your set lcd voodoo
  }
}

if((millis - incLastDebounce) > debounceDelay) {
  if(incReading != buttonState) {
    incButtonState = incReading;
  }
    if(buttonState == HIGH) {
      // here you can put the lcd code
      change = change + 500;
      if(change == 10500){
        change = 0;
       }
    }

  }


incPreviousState = incReading;
setPreviousState = setReading;
}

hopefully you can find the problem and help.

melpomene · Accepted Answer · 2016-06-11 18:16:17Z

2

Looks like you're missing parentheses after millis, so instead of calling the function, you're trying to do arithmetic with its memory address.

This will probably work better:

if ((millis() - incLastDebounce) > debounceDelay) {

answered Jun 11, 2016 at 18:16

melpomene

86.2k8 gold badges95 silver badges154 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

arduino if statement giving error

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related