[C++] Struct aktualisiert sich nur nach cout

DarkSyranus

Moin.
Ich stehe wiedereinmal vor einem abstrakten Rätsel.
Ich habe ein eigenes Struct.

Code

struct tPosition
		{
			short Color;
			short X;
			short Y;
		};

In einer anderen Funktion benutze ich nun dieses Struct um einen Wert zu berechnen.

Code

/**
	* This function checks if a move is valid.
	*/
	bool cField::IsValidMove(tMove Move)
	{
		tPosition Position;


		cout << " >> Is valid move ..." << endl;


		// check if the stones are within the field and free
		if(!this->IsEmptyField(Move.First) || !this->IsEmptyField(Move.Second))
		{
			return false;
		}


		// check if the stones halfes are next to each other
		for(short i = 0; i < 6; i ++)
		{
			// get position of next neighbour stone
			Position = this->GetNextStoneInDirection(Move.First, i);


			// DAS IST DIE BOESE ZEILE
			cout << "Position X: " << Position.X << "  Y: " << Position.Y << endl;


			// check positions
			if((Move.Second.X == Position.X) && (Move.Second.Y == Position.Y))
			{
				cout << "Stone found" << endl;


				break;
			}


			// not located next to each other
			if(i == 5)
			{
				return false;
			}
		}


		cout << "Valid move" << endl;


		return true;
	}

Alles anzeigen

Das funktioniert auch soweit, aber nur, solange die Zeile nach // DAS IST DIE BOESE ZEILE im Code vorhanden ist.
Wenn ich den cout Befehl rausnehme, ist die Überprüfung immer false.

Der Fehler ist für mich völlig unverständlich ...
Hat da Jemand eine Idee, ob es auch ohne die cout Zeile geht?

jojo87

Bin jetzt kein Held c++ Held, aber wird hier

Code

Position = this->GetNextStoneInDirection(Move.First, i);

ein ungültiger Wert für Position vergeben?

Vielleicht zeigst du mal die Funktion?

Gruß,
Jojo

DarkSyranus

Code

/**
	* This function returns the next stone in direction.
	*/
	tPosition cField::GetNextStoneInDirection(tPosition Position, short Direction)
	{
		cout << " >> Get next stone in direction ..." << endl;


		// check if it is a valid field
		if(this->IsValidField(this->Field[Position.Y +this->Directions[Direction].Y][Position.X +this->Directions[Direction].X]))
		{
			return this->Field[Position.Y +this->Directions[Direction].Y][Position.X +this->Directions[Direction].X];
		}


		return this->Field[0][0];
	}

Alles anzeigen

Daran kann es aber eigentlich nicht liegen, da das Problem nur besteht, wenn man kein cout benutzt.

Edit: Habs hinbekommen, GetNextStoneInDirection brauchte noch einen & davor, fragt mich aber bitte nicht warum.
Falls jemand das eklären kann, würde ich es gerne wissen.

The User

du meinst tposition& cField::GetNextStoneInDirection ?
Dadurch verweist die Rückgabe auf den in der Funktion ausgewählten Stein, sie ist keine Kopie.
Dann nehme ich an, dass der Zuweisungsoperator operator= vom Compiler für tPosition falsch automatisch definiert wurde.
Wenn du es ohne & haben möchtest, probier mal in der Klasse tPosition den operator= neu zu definieren, daran könnte es liegen.
Einfach:

Code

tPosition& tPosition::operator=(tPosition arg)
{
Color = arg.Color;
X = arg.X;
Y = arg.Y;
return *this;
}

Nicht sicher, dass es daran liegt.

Viele liebe Grüße
The User

Tags