Run-Time Check Failur #3 the variabel 'Eingabe' is beeing used without being defined

Ich habe die Variable global definiert, also dürfte der Fehler garnicht kommen. Beim Kompilieren tritt kein Fehler auf.
Ich habe mal den Quelltext von der entsprechenden Funktion eingefügt.
Es handelt sich um ein Mitglieder-Verwaltungsprogramm, in der entsprechenden Funktion soll man neue Mitglieder einlesen können.

--------------------------------------------------------------------

void MitgliedNeu()
{
char Eingabe;
int Zaehler;

do
{
system("cls");
cout << "\n\tNeues Mitglied erfassen:";
cout << "\n\t~~~~~~~~~~~~~~~~~~~~~~~~";

cout << "\n\nBitte geben Sie nacheinander die gefragten Daten ein.\n\n";

for (Zaehler=0; Mitglied[Zaehler].Name[0] == ' ' && Zaehler < 50; Zaehler++)
{
cout << "";
cout << "Name:\t\t\t";
for (int x=0; x<20;x++)
{
cin >> Mitglied[Zaehler].Name[x];
}

cout << "\n\nVorname:\t\t";
for (int x=0; x<20;x++)
{
cin >> Mitglied[Zaehler].Vorname[x];
}

cout << "\n\nStrasse und Nr.:\t";
for (int x=0; x<35;x++)
{
cin >> Mitglied[Zaehler].StrasseNr[x];
}

cout << "\n\nPLZ:\t\t\t";
cin >> Mitglied[Zaehler].PLZ;

cout << "\n\nOrt:\t\t\t";
for (int x=0; x<20;x++)
{
cin >> Mitglied[Zaehler].Ort[x];
}

cout << "\n\nGeburtsjahr:\t\t";
cin >> Mitglied[Zaehler].GebJahr;

cout << "\n\nGeburtsmonat:\t\t";
cin >> Mitglied[Zaehler].GebMonat;

cout << "\n\nGeburtstag:\t\t";
cin >> Mitglied[Zaehler].GebTag;

cout << "\n\nMitglied seit (Monat):\t";
for (int x=0; x<10;x++)
{
cin >> Mitglied[Zaehler].MitgliedMon[x];
}

cout << "\n\nMitglied seit (Jahr):\t";
cin >> Mitglied[Zaehler].MitgliedJahr;

cout << "\n\nTelefonnr:\t\t";
cin >> Mitglied[Zaehler].Tel;

cout << "\n\nBeitrag gezahlt? (1, sonst 0):\t";
cin >> Mitglied[Zaehler].Beitrag;

cout << "\n\nMoechten Sie ein weiteres Mitglied einlesen?";
cin >> Eingabe;

}
}while (Eingabe == 'j');

Menu();
}

char Eingabe = 0;

Run-Time Check Failur #3 the variabel 'Eingabe' is beeing used without being defined

Die Fehlermeldung ist wohl etwas irreführend, definiert ist die Variable jedenfalls. Sie ist höchstens nicht initialisiert. Das aber auch nur, wenn die äussere for Schleife nicht mindestens einmal durchlaufen wird oder 'cin >> Eingabe' fehlschlägt. Ersteres kannst du dir selbst beantworten, indem du herausfindest, ob

Mitglied[Zaehler].Name[0] == ' '

vor dem ersten Durchlauf erfüllt ist.

sarah1986 schrieb:

Sobald ich der Variable bei der Deklaration einen Wert übergebe, funktioniert mein Programm nicht mehr, ich habe vorher ein Menu, und wenn man "1" drückt, soll die Funktion aufgerufen werden, allerdings passiert das dann nicht mehr, es öffnet sich dann erneut das Menu.

Das Verhalten kann ich ehrlich gesagt nicht nachvollziehen. Wahrscheinlich ist aber, dass du noch irgendwo anders undefiniertes Verhalten hast. Zudem kann ich den Sinn folgenden Ablaufs nicht erkennen:

do
{
    [...]
    for (Zaehler=0; Mitglied[Zaehler].Name[0] == ' ' && Zaehler < 50; Zaehler++)
    {
        [...]
        cin >> Eingabe;
    }
}while (Eingabe == 'j');

