Ich hab ein kleines Lotto-Programm geschrieben. Zweck des Programms ist es, nach der Eingabe von z.B. 12 verschiedenen Zahlen, alle Möglichen 6er-Zahlenkombinationen aufzulisten.

Beispiel:

Meine gewählten Zahlen sind: 1, 2, 3, 4, 5, 6, 7, 8, 9, 10 ,11, 12

Die Ausgabe des Programms:

Kombination Nr.1: <1> <2> <3> <4> <5> <6>
Kombination Nr.2: <1> <2> <3> <4> <5> <7>
Kombination Nr.3: <1> <2> <3> <4> <5> <8>
Kombination Nr.4: <1> <2> <3> <4> <5> <9>
Kombination Nr.5: <1> <2> <3> <4> <5> <10>
Kombination Nr.6: <1> <2> <3> <4> <5> <11>
Kombination Nr.7: <1> <2> <3> <4> <5> <12>
Kombination Nr.8: <1> <2> <3> <4> <6> <7>
Kombination Nr.9: <1> <2> <3> <4> <6> <8>
Kombination Nr.10: <1> <2> <3> <4> <6> <9>
Kombination Nr.11: <1> <2> <3> <4> <6> <10>
Kombination Nr.12: <1> <2> <3> <4> <6> <11>
Kombination Nr.13: <1> <2> <3> <4> <6> <12>
Kombination Nr.14: <1> <2> <3> <4> <7> <8>

.... usw bis
Kombination Nr.X: <7> <8> <9> <10> <11> <12>

Mein Problem:

Bei der Ausgabe von z.B. 9 oder 10 vorher eingegeben Zahlen funktioniert alles ohne Probleme.
Bei der Eingabe von 11 Zaheln und mehr, fängt mein Problem an. Ab einer Kombination Nr.xy werden alle vorher aufgelisteten Kombination von den neuen Kombinationen überschrieben.

Z.B wird Kombination Nr.1 gelöscht sobald Kombination Nr.xy+1 ausgegeben wird und Kombination Nr.2 wird gelöscht, sobald Kombination Nr.xy+2 hinzukommt.

Ich habe seit mehereren Tagen alles ausprobiert, aber ich krieg es nicht hin alle Kombinationen aufzulisten.

Ich muss auch erwähnen, dass ich ein absoluter Neuling bin, was auch evtl. meine Denk- bzw Programmierfehler erklärt

mfg

Das ist in der Tat ein merkwüdiges Verhalten. Noch merwürdiger wäre aber, wenn jemand ohne ein Stück Code dazu etwas sagen könnte....

Also ein bißchen mehr Informationen können nicht schaden, denn Du hast nur über die Funktionalität Deines Programms Auskunft gegeben, nicht über die Realisierung.

Wenn Du den Code nicht preisgeben willst, dann probier es mit einem Debugger....

Grüße
Richie

P.S.: Vor dem Posten den Code aufs Wesentliche kürzen. Vielleicht entdeckst Du dann sogar selber den Fehler...

Die folgendes könnes sollte. Übergenen wird ein vector mit einer beliebegen anzahl von elementen (überprüfung auf doppelte!) und als 2 parameter anzahl der kombinations größe.

void ausgabe_kombinationen(vector<int> Zahlenfeld,int kombigroesse);

P.S. wenn du es nicht schaffst einfach sagen, dann post ich dir den restlichen code, aber lieber mal selber probieren

mfg spjoe

Irgendwie finde ich das der Fehler überall sein könnte, deswegen poste ich den gesamten Code rein:

#include <iostream.h>

int Stelle1, Stelle2, Stelle3, Stelle4, Stelle5, Stelle6;
int max6, max5, max4, max3, max2;

int main()

