Nie widzę, żebyś ustawiał gdziekolwiek w kodzie zmienną wybor. Domyślnie ma ustawioną wartość zero (bo to nie wiedzieć czemu zmienna globalna), więc switch wykona blok default i case '5', gdzie jest jest wywołanie funkcji exit (której w C++ nie powinno się używać).
wybor jest typu int, a w switchu testujesz literały znakowe. Bałamutne to...