Вопрос по Ардуино. Хелп плиз-((

[CyberSubaru]

Плата: фридуино на 168й меге.
Пишу программку общения с ком-портом.
Суть - в порт передаются команды типа ATS12345
Программка их парсит побайтно и в зависимости от команды выполняет те или иные действия.

Уперся я о непонятный камень:
код работает нестабильно.

в монитор порта вбиваю ATS
в 50% случаев (примерно)
код доходит до выполнения Serial.println("SET COMMAND /PIN ");
НО(!!!!)
Стабильно в цикле забивания массива выводит именно те байты, что передавались. Но if и switch работают по-китайски.

И еще нюанс такой: Serial.print как-то больше способствует корректной работе кода с массивом.

В чем может быть трабл? Уже идей никаких нет.

PHP код:

void setup() 
{ 
    Serial.begin(115200); 
//  Serial.println("ECU READY"); 
} 


int cmd[64];
int i;
int pin;
int maxdata;

void loop() 
{ 
  
  maxdata=0;
  
   for(i=0;i<64;i++) {cmd[i]=0;}
   
   i=0;

   while (Serial.available()>0) 
      {  
       cmd[i]=Serial.read();
       Serial.print(cmd[i],BYTE);   
       i++;
       maxdata++;
       } 
   
 
   if ((cmd[0]==65)and(cmd[1]==84))
   {
     // Serial.println("AT OK"); 

       switch (cmd[2])
       {
       case 73: {      //'I'
               
                };
               break;
       
       case 71: {        //'G'
                  pin=cmd[3];
                };
                break;     
       
       case 83:   {      //'S'
                        Serial.println("SET COMMAND /PIN "); 
                         pin=cmd[3];
                        // Serial.println(pin,BYTE); 
                        // Serial.println(cmd[4],BYTE); 

                         
                         switch (cmd[4])
                         {
                           case 1: pinMode(pin, INPUT); break;
                           case 2: pinMode(pin, OUTPUT); break;
                           case 3: digitalWrite(pin,cmd[5]);break;
                           case 4: analogWrite(pin,cmd[5]);break;
                         }  

                   };
               break;
       }
    }
 

} 


[ancc]

ваще seral.println Никогда не использовал. только serial.print.

и еще. массив cmd[i] очищается если ни одно из условий не выполняется или нет?

может жестко проверять при приеме данных из порта по маске команды и как только расхождение обнаружилось сразу массив очищать.

[ancc]

и еще. про serial.flush после каждого приема верной комманды не помешает вроде. Хотя тут я хз. вроде не надо.

[CyberSubaru]

Сообщение от ancc

ваще seral.println Никогда не использовал. только serial.print.

и еще. массив cmd[i] очищается если ни одно из условий не выполняется или нет?

может жестко проверять при приеме данных из порта по маске команды и как только расхождение обнаружилось сразу массив очищать.

да пока только что и вбиваю одно и тоже - ATS
так что обработка ИНАЧЕ смысла пока не имеет.
вот результат нескольких подряд вбитых "ATS1234":

AT

AT OK
S1234AT

AT OK
S1234AT

AT OK
S1234ATS1234ATS

AT OK
SET COMMAND /PIN


1234ATS

AT OK
SET COMMAND /PIN


1234ATS1234ATS1234ATS

AT OK
SET COMMAND /PIN


1234

добавил перевод каретки

Сообщение от ancc

и еще. про serial.flush после каждого приема верной комманды не помешает вроде. Хотя тут я хз. вроде не надо.

с ним вообще работает криво....

[ancc]

выберете жестко формат комманды AT xxxxx <cr>
и по этой маске команды парсите. как только у вас расхождение команды с маской буфер очищаете и ничего в него не пишете пока первый символ "A" не появится. С этого момента начинаете писать в буфер и снова сличать с маской.

[CyberSubaru]

Сообщение от ancc

выберете жестко формат комманды AT xxxxx <cr>
и по этой маске команды парсите. как только у вас расхождение команды с маской буфер очищаете и ничего в него не пишете пока первый символ "A" не появится. С этого момента начинаете писать в буфер и снова сличать с маской.

Читайте, пожалуйста, внимательнее!

вот результат нескольких подряд вбитых "ATS1234"

Команды вбиваю в монитор порта (в ардуино который) - одну и туже.
Результат работы кода - разный.
Код элементарный - читаем байты с порта, пока есть данные, забиваем их в массив и далее смотрим, если первые два AT, то смотрим дальше. Вот это условие при одинаковых входных данных срабатывает не всегда. И Switch-case тоже срабатывает не всегда, при одинаковых входных данных.
Вопрос - почему такое может быть?

[ancc]

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

и я бы посоветовал комманду заканчивать <cr> (ентер грубо говоря).

[CyberSubaru]

Сообщение от ancc

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

и я бы посоветовал комманду заканчивать <cr> (ентер грубо говоря).

Вы код программы читали? Складывается такое впечатление, что вместо помощи, Вы издеваетесь.

[ancc]

да и не думал вообще-то. Подскажите вот у вас в программе:

((cmd[0]==65)and(cmd[1]==84))

я правильно понимаю что начало команды вы хотите увидеть обязательно в первом элементе массива?

[CyberSubaru]

Сообщение от ancc

да и не думал вообще-то. Подскажите вот у вас в программе:

((cmd[0]==65)and(cmd[1]==84))

я правильно понимаю что начало команды вы хотите увидеть обязательно в первом элементе массива?

верно.
потому что это и предполагается.
вот когда это условие срабатывает - необязательно срабатывает следующее условие проверки элементов массива, будто он где-то повреждается.