Text in 2D Arrays, stm32f4xx

[Ma B. (drumstick)]

Guten Tag

Bei meinem Projekt(c, mdk5, Keil, stm32f4xx) geht es darum eine kleine 
Bedienung zu erstellen. Es soll ein Display angesteuert werden. Die 
Texte sollen in ein FRAM abgelegt werden. Da sechs Sprachen erforderlich 
sind, dachte ich an ein 2D Array. Das Display hat 2 Zeilen, 16 Zeichen, 
ASCII-Code. Um es einfach zu halten stelle ich mir folgendes vor:



Dies wäre die Struktur, die dann ins FRAM abgelegt wird. (Auszugsweise)

#define STRING_LENGTH 17
#define CNT_LANGUAGES 6


typedef struct  {
                  size_t        SetupVersion;
                  char          txt_run_RunOpen[CNT_LANGUAGES][STRING_LENGTH];
                }
                tSetup;


// ---- exported data --

extern tSetup      setup;

Und im Init_Setup würde dann das Array beschrieben.

void Setup_Init(void)
{
   setup.txt_run_RunOpen[0][0] = {{'M'}};
   setup.txt_run_RunOpen[0][1] = {{'a'}};
   setup.txt_run_RunOpen[0][2] = {{'c'}};
   setup.txt_run_RunOpen[0][3] = {{'h'}};
   setup.txt_run_RunOpen[0][4] = {{'i'}};
   setup.txt_run_RunOpen[0][5] = {{'n'}};
   setup.txt_run_RunOpen[0][6] = {{'e'}};
   setup.txt_run_RunOpen[0][7] = {{' '}};
   setup.txt_run_RunOpen[0][8] = {{'R'}};
   setup.txt_run_RunOpen[0][9] = {{'u'}};
   setup.txt_run_RunOpen[0][10] = {{'n'}};
   setup.txt_run_RunOpen[0][11] = {{' '}};
   setup.txt_run_RunOpen[0][12] = {{'o'}};
   setup.txt_run_RunOpen[0][13] = {{'p'}};
   setup.txt_run_RunOpen[0][14] = {{'e'}};
   setup.txt_run_RunOpen[0][15] = {{'n'}};
}

usw.

Micht dünkt dies sehr aufwendig! Andererseits ist das Senden der ASCII 
Zeichen auf den i2c Bus einfach.

for(x = 0;x <= 15; x++)
  {
    LCD_wr_Data(setup.txt_run_RunOpen[0][x]);
  }

Hat jemand eine Idee, wie man dies vereinfachen kann.

Vielen Dank schon mal

[Ma B. (drumstick)]

oder andere Frage, wie kriege ich

#define STRING_LENGTH 17
#define CNT_LANGUAGES 6

typedef struct  {
                  size_t        SetupVersion;
                  char          txt_run_RunOpen[CNT_LANGUAGES][STRING_LENGTH];
                }
                tSetup;

extern tSetup      setup;


setup.txt_run_RunOpen[CNT_LANGUAGES][STRING_LENGTH] = 
{{"Machine run open"},
 {"Anlage oeffnet  "},
 {"Macchina aprire "},
 {""},
 {""},
 {""}};

dazu, dass ich ASCII Zeichen ausgeben kann?

for(x = 0;x <= 15; x++)
  {
    LCD_wr_Data(setup.txt_run_RunOpen[0][x]);
  }

Welche Methode ist speichertechnisch die beste Lösung?

Vielen Dank

Grüsse

[Ma B. (drumstick)]

oder hat mir jemand ein Beispiel wie dies gelöst werden könnte? Ich habe 
nichts gefunden, wo genau dieses Problem beschreibt!

Vielen Dank

Grüsse

MC Bu

[Arne (Gast)]

hab sowas in der Art mal in D/GB für ein Prüfgerät programmiert. Habe 
eine Tabelle wie Du im 2. Post angelegt. Ist auch einfacher zu 
editieren.

[Ma B. (drumstick)]

Ok, vielen Dank!

Hast Du die Zeichen in den Ascii Code gewandelt für die Ausgabe?

Danke und Gruss

MC Bu

[Route_66 H. (route_66)]

MC Bu schrieb:
> Hast Du die Zeichen in den Ascii Code gewandelt für die Ausgabe?

Die Frage habe ich nicht verstanden. Die Zeichen sind doch bereits im 
ASCII Code!?

[Dr. Sommer (Gast)]

MC Bu schrieb:
> Die Texte sollen in ein FRAM abgelegt werden.
Wozu denn der Aufwand... Warum nicht einfach alle Texte ins Flash und 
die der gewünschten Sprache direkt von da direkt auf den I²C oder 
wasauchimmer ausgeben?

[Ma B. (drumstick)]

@ Dr. Sommer

Ja, diesbezüglich habe ich meine Meinung auch geändert. Die Texte werden 
ja nicht mehr verändert und es muss nur darauf zugegriffen werden.

