Thread: Random Number problem in number guessing game...

Here is my source:

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]

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.

Any help would be GREATLY appreciated. Thanks.

EDIT: included the full source.