Verschachtelte Progmem-Arrays auslesen

Christian schrieb:
> l_pgmpointer = (PGM_P) pgm_read_word( &menu[p_menu][i+1] ); //wenn ich hier 
statt i+1 eine konstante Zahl hinschreibe, funktioniert alles

Probier statt
"(&menu[p_menu][i+1])"

mal die Version ohne doppelte Adressauflösung:

"(menu[p_menu] + i + 1)"

Dadurch steht im Argument von pgm_read_word() gleich eine Adresse. Ein 
verschachteltes Array ist ja eigentlich nix anderes als ein 
Pointer-Pointer auf ein Word. Die werden vom Compiler nur voneinander 
Unterschieden, damit der Benutzer/Programmierer besser zurechtkommt.

Ich vermute nämlich eine "falsche" Makro-Umsetzung innerhalb von 
pgmspace, wobei das doch eigentlich auch für zweidimensionale Arrays 
gehen sollte?!

mfg mf

Christian schrieb:

> Das Menü flimmert jetzt allerdings

Ich hab dein Programm nicht analysiert.
Aber Flimmern auf einem LCD ist meistens ein Zeichen für
   Löschen + Neu beschreiben
Und das ständig

Christian schrieb:

> Irgendwas passt nicht...

Da passt vieles nicht

Das waren die Daten

1

static uint8_t menu_entry[50][4] =

2

    {

3

      {8,0,0,0},

4

      {9,0,0,0},

5

      {10,0,0,0},

6

      {11,0,0,0},

7

      {12,0,0,0},

8

      {13,0,0,0},

9

      {14,0,0,0}

10

    };

11

12

static uint8_t menu[20][10] PROGMEM = 

13

    {

14

      {7,0,1,2,3,4,5,6,7}

15

    };

was es mit der menu Variablen auf sich hat, hat ich ehrlich gesagt nicht 
wirklich auf Anhieb durchschaut. Für meinen Geschmack verwendest du da 
ein wenig zu oft zu sehr unstrukturierte uint8_t Arrays, als ob structs 
nie erfunden worden wären

1

void lcd_write_menu(uint8_t p_menu)

2

  {

3

4

  PGM_P l_pgmpointer;

5

  uint8_t l_index;

6

7

  for(uint8_t i=0; i<pgm_read_byte(menu[p_menu]); i++)

Schreibs aus:   & menu[p_menu][0]

menu ist ein 2D Array. Und da es sich nicht ändert, ist es eine gute 
Idee diesen Wert erst einmal vorab zu bestimen (Dein { Schema macht mich 
wahnsinng, das du da noch siehst welche } zu welcher { gehört ... 
Respekt )

1

void lcd_write_menu(uint8_t p_menu)

2

{

3

  PGM_P l_pgmpointer;

4

  uint8_t l_index;

5

6

  uint8_t entryCount = pgm_read_byte( &menu[p_menu][0] );

7

8

  for( uint8_t i = 0; i < entryCount; i++ )

Sicherheitshalber auch kontrollieren (mit einer Ausgabe) ob da 
tatsächlich die 7 daher kommen.

Wie gehts weiter?

     {7,0,1,2,3,4,5,6,7}

nach der 7 stehen dann die 7 Nummern der Menüpunkte. Die liest du mit

1

    l_pgmpointer = (PGM_P) pgm_read_byte( menu[p_menu]+1+i );

wieder: Schreibs aus! Ausserdem kriegst du von dort keinen Pointer. Wenn 
du die Zahlen der Reihe nach ausliest, hast du erst mal nur die Zahlen

1

     entryIndex = pgm_read_byte( &menu[p_Menu][i+1] );

Mit diesem Index kannst du jetzt ins menu_entry Array gehen und dort die 
Zeile auswählen. Spalte 0 sagt dir welche Textnummer anzuzeigen ist

1

     textIndex = pgm_read_byte( &menu_entry[ entryIndex ][0] );

mit diesem text Index gehts dann weiter ins strd Array (das diesmal 
Pointer enthält) aus dem man den Pointer zum Text bekommt

1

     textPointer = pgm_read_word( &strd[ textIndex ] );

und mit diesem Pointer kann dann der Text ausgegeben werden.

1

void lcd_write_menu(uint8_t p_menu)

2

{

3

  uint8_t entryIndex, textIndex;

4

  PGM_P textPointer;

5

6

  uint8_t entryCount = pgm_read_byte( &menu[p_menu][0] );

7

8

  for( uint8_t i = 0; i < entryCount; i++ )

9

  {

10

    entryIndex = pgm_read_byte( &menu[p_Menu][i+1] );

11

    textIndex = pgm_read_byte( &menu_entry[ entryIndex ][0] );

12

    textPointer = (PGM_P)pgm_read_word( &strd[ textIndex ] );

13

14

    lcd_write_menu_entry( l_pgmpointer , i );

15

  }

16

17

  ...

18

}

Wenn du den Überblick verlierst, dan mal dir deine Datenstruktur auf und 
verbinde die Einzelteile mit Pfeilen, die dich bei der Programmierung 
leiten, was du von wem bekommst und was du damit tun musst.

Noch ein Hinweis:
Man kann sich Indizes (als uint8_t) auch ganz einfach auf einem LCD 
ausgeben lassen um zu kontrollieren, ob man das richtige gelesen hat 
oder nicht. Alle Indizes müssen genau so aus den entsprechenden Arrays 
gelesen werden, wie du sie beim Aufbau der Datenstruktur angegeben hast. 
Lediglich Pointerwerte sind eher sinnlos beim Ausgeben.

PS: was sagt der Prefix l_ aus?
Sag jetzt bitte nicht, das der eine lokale Variable kennzeichnet.

Christian schrieb:
> Hm, deine Version funktioniert aber auch nicht wie gewollt, es zeigt
> mehrmals den Text aus strd[0] an, und dann mehrmals den Text aus
> strd[15].

Tja. Im Gegensatz du dir muss ich blind programmieren.
Du hast die einzelnen Indizes: Gib sie dir am LCD aus.

> Und ja, das l_ zeigt eine lokale Variable an.

Spar dir das.

Christian schrieb:
> Ok, hat sich erledigt, das Array menu_entry war noch nicht im Flash
> Bereich, weil ich ja schrittweise umstellen wollte...

OK. Hab ich auch übersehen, dass da ein PROGMEM fehlt.
Wie gesagt: Ich muss blind programmieren. Direkt hier im Forum 
Eingabefeld. Da passiert schon mal das eine oder andere