WiNAVR bringt Fehler (Anfängerproblem)

[Horst (Gast)]

Ich arbeite gerade das WinAVR (Tutorial) durch und compiliere das 
Programm aus dem Beispiel.

Es kommt folgender Fehler:
main.c:12: warning: 'i' may be used uninitialized in this function

Die Zeile 12 lautet:
unsigned char i;

Was ist das Problem?

Kompletter Code falls relevant:
#include <avr/io.h>
#include <util/delay.h>

// Sollte normalerweise schon im Makefile definiert sein.
// In dem Fall hier einfach löschen.
//#define F_CPU 7372800

void ioinit(void);

// Hauptprogramm
int main(void) {
    unsigned char i;

    ioinit();

    // Endlosschleife
    while (1) {
        PORTB = i;
        i++;

        // Eine Sekunde warten
        _delay_ms(1000);
    }

    return 0;
}

// Initialisierung
void ioinit(void)
{
    DDRB = 0xff;    // PortB als Ausgang deklarieren
    PORTB = 0x00;   // Ports auf LOW schalten
}

[Peter II (Gast)]

das ist eine Warnung die sagt das in i kein sinnvoller inhalt drin ist.

[Karl H. (kbuchegg)]

Horst schrieb:

> Es kommt folgender Fehler:
> main.c:12: warning: 'i' may be used uninitialized in this function
>
> Die Zeile 12 lautet:
> unsigned char i;

Hier wird i definiert. Und als funktionslokale Variable hat sie keinen 
definierten Wert, sondern irgendeinen. Was halt zufällig an dieser 
Stelle im Speicher stand.

> Was ist das Problem?

Hier:

>     while (1) {
>         PORTB = i;

welchen Wert hat denn i hier?

[Floh (Gast)]

Horst schrieb:
> Was ist das Problem?

Welchen Wert soll i zu Beginn haben? -> Hinschreiben. :-)

[Horst (Gast)]

Das ging ja fix, Danke :-)

Das habe ich verstanden
unsigned char i=0;    bringt auch keinen Fehler mehr, aber

Beim Start sollte der AVR doch alles auf 0 haben, oder nicht?
Hätte das Programm auch ohne Zuweisung funktioniert?
MUSS man eine Variable immer definieren?
Ist die Zeile so korrekt? (im Tutorial steht ja nicht die Lösung)

[Peter II (Gast)]

Horst schrieb:
> Beim Start sollte der AVR doch alles auf 0 haben, oder nicht?
oder nicht ist richtig.

nur globale Variablen sind 0 bei funktionslokalen steht irgendetwas 
drin.

[Horst (Gast)]

Peter II schrieb:
> nur globale Variablen sind 0 bei funktionslokalen steht irgendetwas
> drin.

Wir würde ich denn eine globale Variable definieren?

[Karl H. (kbuchegg)]

> Wir würde ich denn eine globale Variable definieren?

#include <avr/io.h>
#include <util/delay.h>

// Sollte normalerweise schon im Makefile definiert sein.
// In dem Fall hier einfach löschen.
//#define F_CPU 7372800

void ioinit(void);

unsigned char i;    // <------- jetzt ist sie global und wird automatisch
                    // mit 0 initialisiert


// Hauptprogramm
int main(void) {

    ioinit();

    // Endlosschleife
    while (1) {
        PORTB = i;
        i++;

        // Eine Sekunde warten
        _delay_ms(1000);
    }

    return 0;
}

Gewöhn dir aber trotzdem an, funktionslokale Variablen vor ihrer ersten 
Verwendung zu initialisiern. Denn einfach alles global zu machen ist 
keine Strategie, die du verfolgen solltest. Ausserdem kostet es dir 
nicht viel.

...
// Hauptprogramm
int main(void) {
    unsigned char i = 0;
  ...

und das Thema ist (in diesem Fall) gegessen.

[Horst (Gast)]

Danke Danke für die lehrreichen Infos :-)

[Noname (Gast)]

>Ausserdem kostet es dir nicht viel.

Jetz isser unte de Baliner jejangen, der Kalle. ;-)

[Stefan E. (sternst)]

Peter II schrieb:
> Horst schrieb:
>> Beim Start sollte der AVR doch alles auf 0 haben, oder nicht?
> oder nicht ist richtig.
>
> nur globale Variablen sind 0 bei funktionslokalen steht irgendetwas
> drin.

Das stimmt nicht ganz. Lokale Variablen mit static storage werden auch 
implizit mit 0 initialisiert.