Помимо балансировки фигурных скобок, обратите внимание, что избыточно иметь if (iChoice==2)
при case 2:
, потому что ваш оператор switch включен iChoice
. То же самое касается iChoice==3
.
Edit:
Для уточнения необходимо написать
case 2:
if (iChoice == 2)
{
//...
}
в
case 2:
//...
Но это не то, что мешает вашему коду компилироваться. Как отмечали другие, ваши скобки не сбалансированы, и вы помещаете некоторые из своих операторов case в цикл, а другие нет.