Arduino.ru

Может поможет?

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

Попробуйте в начало функции вставить строчку

while(Serial.available() < 4); - это не сделает код правильным, но может быть заработает, и Вы разбиретесь что к чему.

Спасибо всем за советы, прошерстил кучу информации, но "воз и ныне там". В конце концов вытянул библиотечный скетч ReadASCIIString, собрал схемку управления светодиодом, загрузил в уно и нифига не получилось (афигеть!!!) нет регулировки яркости/цвета и нет ответа на мониторе. Пробовал несколько разных плат и все повторилось(платы рабочие). Как решить эту проблему с пакетами, подскажите, может я че пропускаю, но библотека должна ведь работать(я проверял не все скетчи и ввроде все работало). И последнее, программа ардуинка версия 1,8,0.

Хм. Эк Вы мечетесь-то. А можно глянуть на полный  код?

serialEvent не работает как вы ожидаете. Можете легко это проверить

void serialEvent()
{
  Serial.println(Serial.available());
}

Потом отправьте z123 и посмотрите что вам вернется ;)

Хотите использовать serialEvent вот почти стандартный пример из ардуино иде

String inBuff = "";
bool buffComplete = false;

void setup()
{
  Serial.begin(9600);
}

void loop()
{
  if(buffComplete)
  {
    char k = inBuff[0];
    byte n1 = inBuff[1]-48;
    byte n2 = inBuff[2]-48;
    byte n3 = inBuff[3]-48;
    buffComplete = false;
    inBuff = "";
    Serial.println(k);
    Serial.println(n1);
    Serial.println(n2);
    Serial.println(n3);
  }
}

void serialEvent()
{
  while(Serial.available())
  {
    char inChar = (char)Serial.read();
    inBuff += inChar;
    if (inChar == '\n')
      buffComplete = true;
  }
}

Кому не нравится String можно заменить на массив. Принцип не изменится, только за указателем на элемент массива надо следить.

