while løkker problem

Du skal nok gøre sådan her i stedet

 while(guess != number)
 {
  cin >> guess;
 
  if(guess > number)
    cout << "Too high" << endl;
  else if(guess < number)
    cout << "Too low" << endl;
  else if(guess == number)
    cout << "Correct" << endl;
 }

Hvis du gør sådan her:

while(x < y)
   er det kun denne her linje der bliver gentaget
   og ikk denne her

du skal gøre sådan her:

while(x < y)
{
   så bliver denne linje
   og denne her gentaget
}
og denne her (der er uden for { og }) gør ikke

Desuden chekker du kun om guess == number den første gang.

prøv at forestille dig du gætter for lavt. Så bliver disse to linjer:

cout << "Too low, Try again!" << endl;
cin >> guess;

gentaget indtil guess < number bliver false (når guees ikke er < number)

vis dit gæt derefter er for højt(rimlig stor chance) bliver disse to linjer gentaget

cout << "Too high, Try again!" << endl;
cin >> guess;

indtil dit guess enten er = eller lavere end dit number, og derefter lukker programmet.

Vis du selv vil finde ud af hvordan man kan gøre, skal du ikke læse det næste.
.
.
.
.

Jeg ville kun bruge en while løkke:

int number = 2;
int guess;
cout << "Now i'm thinking of a number from 1-15" << endl;
cout << "Your guess: " << endl;
cin >> guess;
while(number != guess)
{
   if(guess < number)
   {
      cout << "Too low, Try again!" << endl;
      cin >> guess;
      break;
   }
   if(guess > number)
   {
      cout << "Too high, Try again!" << endl;
      cin >> guess;
      break;
   }
}
//vis vi når her til er guess == number
cout << "Good! Your right!" << endl;

Jeg har ikke prøbet at compilere det, så der er sikkert nogle fjel. Men så vidt jeg lige kan se burde det virke.

Der er sikkert også bedre måder at gøre det på.
.:edit:.
Martins måde er nok lidt lettere at gennemskue. Jeg ville i hvert fald fortrække den, men nu har du 2 at vælge mellem

Tak begge to je finder dog stadig små fejl...
Ved Martins skriver den ikke noget første gang man indtaster sit gæt.
Og ved Andreases kører den kun to gange?
kan det være noget med resten af koden?
jeg smækker den lige ind

#include <iostream>
#include <windows.h>
#include <string>
#include <stdlib.h>
using namespace std;
void TegnFarveStreng(string Tekst, int X, int Y, WORD farve)
{
    HANDLE UdHandle;               
    COORD position = {X, Y};    

    UdHandle = GetStdHandle(STD_OUTPUT_HANDLE);   
SetConsoleTextAttribute(UdHandle, farve); 

SetConsoleCursorPosition(UdHandle, position); 

cout << Tekst;

}
int main(int argc, char ** argv)
{
#if defined(_WIN32)
    cout << "Hello Windows user" << endl;
#elif defined(__linux__)
    cout << "Hello Linux user" << endl;
#else
    #error Unsupported operating system
#endif
    Sleep(2000);
    cout << "This is a little console program, I made it most for fun." << endl;
    Sleep(2200);
    cout << "I'we only made a few things for it tho :) " << endl;
    Sleep(1000);
    cout << "It's made most for Windows, so i don't know if it works for Linux" << endl;
    cout << "Tho, there is some things which Linux should be able to run from this, but" << endl;
    Sleep(1000);
    cout << "Here Goes!" << endl;
    cout << "Your Name: " << endl;
    string YourName;
    getline (cin, YourName);
    cout << "Hello " << YourName << endl;
    Sleep(1500);
int number = 2;
int guess;
    cout << "Now i'm thinking of a number from 1-15" << endl;
    cout << "Your guess: " << endl;
    cin >> guess;
while(number != guess)
{
   if(guess < number)
   {
      cout << "Too low, Try again!" << endl;
      cin >> guess;
      break;
   }
   else if(guess > number)
   {
      cout << "Too high, Try again!" << endl;
      cin >> guess;
      break;
   }
}
cout << "Good! Your right!" << endl;

    return 0;
}

Dette virker:

#include <iostream>
#include <cstdlib>
#include <ctime>

int main(int argc, char ** argv)
{
    srand((unsigned)time(0));
    int number = (rand()%15)+1;

    int guess;
    std::cout << "Now i'm thinking of a number from 1-15\nYour guess: " << std::endl;
    std::cin >> guess;
    while(number != guess)
    {
       if(guess < number)
       {
          std::cout << "Too low, Try again!" << std::endl;
          std::cin >> guess;
       }
       else if(guess > number)
       {
          std::cout << "Too high, Try again!" << std::endl;
          std::cin >> guess;
       }
    }
    std::cout << "Good! Your right!" << std::endl;

    return 0;
}

Det var bare brugen af 'break' der var gal.

Dette virker:

Kode 

#include <iostream>
#include <cstdlib>
#include <ctime>

int main(int argc, char ** argv)
{
    srand((unsigned)time(0));
    int number = (rand()%15)+1;

    int guess;
    std::cout << "Now i'm thinking of a number from 1-15\nYour guess: " << std::endl;
    std::cin >> guess;
    while(number != guess)
    {
       if(guess < number)
       {
          std::cout << "Too low, Try again!" << std::endl;
          std::cin >> guess;
       }
       else if(guess > number)
       {
          std::cout << "Too high, Try again!" << std::endl;
          std::cin >> guess;
       }
    }
    std::cout << "Good! Your right!" << std::endl;

    return 0;
}

Det var bare brugen af 'break' der var gal.

Jeg kan godt se nu at break ikke var nødvendig, men burde det ikke virke med break alligevel?