C++ Probleme mit Variablen

[Matze (Gast)]

Hallo Community,

ich finde meinen Fehler einfach nicht.

Folgender Code erzeugt folgende Ausgabe:

#include <iostream>
#include <cstdlib>
using namespace std;

int main()
{
    unsigned int eancode;
    unsigned int ziffern[12];

    cout << "Bitte 13-steilligen EAN-Code eingeben" << endl;
    cin >> eancode; //Beispiel: 40 1270096310 9
    cout << "Folgender EANCode wurde eingegeben:" << eancode << endl;
    cout << "#################################" << endl;

    int i = 0;
    while(i!=12)
    {
        ziffern[i] = eancode%10;
        eancode/=10;
        cout<<"Ziffer " << i << "->" << ziffern[i]<<endl;
        i++;
    }

    system("PAUSE");
    return 0;
}

Bitte 13-steilligen EAN-Code eingeben
4012700963109 <- Eingegeben und Enter
Folgender EANCode wurde eingegeben: 4294967295
################################
Ziffer 0->5
Ziffer 1->9
Ziffer 2->2
Ziffer 3->7
Ziffer 4->6
Ziffer 5->9
Ziffer 6->4
Ziffer 7->9
Ziffer 8->2
Ziffer 9->4
Ziffer 10->0
Ziffer 11->0
Drücken Sie eine beliebige Taste . . .

Ich versteh den Fehler einfach nicht. Bitte helft mir!

Gruß Matze

[Peter II (Gast)]

unsigned int eancode;
cin >> eancode;

wie groß darf int wohl maximal werden?

[Matze (Gast)]

4294967295 < 4012700963109.

Mist! Vielen Dank fürs Augen öffnen.

Gruß,
Matze

[Karl H. (kbuchegg)]

Und aufpassen. Nicht alles was 'Code' oder 'Nummer' heisst, ist auch 
tatsächlich numerisch.
Eine Telefon-Nummer heisst zwar 'Nummer', trotzdem ist es unklug dafür 
einen int oder long zum Speichern zu nehmen. Sieh derartige Dinge besser 
als String an. Dann klappt das zb auch mit der EIngabe von

[phone] / 09

und dein Benutzer ist happy, wenn er Telefonnummern so eingeben kann, 
wie er es gewohnt ist.

Selbiges auch hier mit

40 1270096310 9

Das ist recht offensichtlich nicht mehr einfach nur eine Nummer.

Selbiges zb bei Kreditkartennummern oder EBAN, PIC

[Matze (Gast)]

Danke auch für deine Info! Ich habe die Aufgabenstellung dann wohl etwas 
komplizierter gelöst, als nötig... vielleicht wäre Char wohl die 
einfachere Lösung gewesen.

Falls es jemanden interessiert, hier die Lösung meiner Aufgabe.

#include <iostream>
#include <cstdlib>
using namespace std;

