Getting message 'error compiling for board' for reasons unknown

Projects Programming
1

So for some reason I can't upload this to my Arduino Uno. It's uploading to the right port but it keeps giving me this message.
My code is:

void setup() {

  int input;
  
  Serial.begin(34800);


  void loop(); {

  
    {
      if (Serial.available())
        (input) == Serial.read();
      if (input == "ARMUP")
        digitalWrite(1, HIGH);
      digitalWrite(4, HIGH);
    }


    {
      if (input == "ARMDOWN");
      digitalWrite(2, HIGH);
      digitalWrite(3, HIGH);
    }
  }
}

The error messages state things like: warning: ISO C++ forbids comparison between pointer and integer [-fpermissive]

if (input == "ARMDOWN");
and In function main':, and undefined reference to loop'

Does anyone know why this is happening?

2

There are a lot of problems here:

A very helpful troubleshooting tool is the Auto Format feature (Tools > Auto Format in the Arduino IDE or Ctrl + B in the Arduino Web Editor). If you do an Auto Format and then compare the resulting indentation to your intended program structure, it will quickly point you to where there is a missing or extra brace.

Another useful feature of the Arduino IDE/Arduino Web Editor is that when you place the cursor next to one bracket, it puts a box around the matching bracket. In the Arduino IDE, if the cursor is next to the closing bracket and the opening bracket is off the screen then it will show the opening bracket line in a tool tip after a short delay.

After doing that, do you notice the serious problem with your code?

BobZhao:

void loop(); {

Please compare this to your setup() function and all the function definitions in the example sketches (File > Examples). Do you see something different about your loop() definition?

BobZhao:

  void loop(); {

{

Don't randomly add extra braces ({). They don't serve any purpose and only make your code more difficult to understand and more likely to get bugs. You should understand what every line of code does and any line of code that serves no purpose should be removed.

BobZhao:

(input) == Serial.read();

Now you have useless parentheses. You need to be more methodical and organized in the code you write. Don't just add things randomly. You need to understand what every single character you add to your code does. If you don't understand, research and experiment until you do.

BobZhao:

        (input) == Serial.read();

if (input == "ARMUP")

Please spend some time studying this documentation until you understand why that code makes no sense, for multiple reasons: