LCD Display - Ausgabe mittels Char Array

[Elektrotiger (Gast)]

Ein Array von Charactern soll am LCD Display ausgegeben werden.
Aktuell wird nur der Erste Buchstaben angezeigt.
lcd_string(const char* data) ist der Befehl dazu.
Nach Tastendrücken, wird das Array Lauftext aktualisiert. Dabei wird 
jeder Buchstaben bei hinzufügen eines neuen Buchstabens bei der Ausgabe 
um eins nach Rechts verschoben und die 2. Stelle bleibt immer leer!

Ausgabebeispiel: A
                 B A
                 B BA
                 C BBA

char lauftext[16];
char* plauftext = lauftext;
 
void text(char* test)
{
  for(uint8_t j=6;j<8;j++)
  {
    for(uint8_t i=2;i<16;i++)
    {
      test[i+1]=test[i]; // Zeile um eins verschieben
    }
    test[2]=test[0];     // 2. Feld immer leer !!
    if((adc[j]<256) && flag_high[j])
    {
      if(j==6) test[0]='A';
      else if(j==7) test[0]='B';
      flag_high[j]=0;
    }
    else if(adc[j]>255)flag_high[j]=1;
  }
}

int main()
{ [...]
          case 1:  // Standardbetrieb
  {
            for(uint8_t m=0; m<ADCINPUTS; m++)
      {
        adc[m] = ADC_Read_Avg(m,5);
      }
      text(plauftext);  // lauftext array wird aktualisiert
      lcd_setcursor(0,1);
      lcd_string(plauftext); // Displayaktualisierung
        }
}

[Karl H. (kbuchegg)]

    for(uint8_t i=2;i<16;i++)
    {
      test[i+1]=test[i]; // Zeile um eins verschieben
    }

Vorsicht!
Array Überlauf!

Dein Array wurde mit einer Länge von 16 definiert.
Hier wird i als Maximalwert den Wert 15 annehmen. D.h. i+1 ergibt 16. 
Ein Array Element mit dem Index 16 existiert aber nicht.

    test[2]=test[0];     // 2. Feld immer leer !!

das ist nicht leer.
Nur weil du am LCD nichts sehen willst, heist das noch lange nicht, dass 
das Array Element leer ist. Ein Leerzeichen ist ein Buchstabe wie jeder 
andere auch. Und wenn du ein Leerzeichen haben willst, dann musst du 
auch eines an die entsprechende Stelle im Array einfügen.
Ansonsten steht bei einem global allokierten char-Array lauter 0-Bytes 
in den entsprechenden Array Elementen. Und ein 0-Byte in einem String 
bedeutet nun mal, dass der String an dieser Stelle zu Ende ist. Hast du 
also von Anfang an in lauftext[1] ein 0-Byte stehen, dann verändert sich 
das durch deine Manipulationen nicht. Das 0-Byte bleibt genau dort 
stehen und somit ist für alle String-Funktionen dort der String zu Ende.

[Karl H. (kbuchegg)]

char lauftext[16];
char* plauftext = lauftext;

die Pointer Variable braucht kein Mensch.

      text(lauftext);

ist völlig ok. Siehe jedes noch so grindige C-Lehrbuch über Array 
Handling.

[Elektrotiger (Gast)]

Das ist mir alles klar. Und nein, dass Array hat 16 Elemente, wird 
allerdings von 0 angefangen zu zählen. Also ist es richtig bis i<16 zu 
laufen.

Und mir ist auch bewusst, dass Leerzeichen auch keine Eigentliche 
Leerstelle ist. Es soll, ja auch ein Leerzeichen an dieser Stelle 
ausgegeben werden ;).

[Karl H. (kbuchegg)]

Elektrotiger schrieb:
> Das ist mir alles klar. Und nein, dass Array hat 16 Elemente, wird
> allerdings von 0 angefangen zu zählen. Also ist es richtig bis i<16 zu
> laufen.

so, so.
i wird maximal 15. Was ergibt dann i + 1

    for(uint8_t i=2;i<16;i++)
    {
      test[i+1]=test[i]; // Zeile um eins verschieben
           ***
           ***
    }

Im letzten Schleifendruchlauf läuft es dann auf

     test[15+1] = test[15];

hinaus.
15+1 ergibt 16. Also

     test[16]=test[15];

existiert test[16]?

Nein.


> Und mir ist auch bewusst, dass Leerzeichen auch keine Eigentliche
> Leerstelle ist. Es soll, ja auch ein Leerzeichen an dieser Stelle
> ausgegeben werden ;).

Und wo machst du das.
In dem Codeschnipsel ist das nicht erkennbar.

[Elektrotiger (Gast)]

Karl Heinz schrieb:
> ist völlig ok. Siehe jedes noch so grindige C-Lehrbuch über Array
> Handling

Nicht, wenn ein Mikrocontroller begrenzten Speicher besitzt

[Karl H. (kbuchegg)]

Elektrotiger schrieb:
> Karl Heinz schrieb:
>> ist völlig ok. Siehe jedes noch so grindige C-Lehrbuch über Array
>> Handling
>
> Nicht, wenn ein Mikrocontroller begrenzten Speicher besitzt

