Error else without a previous if перевод

#include <stdio.h>
#define foo(x) if (x>0) printf("Ouchn");

int main()
{
    int a = 4;
    int b = -3;

    if(a>b)
        foo(b) ;
    else
        printf("Arrg!n");
    printf("thanksn");

    return 0;
}

if(a>b){
    if(x>0){
        printf("Ouchn");
    }
}
else{
    printf("Arrh!n");
}
printf("thanksn");    

3 ответа

Лучший ответ

if(a>b)
    if (b>0) printf("Ouchn"); ; 
else 
    printf("Arrg!n");
//...

if(a>b){ /*braces inserted to show the interpretation*/
    if (b>0) printf("Ouchn");
    else printf("Arrg!n");
}

#define foo(x) if ((x)>0) printf("Ouchn"); else
//a ; after the macro else would complete the `else` with an empty branch

#define macro() do{/*macro_body*/}while(0)

#define foo(x) do{ if ((x)>0) printf("Ouchn"); }while(0)

if(a>b) { 
    foo(b) ; /* keep inside { } */
}
else {
    printf("Arrg!n");
}

int main()
{
    int a = 4;
    int b = -3;

    if(a>b)
        if (b>0) printf("Ouchn"); ; /* extra semicolon causes the issue */
    else /* there is no if for this else block, previous one terminated by extra ; in above if */
        printf("Arrg!n");
    printf("thanksn");

    return 0;
}

#include <stdio.h>
#define foo(x) if ((x)>0) printf("Ouchn") /* if condition was wrong.. instead of x use (x) */
int main(void)
{
    int a = 4;
    int b = -3;
    if(a>b)
    { /* always keep curly braces even though there is only one statement after if */
        foo(b);
    }
    else
    {
        printf("Arrg!n");
    }
    printf("thanksn");
    return 0;
}

#define foo(x) 
    do {                   
        if((x) > 0)         
        printf("Ouchn");  
    } while(0)

if (a > b)
        if (b > 0) printf("Ouchn");;
    else
        printf("Arrg!n");

Arduino.ru

Помогите с кодом, пожалуйста.

• Войдите или зарегистрируйтесь, чтобы получить возможность отправлять комментарии

Подскажите с ошибками в коде.

• Войдите или зарегистрируйтесь, чтобы получить возможность отправлять комментарии

Убрать из строки 41 левый символ

• Войдите или зарегистрируйтесь, чтобы получить возможность отправлять комментарии

А какой именно из них?

• Войдите или зарегистрируйтесь, чтобы получить возможность отправлять комментарии

убрал. просто очистил строку.

но вылезло следующее. ISO C++ forbids comparison between pointer and integer

• Войдите или зарегистрируйтесь, чтобы получить возможность отправлять комментарии

• Войдите или зарегистрируйтесь, чтобы получить возможность отправлять комментарии

Спасибо. Ошибок больше нет. Но и желаемого результата тоже=(

В данный момент, отрезок кода, имеет такой вид:

Но Переменная всё равно не выводится. Символы с клавиатуры идут. Затем я нажимаю «В». и ничего не происходит.

Я ожидал увидеть так.

я нажимаю символы. После чего Проверяется условие, есть ли в буфере В. Если оно есть, то Печатаются все символы, набранные до В и само В.

Я изначально не так наваял код?

• Войдите или зарегистрируйтесь, чтобы получить возможность отправлять комментарии

Если B латинская, то всё должно работать.

Одно непонятно, зачем вводить с матричной клавиатуры и из Serial.

• Войдите или зарегистрируйтесь, чтобы получить возможность отправлять комментарии

То, что символы введённые с клавиатуры, печатаюся, это временное явление — потом оно уберётся. что бы меньше шелухи поступало в процессинг.

По поводу В. Да, она была латинской. Заменил на другой символ. Ничего не вышло.

Даже более того, такое ощущение, что он не сравнивает.

После этого добавил в действия при совпавшем условии println. Но не тут-то было. не пишет.

• Войдите или зарегистрируйтесь, чтобы получить возможность отправлять комментарии

Проблема не решена. Всё зависло в воздухе. помогите, кто знает. светч выглядит сейчас так.

у меня просто прям чутьё, что именно в этом месте загводка. — while (Serial.available())

Но понять почему, я не могу.

• Войдите или зарегистрируйтесь, чтобы получить возможность отправлять комментарии

Исключите из лексикона слово процессинг, потому что ArduinoIDE — это не процессинг.

Вы нигде не написали, что и на какой клавиатуре нажимаете. Какой используете монитор для связи с ардуиной. Без этого сложно что либо сказать, потому что скетч вполне нормально выглядит.

Лично я закомментарил бы строки 35-40 (включительно и именно так). Потом добавил бы строку очистки входного буфера после строки 45. Типа:

Затем загрузил бы скетч и вводил в мониторе «123B», кнопка send. Если не получил в ответ ничего, то закомментарил бы строки 43, 48-51.

Программирование ладно, но с логикой надо дружить.

• Войдите или зарегистрируйтесь, чтобы получить возможность отправлять комментарии

Используется матричная клавиатура. Нажимаю на ней символы и затем В. — по условию, должны печататься все ранее нажатые символы(если я правильно понимаю).

По поводу закомментировать строчки — попробовал, не вышло. очистка переменной — да, забыл)

