AVR GCC - Einfache Stringmanipulation

Peter Rachow schrieb:
> Ich stehe gerade mal heftig auf dem Schlauch. Hilft mir jemand runter?
> ;-)

sollte gehen, sicher das der Fehler nicht woanders ist?

Man kann es auch lesbar schreiben

1

vfo_str[4] = vfo_num + 'A';

;

Braucht lcd_put_string einen nullterminierten String ?
0 1 2 3 4 5
V F 0 _ X /0

bundesNetzAgent schrieb:
> Braucht lcd_put_string einen nullterminierten String ?
> 0 1 2 3 4 5
> V F 0 _ X /0

hat er doch.

Mit der Definition und Initialisierung

1

char *vfo_str = "VFO  ";

reservierst Du auf dem Stack nur 5 Zeichen (einschl. dem 
Nullterminator).
Garantiert ist jedenfalls nur diese Länge - mehr ist ja eigentlich auch 
nicht nötig. Das Du da noch Zeichen anfügen willst, weiß der Compiler 
nicht.

Ooops. Sorry.

Du reservierst nur einen Zeiger auf dem Stack der auf einen Bereich, der 
jedoch nur für 5 Zeichen ausgelegt sein muss, zeigt. Wenn Du da was 
anhängst, kann das schiefgehen. Geht es hier wohl auch.

Klaus schrieb:
> Mit der Definition und Initialisierung
> char *vfo_str = "VFO  ";
>
> reservierst Du auf dem Stack nur 5 Zeichen (einschl. dem
> Nullterminator).

ich zähle 6 Zeichen (das sind 2 Leerzeichen!)

das Problem sehe ich darin, das man die Daten nicht mainipulieren darf.

teste mal mit:

1

void show_vfo(int vfo_num)

2

{

3

    char vfo_str[] = "VFO  ";

4

    vfo_str[4] = vfo_num + 'A';

5

    lcd_put_string(x0 + 2, y0 + 2, vfo_str, 1, DARKBROWN, LIGHTGREEN);

6

}

beim PC können sie durchaus im Read-Only Ram liegen. Meist wird es zwar 
gehen aber zulässig dürfte es nicht sein.

1

char *vfo_str = "VFO  ";

deklariert einen Pointer auf ein (u.u. nicht beschreibbares 
String-Literal)

Probier stattdessen mal:

1

char vfo_str[] = "VFO  ";

Dies deklariert einen beschreibbaren Array, der mit "VFO  " 
initialisiert wird.

Nochmal oops.

Wie ich gerade sehe, habe ich mich verzählt. Es sind 6 Zeichen.

Was heisst, denn "es funktioniert nicht"?

1

void show_vfo(int vfo_num)

2

{

3

    char vfo_str[] = "VFO  ";

4

5

    vfo_str[4] = vfo_num + 'A';

6

    //lcd_put_string(x0 + 2, y0 + 2, vfo_str, 1, DARKBROWN, LIGHTGREEN);

7

    puts(vfo_str);

8

}

Pointerversion macht auch PC Probleme.

Mac hat des Pudels Kern abgeschossen. Der Punkt ist, dass der Zeiger per 
Definition nicht zwingend auf einen beschreibbaren Bereich zeigt.

Nochmal Sorry. Ich war wohl unaufmerksam.

Hi zusammen, danke für Eure zahlreichen Rückmeldungen und die 
Erklärungen. Dann mache ich das mit dem Array.

Viele Grüße

Peter

Ich fürchte Falks Einwand ist tatsächlich maßgeblich.

Beitrag "Re: Lifetime String Literal in C"

Tcf K. schrieb:
> Fehlt da nicht ein cast auf char?

nein.

Das da mit einem 'alten "Turbo C"-Compiler' funktioniert, wundert mich 
nicht. Mit dem Compiler aus MS VS6 gehts auch noch, mit dem aus VS2005 
(oder wars doch erst VS2008?) geht es nicht mehr, da gibt es dann eine 
'access violation' zur Laufzeit.