LinkBack URL
About LinkBacks
Random Number problem in number guessing game... Here is my source:
Basically this is a number guessing game which generates a random number at start-up and asks the user for input. The problem i have is that no matter what number the user inputs, it will always give the "your number is too low" MessageBox. In this example, i set it to generate a number between 0 and 25 (using code from this very web site) but even if the user types in his/her guess as "9999" it will still give the "Your number is too low" message.Code:#include "stdafx.h" #include "EDRs Number Guessing Game.h" #include "EDRs Number Guessing GameDlg.h" #include <stdio.h> #include <stdlib.h> #include <time.h> #ifdef _DEBUG #define new DEBUG_NEW #undef THIS_FILE static char THIS_FILE[] = __FILE__; #endif unsigned number, tries = 0; char buffer[32]; int rand_mid(int low, int high); ///////////////////////////////////////////////////////////////////////////// // CAboutDlg dialog used for App About class CAboutDlg : public CDialog { public: CAboutDlg(); // Dialog Data //{{AFX_DATA(CAboutDlg) enum { IDD = IDD_ABOUTBOX }; //}}AFX_DATA // ClassWizard generated virtual function overrides //{{AFX_VIRTUAL(CAboutDlg) protected: virtual void DoDataExchange(CDataExchange* pDX); // DDX/DDV support //}}AFX_VIRTUAL // Implementation protected: //{{AFX_MSG(CAboutDlg) //}}AFX_MSG DECLARE_MESSAGE_MAP() }; CAboutDlg::CAboutDlg() : CDialog(CAboutDlg::IDD) { //{{AFX_DATA_INIT(CAboutDlg) //}}AFX_DATA_INIT } void CAboutDlg::DoDataExchange(CDataExchange* pDX) { CDialog::DoDataExchange(pDX); //{{AFX_DATA_MAP(CAboutDlg) //}}AFX_DATA_MAP } BEGIN_MESSAGE_MAP(CAboutDlg, CDialog) //{{AFX_MSG_MAP(CAboutDlg) // No message handlers //}}AFX_MSG_MAP END_MESSAGE_MAP() ///////////////////////////////////////////////////////////////////////////// // CEDRsNumberGuessingGameDlg dialog CEDRsNumberGuessingGameDlg::CEDRsNumberGuessingGameDlg(CWnd* pParent /*=NULL*/) : CDialog(CEDRsNumberGuessingGameDlg::IDD, pParent) { //{{AFX_DATA_INIT(CEDRsNumberGuessingGameDlg) m_input = 0; //}}AFX_DATA_INIT // Note that LoadIcon does not require a subsequent DestroyIcon in Win32 m_hIcon = AfxGetApp()->LoadIcon(IDR_MAINFRAME); } void CEDRsNumberGuessingGameDlg::DoDataExchange(CDataExchange* pDX) { CDialog::DoDataExchange(pDX); //{{AFX_DATA_MAP(CEDRsNumberGuessingGameDlg) DDX_Text(pDX, IDC_INPUT, m_input); //}}AFX_DATA_MAP } BEGIN_MESSAGE_MAP(CEDRsNumberGuessingGameDlg, CDialog) //{{AFX_MSG_MAP(CEDRsNumberGuessingGameDlg) ON_WM_SYSCOMMAND() ON_WM_PAINT() ON_WM_QUERYDRAGICON() ON_BN_CLICKED(IDC_EXIT, OnExit) ON_BN_CLICKED(IDC_SOLVE, OnSolve) //}}AFX_MSG_MAP END_MESSAGE_MAP() ///////////////////////////////////////////////////////////////////////////// // CEDRsNumberGuessingGameDlg message handlers BOOL CEDRsNumberGuessingGameDlg::OnInitDialog() { CDialog::OnInitDialog(); // Add "About..." menu item to system menu. // IDM_ABOUTBOX must be in the system command range. ASSERT((IDM_ABOUTBOX & 0xFFF0) == IDM_ABOUTBOX); ASSERT(IDM_ABOUTBOX < 0xF000); CMenu* pSysMenu = GetSystemMenu(FALSE); if (pSysMenu != NULL) { CString strAboutMenu; strAboutMenu.LoadString(IDS_ABOUTBOX); if (!strAboutMenu.IsEmpty()) { pSysMenu->AppendMenu(MF_SEPARATOR); pSysMenu->AppendMenu(MF_STRING, IDM_ABOUTBOX, strAboutMenu); } } // Set the icon for this dialog. The framework does this automatically // when the application's main window is not a dialog SetIcon(m_hIcon, TRUE); // Set big icon SetIcon(m_hIcon, FALSE); // Set small icon // TODO: Add extra initialization here srand(time(NULL)); number = rand_mid(0, 25); return TRUE; // return TRUE unless you set the focus to a control } void CEDRsNumberGuessingGameDlg::OnSysCommand(UINT nID, LPARAM lParam) { if ((nID & 0xFFF0) == IDM_ABOUTBOX) { CAboutDlg dlgAbout; dlgAbout.DoModal(); } else { CDialog::OnSysCommand(nID, lParam); } } // If you add a minimize button to your dialog, you will need the code below // to draw the icon. For MFC applications using the document/view model, // this is automatically done for you by the framework. void CEDRsNumberGuessingGameDlg::OnPaint() { if (IsIconic()) { CPaintDC dc(this); // device context for painting SendMessage(WM_ICONERASEBKGND, (WPARAM) dc.GetSafeHdc(), 0); // Center icon in client rectangle int cxIcon = GetSystemMetrics(SM_CXICON); int cyIcon = GetSystemMetrics(SM_CYICON); CRect rect; GetClientRect(&rect); int x = (rect.Width() - cxIcon + 1) / 2; int y = (rect.Height() - cyIcon + 1) / 2; // Draw the icon dc.DrawIcon(x, y, m_hIcon); } else { CDialog::OnPaint(); } } // The system calls this to obtain the cursor to display while the user drags // the minimized window. HCURSOR CEDRsNumberGuessingGameDlg::OnQueryDragIcon() { return (HCURSOR) m_hIcon; } void CEDRsNumberGuessingGameDlg::OnExit() { // TODO: Add your control notification handler code here OnOK(); } void CEDRsNumberGuessingGameDlg::OnSolve() { // TODO: Add your control notification handler code here if(m_input < number) { MessageBox("Sorry, but your number is a little bit too low. Please try again.", "A Little Low", MB_ICONINFORMATION); tries++; } else { MessageBox("Sorry, but your number is a little bit too high. Please try again.", "A Little High", MB_ICONINFORMATION); tries++; } if(m_input == number) { sprintf(buffer, "Congradulations! You solved the number which was %d, and it only took you\n %d tries.", number, tries); MessageBox(buffer, "You guessed right!", MB_ICONINFORMATION); } } int rand_mid(int low, int high) { return low + rand()%(high - low + 1); } [/b]
Any help would be GREATLY appreciated. Thanks.
EDIT: included the full source.
