Übergabe String Array hakt

[Manfred L. (manni)]

Hallo,

ich will das string array tText[] an die Funktion Menu_Set4LinesX() 
übergeben, die die einzelnen strings dann aus dem PROGMEM lesen und via 
printf() ausgeben soll.

Der Compiler meckert zwar nicht, aber es funktioniert leider nicht, da 
das Programm nur kryptische chars ausdruckt. Irgendwo habe ich mich mit 
den string pointern wohl verhaspelt.

Ich würde mich freuen, wenn ihr mir weiter helfen könnt.

Grüße
Manni

P.S.: ich verwende das Atmel Studio 7

/*
** CPU ist ATmega644
*/

#define SYSCLOCK    20000000  // Crystal Frequency in Hz
#define F_CPU      SYSCLOCK  // Crystal Frequency in Hz
#define PROGMEM_MAXLEN  20

#include <stdio.h>
#include <avr/io.h>
#include <string.h>
#include <avr/pgmspace.h>
#include <util/delay.h>


void Menu_Set4LinesX (char** cString);
void Menu_SetStringFromProgMemX (char* str);
char *Menu_ProgMem_GetTextX (char* str);

const char tText1[]          PROGMEM = "Text1";
const char tText2[]          PROGMEM = "Text2";
const char tText3[]          PROGMEM = "Text3";
const char tText4[]          PROGMEM = "Text3";
const char* const tText[]      PROGMEM = {tText1, tText2, tText3, tText4};


int main (void)
  {
  while (1)
    {
    Menu_Set4LinesX ((char**) tText);
    _delay_ms (2000);
    }
  }


void Menu_Set4LinesX (char** cString)
  {
  Menu_SetStringFromProgMemX (cString[0]);
  Menu_SetStringFromProgMemX (cString[1]);
  Menu_SetStringFromProgMemX (cString[2]);
  Menu_SetStringFromProgMemX (cString[3]);
  return;
  }

void Menu_SetStringFromProgMemX (char* str)
  {
  char *Text;
  Text = Menu_ProgMem_GetTextX (str);
  printf ("%s\n", Text);
  return;
  }

char *Menu_ProgMem_GetTextX (char* str)
  {
  static char    Text[PROGMEM_MAXLEN];
  char      *pText;
  uint8_t      len = 0;

  pText = Text;
  while (1)
    {
    *pText = pgm_read_byte (str++);
    if (*pText == '\0')  return Text;
    if (len == (PROGMEM_MAXLEN-1))
      {
      *pText = '\0';
      return Text;
      }
    len++;
    pText++;
    }
  }

[Sebastian (Gast)]

Bei dir sind nicht nur die einzelnen Strings in PROGMEM, sondern auch 
das Array mit den Zeigern auf die Strings. Also musst du auch den 
Zugriff auf die Zeiger über pgm_get_... laufen lassen, und nicht einfach 
nur Menu_SetStringFromProgMemX (cString[0]) ...

LG, Sebastian

[Manfred L. (manni)]

Sebastian schrieb:
> Bei dir sind nicht nur die einzelnen Strings in PROGMEM, sondern auch
> das Array mit den Zeigern auf die Strings.

Ich danke Dir vielmals für diesen hilfreichen Hinweis.
Das ist mir einfach durchgerutscht :-( Es war einfach mal wieder zu heiß 
heute.

Ich habe es der Einfachheit halber geändert auf:

const char* const tText[] = {tText1, tText2, tText3, tText4};

und damit funktionierte es sofort.

Nochmals vielen Dank !

Grüße Manni

[Mombert H. (mh_mh)]

Manfred L. schrieb:
> und damit funktionierte es sofort.

Super! Dann kannst du dich als nächstes um diesen Cast kümmern.
Manfred L. schrieb:
> Menu_Set4LinesX ((char**) tText);
Immer wenn du ein const wegcastest, machst du wahrscheinlich etwas 
falsch, oder handelst grob fahrlässig. Um den Cast los zu werden, musst 
du die aktuell fehlenden const in deinen Funktionsparametern hinzufügen.

[Manfred L. (manni)]

Mombert H. schrieb:
> oder handelst grob fahrlässig

Ja, Du hast vollkommen recht. Mit den const casts stehe ich sowieso 
immer auf Kriegsfüßen. Hast Du eine Idee, wie ich das C-mäßig "sauberer" 
hinkriege ?

[Mombert H. (mh_mh)]

Manfred L. schrieb:
> Mombert H. schrieb:
>> oder handelst grob fahrlässig
>
> Ja, Du hast vollkommen recht. Mit den const casts stehe ich sowieso
> immer auf Kriegsfüßen. Hast Du eine Idee, wie ich das C-mäßig "sauberer"
> hinkriege ?

Ja, füge die fehlenden consts in deinen Funktionsparametern hinzu, also 
z.B. hier:

void Menu_SetStringFromProgMemX (hier char oder hier* str)