Array Probleme

[Chris (Gast)]

Hallo, ich habe folgendes Programm:

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

#define F_CPU 8e6


int main(void)
{
  DDRA=0xFF;
  DDRB=0x0F;
  PORTA=0x00;
  PORTB=0x00;
  DDRC |=(1<<DDC6);
  PORTC |= (1<<PC6); //Empfangen
  
  //UART Config
  UBRRL=51; //Baudrate 9600  
  UCSRB|=(1<<RXCIE)|(1<<RXEN)|(1<<TXEN);
  UCSRC|=(1<<URSEL)|(1<<UCSZ1)|(1<<UCSZ0);
  
  
  char element[] = { 1, 2, 4, 8, 16, 32, 65, 128 };
  
    while(1)
    {
    
    if(element[8]==128)
    {
      UDR='A';
      PORTA=0x01;
    }
    else
    {
      UDR='B';
      PORTA=0x02;
    }
    _delay_ms(5000);
        
    }
}

dieses ist total simpel, jedoch wird bei der IF bedingung immer der else 
Zweig ausgeführt, obwohl das 8. Element 128 ist. Warum funktioniert das 
nicht?

[Karl H. (kbuchegg)]

Wir fangen in C bei 0 zu zählen an

Zähl mal mit

  char element[] = { 1, 2, 4, 8, 16, 32, 65, 128 };

                     0  |  |  |  |   |    |   |
                        1  |  |  |   |    |   |
                           2  |  |   |    |   |
                              3  |   |    |   |
                                 4   |    |   |
                                     5    |   |
                                          6   |
                                              7

Das Array hat zwar 8 Elemente. Aber der höchste Index ist 7

   if(element[8]==128)

Mööp. Es gibt kein Array-Element mit dem Index 8

[Peter II (Gast)]

Chris schrieb:
> obwohl das 8. Element 128 ist.

ist es nicht.

char element[] = { 1, 2, 4, 8, 16, 32, 65, 128 };

char[0] = 1
char[1] = 2
...
char[7] = 128

[Johann L. (gjlayde)]

Einfach Warnungen beachten:

<stdin>:1:1: warning: overflow in implicit constant conversion [-Woverflow]
 char element[] = { 1, 2, 4, 8, 16, 32, 65, 128 };
 ^

[Rufus Τ. F. (rufus)]

Johann L. schrieb:
> Einfach Warnungen beachten:

Das ist aber ein ganz anderes Problem und hat nichts mit dem Array 
oder dem Zugriff darauf zu tun.

[Johann L. (gjlayde)]

Rufus Τ. Firefly schrieb:
> Johann L. schrieb:
>> Einfach Warnungen beachten:
>
> Das ist aber ein ganz anderes Problem und hat nichts mit dem Array
> oder dem Zugriff darauf zu tun.

Nö, Warnungen nicht zu beachten ist quasi immer ein Problem!

Ins Array wird nämlich keine 128 geschrieben, sondern eine -128.  Mit 
ein paar mehr Zeilen sind wie Warnungen:

<stdin>: In function 'main':
<stdin>:6:3: warning: overflow in implicit constant conversion [-Woverflow]
   char element[] = { 1, 2, 4, 8, 16, 32, 65, 128 };
   ^
<stdin>:11:5: warning: comparison is always false due to limited range of data type [-Wtype-limits]
     if (element[8]==128)

Die Korrektur besteht also zunächst mal darin, einen unsigned char (oder 
uint8_t) für element[] zu verwenden.  Danach werden die Warnmeldungen 
zu:

<stdin>: In function 'main':
<stdin>:11:16: warning: array subscript is above array bounds [-Warray-bounds]
     if (element[8]==128)
                ^

Jeder Warnung zeigt ein Problem auf und weist in Richtung der Lösung. 
Nachdem alle Warnungen behoben sind, funktioniert zumindest dieser Teil 
wie er soll.

Wie kommst du darauf, daß diese Warnungen auf ganz andere Probleme 
zeigen?

