Igen, igen, igen og igen

Men jeg har et nyt spørgsmål til jer. Hvorfor må jeg ikke kalde funktionerne one-/twoplayer() i dette script?

Kode 

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

int mode;
int start;
int view;
string slut;


int main()
{
   system("cls");
   cout << "Tryk:" << endl;
   cout << "1) For at starte et nyt spil" << endl;
   cout << "2) For at se Hjalp" << endl;
   cout << "3) For at slutte" << endl;
   cin >> view;
   switch(view) {
   case 1:
      system("cls");
      cout << "Tryk:" << endl;
      cout << "1) For at spille alene mod computeren" << endl;
      cout << "2) For at spille mod en kammerat" << endl;
      cin >> mode;
      switch(mode) {                                        _  _
        case 1: oneplayer(); break; // <-her |_| |_ |_|
        case 2: twoplayer(); break; // <-her |  | |_ |\        default: cout << "Du trykkede hverken 1 eller 2" << endl; system("pause"); main();
      }
      break;
   case 2:
      system("cls");
      cout << "Blablabla" << endl;
      system("pause");
      main();
   case 3:
      cout << "Er du sikker j/n" << endl;
      cin >> slut;
      if (slut == "j") {
          exit(0);
      } else { main(); }
   default:
      system("cls");
      cout << "Du trykkede hverken paa 1, 2 eller 3" << endl;;
      system("pause");
      main();
   }
   return 0;
}
int oneplayer(void)
{
   return 0;
}
int twoplayer()
{
   return 0;
}

MVH. qff

Prøv at rykke funktionerne op over dit int main()

-----
Email: christian [at] steffweb [dot] dk
MSN: christian [at] steffweb [dot] dk

Hvorfor kan jeg ikke få koden til at virke???
|
\\ /

Kode 

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

int main()
{
      int view, mode, start;
      cout << "Tryk:" << endl;
      cout << "1) For at starte et nyt spil" << endl;
      cout << "2) For at se Hjalp" << endl;
      cin  >> view;
      if(view == 1) {
              system("cls");
              cout << "Tryk:" << endl;
              cout << "1) For at spille alene mod computeren" << endl;
              cout << "2) For at spille mod en kammerat" << endl;
      } else if(view == 2) {
              system("cls");
              cout << "Blablabla";
              system("pause");
      } else {
              system("cls");
              cout << "Du trykkede hverken paa 1 eller 2";
              system("pause");
      }
      return 0;
}

Jeg må være komplet idiot, koden virker så simpel, men nåja der er jo nogen der skal leve af at hjælpe mig af med det problem.


MVH. qff

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

int view;
int mode;
int start;


int main()
{
      
      cout << "Tryk:" << endl;
      cout << "1) For at starte et nyt spil" << endl;
      cout << "2) For at se Hjalp" << endl;
      cin  >> view;
      if(view == 1) {
              system("cls");
              cout << "Tryk:" << endl;
              cout << "1) For at spille alene mod computeren" << endl;
              cout << "2) For at spille mod en kammerat" << endl;
      } else if(view == 2) {
              system("cls");
              cout << "Blablabla";
              system("pause");
      } else {
              system("cls");
              cout << "Du trykkede hverken paa 1 eller 2";
              system("pause");
      }
      return 0;
}

Virkede da jeg ændrede det til sådan her

Men jeg har et nyt spørgsmål til jer. Hvorfor må jeg ikke kalde funktionerne one-/twoplayer() i dette script?

Kode 

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

int mode;
int start;
int view;
string slut;


