Guess the number

[Actionman1995]

Just made a CMD guess the number game, would like some constructive criticism or evaluation.

Thanks in advance, Actionman1995

#include <cstdlib>
#include <iostream>

using namespace std;

int computerGuess(int high, int low)
{
    return ((high-low)/2)+low;
}

int generateNumber()
{
    srand(time(0));
    return rand()%100;
}

int displayMenu()
{
    int value;
    cout << " Welcome to Guess the Number\n"
         << " Please Choose a Game Type: ";
         cout << "\n\n";
         cout << "1) Human Guessing\n"
              << "2) Computer Guessing\n"
              << "3) Exit\n\n"
              << "Your choice: ";
         cin >> value;
    return value;
}

int main(int argc, char *argv[])
{
    
    int menu, guess, num, counter = 10;
    
        system("CLS");
        menu = displayMenu();
        system("CLS");
        
    if (menu == 1)
     {
        num = generateNumber();
        do
        {
            cout << "Your Guess is: ";
            cin >> guess;
                if (num>guess)
                {
                    cout << "The number is higher\n\n"
                         << "You have " << counter-- << " attempts left\n\n";
                }
                else if (num<guess)
                {                     
                    cout << "The number is lower\n"
                         << "You have " << counter-- << " attempts left\n\n";
                }
                else
                {
                    cout << "Well done, you got it right!\n\n";
                    break;
                }
        }
        while (counter>=0);
      }
      else if (menu==2)
      {
           int high, low;
           char ans;        
           low = 1;
           high = 100;
          do
          {
               cout << "If the computers guess is too high type in [L],\n"
                       "if the computer guess is too low type in [H]\n"
                       "if the answer is correct typr in [Y].\n\n"
                       "Computers Guess: ";
               num = computerGuess(high, low);
               cout << num << "\nLow: " << low << "\nHigh: " << high << "\n";
               cin >> ans;
               
               switch(ans)
               {
                  case 'l':
                  case 'L':
                     high=num;
                     break;
                  case 'h':
                  case 'H':
                     low=num;
                     break;
                  default:
                     cout << "Well done computer!\n";
               }
          }
          while (counter>=0);
      }
    system("PAUSE");
    return EXIT_SUCCESS;
}

[spaggy]

Don't use system("PAUSE");. cin.get(); is lighter and does not pose potential safety issues like the Systeem call

[Zero One]

In order to expand upon spaggy's previous remark, you can check out the following thread:

http://cplusplus.com/forum/beginner/1988/

[Galik]

Avoid sing namespace std;, http://www.parashift.com/c++-faq-lite/coding-standards.html#faq-27.5

You should only call srand(time(0)); once, not every time you want a random number.

You might want to create a function for each menu option.

Some people would use a switch for the menu selection because its a finite int based decision.