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

Frank E. schrieb:

String buf(cbuf);

cbuf ist nicht nullterminiert. Versuch mal

1

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

LG, Sebastian

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:

1

...

2

String buf="";

3

int cnt=0;

4

while(client.available()>0 && cnt<64)

5

{

6

  int b = client.read();

7

  if(b>31 && b<128)

8

  {

9

    buf.concat(char(b));

10

    cnt++;

11

  }

12

  int anf = buf.indexOf(F("/?cmd="));

13

  int end = buf.indexOf(F(" HTTP"),anf);

14

  buf = buf.substring(anf+6,end);

15

16

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

17

}

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 ...

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

Frank E. schrieb:

String buf="";

...

1	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);