Ezt egyszerűbben meg lehet oldani c++-ban?

ÁLLAPOTGÉP!!

Ezt a förtelmet nem akarom látni :(

Csinálj egy értelmes állapotgépet a menünek.

A külsö while ciklus, amit felfedeztél magadnak, az egy kezdetleges formája az úgynevezett "game loop"-nak: [link]

Itt annyit változtatnék, hogy eltüntetném azt a változót és átírnám a feltételt true-ra: [link]

Ezek után csináltam egy minimális refactort, csak kiszerveztem függvényekbe dolgokat: [link]

Itt egy kicsit ki van csavarva a logikát: A belsö két while-nak inkább if-nek kellene lennie. + amikor a vissza gombot beolvasod, ott if helyett while kéne, hisz addig akarsz az aktuális menüponton maradni, amíg meg nem nyomja a 'v'-t: [link]

Itt kéne egy minimális refactor. Ha megnézed, mindenhol ahol a vissza_kiir van hívva, ott mindíg egy ciklusban van -> ezt a ciklust is belerakjuk a ciklusba: [link]

Most itt van egy bug (már a te verziódban is benne volt, de most, hogy letisztult kicsit a kód, elég szembeötlö): Mi történik, ha a menüben nem az 1-est vagy a 2-est nyomjuk, hanem valami mást? Beragadunk egy végtelen ciklusba, úgy hogy nincs kiút. Szerencsére ezt könnyen fixálhatjuk: a legelsö menü kiiratást betesszük a fö ciklusba, így mindíg kiíródik, nem csak akkor, ha elözöleg sikerült az 1-est vagy a 2-est kiválasztani. Így kétszer fog kiíródni, hisz mindegyik menüpont alatt szintén kiíratjuk. Ezeket a sorokat simán kivesszük: [link]

Még itt van egy aprócska javítás: Amíg nem volt a menü kiszervezve egy függvénybe, addig a menü kezelésnél nem volt gond, hogy olyanok a feltételek, hogy valasztas == '1', mivel pár sorral feljebb kellett csak nézni, és már tudtad is, hogy mi az az '1'. Most viszont jó lenne valami értelmes nevet adni az '1'-nek, meg a '2'-nek: [link]