Принятие пакета с Com порта

[Ostapich]

Забыл обнулить i в  последнем условии:

Код:

if(in_data.size()>=sizeof(packet_big)){
                           tmp=in_data.mid(i,sizeof(packet_big)-1);
                           in_data.remove(0,i+(sizeof(packet_big)-1));
                                                       qDebug()<<"5 "<<in_data.size()<<" i "<<i << "    " << tmp;
                           i=0;
                           emit Send_Data(tmp);

Но все равно не помогает)

[titan83]

У меня пошла кровь из глаз... Как я могу это развидеть???
А если серьезно, то телепатов тут нет.
Не делай такую каку, не надо

Код:

for(size_t i=0;i<in_data.size();i++){
                qDebug()<<"2";
                if(in_data[i]==254){
                    qDebug()<<"3";
                     qDebug()<<"  "<<in_data[i];
                    if(in_data[i+1]==232){

Есть вариант:

Код:

if (in_data.indexOf(QByteArray::fromHex("FEE8")) > -1) {
  bla-bla
}

Ну и выложи сюда дебаг вывод подробный, что там в in_data оказывается хотя бы на шаге "1", домысливать за тебя не хочу, честно.

ЗЫ Хорош копипастить с ардуино: не надо size_t, лучше (unsigned) int.

[Ostapich]

Переведите пожалуйста: in_data.indexOf(QByteArray::fromHex("FEE8") ... Прошу прощения, как могу, стараюсь исправиться ))

[titan83]

Переведите пожалуйста: in_data.indexOf(QByteArray::fromHex("FEE8") ... Прошу прощения, как могу, стараюсь исправиться ))

Чувак, твой код - это сплошная боль. Но я так же говнокодил лет 7 назад.
Ты сделай все заново, но:
1. Без копипасты с ардуины.
2. Без потоков, тебе это тут не надо. инфа 146%.
3. Почитай справку, и ты поймешь, что in_data.indexOf(QByteArray::fromHex("FEE8") спасет тебя от трех уровней вложенности как минимум.
4. Делай все по шагам: сначала просто читай все пакет. Убедись, что приходит то, что отправлено. Потом уже начинай менять пакет.
5. Срочно убирай всю эту арифметику указателей. Кодируй, используя ссылки, а не указатели, будет намного надежнее. Сейчас у тебя сделано очень плохо: месиво из структур, классов, у которых ты что-то меняешь, используя указатели.

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

[titan83]

но когда пытаюсь вывести через дебаг массив, белеберду выводит :  "\xFE\xE8\x00\x00\x00\x00\x00\x00"

packet_big packet_one={254, 232, 0, 0, 0, 0, 0, 0, 0};

[Ostapich]

Да  я уже понял, что все ок. Прошу прощения за причиненную боль, но пока как могу, буду исправляться, спасибо за совет)

[titan83]

То, что критику воспринимаешь адекватно - это тебе большой плюс. Думаю, дальше сам вывезешь.

[Ostapich]

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

[kuzulis]

> вид приведу, выложу тут

не, не надо, оставь у себя