@Route66

setup.txt_run_RunOpen[CNT_LANGUAGES][STRING_LENGTH] = 
{{"Machine run open"},
 {"Anlage oeffnet  "},
 {"Macchina aprire "},
 {""},
 {""},
 {""}};

wie geb ich jetzt einzelne Zeichen des Strings aus? Ich hab da einen 
Knopf in der Leitung.

for(x = 0;x <= 15; x++)
  {
    LCD_wr_Data(setup.txt_run_RunOpen[0][x]);
  }

void LCD_wr_Data(uint8_t data)
{
  LCD_wr(true, data);
}

void LCD_wr(bool dataReg, uint8_t data)
{
  ExtPort.pin.P14_RS       = dataReg;
  ExtPort.pin.P15_R_nW     = false;
  ExtPort.pin.P16_E       = true;
  ExtPort.pin.P10_Dnibble = data / 0x10;
  xferToPortexpander();
  
  ExtPort.pin.P16_E       = false;
  xferToPortexpander();
  
  ExtPort.pin.P16_E       = true;
  ExtPort.pin.P10_Dnibble = data & 0x0f;
  xferToPortexpander();
  
  ExtPort.pin.P16_E       = false;
  xferToPortexpander();
}

so erhalte ich nur schwarze Zeichen auf dem Display!

Vielen Dank und Gruss

[Karl H. (kbuchegg)]

MC Bu schrieb:

> wie geb ich jetzt einzelne Zeichen des Strings aus?

So wie immer.

> Ich hab da einen
> Knopf in der Leitung.
>
>

> for(x = 0;x <= 15; x++)
>   {
>     LCD_wr_Data(setup.txt_run_RunOpen[0][x]);
>   }
>

Im Prinzip schon richtig.
Nur gibt man halt nicht einfach fix 15 Zeichen aus. Schliesslich sagt ja 
kein Mensch, dass alle Texte 15 Zeichen lang sind.

C definiert, dass ein String genau dort zu Ende ist, wo sich das '\0' 
Zeichen befindet.

Also

   x = 0;
   while( setup.txt_run_RunOpen[0][x] != '\0' )
   {
     LCD_wr_Data(setup.txt_run_RunOpen[0][x]);
     x++;
   }

oder ein wenig intelligenter unter Verwendung der Pointer-Schreibweise

    const char* ptr = &setup.txt_run_RunOpen[0][0];

    while( *ptr != '\0' )
      LCD_wr_Data( *ptr++ );

> so erhalte ich nur schwarze Zeichen auf dem Display!

Wenn dein allererstes Original funktioniert hat, dann muss auch das 
gehen.


char str[8] = "test";

ist schliesslich auch nichts anderes als

str[0] = 't';
str[1] = 'e';
str[2] = 's';
str[3] = 't';
str[4] = '\0';

nur dass der Compiler die ganze Arbeit für dich in der 
Initialisierungsphase macht.

[Karl H. (kbuchegg)]

Zeig doch bitte mal den Originalcode.

Denn das hier

typedef struct  {
                  size_t        SetupVersion;
                  char          txt_run_RunOpen[CNT_LANGUAGES][STRING_LENGTH];
                }
                tSetup;

extern tSetup      setup;


setup.txt_run_RunOpen[CNT_LANGUAGES][STRING_LENGTH] = 
{{"Machine run open"},
 {"Anlage oeffnet  "},
 {"Macchina aprire "},
 {""},
 {""},
 {""}};

kann er recht offensichtlich nicht sein. Das compiliert nämlich mit 
Sicherheit nicht. Du kannst nicht setup.txt_run_RunOpen einzeln 
initialisieren.

[Ma B. (drumstick)]

Stimmt, hab ich nicht compiliert. Dies bedeutet ich muss den String bei 
der definition initialisieren. Oder ich kann nur einzelne Zeichen laden?

Setup.startup_Text_line1[0][0] = 'M';

Ich möchte aber alle Textelemente in einem Struct vereinen. Wie kann ich 
dies lösen?

Ansonsten funktioniert es so sehr gut.

Vielen, vielen Dank!

Grüsse

MC Bu

[Karl H. (kbuchegg)]

MC Bu schrieb:
> Stimmt, hab ich nicht compiliert. Dies bedeutet ich muss den String bei
> der definition initialisieren.

Es ist das vernünftigste.

> Oder ich kann nur einzelne Zeichen laden?

Kannst du auch. Aber dann ist es erstens keine Initalisierung mehr und 
zweitens machst du dir eine Menge Arbeit

>

> Setup.startup_Text_line1[0][0] = 'M';
>

kann man machen. Man kann aber auch

   strcpy( Setup.startup_Text_line1[0], "Meine Idee" );

machen :-)

