c++ как оптимизировать функцию ?

[proef]

как оптимизировать функцию vibor()?
(проект MSVS 2012)

 void vibor()
	{
		l:
	int ch = _getch();
	

     if(ch ==224)
		switch(ch)
	{
		case 80:
			if(ConsoleCurrent<=4)
			++ConsoleCurrent;
			else if(ConsoleCurrent >4){ConsoleCurrent = 1;}
			DrawMainMenu();
			break;
		case  72:

			if (ConsoleCurrent >=1)--ConsoleCurrent;
			else if(ConsoleCurrent <1){ConsoleCurrent = 4;}
			DrawMainMenu();
			break;
			
			case 13:
				enter(ConsoleCurrent);
				break;
			case 27:{
				int Result = MessageBox(0,L"хотите выйти?",L" ",MB_OK);
				if(Result == IDYES){exit(0);}
				else{ DrawMainMenu();}
				}
				break;
	
			case 75:
	{
		ShowContextMenu();
	}
	
			default :
				goto l;
		}
	 switch(ch)
    {
    case 80: 
		if(ConsoleCurrent<=4) {++ConsoleCurrent;}
		if(ConsoleCurrent >4){ConsoleCurrent = 1;}
       
        
          DrawMainMenu();
        break;
    case 72: 
        if(ConsoleCurrent>=1) {--ConsoleCurrent;}
     if(ConsoleCurrent <1){ConsoleCurrent = 4;}
        DrawMainMenu();
        break;
    case 13: 
        enter(ConsoleCurrent);
        break;

		case 27:
				{
					
				
			
                  int Result = MessageBox(0,L"хотите выйти?",L" ",MB_YESNO);
				if (Result == IDYES){
				exit(0);

				}
				else{ DrawMainMenu();
				}
					
					}
				break;
				case 75:
	{
		ShowContextMenu();
	}
	break;
	default:
		goto l;
	 }}

весь код здесь

[proef]

так то оно работает но как сделать без goto операторов?

[Smogg]

Например, через while:

char TestExit(char ch){
	if (ch == 27)
	{
		int Result = MessageBox(0,L"хотите выйти?",L" ",MB_OK);
		if(Result == IDYES)
			return 0;

	}
	return ch;
}

void vibor()
{
	int ch;
	while(ch = TestExit(_getch()){
		switch (ch){
			case 1:
				....
			break;
			case 2:
				....
			break;
		}
	}
}

[Son Of Pain]

if(ch ==224)
		switch(ch)
	{
		case 80:

Думаю, ее лучше переписать, а не оптимизировать.

[proef]

ну и как её переписать ?
обьясните