Mein momentanes Problem besteht darin, dass ich nicht weiß wie ich dem User nur eine bestimme Zeit gebe, um seine Eingabe zu tätigen, damit die Schlange auch von allein weiter in die bisherige Richtung bewegt wird. Bisher bleibt meine Schleife immer bei _getch() stehen. Ich würde es nun gerne so lösen, dass der Nutzer beispielsweise 1 Sekunde Zeit hat um eine Eingabe zu tätigen, ansonsten soll die Schleife diese Stelle überspringen und die Schlange soll sich weiter in die vorherige Richtung bewegen.

So viel erstmal dazu. Desweiteren wollte ich fragen ob mein bisheriges Vorgehen so richtig war, denn das Spielfeld habe ich bisher mit einer simpelen if begrenzt, mit der ich abfrage ob die Schlange bestimmte Koordinaten überschreitet, wäre es vielleicht Sinnvoller das Ganze in einem 2d-Array zu realisieren?
Ich bin für jede Antwort dankbar

Mein momentanes Problem besteht darin, dass ich nicht weiß wie ich dem User nur eine bestimme Zeit gebe, um seine Eingabe zu tätigen, damit die Schlange auch von allein weiter in die bisherige Richtung bewegt wird. Bisher bleibt meine Schleife immer bei _getch() stehen. Ich würde es nun gerne so lösen, dass der Nutzer beispielsweise 1 Sekunde Zeit hat um eine Eingabe zu tätigen, ansonsten soll die Schleife diese Stelle überspringen und die Schlange soll sich weiter in die vorherige Richtung bewegen.

Also, ganz direkt würde ich es so machen:

for( auto first = std::clock() + CLOCKS_PER_SEC; std::clock() < first; )
    if( kbhit() )
    {
        int i = getch();
        // [...]
        break;
    }

wäre es vielleicht Sinnvoller das Ganze in einem 2d-Array zu realisieren?

Eigentilch solltest du ein eindimesionales Array verwenden dass du mittels einer Brett -Klasse als zweidimensionales Array ansprechbar machst (also mit x- und y-Koordinate).

Die Spiellogik selbst sollte aber in zwei Dimensionen stattfinden. Schreib dir eine auf_brett -Funktion, die prüft ob eine Funktion auf dem Brett ist, und schau ob der Kopf der Schlange noch auf dem Brett ist - natürlich per if -Statement. Die Frage verstehe ich also nicht ganz - if und 2D-Arrays schließen sich nicht aus.

Da ich noch recht unerfahren bin (lerne seit knapp einem Jahr in der Schule c++) kann ich deinen ersten Lösungsvorschlag noch nicht ganz nachvollziehen, könntest du mir diese eventuell etwas besser erklären? Denn clock() und CLOCKS_PER_SEC etc. sagen mir nicht wirklich etwas, da ich diese nicht kenne.

Zum zweiten Punkt, ich habe bisher ein Spielfeld mit einer for erstellet(also die optische Ausgabe des Feldes) und dann per if gesagt:

if(x <= 0 || x >= 24 || y <= 0 || y >= 24)//Spielfeldgröße 25x25
{
//beenden des Spiels
}

somit benutze ich bisher gar keinen Array. Daher war meine Frage ob es vielleicht Sinnvoller wäre einen Array mit einzubinden (hab ich im Anfangspost wohl schlecht formuliert, tut mir leid).

Da ich noch recht unerfahren bin (lerne seit knapp einem Jahr in der Schule c++) kann ich deinen ersten Lösungsvorschlag noch nicht ganz nachvollziehen, könntest du mir diese eventuell etwas besser erklären? Denn clock() und CLOCKS_PER_SEC etc. sagen mir nicht wirklich etwas, da ich diese nicht kenne.

std::clock() gibt die Anzahl der 'Ticks' seit dem Prozessstart zurück. Es gibt CLOCKS_PER_SEC Ticks pro Sekunde.

Du addierst also die Anzahl der Ticks für eine Sekunde auf die aktuelle Tickzahl drauf. Das wird die Tickzahl sein bei der du abbrichst, falls der Benutzer noch keine Eingabe gemacht hat - zum Zeitpunkt der Addition eine Sekunde in der Zukunft.
Und dann prüfst du in einer Schleifen-Abbruchbedingung ob die aktuelle Tickzahl schon bei der Abbruch-Tickzahl ist.
Falls dem nicht so ist, prüfst du ob eine Taste gedrückt wird - und wenn das der Fall ist, fragst du ab welche, mittels getch . Dann verarbeitest du die Eingabe und brichst die Schleife ab.