{	int i;
	int Anfangszahlen;
	cin >> Anfangszahlen;
	float * Zahlen = new float[Anfangszahlen];
	Zahlen[0] = 12.4f;
	delete [] Zahlen;
	cout << "\n";

	for (i=0; i<Anfangszahlen; i++) 
	{
		cout << "Zahl Nr." << i+1 << ": ";
		cin >> Zahlen[i];
	}

	cout << "\n";

Stelle1=0;
Stelle2=1;
Stelle3=2;
Stelle4=3;
Stelle5=4;
Stelle6=5;
max6=Anfangszahlen-1;
max5=Anfangszahlen-2;
max4=Anfangszahlen-3;
max3=Anfangszahlen-4;
max2=Anfangszahlen-5;

	{

		{while (Stelle6<=max6)

				{	
								{
									cout << "\t" << " <" << Zahlen[Stelle1] << "> " 
										 << "\t" << " <" << Zahlen[Stelle2] << "> " 
										 << "\t" << " <" << Zahlen[Stelle3] << "> " 
										 << "\t" << " <" << Zahlen[Stelle4] << "> "
										 << "\t" << " <" << Zahlen[Stelle5] << "> "
										 << "\t" << " <" << Zahlen[Stelle6] << "> "; 
									cout << "\n";
									Stelle6=Stelle6+1;
								}

				}

		}	

	}

	{Stelle5=Stelle5+1;
	 Stelle6=Stelle5+1;

	 {while (Stelle6<=max6)

				{	
								{
									cout << "\t" << " <" << Zahlen[Stelle1] << "> " 
										 << "\t" << " <" << Zahlen[Stelle2] << "> " 
										 << "\t" << " <" << Zahlen[Stelle3] << "> " 
										 << "\t" << " <" << Zahlen[Stelle4] << "> "
										 << "\t" << " <" << Zahlen[Stelle5] << "> "
										 << "\t" << " <" << Zahlen[Stelle6] << "> "; 
									cout << "\n";
									Stelle6=Stelle6+1;
									if (Stelle6>max6)
									{Stelle5=Stelle5+1;
									Stelle6=Stelle5+1;}
								}	
				}
	 }

	}

	{Stelle4=Stelle4+1;
	 Stelle5=Stelle4+1;
	 Stelle6=Stelle5+1;

		{while (Stelle6<=max6)

				{	
								{
									cout << "\t" << " <" << Zahlen[Stelle1] << "> " 
										 << "\t" << " <" << Zahlen[Stelle2] << "> " 
										 << "\t" << " <" << Zahlen[Stelle3] << "> " 
										 << "\t" << " <" << Zahlen[Stelle4] << "> "
										 << "\t" << " <" << Zahlen[Stelle5] << "> "
										 << "\t" << " <" << Zahlen[Stelle6] << "> "; 
									cout << "\n";
									Stelle6=Stelle6+1;
									if (Stelle6>max6)
									{Stelle5=Stelle5+1;
									Stelle6=Stelle5+1;}
									if (Stelle5>max5)
									{Stelle4=Stelle4+1;
									 Stelle5=Stelle4+1;
									 Stelle6=Stelle5+1;}
								}	
				}
		}
	}

	{Stelle3=Stelle3+1;
	 Stelle4=Stelle3+1;
	 Stelle5=Stelle4+1;
	 Stelle6=Stelle5+1;

		{while (Stelle6<=max6)

				{	
								{
									cout << "\t" << " <" << Zahlen[Stelle1] << "> " 
										 << "\t" << " <" << Zahlen[Stelle2] << "> " 
										 << "\t" << " <" << Zahlen[Stelle3] << "> " 
										 << "\t" << " <" << Zahlen[Stelle4] << "> "
										 << "\t" << " <" << Zahlen[Stelle5] << "> "
										 << "\t" << " <" << Zahlen[Stelle6] << "> "; 
									cout << "\n";
									Stelle6=Stelle6+1;
									if (Stelle6>max6)
									{Stelle5=Stelle5+1;
									Stelle6=Stelle5+1;}
									if (Stelle5>max5)
									{Stelle4=Stelle4+1;
									 Stelle5=Stelle4+1;
									 Stelle6=Stelle5+1;}
									if (Stelle4>max4)
									{Stelle3=Stelle3+1;
									 Stelle4=Stelle3+1;
									 Stelle5=Stelle4+1;
									 Stelle6=Stelle5+1;}
								}	
				}
		}
	}

	{Stelle2=Stelle2+1;
	 Stelle3=Stelle2+1;
	 Stelle4=Stelle3+1;
	 Stelle5=Stelle4+1;
	 Stelle6=Stelle5+1;

		{while (Stelle6<=max6)

				{	
								{
									cout << "\t" << " <" << Zahlen[Stelle1] << "> " 
										 << "\t" << " <" << Zahlen[Stelle2] << "> " 
										 << "\t" << " <" << Zahlen[Stelle3] << "> " 
										 << "\t" << " <" << Zahlen[Stelle4] << "> "
										 << "\t" << " <" << Zahlen[Stelle5] << "> "
										 << "\t" << " <" << Zahlen[Stelle6] << "> "; 
									cout << "\n";
									Stelle6=Stelle6+1;
									if (Stelle6>max6)
									{Stelle5=Stelle5+1;
									Stelle6=Stelle5+1;}
									if (Stelle5>max5)
									{Stelle4=Stelle4+1;
									 Stelle5=Stelle4+1;
									 Stelle6=Stelle5+1;}
									if (Stelle4>max4)
									{Stelle3=Stelle3+1;
									 Stelle4=Stelle3+1;
									 Stelle5=Stelle4+1;
									 Stelle6=Stelle5+1;
									if (Stelle3>max3)
									{Stelle2=Stelle2+1;
									 Stelle3=Stelle2+1;
									 Stelle4=Stelle3+1;
									 Stelle5=Stelle4+1;
									 Stelle6=Stelle5+1;}
								}	
				}
		}
	}

	{Stelle1=Stelle1+1;
	 Stelle2=Stelle1+1;
	 Stelle3=Stelle2+1;
	 Stelle4=Stelle3+1;
	 Stelle5=Stelle4+1;
	 Stelle6=Stelle5+1;

		{while (Stelle6<=max6)

				{	
								{
									cout << "\t" << " <" << Zahlen[Stelle1] << "> " 
										 << "\t" << " <" << Zahlen[Stelle2] << "> " 
										 << "\t" << " <" << Zahlen[Stelle3] << "> " 
										 << "\t" << " <" << Zahlen[Stelle4] << "> "
										 << "\t" << " <" << Zahlen[Stelle5] << "> "
										 << "\t" << " <" << Zahlen[Stelle6] << "> "; 
									cout << "\n";
									Stelle6=Stelle6+1;
									if (Stelle6>max6)
									{Stelle5=Stelle5+1;
									Stelle6=Stelle5+1;}
									if (Stelle5>max5)
									{Stelle4=Stelle4+1;
									 Stelle5=Stelle4+1;
									 Stelle6=Stelle5+1;}
									if (Stelle4>max4)
									{Stelle3=Stelle3+1;
									 Stelle4=Stelle3+1;
									 Stelle5=Stelle4+1;
									 Stelle6=Stelle5+1;
									if (Stelle3>max3)
									{Stelle2=Stelle2+1;
									 Stelle3=Stelle2+1;
									 Stelle4=Stelle3+1;
									 Stelle5=Stelle4+1;
									 Stelle6=Stelle5+1;
									if (Stelle2>max2)
									{Stelle1=Stelle1+1;
									 Stelle2=Stelle1+1;
									 Stelle3=Stelle2+1;
									 Stelle4=Stelle3+1;
									 Stelle5=Stelle4+1;
									 Stelle6=Stelle5+1;}
								}	
				}
		}
	}
		}
	}
	}

}