int main()
{
    unsigned __int64 eancode;
    unsigned __int64 ziffern[12];

    cout << "Bitte 13-steilligen EAN-Code eingeben" << endl;
    cin >> eancode; //Beispiel: 40 1270096310 9
    //eancode = 4012700963109; // kann einkommentiert werden, wenn man keine Lust hast die Zahl von Hand einzugeben
    cout << "Folgender EANCode wurde eingegeben:" << eancode << endl;
    cout << "#################################" << endl;

    int i = 0;
    while(i<13) 
    {
        ziffern[i] = eancode%10;
        if(i == 12)
        {
        //Wenn wir bei der letzten Stelle angekommen sind, dürfen wir natürlich den EANCode nicht mehr durch 10 teilen!!!
        }
        else{
        eancode/=10;
        }
        cout<<"Ziffer " << i << "->" << ziffern[i]<<endl; // diese Zeile kann auskommentiert werden, wenn gewünscht!
        i++;
    }

    int summe1;
    int summe2;
    summe1 = (ziffern[12] + ziffern[10] + ziffern[8] + ziffern[6] + ziffern[4] + ziffern[2]); //Summe 1 berechnen
    summe2 = ((ziffern[11] + ziffern[9] + ziffern[7] + ziffern[5] + ziffern[3] + ziffern[1]) * 3); //Summe 2 berechnen
    int gesSumme = summe1+summe2; //Gesamtsumme berechnen
    cout << "Berechne Summe1:" << summe1 << endl;
    cout << "Berechne Summe2:" << summe2 << endl;
    cout << "Berechne Gesamtsumme:" << gesSumme << endl;
    cout << "#############################" << gesSumme << endl;

    //gültige Prüfsumme berechnen
    if(((gesSumme + ziffern[0])%10) == 0) //lässt sich die Gesamtsumme + die Prüfziffer durch 10 ohne Rest teilen, ist die Prüfsumme gültig!!
    {
    cout << "Prüfsumme gueltig!!:" << endl;
    }
    else
    {
    cout << "Prüfsumme ungueltig!!:" << endl;
    }


    system("PAUSE");
    return 0;
}

[Karl H. (kbuchegg)]

Matze schrieb:
> Danke auch für deine Info! Ich habe die Aufgabenstellung dann wohl etwas
> komplizierter gelöst, als nötig... vielleicht wäre Char wohl die
> einfachere Lösung gewesen.

wenn schon, dann ein char Array.
Aber da wir hier bei C++ sind, wäre ein std::string in Kombination mit 
der getline Funktion das Mittel der Wahl.

[nicht"Gast" (Gast)]

Hallo,

was passiert eigentlich, wenn ich ausd versehen zu wenig Stellen 
eingebe? Nur so als Denkanstoß.

Grüße,

[Karl H. (kbuchegg)]

Matze schrieb:

>     unsigned __int64 ziffern[12];

Eine einzelne Ziffer kann maximal den WErt 9 haben!

>         ziffern[i] = eancode%10;

Dafür ist ein __int64 dann schon reichlich gross. Hier hätte es ein 
einfacher int auch getan. Selbst auf einem PC.

[Dr. Sommer (Gast)]

Matze schrieb:
> unsigned __int64 eancode;
>     unsigned __int64 ziffern[12];

Warum verwendest du diese nicht standardisierten Datentypen, anstelle 
von int64_t laut C++ Standard? Warum verwendest du 64bit-Integer für 
einzelne Ziffern, und nicht zB uint8_t (8-Bit-Integer)?

[Matze (Gast)]

Karl Heinz schrieb:
> Matze schrieb:
>
>>     unsigned __int64 ziffern[12];
>
> Eine einzelne Ziffer kann maximal den WErt 9 haben!
>
>>         ziffern[i] = eancode%10;
>
> Dafür ist ein __int64 dann schon reichlich gross. Hier hätte es ein
> einfacher int auch getan. Selbst auf einem PC.

Ja, sorry. Hab ich direkt nach dem Posten auch gesehen, dass das 
Blödsinn ist.

Dr. Sommer schrieb:
> Matze schrieb:
>> unsigned __int64 eancode;
>>     unsigned __int64 ziffern[12];
>
> Warum verwendest du diese nicht standardisierten Datentypen, anstelle
> von int64_t laut C++ Standard? Warum verwendest du 64bit-Integer für
> einzelne Ziffern, und nicht zB uint8_t (8-Bit-Integer)?

Daran hat wohl Google Schuld ;-) Ich hab einfach c++ Datentypen 
gegoogelt und mir den erstbesten Link angeschaut und dort war für mich 
__int64 angegeben. Ich bin mit den Standards von C++ noch nicht ganz so 
bewandert und fange tatsächlich erst an diese Sprache zu lernen.

Aber Danke für deine Anmerkung, ich werd mir wohl mal die Standardtypen 
in einem C++ Buch zu Gemüte führen.