zu viele == in if-bedingung??? --KEIN spaßpost! [gelöst]

[Th69]

Hallo andi,

na dann viel Spaß beim Tippen, wenn du wirklich auf diese Weise alle 4er-Kombinationen abfragen willst

Wie in deinem anderen Beitrag schon vorgeschlagen wurde, benutze Arrays und prüfe mittels Schleifen deine Bedingungen: sind dann weniger als 10 Zeilen!!!

[andi01]

@DStefan:
ja, hat er. ich habe es auh dank ihm verstanden.mich hatte nur interessiert ob das bei != dasselbe ist, jetzt weiß ichs ja
@Th69:
natürlich löse ich das mit Schleifen , aber trotzdem werde ich um den einen oder anderen vergleich mit den nachbarstellen nicht drumherumkommen .

@all:
wer sich für die fertige lösung interessiert(alles außer die funktion gegen den pc zu spielen ist jezt fertig):

.cpp: http://rapidshare.com/files/272969298/Vier_Gewinnt.cpp.html
.exe: http://rapidshare.com/files/272969549/Vier_Gewinnt.exe.html

danke für die ganzen antworten ,
andi01.

[andi01]

so hab sie jetzt soweit fertig gekürzt auf 351 Zeilen:

.cpp: http://rapidshare.com/files/272988186/VierGewinnt.cpp.html
.exe: http://rapidshare.com/files/272988290/VierGewinnt.exe.html

mfg,
andi01.

[Th69]

Und wenn du jetzt noch die if-Orgien in Schleifen umwandelst, dann könnte es langsam was werden...

P.S: Du darfst auch Funktionen mit Parametern erstellen - dann brächtest du nicht doppelten Code in spielerzug() und spielerzug2() -)

[andi01]

so hab jetzt auch noch das mi den parametern eingefügt, bloß die if-anweisungen sin meiner meinung nach eigentlich nicht mehr zu kürzen.

edit: hat sich erledigt, alle fehler gefunden! zetzt noch 316zeilen!

hier nochmal die aktuellesten dateien:

.cpp: http://rapidshare.com/files/273016960/VierGewinnt.cpp.html
.exe: http://rapidshare.com/files/273019320/VierGewinnt.exe.html

nur wie soll ich die if-bedingungen noch stärker kürzen???

mfg,
andi01.

[camper]

if(position[5][a]==' ')
...
		else
		{
		if(position[5][a]!=' '&&position[4][a]==' ')
...		else
		{
		if((position[5][a]!=' ')&&(position[4][a]!=' ')&&(position[3][a]==' '))

Die Bedingungen werden unnötig komplex. In der 2. if-Anweisung wissen wir von vornherein, dass die Bedingung position[5][a]!=' ' immer erfüllt ist, andernfalls würde diese Anweisung nicht ausgeführt werden. Damit reduziert sich das Ganze zu

if(position[5][a]==' ')
			position[5][a]=farbe;
		else if(position[4][a]==' ')
			position[4][a]=farbe;
		else if(position[3][a]==' ')
			position[3][a]=farbe;
		else if(position[2][a]==' ')
			position[2][a]=farbe;
		else if(position[1][a]==' ')
			position[1][a]=farbe;
		else if(position[a][a]==' ')
			position[a][a]=farbe;

Scheint so, als ob die letzte Bedingung fehlerhaft ist (sollte sicher Position[0][a]) und das springt hier direkt ins Auge. Abgesehen davon ist die Struktur so regeläßig, dass die Vereinfachung mittels Schleife offensichtlich sein sollte. Noch besser wäre möglicherweise die Nutzung eines zusätzlichen Feldes, dass sich den Füllstand jeder Spalte merkt. Auf die Weise müssten wir gar nicht mehr suchen, wo der nächste Stein landen würde.

[andi01]

was die letzte bedingung betrifft: jo, habe ich bereits ausgebessert. ich ändere das mit dem if noch schnell und uploade dann nochmal den code.

mfg,
andi01.

[andi01]

so leiderhab ich jetzt einen fehler gefunden: bei der eingabereihe wenn man den stein in position 5 3 einwirft gehts nich, aber das kriege ich sicher noch hin(vor dem umschreiben der if-bedingungen war der aber auch schon).

hier der komplette Code:

.cpp: http://rapidshare.com/files/273036003/VierGewinnt.cpp.html
.exe: http://rapidshare.com/files/273039252/VierGewinnt.exe.html

und hier der verbesserte ausschnitt:

if(reihe==a)
	{
		if(position[5][a]==' ')
		{
			position[5][a]=farbe;
		}
		else if(position[4][a]==' ')
		{
			position[4][a]=farbe;
		}
		else if(position[3][a]==' ')
		{
			position[3][a]=farbe;
		}
		else if(position[2][a]==' ')
		{
			position[2][a]=farbe;
		}
		else if(position[1][a]==' ')
		{
			position[1][a]=farbe;
		}
		else if(position[0][a]==' ')
		{
			position[0][a]=farbe;
		}
	}
	}

ich hoffe ich finde diesen letzten fehler auch noch.

mfg,
andi01.

[Th69]

Und hier als Schleife:

int i=5
while(i >= 0 && position[i][a] != ' ')
  i--;

// oder alternativ: int i; for(i=5; i >= 0 && position[i][a] != ' '; i--);

position[i][a] = farbe;

[andi01]

edit: habe den Fehler jetzt gefunden! der Code funktioniert perfekt!

danke für die ganzen Antworten ,
andi01.