Hi.

The first thing is that you have semi-colons at the end of your define statements. Another thing that is important to know is that in several (or most) of the cases, there's a better approach than using macros. For instance, in this particular case, you could have this:

Code:
#include <iostream>
#include <string>

using namespace std;

const int MIN = 7;
const int MAX = 8;