Error in my code that I can't find

Projects Programming
1

Hi folks, I'm a totally newbie to Arduino but have done some programming looong time ago for ABB industrial robots. (S4)

Now I have a private project with my model railroad and have got some problems.
I get some errors while writing the code for my little program. Well you have to start somewhere;)

Where did I go wrong? Comments were made in Swedish language so I took them out :slight_smile:

This is my sketch and at the bottom is my error readings.

int sensorState = 0;

void setup()
{
pinMode(A0, INPUT);
pinMode(A1, INPUT);
pinMode(A2, INPUT);
pinMode(A3, INPUT);
pinMode(A4, INPUT);
pinMode(A5, INPUT);
pinMode(0, INPUT);
pinMode(1, INPUT);
pinMode(2, OUTPUT);
pinMode(3, OUTPUT);
pinMode(4, OUTPUT);
pinMode(5, OUTPUT);
pinMode(6, OUTPUT);
pinMode(7, OUTPUT);
pinMode(8, OUTPUT);
pinMode(9, OUTPUT);
pinMode(10, OUTPUT);
pinMode(11, OUTPUT);
pinMode(12, OUTPUT);
pinMode(13, OUTPUT);

}

void loop()
//Läs status på givaren om den är påverkad.
//Om den är påverkad sensorState = HIGH;
{
sensorState = digitalRead(A0);
if (sensorState == HIGH);
digitalWrite(2, HIGH);
digitalWrite(10, HIGH);
delay(500);
digitalWrite(10, LOW);
digitalWrite(13, HIGH);
digitalWrite(11, HIGH);
}

{
sensorState = digitalRead(A1);
if (sensorState == HIGH);
digitalWrite(3, HIGH);
}

{
sensorState = digitalRead(A2);
if (sensorState == HIGH);
digitalWrite(4, HIGH);
}

{
sensorStat = digitalRead(A3);
if (sensorState == HIGH);
digitalWrite(5, HIGH);
}

{
sensorState = digitalRead(A4);
if (sensorState == HIGH);
digitalWrite(6, HIGH);
}

{
sensorState = digitalRead(A5);
if (sensorState == HIGH);
digitalWrite(7, HIGH);
}

{
sensorState = digitalRead(0);
if (sensorState == HIGH);
digitalWrite(8, HIGH);
}

{
sensorState = digitalRead(A0);
if (sensorState == HIGH);
digitalWrite(2, LOW);
digitalWrite(3, LOW);
digitalWrite(4, LOW);
digitalWrite(5, LOW);
digitalWrite(6, LOW);
digitalWrite(7, LOW);
digitalWrite(8, LOW);
digitalWrite(11, LOW);
}

{
sensorState = digitalRead(1);
if (sensorState == HIGH);
digitalWrite(9, HIGH);
digitalWrite(12, HIGH);
delay(500);
digitalWrite(12, LOW);
digitalWrite(13, HIGH);
digitalWrite(11, HIGH);
}

{
sensorState = digitalRead(A1);
if (sensorState == HIGH);
digitalWrite(3, HIGH);
}

{
sensorState = digitalRead(A2);
if (sensorState == HIGH);
digitalWrite(4, HIGH);
}

{
sensorStat = digitalRead(A3);
if (sensorState == HIGH);
digitalWrite(5, HIGH);
}

{
sensorState = digitalRead(A4);
if (sensorState == HIGH);
digitalWrite(6, HIGH);
}

{
sensorState = digitalRead(A5);
if (sensorState == HIGH);
digitalWrite(7, HIGH);
}

{
sensorState = digitalRead(0);
if (sensorState == HIGH);
digitalWrite(8, HIGH);
}

{
sensorState = digitalRead(1);
if (sensorState == HIGH);
digitalWrite(3, LOW);
digitalWrite(4, LOW);
digitalWrite(5, LOW);
digitalWrite(6, LOW);
digitalWrite(7, LOW);
digitalWrite(9, LOW);
digitalWrite(13, LOW);
digitalWrite(11, LOW);
}

Error codes.

sketch_nov18a:50: error: expected unqualified-id before '{' token
sketch_nov18a:56: error: expected unqualified-id before '{' token
sketch_nov18a:62: error: expected unqualified-id before '{' token
sketch_nov18a:68: error: expected unqualified-id before '{' token
sketch_nov18a:74: error: expected unqualified-id before '{' token
sketch_nov18a:80: error: expected unqualified-id before '{' token
sketch_nov18a:86: error: expected unqualified-id before '{' token
sketch_nov18a:102: error: expected unqualified-id before '{' token
sketch_nov18a:113: error: expected unqualified-id before '{' token
sketch_nov18a:119: error: expected unqualified-id before '{' token
sketch_nov18a:125: error: expected unqualified-id before '{' token
sketch_nov18a:131: error: expected unqualified-id before '{' token
sketch_nov18a:137: error: expected unqualified-id before '{' token
sketch_nov18a:143: error: expected unqualified-id before '{' token
sketch_nov18a:149: error: expected unqualified-id before '{' token

