Svenska ElektronikForumet

sodjan skrev:"47", är det decimal eller som hex?
Hur ofta är "ibland"?
1 gång av 2?
1 gång av 100?
Mer sällan?

Icecap skrev:"Vad brukar detta bero på?"
Först och främst: Dåligt kommunikationsprotokoll!
Därnäst: Dålig programmering.

Ett bra protokoll bör vara uppbyggd så att det finns en entydig startsekvens och en entydig stoppsekvens.
Däremellan kommer data samt en checksumma.

Checksumman kan t.ex. vara en addition av alla bytes i data-delen, det kan vara en XOR av dom eller annat man kan hitta på. Självklart kan man köra med CRC18 eller CRC32 men det kan kännas lite överkurs.

Jag brukar skiva värden som siffror, alltså ville värdet 47 bli skickat som '4' (0x34) och därefter '7' (0x37). Checksumman ville då bli 0x34 + 0x37 = 0x6B och den hade jag skickat som 2 siffror i HEX.

Oftast inleder jag ett block med STX (0x02) och avslutar med ETX (0x03).

I protokollet bör det vara definierat om ett block som godtas ska ha en acknowledge tillbaka eller om det bara ska kastas av mottagaren. Har man ett kommunikationsförfarande som begär värdet och får svar kan den då bara begära värdet igen om det blev fel på vägen.

Al_Bundy skrev:Du gillar verkligen att kasta skit på min kod utan ens har sett den

sodjan skrev:> "47\r\n" står det.

Står vardå?

Är det alltså texten "47" eller värdet 47 ?

Men tja, någonstans ifrån kommer det ju...

Miramithe skrev:Fullt förståeligt när icecap flera ggr försökt hjälpa och påpekat fel som du avfärdar eller skiter helt i. Skit ska skit ha heter det.

Klas-Kenny skrev:

Al_Bundy skrev:Du gillar verkligen att kasta skit på min kod utan ens har sett den

Då får du väl ta och visa upp lite kod då, så folk slipper sitta och gissa..

Miramithe skrev:Fullt förståeligt när icecap flera ggr försökt hjälpa och påpekat fel som du avfärdar eller skiter helt i. Skit ska skit ha heter det.

Icecap skrev: Al: nej, det gör jag inte!
Och det behövs ju inte, du klarar det helt själv, gång efter gång!

Du har fått råd, du avfärder dom och du hade två möjligheter till varför fenomenet uppstår. Du förkastar den ena orsak, alltså kvarstår det bara den andra, så enkelt är det!

Och mina råd är inte för skoj skull, de är baserat på många års erfarenhet i alla möjliga tänkbara sammanhang.
Men du vet ju bättre än alla andra - i vanlig ordning - och då är det ju konstigt att du väldigt ofta inte förstår vad som går fel, inte kan analysera dig fram till det och samtidig vägrar ta råd och lära dig.

I de senaste par dagar haf jag suttit med kommunikation över serielänk - och det har varit en del knas på vägen. Men jag har löst dom steg för steg, analyserat vad som gick och inte gick och nu anser jag att den krypterade kommunikation jag har mellan 5 trådlösa enheter är på god gång. Och ja, det tar tid, det har tagit tid att planera protokollet, krypteringen, hur värden ska överföras osv. men då det jobb var klart var resten ganska enkelt.

Men om du får värdet 47 utan någon specifik orsak är det antingen fel på hårdvaran eller på programmet. Motbevisa det gärna, jag skulle uppskatta att lära något nytt.

Men om ditt "försvar" är att klaga på att jag påpekar det självklara ställer du dig själv i ett riktigt dåligt ljus.

 static void get_values_from_flik_5()
      {
        int a = 0;
        String text;
        while (a == 0)
        {
          if(Serial.available() > 0)
          {
            text = Serial.readString();
            int firstcolon = text.indexOf(';');
            int secondcolon = text.indexOf(';', firstcolon+1);
            int thirdcolon = text.indexOf(';', secondcolon+1);
            int fourthcolon = text.indexOf(';', thirdcolon+1);
            int firthcolon = text.indexOf(';', fourthcolon+1);
            PHbor = string_to_float(text.substring(0, firstcolon));
            K = string_to_float(text.substring(firstcolon+1, secondcolon));
            Td = string_to_float(text.substring(secondcolon+1, thirdcolon));
            Ti = string_to_float(text.substring(thirdcolon+1, fourthcolon));
            MAX = string_to_float(text.substring(fourthcolon+1, firthcolon));
            Serial.flush();
            Serial.print(PHbor);
            Serial.print(";");
            Serial.print(K);
            Serial.print(";");
            Serial.print(Td);
            Serial.print(";");
            Serial.print(Ti);
            Serial.print(";");
            Serial.print(MAX);
            Serial.print("#");
            Serial.flush();            
            a = 1;                                                                 // Breake loop
          }
        }
      }

Agwan skrev:

Miramithe skrev:Fullt förståeligt när icecap flera ggr försökt hjälpa och påpekat fel som du avfärdar eller skiter helt i. Skit ska skit ha heter det.

Så var det ju inte riktigt. Icecap började ju med att slänga ur sig några förolämpningar vilket ju är perfekt om man vill att den som ställt frågan skall hamna i en sinnesstämning som inte passar för att ta emot efterkommande information, sedan svarade han på en fråga som inte ställts. Han såg ut att svara på frågan hur ett bra protokoll bör vara uppbyggt, inte på om det fanns någon anledning till att just siffran 47 eller möjligen någon annan konstant siffra skulle returneras. Icecaps svar "Då kvarstår dålig programmering." kan ju inte direkt ses som konstruktivt.