Добрый день. Ну посмотреть полный код... да в принципе можно, но ладно мой код не фурыкает, так ведь и библиотечный сериал(см. в предыдущем посте) также не регулирует светодиоды. А что касамо моего кода - это управление станком и связь с иполнительной платой (ардуинка уно) по серинику с ведущей платой. Одиночные команды идут без проблем, появилась необходимость немного переделать управление не одиночными коммандами а пакетом, типа код комманды + код операции + к.с. + подтвеждение. Извиняюсь за длинную писанину, станок пока работает и я пока долбаюсь с прогой(перевод на пакетный режим). Че я выяснил, массив не заполняется значениями в цикле, выводится только первая комманда а операция и т.д. все в нулях :(. Ну и последнее что хочу попробовать, это на другом компе установить идэ ардуинку и посмотрю как будет работать прошивка платы с библиотечной прогой серийника. Скачал еще новую версию 1,8,6. ардуины, может поможет. НА форуме почитал про серийник(его работу) и не видел подобной проблемы, это получается только у меня такая засада, вот досада :(

Именно про это я Вам и говорил. И даже рассказал почему.  Вы сделали то, что я советовал?

Да я проверил ваше предложение но это мало что мне дало. Собственно говоря разницы между цыклами я не увидел, не говоря уже об особенностях их применени. А Вот почему так происходит что-то я не видел вашего расказа, извините наверно слепой стал, если не трудно поясните еще раз

Так вроде пост мой никуда не удалился.     Кстати, а как у Вас вызывается функция void serialEvent()  ?

А мои посты видно? :)

Ребята, да в том-то и дело что и функции и текстовка прописывается (под мою задачу) в корне ничем не отличается от аналогичных и работающих. Я осмелюсь уточнить(ранее писал), залил на уно скетч из библиотеки ReadASCIIString и результат был нулевой!!! Проверил по длинне приема, пакет соответствует а вот ни регулировки светодиода ни ответа на монитор не получил. ПОучается что по иф условие не обрабатывается, т.е. код конца пакета ('\n') почемуто не нашелся(его естественно вставлял в строку). Вчера вечером повозился с ранее написанными скетчам(пакетной оправкой не занимался) и получается что проблема только с пакетами, неважно какой длинны(больше 1 байта). А байт ловит приемник без проблем. Но там нет цыклов реад, на важно каким цыклом читаем (for или while). Надо попробовать читать без них, щас проверю!

А ларчик просто открывался, переустановил ардуинку иде, все заработало. Новую 1,8,6, не ставил, просто переустановил(полностью) 1,8,0, и все! Незнаю почему мне так "повезло" что прога глюканула и довольно интересно, чуть было не полысел :(. Вообщем ребята всем спасибо за помощь.

5N62V кстати про функцию serialEvent()? Что значит как вызывается, я просто ввел это прерывание по наличию в приемном буфере чегото и все!  Функция работает пока буфер не опустеет, ну а дальше уж че надо то и буду делать с полученными данными. 

ну вот куски чего у меня получилось:

И что, этот код работает? - тогда ждем второй серии...

"почему было в цыкле было if (inChar == '\n')" - потому что '\n' - правильно, а 'n' - это чушь.

заклятие невидимости сними с постов - будут видны.

String inBuff = "";
bool buffComplete = false;
byte number = 0; //байт в буфере на момент вызова ф-ии
byte number1 = 0;// кол-во проходов цикла while

void setup()
{
  Serial.begin(9600);
}

void loop()
{
  if(buffComplete)
  {
    char k = inBuff[0];
    byte n1 = inBuff[1]-48;
    byte n2 = inBuff[2]-48;
    byte n3 = inBuff[3]-48;
    buffComplete = false;
    inBuff = "";
    Serial.println(k);
    Serial.println(n1);
    Serial.println(n2);
    Serial.println(n3);
  }
  if(number){
    Serial.print(" number = ");Serial.print(number);Serial.print("  number1 = ");Serial.println(number1);
    number = number1 = 0;
  }
}

void serialEvent()
{number = Serial.available();
  while(Serial.available())
  {
    char inChar = (char)Serial.read();
    //delay(1);
    inBuff += inChar;
    if (inChar == '\n')
      buffComplete = true; 
      number1++;
  }

}

Запускаем, вводим те же z123 (не забывая в настройках монитора выбрать "Новая строка") получаем в монитор порта:

Уважаемый ТС, как смог, объяснил.  У Вас та же проблема.

Это штатное поведение функции SerialEvent() - она вызывается на каждый приходящий символ, так и должно быть.

Я так и понял, просто эту особенность надо юзать осознанно.  Лично я пятьсот раз подумаю о возможных последствиях дерганья программы по прерыванию, особенно когда loop большой и медленный. В конце-концов данные и в сериале могут полежать, никуда не денутся, пока основной цикл не проверит их наличие. 

похоже, вы все-таки не вполне понимаете функцию SerialEvent(). Прерывания в ней не используются, иначе введением delay() вы не смогли бы изменить ее поведение.

Функция SerialEvent() тупо вызывается один раз за loop(), ее использование абсолютно равнозначно добавлению в loop() кода

if (Serial.Availiable() ) { .... }

это следует из описания https://www.arduino.cc/en/Tutorial/SerialEvent - см вторую фразу. Проверка происходит _между_ вызовами loop().

И вот еще цитата оттуда же:

/*
  SerialEvent occurs whenever a new data comes in the hardware serial RX. This
  routine is run between each time loop() runs, so using delay inside loop can
  delay response. Multiple bytes of data may be available.
*/

Если быть точнее, то вот так она вызывается:

int main(void)
{
	init();
	initVariant();
...
	setup();
	for (;;) {
		loop();
		if (serialEventRun) serialEventRun();
	}
        
	return 0;
}
...
void serialEventRun(void)
{
  if (Serial0_available && serialEvent && Serial0_available()) serialEvent();
...
}

Т.е. если у вас в loop-e будет что-то типа while(true) {if (Serial.available() ...) {...} }, то сериаливент вам не нужен в принципе. Да и вообще не особо нужен, по моему мнению. Разве что любителям хардкорного ООП-а она пригодится.

sadman41, b707   - Спасибо! Конструктивно!  И с какого будуна я взял, что эта ф-ция работает по прерыванию?! :)) Где-то читал, так в памяти отложилось. 

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

Так я вам и не запрещаю, для этого у нас Клапауций имеется.

Мне вот проще в лупе держать while, экономить микросекунды и на лету анализировать входящие данные без дополнительного складывания в буфер и повторного прохождения по нему.  Каждому своё.

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

if (Serial.available()>x)function();

Пока входящий в Serial пакет не более 16 байт - и это допустимо.