[C++] как обойтись без goto?

[ZaVyLoN]

Всем HI
только нач ал как говориться делать первые шаги и возник вопрос

вопрос я задал в самом коде в коментах  

сам код :

Код:

void show()
{
	system("cls");
				cout<<"\tИнформация о предприятиях\n\n";
				cout<<"==========================================";
}
int main()
{
	setlocale(0, "rus");
	srand(time(0));

	char k; 
	int x;
	int z, klav;

	Confectionery c;
	c.init();
	c.podschet();
	Meat m;
	m.init();
	m.podschet();
	Bakery b;
	b.init();
	b.podschet();
	Upravlenie u;

	
	system("cls");
	cout<<"\t** Меню **\n"<<endl;
	cout<<" ->  Просмотреть информацию   "<<endl;
	cout<<"    Доходы                   "<<endl;
	cout<<"    Выход                    "<<endl;

		
	int p = 1;
	int m1 = getch();
	do
	{
		if(m1 == 80) 
			p++;
		if(m1 == 72) 
			p--;

		do
		{
			if(p<=0) 
				p = p+3;
		}while (p<=0);
		do
		{
			if(p>=4) 
				p = p-3;
		}while (p>=4);

		if(p == 1)
		{
			system("cls");
			cout<<"\t** Меню **\n"<<endl;
			cout<<" ->  Просмотреть информацию   "<<endl;
			cout<<"    Доходы                   "<<endl;
			cout<<"    Выход                    "<<endl;
			m1 = getch();
			if(m1 == 13) 
			{
				system("cls");
				cout<<"\tИнформация о предприятиях\n\n";
				cout<<"==========================================";
				c.show();
				m.show();
				b.show();

     // вот собоственно как отсюда выйти в главное меню не использую goto , тоже самое и в других 2 -x if()   :confused: :dontknow:
				
				                                   
			}
		}
		if(p == 2)
		{
			system("cls");
			cout<<"\t** Меню **\n"<<endl;
			cout<<"    Просмотреть информацию   "<<endl;
			cout<<" ->  Доходы                   "<<endl;
			cout<<"    Выход                    "<<endl;	
			m1 = getch();
		}
		if(p == 3)
		{
			system("cls");
			cout<<"\t** Меню **\n"<<endl;
			cout<<"    Просмотреть информацию   "<<endl;
			cout<<"    Доходы                   "<<endl;
			cout<<" ->  Выход                    "<<endl;
			m1 = getch();
		}
	}while(m1!=13);
	
	
	return 0;
}

[Dr.Yevhenius]

Всю работу з меню вставь в бесконечный цикл (например, while(true)), а потом, если нужно, командой continue сможешь вернуться в начало, а командой break выйти с цикла и завершить работу. Такой вариант.

P.S. Было бы легче разобраться в коде, если бы не очень важные моменты упустил или заменил, к примеру, троеточием. 

[Алексей++]

встречный вопрос: а где главное меню ?

и вообще, структура кода даже на беглый взгляд корявая... Расскажи задачу

[Sla]

не вижу необходимости здесь в goto
ведь все попадает в цикл
do {
}while(m1!=13);

Если я правильно расшифровал

[ZaVyLoN]

Inkognito,
спс , попробую))

Алексей1153++,
cout<<"\t** Меню **\n"<<endl;
cout<<"    Просмотреть информацию   "<<endl;
cout<<" ->  Доходы                   "<<endl;
cout<<"    Выход                    "<<endl;

есть меню, чтобы стрелочка бегала туда сюда написал,.
при нажатии Enter как для первого случая выводиться инф.
а как потом войти обратно в меню!??

и что корявого!?
подскажи будем исправлять)

[Алексей++]

про стрелочку я не понял, так как программу не запускал )

Корявость - и неуниверсальность тоже - я бы изначально спроектировал в виде классов. Напрашивается сразу класс CMyMenu

Даже неважно, что там, а уже код на порядок станет более лёгким и универсальным

[ZaVyLoN]

-> safas
    asfda
    asdasd

     safas
-> asfda
    asdasd

    safas
    asfda
->asdasd

вот что я имел ввиду.
каждый раз перерисовываешь меню, имитируя движени

[Алексей++]

 Тут даже классы не нужны ИМХО ) Три строки-константы и один бесконечный цикл

для универсальности заменяем константные строки на массив или вектор и всё же завёртываем в класс. )))

[Алексей++]

вот примерное так (писал на коленке в блокноте на сильно уродской клаве на ноутбуке - лишь хочу показать идею, компилировать даже не пытался )) )

Код:

const char* pointer="->\t";
const char* emptypointer="  \t";

const char* menulist[]=
{
	"Просмотреть информацию",
	" Доходы  ",
	"Выход ",
	0
};

void printMenu(int pos_zb=0)
{
	system("cls");
	for(int i=0; menulist[i];i++)
	{
		if(pos_zb==i)
		{
			cout<<pointer;
		}
		else
		{
			cout<<emptypointer;
		}

		cout<<menulist[i]<<endl;
	}
}

int main()
{

	
	system("cls");


	printMenu();

		
	do
	{
		...

		printMenu(текущая выбранная позиция,начинающаяся с 0);

		...		

	}while(m1!=13);
	
	
	return 0;
}

[Sla]

Код:

do
	{
		...
		printMenu(текущая выбранная позиция,начинающаяся с 0);
		...		
m1 = getch();
	}while(m1!=13);

[resource]

Алексей1153++, позволю себе поинтересоваться. Что такое print(0) ?

[Алексей++]

printMenu )

подправил

[resource]

Алексей1153++, понятно. А то я уж подумал грешным делом, что выдруг есть что-то такое, а я и не вкурсе. Серьезно говорю.
Я вот вроде пишу сколько, а буквально только года 2 назад узнал, что printf'ом можно вполне законно выводить строки, не заканчивающиеся нулем. Это как бы шок для меня был. Столько лет printf себе и printf, а тут такое оказывается... С тех пор уже готов к любым неожиданностям 

[Алексей++]

resource, printf -м не пользуюсь

[Алексей++]

printf'ом можно вполне законно выводить строки, не заканчивающиеся нулем

а покажи, интересно, в чём шок )

[resource]

ну насчет шока это я конечно фигурально выразился  
скажем так, я был удивлен.

A format specification, which consists of optional and required fields, has the following form:

%[flags] [width] [.precision] [{h | l | ll | I | I32 | I64}]type

Я как-то никогда не обращал внимание на такое возможное значение "*", полей "width" и ".precision". Вроде простая и элементарная вещь. Но я просто как-то не знал даже о таком. И когда увидел, что в printf пихают не нуль-строку, подумал

бред какой-то

оказалось не бред. Хотя на тот момент, я был в этом уверен. И  сам для себя обнаружил, что столько лет не знал элементарной вещи. Поэтому сейчас и предпочитаю поинтересоваться если вижу что-то непонятно, прежде чем утверждать. А то вдруг опять.... что-то  

похоже наоффтопил маленько...