2

Get rid of all the ; at the end of the lines that use the "if" function.
Look up how to use if in the reference section of this site.
When you want to execute more than one line of code following the if encase them in {}

3

also, you have some typo's:

sensorStat = digitalRead(A3);

and needed brackets to encapsulate all of loop():

int sensorState = 0; 

void setup() 
{
  pinMode(A0, INPUT);
  pinMode(A1, INPUT);
  pinMode(A2, INPUT);
  pinMode(A3, INPUT);
  pinMode(A4, INPUT);
  pinMode(A5, INPUT);
  pinMode(0, INPUT);
  pinMode(1, INPUT);
  pinMode(2, OUTPUT);
  pinMode(3, OUTPUT);
  pinMode(4, OUTPUT);
  pinMode(5, OUTPUT);
  pinMode(6, OUTPUT);
  pinMode(7, OUTPUT);
  pinMode(8, OUTPUT);
  pinMode(9, OUTPUT);
  pinMode(10, OUTPUT);
  pinMode(11, OUTPUT);
  pinMode(12, OUTPUT);
  pinMode(13, OUTPUT);

}



void loop() 
{
  //Läs status på givaren om den är påverkad.
  //Om den är påverkad sensorState = HIGH;
  {
    sensorState = digitalRead(A0);                 
    if (sensorState == HIGH)
    digitalWrite(2, HIGH);                        
    digitalWrite(10, HIGH);         
    delay(500);               
    digitalWrite(10, LOW);                    
    digitalWrite(13, HIGH);                       
    digitalWrite(11, HIGH);                        
  }

  {
    sensorState = digitalRead(A1); 
    if (sensorState == HIGH)      
    digitalWrite(3, HIGH);                        
  }

  {
    sensorState = digitalRead(A2);
    if (sensorState == HIGH)        
    digitalWrite(4, HIGH);         
  }

  {
    sensorState = digitalRead(A3);
    if (sensorState == HIGH)         
    digitalWrite(5, HIGH);         
  }

  {
    sensorState = digitalRead(A4);
    if (sensorState == HIGH)        
    digitalWrite(6, HIGH);         
  }

  {
    sensorState = digitalRead(A5);
    if (sensorState == HIGH)        
    digitalWrite(7, HIGH);         
  }

  {
    sensorState = digitalRead(0);
    if (sensorState == HIGH)        
    digitalWrite(8, HIGH);         
  }

  {
    sensorState = digitalRead(A0);
    if (sensorState == HIGH)                       
    digitalWrite(2, LOW);
    digitalWrite(3, LOW);
    digitalWrite(4, LOW);
    digitalWrite(5, LOW);
    digitalWrite(6, LOW);
    digitalWrite(7, LOW);
    digitalWrite(8, LOW);            
    digitalWrite(11, LOW);                          
  }

  {
    sensorState = digitalRead(1);                 
    if (sensorState == HIGH)
    digitalWrite(9, HIGH);                         
    digitalWrite(12, HIGH);         
    delay(500);               
    digitalWrite(12, LOW);                  
    digitalWrite(13, HIGH);                        
    digitalWrite(11, HIGH);                       
  }

  {
    sensorState = digitalRead(A1); 
    if (sensorState == HIGH)         
    digitalWrite(3, HIGH);                        
  }

  {
    sensorState = digitalRead(A2);
    if (sensorState == HIGH)       
    digitalWrite(4, HIGH);         
  }

  {
    sensorState = digitalRead(A3);
    if (sensorState == HIGH)       
    digitalWrite(5, HIGH);         
  }

  {
    sensorState = digitalRead(A4);
    if (sensorState == HIGH)        
    digitalWrite(6, HIGH);         
  }

  {
    sensorState = digitalRead(A5);
    if (sensorState == HIGH)        
    digitalWrite(7, HIGH);         
  }

  {
    sensorState = digitalRead(0);
    if (sensorState == HIGH)      
    digitalWrite(8, HIGH);         
  }

  {
    sensorState = digitalRead(1);
    if (sensorState == HIGH)                   
    digitalWrite(3, LOW);
    digitalWrite(4, LOW);
    digitalWrite(5, LOW);
    digitalWrite(6, LOW);
    digitalWrite(7, LOW);
    digitalWrite(9, LOW);
    digitalWrite(13, LOW);            
    digitalWrite(11, LOW);                          
  }
}