3. Warum geht mein Programm nicht :( ?

Warum geht mein Programm nicht :( ?

  • B

    ha es funzt danke ich musst die zeilen umbrüche im txt entfernen dann gings.
    Danke

    0
  • B

    *schnief* nun is das Programm fertig aber es ist das nächste Problem wenn ich mehr als 3 namen generieren will dann geht es nicht mehr es ging aber mal habe schon mit 1000 getestet es hat funktioniert. Ich hatte es nun ein bischen Anschaulich gemacht und nun geht es nicht mehr aber ich kann mir nicht erklären warum es nicht geht könnt ihr mir helfen?

    #include <cstdlib>
#include <iostream>
#include <fstream>
#include <string>
#include <stdio.h>
#include <vector>
#include <string>
#include <fcntl.h>
#include <io.h>
using namespace std;

ifstream testdatei;
ifstream testdatei2;
ifstream testdatei3;
string names;
string names2;
string names3;
vector<string> puffer;
vector<string> puffer2;
vector<string> puffer3;
int newtext;
int anzahl;
int i;
vector<long int> randomzahlen;
vector<long int> randomzahlen2;
vector<long int> randomzahlen3;
string generate;

void dateioeffnen ()  //weibliche namen öffnene
{

    testdatei.open ("wnamen.txt", ifstream::in);   //variable für die zu öffnende Datei

    if (0==testdatei)                                // sequenz für das öffnen der datei
    {
       cout << "Erste Datei Fehlerhaft benachrichtigen Sie den Administrator" << endl; //Fehler Benarichtigung 
    }
    else
    {
        cout << "Erste Datei erfolgreich geoeffnet"<<endl;
    }

}
void dateioeffnen2 ()	//männliche namen öffnen
{
	testdatei2.open ("mnamen.txt", ifstream::in);   //variable für die zu öffnende Datei

    if (0==testdatei2)                                // sequenz für das öffnen der datei
    {
       cout << "Zweite Datei Fehlerhafte benachrichtigen Sie den Administrator" << endl; //Fehler Benarichtigung 
    }
    else
    {
        cout << "Zweite Datei erfolgreich geoeffnet"<<endl;
    }
}

void dateioeffnen3 () //nachnamen öffnen
{
	testdatei3.open ("nachnamen.txt", ifstream::in);   //variable für die zu öffnende Datei

    if (0==testdatei3)                                // sequenz für das öffnen der datei
    {
       cout << "Dritte Datei Fehlerhafte benachrichtigen Sie den Administrator" << endl; //Fehler Benarichtigung 
    }
    else
    {
        cout << "Dritte Datei erfolgreich geoeffnet"<<endl;
    }
}
void dateilesen() //weibliche namen lesen 
{

    while(testdatei.good()) {            //while schleife liest datei zeile für zeile aus 
        getline(testdatei, names, ';');  // liest zeilen (zu lesende datei, zu speichernde variable, lese kriterien)
        puffer.push_back(names);          //ausgelesene datein in vector schreiben 
        //cout << names << endl;          // vector ausgabe 

    } 
}

void dateilesen2() //mänliche namen lesen
{

    while(testdatei2.good()) {            //while schleife liest datei zeile für zeile aus 
        getline(testdatei2, names2, ';');  // liest zeilen (zu lesende datei, zu speichernde variable, lese kriterien)
        puffer2.push_back(names2);          //ausgelesene datein in vector schreiben 
        //cout << names << endl;       // vector ausgabe 

    } 
}

void dateilesen3() //nachnamen lesen 
{

    while(testdatei3.good()) {            //while schleife liest datei zeile für zeile aus 
        getline(testdatei3, names3, ';');  // liest zeilen (zu lesende datei, zu speichernde variable, lese kriterien)
        puffer3.push_back(names3);          //ausgelesene datein in vector schreiben 
        //cout << names << endl;    // vector ausgabe

    } 
}   

void random ()                            //random funktion zum generien der zu schreibenden zeilen
{
     cout << "Bitte geben Sie die Anzahl der zu generierenden Namen ein." << endl; 
     cin >> anzahl;
     cout << "Die " << anzahl << " Namen werden generiert." << endl;
     srand (time(NULL));

//randomzeile für weibliche namen 
for (int x=0;x<anzahl;x++)                   // festlung der anzahl der zu generienden namen 
{
randomzahlen.push_back(rand()%puffer.size()); // Zahlen berreich festlegung
//cout << randomzahlen.at(x) << endl;                   
}

//randomzeile für männliche namen
for (int i=0;i<anzahl;i++)                   // festlung der anzahl der zu generienden namen 
{
randomzahlen2.push_back(rand()%puffer2.size()); // Zahlen berreich festlegung
//cout << randomzahlen2.at(i) << endl;                   
}

//randomzeile für nachnamen 
for (int k=0;k<anzahl;k++)                   // festlung der anzahl der zu generienden namen 
{
randomzahlen3.push_back(rand()%puffer3.size()); // Zahlen berreich festlegung
//cout << randomzahlen2.at(i) << endl;                   
}

}

void weiblichgenerieren() //weiblich namen generieren 
{

    fstream d; //stream "intetger"
    d.open("generate.csv", ios::out); //zu schreibende datei
   for ( int x = 0; x < anzahl; x++) d << puffer.at(randomzahlen.at(x)) << ";" << puffer3.at(randomzahlen3.at(x))<<endl;// zufallszeilen auslesen und in datei schreiben

    d.close(); //datei stream schließen
    ifstream LogFile("c:\\generate.csv");//überprüfen ob datei erstellt wurde 
    if(LogFile)
    cout << "Die Datei wurde erstellt bitte bennen Sie Die Datei um" << endl;
}

void maennlichgenerieren()
{

    fstream d;
    d.open("generate.csv", ios::out);
   for ( int x = 0; x < anzahl; x++) d << puffer.at(randomzahlen2.at(x)) << ";" << puffer3.at(randomzahlen3.at(x))<<endl;

    d.close();
    ifstream LogFile("c:\\generate.csv");
    if(LogFile)
    cout << "Die Datei wurde erstellt bitte bennen Sie Die Datei um" << endl;
}

int main(int argc, char *argv[])
{
cout << "~~~~~~~~Herzlich Willkommen beim Namensgenerator~~~~~~~~" << endl << endl; 

cout << "Bitte waehlen Sie aus, welches Geschlecht sie generieren moechten." << endl<< endl; 
anfang:;
cout << "Geben sie fuer Weiblich eine 1 ein oder fuer Maennlich eine 2." << endl;
int geschlechteingabe;
cout << "Bitte geben sie das gewuenschte Geschlecht an: " ; 
cin >> geschlechteingabe;
cout << endl;

if(geschlechteingabe < 1)
{
 cout << "!!!!!!!!ACHTUNG!!!!!!!!!"<<endl<<endl;
 cout << "Sie haben eine falsche Eingabe gemacht bitte versuchen sie es noch einmal." << endl<<endl;
 goto anfang;
}
 else 
 {
      if (geschlechteingabe > 2)
      {
       cout << "!!!!!!!!ACHTUNG!!!!!!!!!"<<endl<<endl;
       cout << "Sie haben eine falsche Eingabe gemacht bitte versuchen sie es noch einmal." << endl<<endl;
       goto anfang;
       }

	else 
	{
	if (geschlechteingabe=2)
		{
            cout<< " Sie haben das maennliche Geschlecht gewählt." << endl <<endl;
			dateioeffnen();
        	dateioeffnen2();
        	dateioeffnen3();
        	dateilesen();
        	dateilesen2();
        	dateilesen3();
			random();
			maennlichgenerieren();
		}
		else 
		{
			if (geschlechteingabe=1)
            {
                cout<< " Sie haben das weibliche Geschlecht gewählt." << endl <<endl;
            	dateioeffnen();
            	dateioeffnen2();
            	dateioeffnen3();
            	dateilesen();
            	dateilesen2();
            	dateilesen3();
            	random();
            	weiblichgenerieren();
                }
		}
	}
}

    system("PAUSE");
    return EXIT_SUCCESS;
}

    und das kommt dabei raus :

    http://www.bilder-hochladen.net/files/a72x-j-jpg.html

    Schon mal ein ganz dickes dann voraus.
    Mfg
    Barti

    0
  • Mod

    Das klingt doch nach einem idealen Problemfall um damit zu lernen, wie man einen Debugger benutzt. Damit sollte sich das Problem statt mit 1000 mal testen mit einmal testen lösen lassen. Ist dann auch ein sehr nützliches Wissen für zukünftige, komplexere Projekte.

    0
  • B

    ich weiss nicht wie das mit debuggen geht ... bzw was man dazu benutzt

    0
  • _

    Barti20 schrieb:

    ich weiss nicht wie das mit debuggen geht ... bzw was man dazu benutzt

    Hier kannst du dich schön einlesen in die Welt des Debuggings:

    http://magazin.c-plusplus.net/artikel/Debuggen mit VCPlusPlus6

    Bezieht sich zwar auf VS6, aber an den Debugging-Mechanismen hat sich wenig geändert.

    0
  • B

    hmm da is nix bei raus gekommen er meint es is alles richtig und nun ?

    0
  • _

    Barti20 schrieb:

    hmm da is nix bei raus gekommen er meint es is alles richtig und nun ?

    Wie meinst du das? Was genau hast du denn jetzt ausprobiert?

    0
  • B

    ich habe mit dem debugger ding von dev c++ jede zeile einzeln überprüft aber er hat mir nirgends gesagt das da irgendwo nen fehler is ich habe nun auch einmal die zeilen auskommentiert wo ich mir gedacht habe das es daran liegen könnte aber es passiert das gleich ...

    0
  • M

    Barti20 schrieb:

    ich habe mit dem debugger ding von dev c++ jede zeile einzeln überprüft aber er hat mir nirgends gesagt das da irgendwo nen fehler is

    Die Fehlerprüfung musst du schon selbst machen. Der Debugger sorgt nur dafür, dass du in Einzelschritten durch dein Programm gehen und dir die Variablenwerte ansehen kannst. Das Denken übernimmt er nicht für dich.

    0
  • _

    Wikipedia bringt es auf den Punkt:

    Wikipedia schrieb:

    Funktionen eines Debuggers:

    * die Steuerung des Programmablaufs, insbesondere durch Haltepunkte und die Einzelschritt-Verarbeitung von Befehlen
    * das Inspizieren von Daten, z. B. die Register, den aktuellen Programmcode als Assembler oder Hochsprachenquelltext, den allgemeinen Daten in festen und flüchtigen Speichern, der Erzeugung von fortgeschrittenen Daten-Interpretationen etwa durch eine Callstack-Funktionalität oder das Anzeigen von Ein-/Ausgabe-Registern, Tabellen und Hochsprachen-Strukturen
    * das Modifizieren von Speichern, z. B. des Hauptspeichers, der externen Ein-/Ausgabe-Zustände und der Register des Prozessorkerns

    http://de.wikipedia.org/wiki/Debugger

    0
  • B

    hmm ok ich habs hin bekommen geht alles nun habe ich schon wieder ein problem...
    Ich glaube langsam werde ich für euch lästig...
    Hatte ja oben die geschlechts Abfrage mit 1 oder 2 abgefragt sollte da jetzt ein buchstabe eingegeben werden verfällt das programm in eine endlisch schleife ich habe das problem jetzt so gelöst :

    if(geschlechteingabe[1] != "w"||"m")
{

 cout << "!!!!!!!!ACHTUNG!!!!!!!!!"<<endl<<endl;
 cout << "Sie haben eine falsche Eingabe gemacht, bitte versuchen sie es noch einmal." << endl<<endl;
 goto anfang; // goto marke setzen
}

    geschlechtseingabe ist nen array...
    also so habe ich ihn gemacht :

    string geschlechteingabe [10];

    aber jetzt ist das Problem auch wenn ich w oder m eingebe verlässt er nicht die if funktion.

    Woran kann das liegen?

    0
  • _

    Barti20 schrieb:

    if(geschlechteingabe[1] != "w"||"m")

    Das geht so nicht (so eine Sprache ist eine Programmiersprache nicht 😉 ). Du musst folgendermaßen notieren:

    if(geschlechteingabe[1] != "w" && geschlechteingabe[1] != "m")

    Und von goto würde ich dir abraten. Mach es lieber mit einer vernünftigen Schleife (mache... solange die Eingabe nicht "w" oder "m" ist => also eine do-while-Schleife).

    0
  • B

    ok danke 🙂 habs jetzt ach mit einer while do schleife gemacht 🙂

    0
  • B

    Wie kann ich verhindern wenn in einen Integer buchstaben eingegeben werden, dass das Programm in eine Endlosschleife springt ?
    Sonder statdessen eine Fehler meldung kommt?
    Wie muss ich daran gehen?
    Also Problem ist wenn man bei mir im Programm bei Anzahl einen Buchstaben eingibt dann wird die Fehler meldung von der If Funktion ausgegeben und das ganze in einer Endlosschleife.
    Die Variable anzahl ist ein Integer.
    So siehts aus:

    void random ()                            //random funktion zum generien der zu schreibenden zeilen
{
     anfang:;
     cout << "Bitte geben Sie die Anzahl der zu generierenden Namen ein." << endl; 
     cout << "Anzahl: ";
     cin >> anzahl;

     if(anzahl<=0)
     {
     cout << endl; 
     cout << "_______________________________________________________________________________" <<endl << endl;
     cout << "!!!!!!!!!!FEHLER!!!!!!!!!!!!!!!" << endl << endl; 
     cout<< "Die eingegebene Anzahl liegt unter 0 bitte Versuchen Sie es noch einmal."<<endl <<endl;
     cout << "_______________________________________________________________________________" <<endl << endl;
     goto anfang;
     }
     cout << "Die " << anzahl << " Namen werden generiert." << endl;
     srand (time(NULL));

//randomzeile für weibliche namen 
for (int x=0;x<anzahl;x++)                   // festlung der anzahl der zu generienden namen 
{
randomzahlen.push_back(rand()%puffer.size()); // Zahlen berreich festlegung
//cout << randomzahlen.at(x) << endl;                   
}

    und ja ich weiß goto Befehle sind net gut 🙂 Das wird auch noch geändert wenn der rest funktioniert 🙂
    Schon mal ein Danke im Voraus

    0
  • _

    Barti20 schrieb:

    Wie kann ich verhindern wenn in einen Integer buchstaben eingegeben werden, dass das Programm in eine Endlosschleife springt ?

    Du musst den Eingabepuffer löschen (=>Suchfunktion).

    0
  • B

    hmm das klappt nicht bleibt das gleich weder mit
    cin.clear();
    noch
    cin.ignore(cin.rdbuf()->in_avail());
    hat mein problem gelöst. 😞

    0
  • _

    Probier's mal so:

    std::cin.clear();
std::cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
    0
  • B

    ahh super es geht.
    Aber was genau machen die beiden zeilen ?
    Würd schon gern wissen wie mein Programm funktioniert 😛

    0
  • _

    Barti20 schrieb:

    ahh super es geht.
    Aber was genau machen die beiden zeilen ?
    Würd schon gern wissen wie mein Programm funktioniert 😛

    Hier findest du alle Antworten: http://www.cppreference.com/wiki/io/start?s[]=cin&s[]=clear

    0
Anmelden zum Antworten