Ich muss nun nur noch dafür sorgen, dass die Schlange länger wird, sie mit sich selbst zusammenstoßen kann und Punkte spawnen.

for( auto first = std::clock() + CLOCKS_PER_SEC; std::clock() < first; )

first ? Echt jetzt?

1.Schlange(Länge bisher 1) automatisch in die vorher gewählte Richtung bewegen
2.Kollision mit Wand ist möglich
3.Score zählen(nur leider steigt der nicht da es noch keine Punkte zum einsammeln gibt

Was noch fehlt:

1.Punkte die spawnen(damit die Schlange diese einsammeln kann)
2.Schlange muss wachsen können
3.Kollision mit Schlangenkopf und Körper muss möglich sein.

Da es sich hier um mein erstes Programm dieser Art handelt (bisher habe ich mein meist nur Konsolenprogramme, die irgendwas berechnet und dann ausgegeben haben, geschrieben) habe ich nun keine Ideen wie ich diese Punkte ins Spiel integriere.

Ich bräuchte nun den ein oder anderen Tipp, wie ich das realisieren kann.

Edit,
Hier mal der bisherige Code:

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

using namespace std;

const int rechts = 'd';
const int links = 'a';
const int hoch = 'w';
const int runter = 's';
const int prechts = 0x4d;         //Hexadezimale codes der Pfeiltasten
const int plinks = 0x4b;
const int phoch = 0x48;
const int prunter = 0x50;

void Bewegung(int, int);

int main()
{
	bool ende = false;
	int x = 12;                      //x Koordniate (erste Definition = Startpunkt)
	int y = 10;                      //y Koordniate (erste Definition = Startpunkt)
	int richtung = 1;
	int punkte = 0;

	cout << "Punkte " << punkte << endl;
	cout <<"XXXXXXXXXXXXXXXXXXXXXXXXX"<<endl;
	for(int i = 1; i < 24; i++)
	{
	cout << "X                      X"<<endl;
	}
	cout <<"XXXXXXXXXXXXXXXXXXXXXXXXX"<<endl;

	do
	{
		Bewegung(x, y);
		cout << "o";

		if(kbhit())
		{
			int taste = getch();
			if(!taste || taste == 0xe0))
				taste = getch();

			if(taste == hoch && richtung != 2 || taste == phoch && richtung != 2)
			{
				richtung = 1;
				--y;
			}

			else if(taste == runter && richtung != 1 || taste == prunter && richtung != 1)
			{
				richtung = 2;
				++y;
			}

			else if(taste == rechts && richtung != 4 || taste == prechts && richtung != 4)
			{
				richtung = 3;
				++x;
			}

			else if(taste == links && richtung != 3 || taste == plinks && richtung != 3)

			else
				continue;

			Sleep(150);
		}

		else if(!kbhit())
		{
			if(richtung == 1)
			{
				--y;
			}
			if(richtung == 2)
			{
				++y;
			}
			if(richtung == 3)
			{
				++x;
			}
			if(richtung == 4)
			{
				--x;
			}
			Sleep(150);
		}

		system("cls");

		if( x <= 0 || x >= 24 || x <= 1 || y >= 25)
		{
			cout << "Game Over!" << endl << endl;
			cout << "Du hast " << punkte << " Punkte erreicht" << endl << endl;
			system("pause");
			return 0;
		}

		cout << "Punkte " << punkte << endl;
		cout <<"XXXXXXXXXXXXXXXXXXXXXXXXX"<<endl;
		for(int i = 1; i < 24; i++)
		{
			cout << "X                      X"<<endl;
		}
		cout <<"XXXXXXXXXXXXXXXXXXXXXXXXX"<<endl;

	}
	while(true);

	return 0;
}

void Bewegung(int b, int h)
{
	COORD punkt;
	punkt.X = b;
	punkt.Y = h;
	SetConsoleCursorPosition(GetStdHandle(STD_OUTPUT_HANDLE), punkt);
}

Edit durch Arcoth: Code formatiert.

• "XXXXXXXXXXXXXXXXXXXXXXXXX" ggf. durch std::string(30, 'X') ersetzen.

