#include<iostream>
#include<conio.h>
using namespace std;

int main()
{
int a=3, b=3, c=3;
if(a==b==c)
{
cout<<"test\n";
}
getch();
}

der Code compilert fehlerfrei und gibt auch keine warnings etc. aus.

im viergewinntcode is das genau dasselbe nur mit noch mehr == hintereinander und anderen variablennamen, deshalb sollte dieser codeschnipsel als beispiel für das problem ausreichen.

Wie löst man dieses Problem?

mfg,
andi01.

Das was du da hast evaluiert zu:

((a==b)==c)
(true==c)
false

MfG SideWinder

bb

a==b && a==c

Das was du da hast evaluiert zu:

((a==b)==c)
(true==c)
false

MfG SideWinder

danke, jetzt hab ichs verstanden.

danke für die antworten,
andi01.

warum sollte das gehen?

weil das programm bei folgendem schnipsel test ausgibt:

#include <windows.h> 
#include <iostream>
#include<conio.h>

using namespace std; 
int main() 
{
	int a=3;
	int b=4;
	int c=5;
	if(a!=b!=c)
	{
		cout<<"test\n";
	}
	getch();
}

jetzt will ich nur wisse ob er das auch wieder falsch interpretiert und durch zufall dest ausgibt zB weil der erste teil stimmt oder ob er das richtig interpretiert.ich müsste das deshalb wissen weil bei den bedingungen im echten code viel != stehen und der pc das nicht falsch interpretieren darf.

mfg,
andi01.

true.

jetzt weiß ich wenigstens dass es so ist. aber mal rein interessehalber: warum? wie interpretiert es der complier denn dann?

mfg,
andi01.

danke für die ganzen antworten,
andi01.

ich verstehe zwar nicht so ganz warum, aber der kompiler scheint es schonmal falsch zu interpretieren, danke.

jetzt weiß ich wenigstens dass es so ist. aber mal rein interessehalber: warum? wie interpretiert es der complier denn dann?

mfg,
andi01.

Na, das hat doch unskilled prima erklärt: Nacheinander. Wobei nach jeder Evaluation das Ergebnis für die nächste verwendet wird.

Stell's dir wie Funktionsaufrufe vor (du weißt doch: Operatoren kann man überladen) Dann ist:

if(a == b == c)

dasselbe wie:

if(operator==(operator==(a, b), c))

Wobei operator==() natürlich bool liefert, das in diesem Ausdruck zu int konvertiert wird.

Macht's das jetzt irgendwie klarer?

Stefan.

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!!!

@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.

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

mfg,
andi01.

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

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.

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.

mfg,
andi01.

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.

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;

danke für die ganzen Antworten ,
andi01.