complier berreit - einfach makieren und ausführen

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

using namespace std;

HANDLE hOuput = GetStdHandle(STD_OUTPUT_HANDLE);

void gotoxy (int x, int y)
{
 COORD pos;
 pos.X = x;
 pos.Y = y;
 SetConsoleCursorPosition(hOuput,pos);
}

void show_arrow( int x, int y )
{
 gotoxy(x,y);
 cout << "²";
}

void kill_arrow ( int x, int y )
{
 gotoxy (x, y);
 cout << " ";
}

void menu()
{
 int x = 15;
 int y = 15;

 cout<<"\n\n\n\n\n\n\n\n\n\n\n\n\n\n\n";
 cout<<"                 Spiel Starten \n";
 cout<<"                 Weiter Spielen \n";
 cout<<"                 Spiel Beenden  \n";

 /////////////////////////////////////////////
 gotoxy(15,15);
 while(1)
 {	
 show_arrow ( x, y );
 Sleep(100);
 kill_arrow ( x, y );

 if(GetAsyncKeyState(VK_UP) & 1 )
 {                       
 --y;
 }

 if(GetAsyncKeyState(VK_DOWN) & 1 )
 {
 ++y;
 }

 if (y == 15)
 {
  if(GetAsyncKeyState(VK_RETURN) & 1 )
 {   
  system("cls");      
  cout<<"auswahl 1"; 
  Sleep(1000);
  system("cls");
  menu();     
 } 
 }

 if (y == 16)
 {
  if(GetAsyncKeyState(VK_RETURN) & 1 )
 {   
  system("cls");      
  cout<<"auswahl 2"; 
  Sleep(1000);
  system("cls"); 
  menu();
 } 
 }

 if (y == 17)
 {
  if(GetAsyncKeyState(VK_RETURN) & 1 )
 {     
  system("cls");      
  cout<<"auswahl 3"; 
  Sleep(1000);
  system("cls"); 
  menu();    
 } 
  }

 //////////////////////////////////////////////
 if (y == 14)
 {
  y = 15;
 }

 if (y == 18)
 {
  y = 17;
 }

 }
}

int main()
{
 menu();
}

wer will darf ihn auch verändern

Hey nicht schlecht!

Genau...

while(1) { ... }

Der Prozessor schmort, obwohl niemand am Rechner sitzt...
Außerdem ist es win-only, k.A. wer da ein Consolen-Menu braucht.
Zu guter letzt ist es C und sollte in ein anderes Forum verschoben werden, da das nix mit std-C++ zu tun hat.

Warum ruft sich menu() eigentlich rekursiv auf? Das lässt sich mit break bzw. continue auf jeden Fall besser lösen.

Die ganzen if-Abfragen sind in einer switch-Verzweigung auf jeden Fall besser aufgehoben. Und hOutput kann man als globale Variable auch vollständig vermeiden.

Aber lass dich davon nicht entmutigen, jeder fängt mal klein an.

Nice.

Bye.

Die "Nice, cool und der Anno" - Typ sind alle er >.>

Der Code ist einfach grausig... mehr nicht.

Sowas dauert bestimmt keine 3h

Mjoa, mjoa schrieb:

Hey nicht schlecht!

Genau...

while(1) { ... }

Der Prozessor schmort, obwohl niemand am Rechner sitzt...

Nö, da ist ein Sleep(100) drin.