Operacja na stringu

co zrobiłem źle/niepoprawnie/bezmyślnie

Przede wszystkim źle/niepoprawnie/bezmyślnie (niepotrzebne skreślić - maksymalnie 2) piszesz w tym temacie.
_{» Porady dla użytkowników cpp0x.pl ♦} Zakładanie tematów na forum ^zalecenie

Co do kodu: wiele rzeczy można w nim zapisać krócej. Na przykład switcha zmieniającego kolejność wyświetlania wariantów, funkcję NumberOfLines() można wywalić i liczyć pytania przy faktycznym wczytywaniu, itd..

Wydawało mi się, że lepiej zliczyć pytania przy włączaniu programu(zrobić to raz) niż liczyć to przy każdym losowaniu pytania. Sądziłem, że zaoszczędzę na czymś... sam nie wiem na czym, może optymalności - trudno mi powiedzieć bo to dopiero 3 dnień mój z c++.
A co do skracania kodu to.. czy krócej znaczy "lepiej/optymalniej"?(niestety tego nie wiem).

lepiej zliczyć pytania przy włączaniu programu(zrobić to raz) niż liczyć to przy każdym losowaniu pytania.

A nie lepiej wczytać i zliczyć pytania raz, na początku programu?

Tak początkowo chciałem zrobić lecz pomyślałem, że jeśli pytań będzie dużo program będzie rezerwował sporo pamięci(w sumie przy dzisiejszych komputerach to pikuś, lecz troche się obawiałam tej metody) dlatego też postanowiłem na początku zliczyć pytania a następnie losując numer pytania wczytując daną linie z pytaniem(miałem z tym sporo kłopotów).