int main()
{
   system("cls");
   cout << "Tryk:" << endl;
   cout << "1) For at starte et nyt spil" << endl;
   cout << "2) For at se Hjalp" << endl;
   cout << "3) For at slutte" << endl;
   cin >> view;
   switch(view) {
   case 1:
      system("cls");
      cout << "Tryk:" << endl;
      cout << "1) For at spille alene mod computeren" << endl;
      cout << "2) For at spille mod en kammerat" << endl;
      cin >> mode;
      switch(mode) {                                        _  _
        case 1: oneplayer(); break; // <-her |_| |_ |_|
        case 2: twoplayer(); break; // <-her |  | |_ |\        default: cout << "Du trykkede hverken 1 eller 2" << endl; system("pause"); main();
      }
      break;
   case 2:
      system("cls");
      cout << "Blablabla" << endl;
      system("pause");
      main();
   case 3:
      cout << "Er du sikker j/n" << endl;
      cin >> slut;
      if (slut == "j") {
          exit(0);
      } else { main(); }
   default:
      system("cls");
      cout << "Du trykkede hverken paa 1, 2 eller 3" << endl;;
      system("pause");
      main();
   }
   return 0;
}
int oneplayer(void)
{
   return 0;
}
int twoplayer()
{
   return 0;
}

MVH. qff

Prøv at se på linjen hvor der står "switch(mode)" den ser sådan her ud:

switch(mode) { _ _
du skal bare fjerne "_ _" så skulle den gerne virke det gør den ihvertfald her.

PS: _ _ står længere henne i eksemplet bare for at vise dig det


[Redigeret d. 22/02-04 21:32:36 af KOEN]

Hvad er det du ikke kan få til at virke? Jeg kan ikke rigtigt se at der er noget problem.

Hvad er det du ikke kan få til at virke? Jeg kan ikke rigtigt se at der er noget problem.

Jeg har heller ikke noget problem hvis jeg bare kopier hele koden. Hvad er der galdt?

Jeg har heller ikke noget problem hvis jeg bare kopier hele koden. Hvad er der galdt?

Mystisk kan I få den til at vise "Blablabla" osv.?


MVH. qff

Sorry guys, men det var en fejl i min compiler...
Fattede heller ikke at den ikke skulle virke, LORTE COMPILER TING... Nåja


MVH. qff

Men jeg har et nyt spørgsmål til jer. Hvorfor må jeg ikke kalde funktionerne one-/twoplayer() i dette script?

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

int mode;
int start;
int view;
string slut;


int main()
{
   system("cls");
   cout << "Tryk:" << endl;
   cout << "1) For at starte et nyt spil" << endl;
   cout << "2) For at se Hjalp" << endl;
   cout << "3) For at slutte" << endl;
   cin >> view;
   switch(view) {
   case 1:
      system("cls");
      cout << "Tryk:" << endl;
      cout << "1) For at spille alene mod computeren" << endl;
      cout << "2) For at spille mod en kammerat" << endl;
      cin >> mode;
      switch(mode) {                                        _  _
        case 1: oneplayer(); break; // <-her |_| |_ |_|
        case 2: twoplayer(); break; // <-her |  | |_ |\        default: cout << "Du trykkede hverken 1 eller 2" << endl; system("pause"); main();
      }
      break;
   case 2:
      system("cls");
      cout << "Blablabla" << endl;
      system("pause");
      main();
   case 3:
      cout << "Er du sikker j/n" << endl;
      cin >> slut;
      if (slut == "j") {
          exit(0);
      } else { main(); }
   default:
      system("cls");
      cout << "Du trykkede hverken paa 1, 2 eller 3" << endl;;
      system("pause");
      main();
   }
   return 0;
}
int oneplayer(void)
{
   return 0;
}
int twoplayer()
{
   return 0;
}

MVH. qff

Og bare rolig det har ikke noget at gøre med HER kommentaren.

MVH. qff

Du skal have en definition af oneplayer() og twoplayer() inden du kalder dem. Så enten flytter du dem op før main, eller du laver prototyper, før main:

int oneplayer(void);
int twoplayer(void);

BorlandC vil ikke finde sig i at man kalder main() fra main(), gcc ser ikke ud til at have problemer, mystisk.

Nogle kompilere på nogle systemer skriver først til skærmen når der kommer en "\\n" eller endl, så hvis du har:

cout << "whatever";
system("pause");

Så er det ikke sikkert at den skriver "whatever" inden kaldet til system(...), du kan løse det ved at skrive:

cout << "whatever";
cout.flush();
system("pause");

Du bør nok finde en mere elegant måde at lave "hit any key to continue" på. Nogen systemer vil lave en ny dosbox for hver system() kald.