PSTR() und String zusammensetzten?

[hal (Gast)]

Hallo,
im C-Code für einen Atmel Mega328P muss ich einen String auf einen 
PSTR() und einen String zu einen String zusammensetzten.

Gibt es dafür bereits eine elegante Prozedur?

Bisher mache ich das so ...

// ...
  // const char *data
  // const char text[6]
  // ...
  
  // Textlängen
  uint8_t count_data = string_len_P(data);
  uint8_t count_text = string_len(text);
  uint8_t count_str  = count_data + count_text;
  
  // String zusammensetzen
  char str[count_str+1];
  
  uint8_t cnt = 0;
  
  for(uint8_t i=0;i<count_data;i++)
  {
    str[cnt++] = pgm_read_byte(data++);
  }
  
  for(uint8_t i=0;i<count_text;i++)
  {
    str[cnt++] = text[i];
  }
  
  str[cnt] = '\0';

// ...

uint8_t string_len(const char *str)
{
  uint8_t i = 0, 
      j = 0;
  
  while(str[i++]!='\0')
  {
    j++;
  }
  
  return j;
}

uint8_t string_len_P(const char *data)
{
  uint8_t i = 0;
  
  while(pgm_read_byte(data++))
  {
    i++;
  }
  
  return i;
}

danke

Gruß
hal

[Stefan F. (Gast)]

Vielleicht gefällt dir strcat() und strcat_P(), sowie strlen() und 
strlen_P(). Damit kannst du dir deine Hand-gedengelten for-Schleifen 
sparen.

https://www.nongnu.org/avr-libc/user-manual/group__avr__string.html#ga1fd2a6e188f02599e5eeb17519f67f3e

https://www.nongnu.org/avr-libc/user-manual/group__avr__pgmspace.html#ga5c1bae2b20f7d5ec00411c6378b8ed70

In den beiden Dateien sind noch viele andere nützliche Funktionen für 
C-Strings.

[foobar (Gast)]

Beim avr-glibc gibt es die _P-Funktionen für progmem-Strings 
(strlen/strlen_P, strcpy/strcpy_P, etc).  Bei den printf-Funktionen 
zusätzlich zum %s das %S für progmem-Strings.  Die einfachste Lösung 
dürfte:

sprintf(str, "%S%s", data, text)

sein.

Btw, verkneif dir sowas wie "char str[count_str+1];" mit berechnetem 
count.  Nimm ne feste Größe und stell sicher, dass das Ergebnis 
reinpasst.

[hal (Gast)]

OK, alles klar ... danke.

  char str[20];
  
  strcpy_P(str, data);
  strcat(str, text);
  
  uint8_t count = strlen(str);

foobar schrieb:
> Btw, verkneif dir sowas wie "char str[count_str+1];" mit berechnetem
> count.  Nimm ne feste Größe und stell sicher, dass das Ergebnis
> reinpasst.

Was ist der Nachteil?

[foobar (Gast)]

> Was ist der Nachteil?

Ist, als würdest du dir beim Autofahren den Sicherheitsgurt auf's 
T-Shirt malen statt ihn anzulegen - sieht nett aus, hilft im Ernstfall 
(zu großer count) aber nicht.  Ändert nur den Namen: aus Buffer- wird 
Stack-Overflow.

Du kommst nicht drumherum, dir Gedanken zu machen, welche Größe maximal 
erlaubt/möglich ist und das entsprechend zu behandeln.

Bei erfahreneren Programmierern wirst du dieses Konstrukt nur sehr 
selten finden - insb nicht auf Systemen mit kleinem festen Stack.