вот, например,


Задача стояла такая:
модем на запрос АТ команды выдает ответ. Сначала пишет саму команду, потом ответ. В ASCII это выглядит так:


41 54 2B 43 50 42 53 3D 3F 0D 0D 0A 45 52 52 4F 52 0D 0A 00
A T + C P B S = ? CR CR LF E R R O R CR LF

Моя задача была сложить в массив data[] все после символа 0x0A, т.е. ERROR

итог, код рабочий. Но хороший ли тон так писать- через флаги.

Хороший до тех пор пока не появляется 1024 флага в программе из 100 строк.
Кстати, про goto - это был пиар блочного кодирования, совсем не обязательно его игнорировать и кривиться, что мол читабельность программы снижается итд итп - это ведь все ложь.

Скажем так - можно, но осторожно.
Чтобы лапши не накрутить.

Альтернатива флагам и поллингу - запуск callback-функций непосредственно в обработчике прерываний.
Что лучше, что хуже - это уже каждый решает индивидуально и в зависимости от.

Названия только таким переменным надо давать осмысленные. Мне, например пришлось прочитать весь фрагмент кода, чтоб понять, что flag должна называться newLineReceived.

Можно использовать и флаги и goto. Насчет названия, пожалуй, соглашусь.

хорошо, был ли вариант переписать данную задачу без флагов и тому подобное. Только с if или тому подобным?
Сами как решили бы подобную задачу? Тоже с флагами?

Методом конечных автоматов.

Я бы первым делом отключил эхо. И точно не "сливал" бы приходящие от модема ответы - их парсить надо внимательно, а не полагаться на авось.

А что касается подобного использования флага, так это просто вариант стейт-машины на два состояния. Никакого внутреннего отторжения не вызывает.

Я бы заполнил сначала весь буфер, а потом бы воспользовался функцией strchr.

т.е. конечные автоматы с
case swith?

strchr изучу, спасибо.

эхо не хочу отключать- где- то оно надо- контролировать запросы хоть

Это тут ни причем.

Bill,
как же ни при чем?

Классический конечный автомат

Я же не конечный автомат имел в виду.

Напрасно. Тем более, что сейчас вы их просто игнорируете (полагая при этом что они есть), а не контролируете.

aaarrr
как в даташите эхо это называется правильно (мы про q2686 говорим, или в общем)? Пробежался сейчас- там только echo cancelation есть- но это для голоса. В каком разделе оно отключается?

Хотел написать , но резидент опередил , очень много задачь рашается автоматами. Причем каждый сам для себя определяет, как он пишет. И вырабатывает приемы. Ваш флажек - это просто автомат с 2 состояниями. На мой взгляд использование case, не самый лучший стиль. Есть подводные камни. Лучше все же if - вы больше владеете ситуацией, причем, на самом деле, все, в конце концов, транслируется в одинаковый код.

Q2686 был сейчас упомянут впервые. А если вообще, то ATE0.

firstvald спасибо. я правильно понимаю применение автомата- я его правильно привел? Что в нем Билл не то нашел?

aaarrr да, спасибо, найду поиском эту команду, посмотрю подробнее.

ну и, кстати, распарсить ф-ей strchr тоже идея хорошая.

Не забудьте, что все функции str* ожидают увидеть в конце строки 0. Иначе будут долго ходить по памяти. Можно в линейном буфере закончить нулем последовательность, если, конечно, буфер не заполнен.

Я против автомата ничего не имею. Он действительно реализуется с помощью оператора switch.