mfg spjoe

Ich habe es mir jetzt nicht genauer angeschaut, aber an welcher Stelle gibst Du denn den Speicher frei

Was ich damit meine ist, dass Du delete viel zu früh aufrufst. Du gibst den reservierten Speicherbereich wieder frei.

Ich habe es nicht weiter getestet, aber so etwas kann sehr unschönes Programmverhalten auslösen.

Ansonsten gewöhne Dir ein paar mehr Funktionen an, oder besser, wenn Du schon C++ benutzt Objktorientierte Programmierung.

Grüße
Richie

Ok ich werde mir eure Tipps mal genauer anschaun und versuchen diese in mein Programm umzusetzen.

Wo meinst du ist es am besten den Speicherbereich wieder freizugeben.

Danke

mfg spjoe

Ein delete solltest Du dann aufrufen, wenn Du das Objekt nicht mehr benötigst.
D.h.: Hier am Besten am Ende des Programms als letzte Anweisung in main.

Grüße
Richie

ohne berücksichtigung der reihnfolge(lotto) ist die formel für die anzahl der möglichkeiten: n über k, sprich 12 über 6 = 924

mfg spjoe

Genau das ist richtig.

Richie_Gecco schrieb:

Hi!

Ein delete solltest Du dann aufrufen, wenn Du das Objekt nicht mehr benötigst.
D.h.: Hier am Besten am Ende des Programms als letzte Anweisung in main.

Grüße
Richie

Ok habe es geändert. Leider besteht das Problem immer noch.

Kannst Du bitte nochmal genau beschreiben, was Dein Problem ist?

Ich habe das gerade mal mit Watcom durchkompiliert. Nach der delete Änderung sieht die Ausgabe recht gut aus.
Das Einzige, was anzumerken wäre, dass bei mir die Windows-Konsole die ersten Ausgaben wegschneidet. Aber das ist normal.

Kannst Du bitte mal ganz genau erklären, was Du mit löscht die ersten Kombinationen meinst?

Grüße
Richie

Das Einzige, was anzumerken wäre, dass bei mir die Windows-Konsole die ersten Ausgaben wegschneidet. Aber das ist normal.

Genau das war/ist mein Problem- Aber wenns normal ist, muss ich wohl damit leben. Oder gibt es einen Ausweg, dass ich alle Kombinationen trotzdem irgendwie aufgelistet bekomme.

Du kannst die größe der Konsole ändern (unter XP getestet). Und zwar links klick auf das Symbol in der oberen linken Ecke. Dort Eigenschaften -> Layout und unter Fensterpuffergrüße kannst Du die Anzahl von Zeilen eingeben.

Ansonsten Ausgabe in eine Datei umlenken.

D.h. In der Konsole:
Programmname > Ausgabe.txt

Dann kannst Du das bequem weiterverwerten.

Grüße
Richie

Danke auch an alle anderen, die mir geholfen haben.

mfg

Und bleib am Ball!