Помогите пожалуйста исправить код программы на visual c++!!

не так тут то, что ваш код - на C#, а вы пишете, что нужен C++
исключение следует выбрасывать перед изменением значения.
шестнадцатеричный - хз как тут вообще применить это определение, но это явно не диапазон [-16, 16]
Код C#:
struct Range
{
private readonly int _min;
private readonly int _max;

public Range(int min, int max)
{
if (min > max)
throw new ArgumentException();

_min = min;
_max = max;
}

public bool Contains(int value)
{
return value >= _min && value <= _max;
}

public override string ToString()
{
return string.Format("[{0}, {1}]", _min, _max);
}
}

class Counter
{
private Range _range;
private int _value;

public Counter(Range range, int value)
{
if (!range.Contains(value))
throw new ArgumentException(string.Format("Value {0} not in {1}", value, range));

_range = range;
_value = value;
}

public int Value
{
get { return _value; }
}

public void Increment()
{
SetValue(Value + 1);
}

public void Decrement()
{
SetValue(Value - 1);
}

private void SetValue(int next)
{
if (!_range.Contains(next))
throw new InvalidOperationException();

_value = next;
}
}

а что вообще такое шестнадцатеричный счётчик? почему он у тебя имеет границы от -16 до +16? вообще-то шестнадцатеричное число может быть любым, система счисления его не ограничивает.. .

в конструкторе public Counter16(), чтобы не дублировать код, просто вызывай public Counter16(0)
а в конструкторе public Counter16(int counter) , соответственно, просто вызывай public Counter16(counter, -16, 16)

я бы функцию private void CheckBounres() переписал как public bool CheckBounres(int value), и пусть возвращает true или false - после простой проверки необязательно кидать исключение.

тогда бы ты смог в методах dec и inc сначала проверять полученное значение на корректность, а потом уже думать, менять ли текущее значение. А то ты сначала переменную запорол некорректным значением, а потом исключение кидаешь.