How to solve this problem ? I get this message whenever I compile it in arduino compiler ? expected initializer before '-' token

Question

0.00/5 (No votes)

See more:

#include <SoftwareSerial.h>
int PulseSensorPurplePin-0;
int Signal;
int Threshold - 550;
int S;
Softwareserial mySerial(10, 11);
void setup()
( pinMode (LED13, OUTPUT);
  Serial.begin(9600);
  mySerial.begin(9600); )
void loop()
( Signal = analogRead(PulseSensorPurplepin);
  S = (Signal / 8); Serial.print("Your heart Beat is : ");
  Serial.printIn(S);
  mySerial.printIn(S);
  if (Signal > Threshold)
  ( digitalWrite(LED13, HIGH);
  ) else
    (
      digitalWrite(LED13, LOW);
    )
    delay(1000);
  )

What I have tried:

can someone help me solve this problem ? thank you

Posted 16-Apr-22 13:20pm

Shieka Alshehhi

Updated 12-Jul-22 20:11pm

Add a Solution

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Dave Kreskowiak · Answer 1 · 2022-04-16T14:57:00

Solution 1

Variable names cannot have '-' characters in them.

C

#include <SoftwareSerial.h>
int PulseSensorPurplePin-0;    <-- invalid name
int Signal;
int Threshold - 550;           <-- invalid name
int S;

These should be:

C

#include <SoftwareSerial.h>
int PulseSensorPurplePin;
int Signal;
int Threshold;
int S;

Posted 16-Apr-22 14:57pm

Dave Kreskowiak

Comments

Shieka Alshehhi 16-Apr-22 21:09pm

thank you , but i got a new error which says " initializer provided for function" for this line :
( pinMode (LED13, OUTPUT);

Dave Kreskowiak 16-Apr-22 21:31pm

I didn't catch this before, but you're have parenthesis where curly braces should be:

#include <SoftwareSerial.h>
int PulseSensorPurplePin-0;
int Signal;
int Threshold - 550;
int S;
Softwareserial mySerial(10, 11);
void setup()
{
    pinMode (LED13, OUTPUT);
    Serial.begin(9600);
    mySerial.begin(9600);
}

void loop()
{
    Signal = analogRead(PulseSensorPurplepin);
    S = (Signal / 8);
    Serial.print("Your heart Beat is : ");
    Serial.printIn(S);
    mySerial.printIn(S);
    
    if (Signal > Threshold)
    {
        digitalWrite(LED13, HIGH);
    }
    else
    {
        digitalWrite(LED13, LOW);
    }

    delay(1000);
}

You REALLY need a beginners book on writing code for an Arduino. Both of these error are really easy to spot and fix.

CPallini 17-Apr-22 5:13am

5."You REALLY need a beginners book on writing code for an Arduino. Both of these error are really easy to spot and fix"
Indeed.
Mostly considering that Arduino IDE does provide an application skeleton with curly braces at the right places...

bonezegei · Answer 2 · 2022-07-12T20:11:00

Solution 2

#include <softwareserial.h>

int PulseSensorPurplePin = A0; //not =0 since you are reading Analog Value the value 0 for input pin is at Digital pin 0 "A0" is used instead
int Signal;
int Threshold = 550; //variable should be initialize this way i think
int S;
int LED13=13; //you should declare LED 13 if its connected to pin 13

SoftwareSerial mySerial(10, 11);

void setup(){
pinMode (LED13, OUTPUT);
Serial.begin(9600);
mySerial.begin(9600);

}

void loop(){
Signal = analogRead(PulseSensorPurplePin);

S = (Signal / 8);

Serial.print("Your heart Beat is : ");

Serial.println(S,DEC); //it should be Serial.println not Serial.printIn
mySerial.println(S,DEC); //the data type I think should also be declared as Decimal or DEC

if (Signal > Threshold){
digitalWrite(LED13, HIGH);
}
else{
digitalWrite(LED13, LOW);
}

delay(1000);
}

But I think this program does not provide a relevant heartbeat result since the code only reads analog value from A0 and compare it to a threshold. I think most heartbeat sensor runs on a one wire interface which is a digital type of data transmission serially.

Posted 12-Jul-22 20:11pm

bonezegei

Updated 13-Jul-22 4:57am

v2

Comments

CHill60 13-Jul-22 7:46am

The code

int PulseSensorPurplePin = A0;

results in the error "main.cpp:3:26: error: ‘A0’ was not declared in this scope"

bonezegei 13-Jul-22 7:55am

Oh I found additional Errors

//This are typo errors from my code
SoftwareSerial mySerial(10, 11);
Signal = analogRead(PulseSensorPurplePin);

if that code was compiled with arduino
try checking if what board was set to your IDE this code will nost likely work uno mega Boards like arduino Uno and arduino mega..

If the board you chose was a different one refer to the pinout of the analog pin.. some boards does not have this type of pins thats why that type of error occurs..

bonezegei 13-Jul-22 7:57am

#include <softwareserial.h>

int PulseSensorPurplePin = A0; //not =0 since you are reading Analog Value the value 0 for input pin is at Digital pin 0 "A0" is used instead
int Signal;
int Threshold = 550; //variable should be initialize this way i think
int S;
int LED13=13; //you should declare LED 13 if its connected to pin 13

SoftwareSerial mySerial(10, 11);

void setup(){
pinMode (LED13, OUTPUT);
Serial.begin(9600);
mySerial.begin(9600);

}

void loop(){
Signal = analogRead(PulseSensorPurplePin);

S = (Signal / 8);

Serial.print("Your heart Beat is : ");

Serial.println(S,DEC); //it should be Serial.println not Serial.printIn
mySerial.println(S,DEC); //the data type I think should also be declared as Decimal or DEC

if (Signal > Threshold){
digitalWrite(LED13, HIGH);
}
else{
digitalWrite(LED13, LOW);
}

delay(1000);
}

bonezegei 13-Jul-22 7:59am

the code above works fine when i compiles it

but the board was set to an arduino uno
if the problem still exist even the board was set to AVR microcontroller try adding #include <arduino.h>