hoger en lager

Status
Niet open voor verdere reacties.

justdavid

Gebruiker
Lid geworden
30 jul 2006
Berichten
37
ik wil me hoger en lager verbeteren :P

wie heeft tips ???
 
Laatst bewerkt door een moderator:
ik heb niet zo'n zin om hem door te kijke maar misschien heb je er iets aan om hem te vergelijken met die van een ander ^^

de mijne:
Code:
#include <iostream>
#include <conio.h>
#include <windows.h>
#include <cstdlib>
#include <ctime>

using std::cout;
using std::cin;

int getNumber();
int getGuess();
int askToPlayAgain();

int turns;
bool wantToPlay = true;

int main()
{
    cout << "\tWelcome to Higher Lower.\n\n";
    
    while(wantToPlay == true)
    {
        int number = getNumber();
        
        turns = 0;
    
        cout << "The computer has picked a number between 1 and 100.\n";
        cout << "Try to guess that number.\n";
    
        int guess;
    
        while(guess != number)
        {
                guess = getGuess();
    
                if(guess < number)
                {
                         cout << "Higher!\n";
                }
    
                if(guess > number)
                {
                         cout << "Lower!\n";
                }
                
        }
    
        cout << "\n\nCongratulations, you have guessed the number in " << turns << " turns!\n";
        
        wantToPlay = askToPlayAgain();
    }
    
    cout << "\n\n\tGoodbye.";
    
    getch();
    
    return 0;
}

int getNumber()
{
    int x;
    srand(time(0));
    x = (rand() % 100) + 1;
    
    return x;
}

int getGuess()
{
    int x;
    cout << "\nYour guess: ";
    cin >> x;
    turns++;
    
    return x;
}

int askToPlayAgain()
{
     char x;
     bool y;
     while(x != 'y' && x != 'Y' && x != 'n' && x != 'N')
     {
             cout << "Do you wish to play again? (y/n)";
             cin >> x;
     }
     
     if(x == 'n' || x == 'N')
     {
          y = false;
     }
     
     if(x == 'y' || x == 'Y')
     {
          y = true;
          system("CLS");     
     }
     
     return y;
}
 
ik heb niet zo'n zin om hem door te kijke maar misschien heb je er iets aan om hem te vergelijken met die van een ander ^^

ik zal het vergelijken :P
(vind nederland wel wat makkelijker)

Ach Nederlands is misschien makkelijker, maar _goed_ Nederlands is voor veel mensen nog aardig lastig...

Waarom gebruik je BEGIN als functie en roep je daarvandaan RESULTATEN aan waarin ook nog de invoer voor een nieuwe 'gok' gedaan wordt. Het zal wel werken op deze manier maar echt leesbaar is het niet. Lijkt me een geforceerd recursief algoritme.
Ik zou iets maken met: Initialiseer, Loop terwijl nog levens, en in de loop invoer ophalen, vergelijken/verwerken en resultaat tonen.

Nog een tip: de random functie levert een getal op in het bereik [0, maxrandom - 1]
Dus maxrandom is geen geldige keuze.
 
Wat mij direct opvalt is dat je bij vrijwel elke controle de Resultaten() opnieuw aanroept. Zou je niet gewoon deze aan het eind van je functie opnieuw kunnen aanroepen, aangezien je hem nu óveral aanroept? :)
 
hah! dik gestolen :P
jah nu andere post veranderd ermaar copyright neer gezet..
boeit me niet als mensen het gebruiken ofzo maar het al vaker gehad dat je iets post ofzo om mensen te helpen en dat je later op een andere website ziet ofzo van dit is mijn code blablabla :/ kank niet lijen xD

ook al is het maar iets simpels
 
Code verwijderd. Blijkbaar niet van jezelf. Mocht je zelf iets gemaakt hebben mag je daar een topic over openen
 
Status
Niet open voor verdere reacties.
Terug
Bovenaan Onderaan