An deiner Stelle würde ich mich mit deinen begrenzten mangelhaften C 
Kenntnisse nicht mit mir anlegen.

[Elektrotiger (Gast)]

Karl Heinz schrieb:
> Und wo machst du das.
> In dem Codeschnipsel ist das nicht erkennbar.

Kann schlecht 1000 Zeilen Code zeigen :(.
Aber danke, habe in der Schleife nicht mehr daran gedacht. Den Überlauf 
muss ich abfangen.
Leerzeile wird vorher im Programm schon definiert.

Das löst, aber alles mein Problem bisher nicht. Es wird immer noch nur, 
das Erste Symbol angezeigt.

[Karl H. (kbuchegg)]

Elektrotiger schrieb:

> Kann schlecht 1000 Zeilen Code zeigen :(.

Dann speck auf das wesentliche ab, aber so dass der Code vollständig 
ist.

> Leerzeile wird vorher im Programm schon definiert.
Das kann ich dir glauben oder auch nicht.
Wie hast du die Leerzeile definiert?

> Das löst, aber alles mein Problem bisher nicht. Es wird immer noch nur,
> das Erste Symbol angezeigt.

Weil dein Programm fehlerhaft ist.
Ganz einfach.

[Karl H. (kbuchegg)]

OK.
Machen wirs kurz

void text(char* test)
{
  for(uint8_t j=6;j<8;j++)
  {
    for(uint8_t i=2;i<14;i++)
//                    **                <------
    {
      test[i+1]=test[i]; // Zeile um eins verschieben
    }
    test[15] = '\0';                    // <-------
    test[1] = ' ';                      // <-------
    test[2] = test[0];
...

ist zwar ein wenig Overkill, die Sicherheitsterminierung bzw. das 
Leerzeichen jedesmal wieder neu einzufügen, aber im Vergleich zum 
lcd_string kostet das praktisch nichts. Durch die 2 Schleifendurchläufe 
weniger wird es auf jeden Fall wieder aufgefangen.

[Vlad T. (vlad_tepesch)]

mir ist der gesamte sinn und zweck des Programmes nicht klar.

In einem Zyklus, werden zuerst alle ADC-Inputs eingelesen (scheinen ja 
mindestens 8 zu sein)

dann wird für den input 6 und 7 jeweils einmal alle character nach 
rechts geschoben und nur ein buchstabe ausgegeben, wenn eine Flanke 
hoch->niedrig auftaucht.

Das heißt, das alte Zeichen wird jeweils kopiert und eventuell 
geupdated.
Ich glaube nicht, dass man dem Display dann irgendwelche sinnvollen 
Informationen entnehmen kann, weil man nicht weiß, ob das das Zeichen 
jetzt da noch vom letzten mal steht, oder in diesem Frame neu gesetzt 
wurde, außerdem wird es in jedem Frame 2 mal bewegt, aber nur der adc[7] 
input wird vom anderen abgetrennt dargestellt ....

komisches Interface.



Ansonsten würde ich dem TO empfehlen ein wenig mehr Demut an den Tag zu 
legen.
Als Anfänger kann man nicht alles wissen, das wirft dir auch niemand 
vor, aber dann die Helfenden belehren zu wollen, und dann noch in so 
einem Ton...

[Elektrotiger (Gast)]

Funktioniert leider nicht. Sobald ich eine Zuweisung für das Array 
festlege. Wie z.B. lauftext[1]=' ';
Schwimmt das Display, sprich es gibt Zufällige Sonderzeichen und dass so 
schnell hintereinander neue, dass man da nichts mehr erkennt.

char lauftext[16];
char* plauftext = lauftext;


void text(char* test)
{

  for(uint8_t i=2;i<15;i++)
  {
    test[i+1]=test[i];  // Zeile um eine Darstellung verschieben
  }
  test[2]=test[0]; // 2. Feld immer leer !!
  for(uint8_t j=6;j<8;j++)
  {
    if((adc[j]<256) && flag_high[j])
    {
      if(j==6) test[0]='A';
      else if(j==7) test[0]='B';
      flag_high[j]=0;
    }
    else if(adc[j]>255)flag_high[j]=1;
  }
}

int main( void )
{
  lauftext [1]=' ';
  lauftext [15]='\0';
[...]
case 1:  // Standardbetrieb
          {
            for(uint8_t m=0; m<ADCINPUTS; m++)
            {
              adc[m] = ADC_Read_Avg(m,5);
            }
            text(plauftext);  // lauftext array wird aktualisiert
            lcd_setcursor(0,1);
            lcd_string(plauftext);
}

[Vlad T. (vlad_tepesch)]

Karl Heinz schrieb:
> ist zwar ein wenig Overkill, die Sicherheitsterminierung bzw. das
> Leerzeichen jedesmal wieder neu einzufügen,

dann reicht aber auch eine 13, wenn du die [15] sowiso überschreibst :)

Aber ich denke, der Puffer sollte eins länger gemacht werden, weil das 
Display ja wahrscheinlich 16 Zeichen hat.

[Elektrotiger (Gast)]

Vlad Tepesch schrieb:
> Das heißt, das alte Zeichen wird jeweils kopiert und eventuell
> geupdated.
> Ich glaube nicht, dass man dem Display dann irgendwelche sinnvollen
> Informationen entnehmen kann, weil man nicht weiß, ob das das Zeichen
> jetzt da noch vom letzten mal steht, oder in diesem Frame neu gesetzt
> wurde, außerdem wird es in jedem Frame 2 mal bewegt, aber nur der adc[7]
> input wird vom anderen abgetrennt dargestellt ....
>
> komisches Interface.

Liegt daran, dass der Prototyp nur 2 angesteuerte ADC Eingänge besitzt 
und die anderen Pins zum testen anderweitig genutzt werden.

Das ergibt so wie es geplant ist für unsere Zwecke Sinn.

Nochmal es geht mir nicht darum den Code schöner zu gestalten, sondern 
Allgemein zu Wissen warum nur das Erste Symbol angezeigt wird.

[Elektrotiger (Gast)]

Vlad Tepesch schrieb:
> Aber ich denke, der Puffer sollte eins länger gemacht werden, weil das
> Display ja wahrscheinlich 16 Zeichen hat.

Richtig, habe ich übernommen.

[Vlad T. (vlad_tepesch)]

Vlad Tepesch schrieb:
> dann reicht aber auch eine 13, wenn du die [15] sowiso überschreibst :)

ach ist ja quatsch, steht ja <14

[Volker S. (vloki)]

De

Elektrotiger schrieb:
> Nochmal es geht mir nicht darum den Code schöner zu gestalten, sondern
> Allgemein zu Wissen warum nur das Erste Symbol angezeigt wird.

Schon mal was von "debuggen" gehört ?

[Vlad T. (vlad_tepesch)]

Elektrotiger schrieb:
> Das ergibt so wie es geplant ist für unsere Zwecke Sinn.

das bezweifele ich, weil sich komplett unterschiedliche Situationen mit 
diesem Output nicht unterscheiden lassen.


zB bei gleichbleibendem Signal 7 kannst du nicht sagen, ob sich 6 
ändert, weil da nur As stehen werden, egal wie sich 6 verändert. high 
geht, low geht, so bleibt oder wild toggelt - sieht alles gleich aus.
vice versa

[Karl H. (kbuchegg)]

Elektrotiger schrieb:
> Funktioniert leider nicht. Sobald ich eine Zuweisung für das Array
> festlege. Wie z.B. lauftext[1]=' ';
> Schwimmt das Display, sprich es gibt Zufällige Sonderzeichen und dass so
> schnell hintereinander neue, dass man da nichts mehr erkennt.

Wer weiss schon, welche sonstigen Array Überläufe du sonst noch so in 
deinem Programm hast.

[Elektrotiger (Gast)]

Vlad Tepesch schrieb:
> zB bei gleichbleibendem Signal 7 kannst du nicht sagen, ob sich 6
> ändert, weil da nur As stehen werden, egal wie sich 6 verändert. high
> geht, low geht, so bleibt oder wild toggelt - sieht alles gleich aus.
> vice versa

Das ist richtig, aber für uns irrelevant bzw sogar gewünscht.
Beim Drücken eines Knopfes liegt High an. Sollte ein Dauersignal 
anliegen, wissen wir, dass ein Fehler vorliegt. Sprich ein Bruch im LWL 
Kabel oder Taste hängt fest ;).

[Karl H. (kbuchegg)]

void text(char* test)
{

  for(uint8_t i=2;i<15;i++)
//                  **
  {
    test[i+1]=test[i];  // Zeile um eine Darstellung verschieben
  }

Immer noch falsch.
Damit überschreibst du dir irgendwann die 0-Terminierung von hier

int main()
{
  ....
  lauftext [15]='\0';

[Elektrotiger (Gast)]

Habs... ist kein Syntaxfehler, sondern Logischer Fehler.
Habe die Leerstelle rekursiv nach rechts weiter kopiert.
Also Element 2 zu 3 und dann 3 zu 4 welches ja schon 2 ist.

[Karl H. (kbuchegg)]

Elektrotiger schrieb:
> Habs... ist kein Syntaxfehler, sondern Logischer Fehler.

Sagen wir mal:
Das kommt auch noch dazu.

(und eigentlich eine Schande für mich, dass ich es nicht gesehen habe).

[Vlad T. (vlad_tepesch)]

Elektrotiger schrieb:
> Habs... ist kein Syntaxfehler, sondern Logischer Fehler.
syntaxfehler hätte der Compiler ja mit größter Wahrscheinlichkeit 
bemängelt, oder eventuell wenigstens gewarnt.

> Habe die Leerstelle rekursiv nach rechts weiter kopiert.
> Also Element 2 zu 3 und dann 3 zu 4 welches ja schon 2 ist.

Nicht die Leerstelle, das entsprechende Zeichen.
nach der ersten Flanke, sollte ja da aber ein Buchstabe stehen, der 
kopiert wird.
Wenn da also vorher ein Fehler war, der dir nur das erste Zeichen 
ausgegeben hat, sollte der jetzt ja immer noch da sein.