Taki oto programik z kilkoma pytaniami. Co o nim sądzicie? Co można w nim zmienić żeby był lepszy?

Question

#include <iostream>
#include <windows.h>
#include <cstdlib>
#include <stdio.h>

using namespace std;

int nr_odpowiedzi[4];

int main()
{
    cout << "Witaj w quizie!" << endl;
    cout << "Za 3 sekundy zobaczysz piewsze pytanie!" << endl;
    Sleep(3000);

    cout << endl;

    cout << "A. Jakiego koloru jest czerwony maluch?" << endl;
    cout << "------------------------------------------" << endl;
    cout << "1. Zielonego" << endl;
    cout << "2. Niebieskiego" << endl;
    cout << "3. Czerwonego" << endl;
    cout << "4. Czarnego" << endl;


    cout << endl << "Jaka jest Twoja odpowiedz: ";
    if (!(cin >> nr_odpowiedzi[0]))
    {
        cerr << "Nie ma takiej odpowiedzi!";
        exit(0);
    }

    switch(nr_odpowiedzi[0])
    {
    case 1:
        {
            cout << "Zla odpowiedz!";
            exit(0);
        }
    break;
    case 2:
        {
            cout << "Zla odpowiedz!";
            exit(0);
        }
    break;
    case 3:
        {
            cout << "Dobra odpowiedz!";

        Sleep(1000);
        system("cls");

        cout << "B. Ile wynosi 2 + 2 * 2?" << endl;
        cout << "------------------------------------------" << endl;
        cout << "1. 6" << endl;
        cout << "2. 8" << endl;
        cout << "3. 4" << endl;
        cout << "4. 10" << endl;

        cout << "Jaka jest Twoja odpowiedz: ";
        if (!(cin >> nr_odpowiedzi[1]))
    {
        cerr << "Nie ma takiej odpowiedzi!";
        exit(0);
    }

    switch(nr_odpowiedzi[1])
    {
    case 1:
        {
            cout << "Dobra odpowiedz!";

        Sleep(1000);
        system("cls");

        cout << "B. Co to jest jest monsun?" << endl;
        cout << "------------------------------------------" << endl;
        cout << "1. Jedzenie" << endl;
        cout << "2. Wiatr" << endl;
        cout << "3. Przesiedlenie" << endl;
        cout << "4. Kara" << endl;

        cout << "Jaka jest Twoja odpowiedz: ";
        if (!(cin >> nr_odpowiedzi[2]))
    {
        cerr << "Nie ma takiej odpowiedzi!";
        exit(0);
    }

    switch(nr_odpowiedzi[2])
    {
    case 1:
        {
            cout << "Zla odpowiedz!";
            exit(0);
        }
    break;
    case 2:
        {
            cout << "Dobra odpowiedz!";

        Sleep(1000);
        system("cls");

        cout << "B. Kiedy zmarl Cezar?" << endl;
        cout << "------------------------------------------" << endl;
        cout << "1. 46r. p.n.e" << endl;
        cout << "2. 145r. p.n.e" << endl;
        cout << "3. 45r." << endl;
        cout << "4. 44r. p.n.e" << endl;

        cout << "Jaka jest Twoja odpowiedz: ";
        if (!(cin >> nr_odpowiedzi[3]))
    {
        cerr << "Nie ma takiej odpowiedzi!";
        exit(0);
    }

    switch(nr_odpowiedzi[3])
    {
    case 1:
        {
            cout << "Zla odpowiedz!";
            exit(0);
        }
    break;
    case 2:
        {
            cout << "Zla odpowiedz!";
            exit(0);
        }
    case 3:
    break;
        {
            cout << "Zla odpowiedz!";
            exit(0);
        }
    break;
    case 4:
        {
            cout << "Dobra odpowiedz!";
        }
    break;
    default:
        {
            cout << "Nie ma takiej odpowiedzi!";
        }

    }

        }
    break;
    case 3:
        {
            cout << "Zla odpowiedz!";
            exit(0);
        }
    break;
    case 4:
        {
            cout << "Zla odpowiedz!";
            exit(0);
        }
    break;
    default:
        {
            cout << "Nie ma takiej odpowiedzi!";
        }
    }

        }
    break;
    case 2:
        {
            cout << "Zla odpowiedz!";
            exit(0);
        }
    break;
    case 3:
        {
            cout << "Zla odpowiedz!";
            exit(0);
        }
    break;
    case 4:
        {
            cout << "Zla odpowiedz!";
            exit(0);
        }
    break;
    default:
        {
            cout << "Nie ma takiej odpowiedzi!";
        }
    }
    }
    break;
    case 4:
        {
            cout << "Zla odpowiedz!";
            exit(0);
        }
    break;
    default:
        {
            cout << "Nie ma takiej odpowiedzi!";
        }
    }

    return 0;
}

 

Answer 1

sprawdzasz jednym warunkiem czy odp jest dobra czy zła

dodatkowa funkcja zapewni, że nikt nie da odpowiedzi, której nie ma na liście

dostosuj go do swoich potrzeb ;p

#include <iostream>
#include <stdio.h>
#include <conio.h>

int dozwoloneOpd()
{
    int key;
    do
    {
        key = _getch();
    }while (key < '0' || key > '4');
    return key - '0';
}

int main()
{
    int odp = dozwoloneOpd();
    if(odp == 3)        //odp jest prawidłowa
    {
        std::cout << "Dobrze";
    }
    else
    {
        std::cout << "Zle";
    }
    return 0;
}