Identical code but one gives error message (???)

Using Arduino Programming Questions
1

Why does these codes gives me this error message:"'evalEast' does not name a type"
Here are the code:
const int east = A1;
const int north = A2;
const int west = A3;
const int south = A4;
const int center = A5;

const int East1a=6;
const int North1a=5;
const int West1a=4;
const int South1a=3;

int valuEast =16;
int valuNorth =15;
int valuWest =14;
int valuSouth =13;

void setup() {
pinMode(east, INPUT_PULLUP);
pinMode(East1a, INPUT);
Serial.begin(9600); }

void loop() {

start:
delay (500);
valuEast = analogRead (east);
Serial.println(valuEast); }

evalEast:
if (valu >=150)goto checkNorth;
if (valu <=50)goto sendEast1a;
goto start;

checkNorth:
delay (5000);
goto start;

sendEast1a:
Serial.println(east);
pinMode (East1a, OUTPUT);
digitalWrite(East1a, LOW);
if (valu <50) goto sendEast1a;
pinMode (South1a, INPUT);
goto start;
}

When these, which to my believe, are very identical:
Here are the code:
const int currentSensor = A0; // the current source is connected to analog pin 0
const int voltSensor = A2; // the voltage source is connected to analog pin 2
int I_sens = 0; // variable to store the value read from the I_sens pin
int V_sens = 2; // variable to store the value read from the V_sens pin

void setup()
{
// initialize digital pin 10, 11, 12 & 13 as outputs.
pinMode(10, OUTPUT); // Unlatch command
pinMode(11, OUTPUT); // Latch command
pinMode(12, OUTPUT); // Led unlatched (green)
pinMode(13, OUTPUT); // Led latched (red)
Serial.begin(9600); // Serial port baud rate
}
void loop()
{
prestart:
V_sens = analogRead(2);
Serial.print("V_sens=");
Serial.println(V_sens);
delay(1000);
{
start:
Serial.println("Off line");
V_sens = analogRead(2);
Serial.print("V_sens=");
Serial.println(V_sens);
if (V_sens<150)goto start;
if (V_sens>=150){Serial.println("On line");}

loadOnLine:
I_sens = analogRead(0);
Serial.print("I_sens=");
Serial.println(I_sens);
if (I_sens>=300)goto Overload;
if (I_sens>=275)goto warning;
delay(1000);
goto loadOnLine;

warning:
Serial.println("Warning");
delay(5000);
goto loadOnLine;

Overload:
Serial.println("Overload");
delay(1000);
goto Overload;
}
}

2

You have a misplaced brace.

void loop() {
     
start:
    delay (500);
    valuEast = analogRead (east);
    Serial.println(valuEast); } // this brace needs to be removed

evalEast:
3

Check your braces. This is the entire loop function as the compiler sees it:

void loop() {
     
start: 
    delay (500);
    valuEast = analogRead (east);
    Serial.println(valuEast); }

Everything after that is sitting outside of any function.

4

Not the cause of the problem, but get rid of the goto statements. Once you have developed bad coding habits (like significant use of goto statements), it's difficult to get rid of them.

5

Thank's guys:
The OldSchoolBob, that I am, have been "P_basic" user for the past 30 years; just discovering C and Arduino.
Best regards!
Robert

6

Be sure that is NOTED; I do understand your so right !!!
YO!