[Rufus Τ. F. (rufus)]

Johann L. schrieb:
> Wie kommst du darauf, daß diese Warnungen auf ganz andere Probleme
> zeigen?

Du reitest auf dem im Array stehenden Wert herum, das ursprüngliche 
Problem des Threadstarters aber lag in seinem fehlerhaften Arrayzugriff 
über die Arraygrenzen hinaus.

[Johann L. (gjlayde)]

Rufus Τ. Firefly schrieb:
> Johann L. schrieb:
>> Wie kommst du darauf, daß diese Warnungen auf ganz andere Probleme
>> zeigen?
>
> Du reitest auf dem im Array stehenden Wert herum, das ursprüngliche
> Problem des Threadstarters aber lag in seinem fehlerhaften Arrayzugriff
> über die Arraygrenzen hinaus.

Nein, ich reite auf jeder Warnung herum, auch auf der, die auf den 
Zugriff jenseits der Arraygrenzen hinweist.

Übrigens wird der der Zugriff elament[8]==128 idR wegoptimiert da der 
Vergleich immer falsch ist (2. Warnung), der Zugriff stellt also 
zunächst kein ein Problem dar, da er nie ausgeführt wird.

[Peter K. (peterka2000)]

Einmal eine ganz Blöde Frage: Das ist doch ein char-Array. Muss ein Char 
nicht mit '' deklariert werden?

[troll (Gast)]

Peter K. schrieb:
> Einmal eine ganz Blöde Frage: Das ist doch ein char-Array. Muss ein Char
> nicht mit '' deklariert werden?

Nein. char blub[42] sagt erstmal nur ein Array aus 8-Bit-Zahlen, je nach 
Compiler signed oder unsigned. Da kannst du ganz normal Zahlen von (wenn 
unsigned) 0-255 reintun oder eben mit 'a', 'b', '0' usw. den ASCII-CODE 
einzelner Buchstaben.

[Peter K. (peterka2000)]

troll schrieb:
> Peter K. schrieb:
>> Einmal eine ganz Blöde Frage: Das ist doch ein char-Array. Muss ein Char
>> nicht mit '' deklariert werden?
>
> Nein. char blub[42] sagt erstmal nur ein Array aus 8-Bit-Zahlen, je nach
> Compiler signed oder unsigned. Da kannst du ganz normal Zahlen von (wenn
> unsigned) 0-255 reintun oder eben mit 'a', 'b', '0' usw. den ASCII-CODE
> einzelner Buchstaben.

Ok, ich bin auch kein C-Profi. Ist mir nur so beim ersten Blick 
aufgefallen.

[Rolf M. (rmagnus)]

Peter K. schrieb:
> troll schrieb:
>> Peter K. schrieb:
>>> Einmal eine ganz Blöde Frage: Das ist doch ein char-Array. Muss ein Char
>>> nicht mit '' deklariert werden?
>>
>> Nein. char blub[42] sagt erstmal nur ein Array aus 8-Bit-Zahlen, je nach
>> Compiler signed oder unsigned. Da kannst du ganz normal Zahlen von (wenn
>> unsigned) 0-255 reintun oder eben mit 'a', 'b', '0' usw. den ASCII-CODE
>> einzelner Buchstaben.
>
> Ok, ich bin auch kein C-Profi. Ist mir nur so beim ersten Blick
> aufgefallen.

Prinzipiell ist char für den Compiler nur ein Integer-Typ wie die 
anderen auch, mit der Ausnahme, daß er ohne [un]signed-Präfix nicht 
zwingend mit Vorzeichen sein muß.
Aber obwohl der Compiler es zulässt, einem char direkt eine Zahl 
zuzuweisen, ist es durchaus sinnvoll diesen Typ (also ohne signed oder 
unsigned davor) nur für Zeichen zu verwenden.

[Joachim D. (Firma: JDCC) (scheppertreiber)]

Eieiei ... was macht den die "65" in dem Array ? ;)