Problem mit Positionsberechnung

GermanFighter

Hi,
ich habe ein Problem mit einer selbsterstellten Klasse und hoffe, dass ihr mir helfen könnt.

//position.h
#ifndef POSITION
#define POSITION

class Position
{
public:
	int x, y;
	Position()
	{
		x = y = 0;
	}

	Position(int i_x, int i_y)
	{
		x = i_x;
		y = i_y;
	}
	void print();
};

#endif

//position.cpp
#include "position.h"
#include <iostream>
#include <conio.h>
using namespace std;

void Position::print()
{
	cout << "X: " << x << "\n"
		 << "Y: " << y  << endl;
}
//position_test.cpp
#include "position.h"
#include <iostream>
#include <conio.h>
using namespace std;

int main()
{
	char c;
    Position spieler;
	do
	{
	    spieler.print();
	    if(kbhit())
		{
			c = getch();
			switch(c)
			{
			case 'w': spieler.x += 1;
			case 'a': spieler.x -= 1;
			case 'd': spieler.y += 1;
			case 's': spieler.y -= 1;
			}
		}
		system("CLS");
	}
	while(true);

	cin.clear();
	cin.get();
	return 0;
}

Wenn ich a oder s drücke funktioniert es, aber bei w und d passiert nichts.

icarus2

Dir fehlen die breaks im switch-Statement:

switch(c)
            {
            case 'w': spieler.x += 1; break;
            case 'a': spieler.x -= 1; break;
            case 'd': spieler.y += 1; break;
            case 's': spieler.y -= 1; break;
            }

Ansonsten, wenn du 'w' drueckst, werden alles Cases nach dem 'w'-Case auch ausgefuehrt.

GermanFighter

Danke jetzt geht's
Gibt es auch eine Möglichkeit die x- und y-Werte zu ändern ohne sie in den Public- sondern in den Privatebereich zu schreiben.

Belli

Mach die Variablen private und stelle stattdessen public Methoden:
void moveLeft();
void moveRight();
...

zur Verfügung, in welchen dann die Variablenwerte geändert werden. Damit stellst Du sicher, dass nicht irgendetwas in diese Variablen geschrieben werden kann, sondern dass die Kontrolle über den Variableninhalt bei Dir (in Deiner Klasse) bleibt.

GermanFighter

Dann kann ich aber nicht so was wie x += 1 machen

Belli

Genau. Stattdessen machst Du dann:

class Position
{
  public:
    ...
    void moveRight(int distance = 1)
    {
      //Hier kannst Du noch distance auf gültige Werte prüfen
      ...
      x += distance;
    }
    ...
  private:
    int x;
    ...
};

...
Position spieler;
...
spieler.moveRight();
...

GermanFighter

Es hat geklappt! Danke!