Ist ein bischen einfacher.
Vielleicht solltest du doch erst mal im C-Buch deiner Wahl nachlesen, 
wie Stringverarbeitung in C überhaupt funktioniert?

> Ich möchte aber alle Textelemente in einem Struct vereinen. Wie kann ich
> dies lösen?

Genau so, wie das da oben gemacht wurde.

[Dr. Sommer (Gast)]

Karl Heinz schrieb:
> kann er recht offensichtlich nicht sein. Das compiliert nämlich mit
> Sicherheit nicht. Du kannst nicht setup.txt_run_RunOpen einzeln
> initialisieren.
man kann aber

typedef struct  {
                  size_t        SetupVersion;
                  char          txt_run_RunOpen[CNT_LANGUAGES][STRING_LENGTH];
                }
                tSetup;

extern tSetup      setup;

tSetup setup = {  42,
        {{"Machine run open"}, 
        {"Anlage oeffnet  "},
        {"Macchina aprire "}, 
        {""},
        {""},
        {""}}};

 machen.

[Ma B. (drumstick)]

Ich kann aber nicht die Stringkette in einem Struct anlegen und 
gleichzeitig initialisieren!

typedef struct  {
                  size_t        SetupVersion;
                  char          startup_Text_line1[STRING_LENGTH];
                }
                tSetup;

Ausser wie es Dr.Sommer vorgemacht hat! Dies würde ja Sinn machen, da 
ich den struct nur für die Texte anlege. Diese Lösung finde ich 
praktikabel.

Das beste wird es sein, ich lese über struct ebenfalls nach. ich 
beschäftige mich leider bisschen zu wenig mit c.

Vielen Dank für Eure Hilfe!

Grüsse

MC Bu

[Arne (Gast)]

MC Bu schrieb:
> ich beschäftige mich leider bisschen zu wenig mit c.
Keep it simple - am Anfang. Optimieren kannst Du immer noch. Hauptsache 
Du hast erstmal was am Laufen.

[Karl H. (kbuchegg)]

MC Bu schrieb:
> Ich kann aber nicht die Stringkette in einem Struct anlegen und
> gleichzeitig initialisieren!

Wo genau machst du das denn?

> Ausser wie es Dr.Sommer vorgemacht hat!

Ja, genau das ist ja auch der Normalfall.

> Das beste wird es sein, ich lese über struct ebenfalls nach. ich
> beschäftige mich leider bisschen zu wenig mit c.

Sieht fast danach aus.

[Ma B. (drumstick)]

Niergends! Das war nur eine Überlegung!


Jetzt siehts bei mir so aus:

im Text.h

typedef struct  {
                  char  startup_Text_line1[STRING_LENGTH];
                  char  operation_Text_line1[STRING_LENGTH];
                  char  operation_Text_line2[STRING_LENGTH];
                }
              tTxt;

extern  tTxt Text;

im Text.c

void Init_Texte(void)
{
  tTxt Text = {  {"Hallo Welt"},
                 {"Operation is"},
                 {"running"}
              };
}

[Dr. Sommer (Gast)]

MC Bu schrieb:
> Jetzt siehts bei mir so aus:
Das klappt nicht. Zuweisungen von structs gehen in C nicht. Nur 
initialisierungen, wie ich sie oben gezeigt hab. Wenn deine Texte im 
Flash sind, kannst du sie eh nicht ändern/zuweisen. Braucht man ja auch 
nicht, da 1x initialisieren und im Read-Only-Flash zu lassen ja reicht.

[Dr. Sommer (Gast)]

Uhps, falsch gelesen, das ist ja die Initialisierung einer lokalen 
Variable... Das geht, ist aber wenig sinnvoll da ineffizient. Und die 
globale "Text" Variable wird gar nicht verwendet...

[Karl H. (kbuchegg)]

MC Bu schrieb:

> im Text.c
>

> 
> 
> void Init_Texte(void)
> {
>   tTxt Text = {  {"Hallo Welt"},
>                  {"Operation is"},
>                  {"running"}
>               };
> }
>

Das ist aber eine lokale Variable.

Auch wenn die zufällig den gleichen Namen hat, wie deine globale, sie 
hat nichts mit der globalen Variablen zu tun.

[Ma B. (drumstick)]

ups ja, jetzt seh ichs

[Karl H. (kbuchegg)]

im Text.h

typedef struct  {
                  char  startup_Text_line1[STRING_LENGTH];
                  char  operation_Text_line1[STRING_LENGTH];
                  char  operation_Text_line2[STRING_LENGTH];
                }
              tTxt;

extern  tTxt Text;

im Text.c

#include "Text.h"

tTxt Text = {  "Hallo Welt",
               "Operation is",
               "running"
            };

und die Funktion Init_Texte braucht keiner.

[Dr. Sommer (Gast)]

> und die Funktion Init_Texte braucht keiner.
Dann noch "const" dran und es landet schön sparsam im Flash.