PCCar.ru - Ваш автомобильный компьютер - Вопрос по Ардуино. Хелп плиз-((

- Arduino проекты (http://pccar.ru/forumdisplay.php?f=195)

- - Вопрос по Ардуино. Хелп плиз-(( (http://pccar.ru/showthread.php?t=12588)

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

Плата: фридуино на 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;

       }

    }

 



}

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

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

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

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

Цитата:

Сообщение от ancc (Сообщение 148830)

да пока только что и вбиваю одно и тоже - 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 (Сообщение 148832)

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

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

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

Цитата:

Сообщение от ancc (Сообщение 148839)

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

Цитата:

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

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

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

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

Цитата:

Сообщение от ancc (Сообщение 148843)

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

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

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

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

Цитата:

Сообщение от ancc (Сообщение 148845)

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