Arduino UIPEthernet: client.read() geht nicht in String

[Frank E. (Firma: Q3) (qualidat)]

In dem folgenden Code-Schnipsel steht eine Lese-Routine (im Loop), um 
einen HTTP-Request in einen String einzulesen. Auf dem Uno mit der Lib 
"Ethernet" funktioniert das, auf dem Nano mit UIPEthernet (wegen des 
anderen Ethernet-Chips), bekomme ich das einfach nicht hin. Bitte mal 
draufsehen:

...
if(client){
  char cbuf[64];
  int cnt = 0;

  while(client.available()>0){
    char c = client.read();
    if(cnt<64){
      cbuf[cnt]=c;
      Serial.println(c); // <-- das wird lesbar ausgegeben
    }
    cnt++;
  }
  String buf(cbuf);
  Serial.println(buf); // <-- das ist immer leer
}

Bei der Ausgabe oben in Zeile 10 ist schön der Anfang des HTTP-Request 
vom Browser zu sehen, aber buf ist immer leer, nicht etwa verkrüppelter 
Text, nein, gähnende Leere. ... warum?

Danke für Tips.

[Sebastian W. (wangnick)]

Frank E. schrieb:
> String buf(cbuf);

cbuf ist nicht nullterminiert. Versuch mal

String buf(cbuf,min(cnt,sizeof(cbuf)));

LG, Sebastian

[Frank E. (Firma: Q3) (qualidat)]

Sebastian W. schrieb:
> cbuf ist nicht nullterminiert. Versuch malString
> buf(cbuf,min(cnt,sizeof(cbuf)));
> LG, Sebastian

Moin. Danke für den Tip, hat aber leider auch nichts gebracht. Nach 
endlosen Versuchen funzt nun dieser Code stabil. Es geht darum, aus dem 
HTTP-Request des Browsers einen GET-Parameter heraus zu fummeln, z.B. 
http://192.168.10.70/?cmd=on

Da klappt jetzt damit zuverlässig:

...
String buf="";
int cnt=0;
while(client.available()>0 && cnt<64)
{
  int b = client.read();
  if(b>31 && b<128)
  {
    buf.concat(char(b));
    cnt++;
  }
  int anf = buf.indexOf(F("/?cmd="));
  int end = buf.indexOf(F(" HTTP"),anf);
  buf = buf.substring(anf+6,end);

  Serial.println(buf); // <-- temporäre Kontollausgabe
}

Möglicherweise hat es auch geholfen, konsequent alle String-Konstanten 
mit F("bla bla") in den Flash zu packen. Obwohl vorher 93% belegt, bzw. 
350 Byte im RAM frei waren, ging es nicht. Jetzt mit fast 600 Byte kein 
Problem mehr ...

[Michael U. (amiga)]

Hallo,

ich nutze die String-Klasse sehr gern, allerdings auf ESP8266/ESP32.
Wenn nicht genug freier Ram verfügbar ist, habe ich diesen Zustand 
selbst da schon erreicht. Es gibt keine Fehlermeldungen, es kommt nur 
ein leerer String zurück, wenn irgendein interner Buffer innerhalb 
einerStringfunktion nicht angelegt werden konnte.
Auf den kleinen AVR vermeide ich String eigentlich generell und nutze 
die C-Stringfunktionen.

Gruß aus Berlin
Michael

[Εrnst B. (ernst)]

Frank E. schrieb:
> String buf="";
...
>     buf.concat(char(b));

Gerade sowas ist bei wenig Speicher ungünstig.
Jedesmal wenn du den String um ein Zeichen verlängerst, muss potentiell 
neuer Ram ge-malloc-ed werden, der bisherige String umkopiert, und der 
alte Speicherplatz freigegeben werden.

Braucht also doppelte RAM-Menge, und fragmentiert den freien Speicher 
evtl. ungünstig.

der Arduino-String hat eine "reserve(bytes)"-Methode, damit kannst du 
vorher deine vermutliche String-Länge kundtun, und dieses wiederholte 
re-alloc verhindern.

>> String buf;
>> buf.reserve(128-31);