Какой монитор для связи с ардуино — это как трактовать? — запустил скетч — нажал монитор порта. проверяю. ничего военного) — или я не понял суть вопроса?

кнопка send — тоже не понял.

• Войдите или зарегистрируйтесь, чтобы получить возможность отправлять комментарии

Стоит отметить, что символы с клавиатуры, в мониотр порта выводятся исправно. Если написать просто условие, что-то на подобие:

То сравнение проходит. следовательно Морока, в строке — while ( Serial .available()), ну или на крайний случай if (inputchar == ‘B’ ). потому что идёт сравнение составляющей массива со строкой — (как-то запутанно). в любом случае, что бы наступил мир и покой для моего головного мозга, мне просто необходимо сравнить символы с клавиатуры со строкой (просто ввести код -больше 3х символов и если он равен, то запустить процедуру) . Мои товарищи подсказать не смогли — разводят руками — мол, должно работать.

Если вы видите простое решение проблемы, будте добры — напишите его пожалуйста (просто кусок скетча, что бы я ег овставить и оно заработало) — я отблагодарю(материально, если потребуется).

• Войдите или зарегистрируйтесь, чтобы получить возможность отправлять комментарии

Вы внимательно посмотрите что куда вы сохраняете, и что куда выводите.

Выводите вы переменную inp. Внее данные попадают из переменной inputchar, которая, в свою очередь получает что-то из Serial.

То есть имеем цепочку Serial->inputchar->inp

Вопрос: где в ней ваша клавиатура и почему вы ждете что введенные с нее символы будут выводится?

Вот этот кусок кода:

Таки может и будет выводить все что прочитало с внешней клавиатуры (коды клавиш). Но СРАЗУ. Без всяких «B» и проч запоминаний. Работа с Serial и работа с клавиатурой — у вас живут отдельной жизнью.

P.S. Кстати «B» и «b» — это разрные символы. Так что смотрите еще и за тем какую b вы шлете в мониторе.

• Войдите или зарегистрируйтесь, чтобы получить возможность отправлять комментарии

Клавиатура в этьой цепочке здесь — char inputchar = Serial.read();

считывание сигнала из монитора порта -поправьте, если я не прав.

В у меня одна и та же — для проверки взял её из массива символов для клавиатуры

• Войдите или зарегистрируйтесь, чтобы получить возможность отправлять комментарии

Клавиатура в этьой цепочке здесь — char inputchar = Serial.read();

Я тут не вижу никакой работы с клавиатурой. Я вижу считывание символа из Serial порта.

Или, если вы настаиваите что это чтение клавиатуры, то устаканьте терминологию. Давайте уберем неоднозначность. Предлагаю «клавиатурой» называй, ту клавиатуру которая подключен к компьютеру, которой вы пользуетесь когда пишите тут на форуме, а ту которая подключена напрямаю к ардуине называть «кейпадом».

Итак, используя эти два понятия «клавиатура» и «кейпад». Попробуйте еще раз описать где и что вы нажимаете, и что ожидаете.

• Войдите или зарегистрируйтесь, чтобы получить возможность отправлять комментарии

Все эти левые символы — ад копипастеров. Пишите код сами.

• Войдите или зарегистрируйтесь, чтобы получить возможность отправлять комментарии

Хорошо. будь по вашему)

Символы с кейпада поступают в сериал порт.

С помощью кейпада нажимаю цифры(любые — допустим 123) — и затем нажимаю (на кейпаде) В.

Цифры выводятся в монитор порта. Они считываются этой командой — char inputchar = Serial.read();

И после нажатия В (на кейпаде) — должна вывестись переменная inp. С ‘B’, или без енё это уже не важно.

Но этого не происходит.

• Войдите или зарегистрируйтесь, чтобы получить возможность отправлять комментарии

Символы с кейпада поступают в сериал порт.

То есть, вы утверждаете, что:

1. Кейпад имеет Serial интерфейс
2. К арудине он подключен двумя проводками на Rx (D0) /Tx (D1) пины (плюс земля, и, возможно питание).

Эти утверждения соотвествуют реальности?

• Войдите или зарегистрируйтесь, чтобы получить возможность отправлять комментарии

Хорошо. будь по вашему)

Да мне вообще-то это не очень-то и нужно Я просто пытаюсь помочь вам.

В первую очередь все эти вопросы нацелены на то, что-бы вы сами над ними задумались. Почему они возникают.

Потому что они могут:
1. Содержать в себе ответы на вашу проблему
2. Быть маркерами того, что на форуме вы тоже делаете «что-то нето». Обязанность за то что бы ОДНОЗНАЧНО и ПОЛНО передать картину — лежит целиком на вас.

А «неоднозначность в описании» может свидетельстововать о «неоднозначности в голове». А это может находить отражение и в коде. Так как на форуме может кто-то и «догадается», то сама ардуина — что ей сказали, то и будет делать