Wozu liest du Eingabe ein, wenn du in der unmittelbaren Schleife nicht darauf reagierst?

btw:
Nutze in Zukunft bitte Code-Tags, sonst geht die Formatierung verloren und der Code wird schwer lesbar.

Wenn ich das nicht programmieren müsste, würde ich es lassen. Da funktioniert einiges nicht, wenn jmd. lust hat, mir zu versuchen zu helfen, dann bitte ich um kurze Info, dann schick ich den Code mal zu. (sind über 500 Zeilen, also zum hier eintragen etwas viel).

Erstmal sieht die Konstruktion "for(x=0;x<WERT;++x) cin<<var[x];" zum Einlesen von String etwas gewöhnungsbedürftig aus - da solltest du besser std::string's für die Namen etc verwenden.

Würde ich gerne tun, aber ich bin noch ziemlich blutiger Anfänger, und soll eine Schulung zu dem Thema machen, da sollen bestimmte Inhalte vorkommen, und nicht sooo tief rein gehen. Aber trotzdem danke für die Tipps.

CStoll schrieb:

Zweitens ist die for()-Schleife überflüssig - schreib einfach deine Position am Anfang der do-while in eine Variable und erhöhe sie bei Bedarf.

Welche der Schleifen? Oder meinst du alle? Wie meinst du das mit, Position in Variable schreiben??????

CStoll schrieb:

Aktuell erwartet das Programm auf jeden Fall 50 Datensätze, solange nicht ein Mitgliedsname vorher schon mit ' ' begonnen hat.

Deshalb frage ich ja am Ende nach weiteren Datensätzen, die muss man ja nicht gleich anlegen.

CStoll schrieb:

(Plan B: Pack' die Mitglieder in einen STL-Container, dann brauchst du dir über den Speicher keine Gedanken zu machen)

Auch davon habe ich leider null Ahnung, und soll es für die Schulung nciht verwenden.

do
{
    [...]
    for (Zaehler=0; Mitglied[Zaehler].Name[0] == ' ' && Zaehler < 50; Zaehler++)
    {
        [...]
        cin >> Eingabe;
    }
}while (Eingabe == 'j');

Die for-Schleife zählt von 0 bis 50, ohne sich darum zu kümmern, was in "Eingabe" steht - die Bedingung der do-while wird erst überprüft, wenn sie damit fertig ist (nach dem Eingeben von 50 Datensätzen) - und wenn du an der Stelle 'j' eingegeben hast, fängt die for-Schleife von vorne an. Was du brauchst, sieht etwa so aus:

Zaehler=0;
do
{
  [...]
  cin>>Eingabe;
  ++Zaehler;
}while(Zaehler<50&&Eingabe=='j');

@Strings: Dazu deklarierst du die Elemente Name, Vorname,... deines Strukturtyps nicht als char[], sondern als std::string - dann verkürzt sich die Eingabe der Werte auf ein einfaches

cout<<"Name:";
cin>>Mitglieder[Zaehler].Name;

PS: Wenn du unbedingt mit char[] arbeiten willst, ist übrigens scanf immer noch besser als alles zeichenweise einzulesen.

Wo liegt der unterschied zwischen string und std::string?

Bei dem einen steht std:: davor und bei dem anderen nicht.
std::string ist ein Container aus der STL für Zeichenketten. Alle Container der STL befinden sich im std Namensraum. Was ein Namensraum ist und wie man auf Bezeichner aus diesem zugreifen kann, findest du auf Humes Homepage. Nur soviel sei gesagt, eine Möglichkeit ist, vor den Bezeichner foo:: zu schreiben, wobei foo der entsprechende Namensraum ist. Eine andere Möglichkeit ist, den Namensraum oder Container in den aktuellen Namensraum einzubinden, so dass du foo:: auch weglassen kannst.

CStoll schrieb:

Was für Fehler gab es denn?

Würde mich auch interessieren. Ich denke, es ist sinnvoller diese zu eliminieren, als die momentane Codestruktur beizubehalten.

CStoll schrieb:

Wenn du unbedingt mit char[] arbeiten willst, ist übrigens scanf immer noch besser als alles zeichenweise einzulesen.

Wobei hier getline die bessere Wahl ist, sofern es nur um Zeichenketten geht.