if(kbhit()) {/* [..] */}
else if(!kbhit())

Was soll das denn?
*  In Zeile 64 fehlt was, das war wohl ein Flüchtigkeitsfehler von dir.
*  COORD ist ein Aggregat. Unterstützt dein Compiler C++11 kannst du also Zeile 119-122 durch  

SetConsoleCursorPosition(GetStdHandle(STD_OUTPUT_HANDLE), {b, h});

ersetzen.
*  73-88 solltest du durch ein switch-Statement ersetzen.  

switch (richtung)
{
case 1: --y; break;
case 2: ++y; break;
case 3: ++x; break;
case 4: --x; break;
}

*  Warum die Endlosschleife als `do ... while(true)` schreiben? Da kriege ich immer das Gefühl, die Schleifenform hat er aus guten Grund gewählt, weil die Abbruchbedingung erst nach dem ersten Schleifendurchlauf geprüft werden soll. `for(;;)` ist da geläufiger. Abgesehen davon solltest du immer versuchen, Schleifen eine Abbruchbedingung zu geben.
*  In `x <= 0 || x >= 24 || x <= 1 || y >= 25` steckt wohl auch ein Flüchtigkeitsfehler.
*  `return 0;` am Ende der `main` -Funktion ist überflüssig.

Jedenfalls:

Arcoth schrieb:

Hab mir mal die Freiheit genommen deinen Code zu formatieren.
Kommentare zum Code:

• "XXXXXXXXXXXXXXXXXXXXXXXXX" ggf. durch std::string(30, 'X') ersetzen.

if(kbhit()) {/* [..] */}
else if(!kbhit())

  
Was soll das denn?
*  In Zeile 64 fehlt was, das war wohl ein Flüchtigkeitsfehler von dir.
*  COORD ist ein Aggregat. Unterstützt dein Compiler C++11 kannst du also Zeile 119-122 durch  

SetConsoleCursorPosition(GetStdHandle(STD_OUTPUT_HANDLE), {b, h});

  
ersetzen.
*  73-88 solltest du durch ein switch-Statement ersetzen.  

switch (richtung)
{
case 1: --y; break;
case 2: ++y; break;
case 3: ++x; break;
case 4: --x; break;
}

*  Warum die Endlosschleife als `do ... while(true)` schreiben? Da kriege ich immer das Gefühl, die Schleifenform hat er aus guten Grund gewählt, weil die Abbruchbedingung erst nach dem ersten Schleifendurchlauf geprüft werden soll. `for(;;)` ist da geläufiger. Abgesehen davon solltest du immer versuchen, Schleifen eine Abbruchbedingung zu geben.
*  In `x <= 0 || x >= 24 || x <= 1 || y >= 25` steckt wohl auch ein Flüchtigkeitsfehler.
*  `return 0;` am Ende der `main` -Funktion ist überflüssig.

Erstmal danke fürs formatieren

Zum string, leider habe ich strings noch nicht wirklich gelernt, daher habe ich versucht das meiste über eine for zu erstellen nur dieses Stück habe ich dann seperat geschrieben.

Ist das mit dem !kbhit() so nicht richtig? Ich hatte das so geplant, dass das Programm fals keine Eingabe erfolgt dort hin springen soll und dann die Bewegung in die vorherige Richtung fortgesetzt wird.
Naja aber du hast recht, ich habe es mal durch ein normales else ersetzt.

Danke für den Tipp, ich werde das mal ausprobieren.

Das mit dem switch werde ich definitiv tun, wird den Code etwas übersichtlicher machen.

Die do while kommt daher, dass ich noch ein manuelles Beenden der Schleife ermöglichen möchte, sobald der Rest funktioniert. Ich werde also am Ende noch eine Bedingung wie "!= beenden" oder ähnliches einsetzen.

Dabei handelt es sich tatsächlich um keinen Fehler, In der obersten Zeile wird der Punktestand ausgegeben, somit ist die Y-Achse um eins nach unten verschoben.

Unser Lehrer zieht uns immer nen Punkt in Tests etc. ab wenn wir das return 0; nicht schreiben, daher hab ich es mir angewöhnt dieses auch bei privaten Projekten zu setzen, dann vergesse ich es auch nicht bei Schulaufgaben.

Ich werde mich jetzt nochmal dran setzen, vielen Dank für die Hilfe