Пока же, вы сделали основные ошибок новичка:
1. Назвали тему черти-как. Как вы думете, в форуме «программирование» много тем, которые по сути не являются просьбой о помощи с кодом? так что, вообще все темы так называть? Как потом ориентироваться? Зачем тогда вообще придумали название тем? Я, обычно, вообще темы типа «помогите новичку», «помогите с кодом», «не работает скетч» — игнорирую.
2. Ни фига не описали какое у вас железо (какая плата, что за кейпад, что за либы используете, что и куда подключаете. ): в резульатате нужно либо догадываться/предполагать, либо вытягивать из вас как из партизана на допоросе.

Не сделали ошибок «новичка» (в плохом смысле):
1. Нормально вставили код
2. Явно пытаетесь сами что-то делать, а не просто «а. помогите».
3. Пытаетесь выполнять советы, а не спорить и указывать другим что нужно делать (бывают и такие персонажи

Источник

Else without a previous if error

My compiler is giving else without previous if error in the else if part.

The if part can be clearly seen. Is it due to the declaration in between. But why should a declaration cause problem? How to get around it?

8 Answers 8

int temp=number[i]; is set at an invalid postition. Move in front of the if or inside else .

The definition int temp = number[i]; follows after the previous if and prevents the else from belonging to it, or anything else. Hence the error. You’d have to move the definition before the if statement, or otherwise rethink the logic.

Let’s look at the code where the error occurs:

You can only use else if it is the statement that immediately follows the if . And it is not. You have declared a new variable, temp .

is at the wrong position.

Now there must immediately an:

If there is any other statement, you are outside your if -block and else is not allowed.

The reason this is causing a problem is simply because it breaks the syntax rules for the language. The compiler assumes there is no else block if it doesn’t follow immediately after the if block. You can put whitespace in between, but absolutely nothing else.

You’ve got 4 possibilities for where you could put that declaration line:

1. Before the whole if/else structure.
2. After the whole if/else structure.
3. Within the if block.
4. Within the else block.

I’d recommend the 1st one, or possibly the 4th (depending on whether you’re using the temp variable later).

If you think about it logically, trying to put the declaration between the if and else blocks doesn’t make sense. It makes it unclear if and when it actually gets executed in relation to the rest of the control structure.

Источник

C++ error ‘else’ without a previous ‘if’ [duplicate]

Im new to C++ and have been working on an assignment

Ok so this runs, but it doesn’t do the calculations when the window comes up I type in A B or C and then the length of the words and it just says pay=0

here is the new code:

4 Answers 4

You have a semicolon after the first if (at the end of the line). Remove that and you’re done.

As others have noted, there are other extra semicolons and errors. Be careful!

You have to remove the ‘;’ at the end of (and also replace the ‘=’ with ‘==’)

as at the end of

Explanation

The ; marks the end of a statement. So here it ends the if immediately. The <. >that follows will be executed not only when the condition is true, but always. Because if you put a <. >block anywhere in your code without putting a condition in front of it, it will be executed always.

So when you write if (condition); < statements; >, the program will evaluate the condition expression, and after that it will always execute < statements >, unconditionally. If you remove that ; before the <. >, that will work as intended, and execute the <. >block only if the condition was true.

The = is the assignment operator, not for comparison. So authorLevel = ‘C’ will set the value of authorLevel to C . Inside the if , the outcome of the assignment is used as the condition. The outcome of an assignment is the value that was assigned, so in this case ‘C’ . And since ‘C’ is neither null nor 0, it will be considered as true. So mistaking the operator causes several problems at the same time, be careful with it.

Источник

Else without previous if arduino

I’m just starting out at C++ and I keep getting the error: ‘else’ without a previous ‘if’ even though I’m sure it should work.
Here’s my code:

Please use the code tags when posting code.

In either case: the ‘if’ statement doesn’t need a semicolon (‘;’).

Also: the ‘else’ statement doesn’t need a condition. «age >=100» isn’t being evaluated. Simply putting «else» will include all «>=80» cases right now.

Your ifs and else-ifs should not have semicolons after them. Also, your first if is missing a closing > and you have one closing > too many at the end. Finally, an else clause shouldn’t have a condition, so get rid of the ( ) and everything in between.

EDIT: Infernal ninjas.

What does your updated code look like?

EDIT: Also tried without the ( ), but it’s the same.

Leave the part in bold out, then it works.

You forgot to remove the condition and ; on line 22.

EDIT: My ninja skills are dropping.

It works fine for me if you make the correction hanst99 indicated.

I notice you still have a semi-colon after else (age>=100) Remove it and things should work better. Also, you’re not checking if age is between 80 and 99.

EDIT: I tried it with else if (age >=100) and it worked great.

wouldnt a switch (case) be easier than multiple if else? just a suggestion.

Switch doesn’t work with conditionals. You could, however, transform the age value:

sorry my bad. set up the switch wrong, this would work, correct me if im wrong

No, again: switch doesn’t take conditionals. In fact, nothing takes conditionals in the form you just posted. In your code, lines 4, 8, and 12 are the same as typing «0;» or «1;» in your code. Even if it does compile, it will simply ignore those lines.

If you really want to use a switch, do this:

Источник