Error: Expected primary-expression before ',' token

Projects Programming
1

hello there. Im currently working on my university project. Im working on a smart traffic light that can indicate the traffic. Right now im stucked on the coding and i cant seem to understand the error. Help me pls :frowning:

here is the code where there is an error

void set3()
{
  lcd.setCursor(colx, 0);
  lcd.print("NA ");
  lcd.setCursor(colx, 1);
  lcd.print("OK!!");
}

here is the error

error: expected primary-expression before ',' token

   lcd.setCursor(colx, 0);

                     ^

jajel:73: error: expected primary-expression before ',' token

   lcd.setCursor(colx, 1);
2

Your error is more than likely caused by some code that you have not shown. Please provide your complete code.

3

sorry for that. here is my full code

#define colx
#include <LiquidCrystal.h>

int inputPin1 = 9;              //choose the input pin (for PIR1 sensor)
int inputPin = 8;               // choose the input pin (for PIR sensor)
int pirState = LOW;             // we start, assuming no motion detected
int sensor1, sensor2;
LiquidCrystal lcd(7, 6, 5, 4, 3, 2);

int val = 0;                    // variable for reading the pin status
int val1 = 0;

void setup() {
  pinMode(sensor1, INPUT);
  pinMode(sensor2, INPUT);
  lcd.begin (16, 2);
  lcd.setCursor(0, 0);
  lcd.print("Sensor1:");
  lcd.setCursor(0, 1);
  lcd.print("Sensor2:");
}

void set1()
{
  lcd.setCursor(colx, 0);
  lcd.print("OK!!");
  lcd.setCursor(colx, 1);
  lcd.print("OK!!");
}
void set2()
{
  lcd.setCursor(colx, 0);
  lcd.print("OK!!");
  lcd.setCursor(colx, 1);
  lcd.print("NA ");
}
void set3()
{
  lcd.setCursor(colx, 0);
  lcd.print("NA ");
  lcd.setCursor(colx, 1);
  lcd.print("OK!!");
}
void default()
{
  lcd.setCursor(colx, 0);
  lcd.print("NA ");
  lcd.setCursor(colx, 1);
  lcd.print("NA ");
}

void set1()
{
  lcd.setCursor(colx, 0);
  lcd.print("OK!!");
  lcd.setCursor(colx, 1);
  lcd.print("OK!!");
}
void set2()
{
  lcd.setCursor(colx, 0);
  lcd.print("OK!!");
  lcd.setCursor(colx, 1);
  lcd.print("NA ");
}
void set3()
{
  lcd.setCursor(colx, 0);
  lcd.print("NA ");
  lcd.setCursor(colx, 1);
  lcd.print("OK!!");
}
void default()
{
  lcd.setCursor(colx, 0);
  lcd.print("NA ");
  lcd.setCursor(colx, 1);
  lcd.print("NA ");
}

void loop()
{
  int senread1 = digitalRead(sensor1);
  int senread2 = digitalRead(sensor2);

  if (senread1 == HIGH && senread2 == HIGH )
  {
    set1();
  }
  else if (senread1 == !HIGH && senread2 == HIGH )
  {
    set2();
  }
  else if (senread1 == HIGH && senread2 == !HIGH )
  {
    set3();
  }
  else
  {
    default();
  }

  delay(1);
}
5

Delta_G:

#define colx

You've defined colx as nothing. #define things get replaces before compilation. So that line when the compiler sees it looks like

lcd.setCursor( , 0);

And now you see the error

YOU ARE MY HERO MAN. That did the trick. Thank you thank you thank you thank you

6

(deleted)