Wysypujący się program.

Witam!

Jestem totalnie nowy w C++ i chciałem tak dla własnej satysfakcji stworzyć sobie quiz, niestety wysypuje się na drugim pytaniu. Kod źródłowy quizu:

#include <iostream>
#include <conio.h>
#include <algorithm>
#include <string>
#include <cstdlib>
#include <string>

using namespace std;

int punkty = 0, i;

string baza();
string pytanie();
string wybor();
string koniec();
string pyt[40], odp[40], odpowiedz;

char t, przycisk, dowolny;

int main()
{
    baza();

    // QUIZ //
    cout << "Wybierz temat:" << endl;
    cout << "1. Sport" << endl;
    cout << "2. Religie" << endl;
    cout << "3. Nauki scisle" << endl;
    cout << "4. Historia Polski XIX i XX wieku" << endl << endl;

    cout << "Aby wybrac temat, wpisz numer jemu odpowiadajacy: "; t = getch();
    wybor();

    return 0;
}


// FUNCKJE

string baza() // BAZA PYTAŃ I ODPOWIEDZI


string pytanie()
{
        system("cls");
        cout << pyt[i] << endl << "Wpisz odpowiedz: "; cin >> odpowiedz;

        if(odpowiedz == odp[i]) {punkty++;} 
        else {cout << "Poprawna odpowiedz: " << odp[i];}

        transform(odpowiedz.begin(), odpowiedz.end(), odpowiedz.begin(), ::toupper); if(odpowiedz == odp[i]) {punkty++;} else{}
        system("cls");
}

string wybor()
{
    if(t == '1') {for(int i = 0; i < 10; i++) {pytanie();}}
    if(t == '2') {for(int i = 10; i < 20; i++) {pytanie();}}
    if(t == '3') {for(int i = 20; i < 30; i++) {pytanie();}}
    if(t == '4') {for(int i = 30; i < 40; i++) {pytanie();}}
}

Podejrzewam, że program się wysypuje z powodu tej ostatniej funkcji, ale nie wiem, jak to zmienić. Prosiłbym o podpowiedź lub o jedną linię kodu (chociaż sam wolałbym do tego dojść), która nakieruje mnie na rozwiązanie.

zmienna i globalna jest nie zainicjalizowana. Przestan uzywac zmiennych globalnych to problem zniknie

edit. Tak dawno nie uzywalem zmiennych globalnych (bo jest to zle) ze zapomnialem, ze bedzie zainicjalizowana zerem (w tym przypadku) nadal. Usun globalne zmienne to problem zniknie. Do tego nazywaj zmienne sensownie

Nie rozumiem dlaczego sądzisz że i z pytanie to jest i z pętli w wybor.

Jeśli definiujesz funkcję jako zwracającą Typ to musisz ten typ zwrócić. Masz tu UB.

Porada dla Ciebie na czas, kiedy nie wiesz co robisz:

• Zamiast type array_name[size] używaj std::array<type, size> array_name;
• zamiast array_name[index] używaj array_name.at(index)

Te zmiany pozwolą Ci wyłapać wszystkie głupie błędy związane z odwoływaniem się do nieistniejących elementów.

PS. nie używaj zmiennych globalnych.

Każdy pisze, żebym nie używał zmiennych globalnych. Może mi ktoś wyjaśnić dlaczego? Przepraszam, że tak głupio pytam, ale jestem newbie w tym temacie.

1. utrudniasz czytanie kodu innym. Widząc zmienną globalną nie masz pojęcia jaki jest jej udział w wykonaniu innych funkcji. Czy jeśli zmienię ImieStefana to czy funkcja mojsinus(x) będzie zwracać inne wyniki? Z perspektywy czytającego każda zmienna globalna to dodatkowy parametr do wszystkich istniejących funkcji.
2. utrudniasz testowanie. Zamiast sprawdzić, że mojsinus(π) == 0, musisz zastanowić się jak poustawiać zmienne globalne, aby otrzymać poprawny wynik
3. utrudniasz/uniemożliwiasz wielowątkowość. Jeśli podczas swojej pracy operujesz na zmiennych globalnych, a jakaś inna funkcja z innego wątku również używa tych samych...

Wyobraź sobie że masz duży, duży projekt. Nazwy mogą się zacząć w pewnym momencie pokrywać. Przykładowa sytuacja: masz zmienną globalną faultId, oraz funkcję która przyjmuje parametr o takiej samej nazwie. W procesie refaktoringu usuwasz go z funkcji, ale przez nieostrożność nie usunąłeś go z wszystkich wystąpień w funkcji - kompilator Cię o tym nie poinformuje bo z jego perspektywy wszystko jest OK, a u Ciebie w programie zaczynają się cuda :)
Generalnie sytuacja jest trochę nadmuchana, ale jak najbardziej możliwa. Chodzi przede wszystkim o utrzymanie porządku w kodzie.

Dodałbym jeszcze:
4. dobrą praktyką jest największe możliwe ograniczanie scope każdego obiektu
5. utrudniasz kompilatorowi optymalizacje kodu

Łał, dziękuję serdecznie za tak szybkie odpowiedzi. Teraz już będę wiedział, dlaczego nie warto używać zmiennych globalnych dla funkcji.