Sprawdzenie zadania, optymalizacja kodu.

Question

Witam ponownie. Dzięki wam problem z tablicą jest już nieaktualny (ehh określenie 1 komórki a chęć korzystania z 2)

Treść zadania:

Napisz program udostepniajacy opcje sumowania wynikow ankiety, w ktorej moga wysatpic
trzy rozne wartosci. Dane wejsciowe wprowadzane do programu to pytanie ankietowe
oraz trzy mozliwe odpowiedzi. Pierwszej odpowiedzi przypisywana jest wartosc 1, drugiej 2,
a trzeciej 3. Odpowiedzi sa sumowane do chwili, w ktorej uzytkownik wprowadzi 0.
- wtedy program powinien pokazac wyniki ankiety. Postaraj sie wygenerowac wykres slupkowy
pokazujacy wyniki przeskalowane w taki sposob, aby zmiescily sie na ekrane bez wzgledu na
liczbe udzielonych odpowiedzi.

Wykres słupkowy na chwilę obecną odpuszczam.

Mój kod:

#include <iostream>
#include <string>

using namespace std;

int main()
{
    string pytanie;
    string odpowiedz[3];
    cout<<"Wprowadz pytanie do ankiety: ";
    getline(cin, pytanie, '\n');

    for(int i=0; i<3; i++)
    {
        cout<<"Podaj: "<<i+1<<" odpowiedz: ";
        getline(cin, odpowiedz[i], '\n');
    }
    
    cout<<"\nWitam ankieterow\n\n";
    cout<<"Pytanie:\n";
    cout<<pytanie<<endl;
    cout<<endl<<"Odpowiedzi:"<<endl;

    for(int i=0; i<3; i++)
    {
        cout<<i+1<<": "<<odpowiedz[i]<<endl;
    }
    int wybor_odpowiedzi[3]={};
    int numer;

    while(true)
    {
        cout<<"Wprowadz numer odpowiedzi: ";
        cin>>numer;

    switch (numer)
    {
    case 1:
            cout<<"wybrales: "<<odpowiedz[0]<<endl;
            wybor_odpowiedzi[0]=wybor_odpowiedzi[0]+1;
            break;
    case 2:
            cout<<"wybrales: "<<odpowiedz[1]<<endl;
            wybor_odpowiedzi[1]=wybor_odpowiedzi[1]+1;
            break;
    case 3:
            cout<<"wybrales: "<<odpowiedz[2]<<endl;
            wybor_odpowiedzi[2]=wybor_odpowiedzi[2]+1;
            break;
    default:
    cout<<"Nieprawidlowy numer.\n";
    break;
    }
    
    if(numer==0)
    {
        cout<<"Wybrales 0, czas na wyniki ankiety\n";
        cout<<"Odpowiedz 1 padla: "<<wybor_odpowiedzi[0]<<" razy.\n";
        cout<<"Odpowiedz 2 padla: "<<wybor_odpowiedzi[1]<<" razy.\n";
        cout<<"Odpowiedz 3 padla: "<<wybor_odpowiedzi[2]<<" razy.\n";
        break;
    }
    }
}

Prosiłbym o wskazówki czego wystrzegać się na przyszłość. Jak skrócić kod. Zastanawiam się nad treścią zadania. Mam nadzieję że dobrze je zinterpretowałem.

Z góry dziękuje za pomoc.

Answer 1

1. int wybor_odpowiedzi[3]={}; możesz zmienić na po prostu int wybor_odpowiedzi[3]

2. Zamiast tego całego switcha możesz zrobić coś takiego:

cout<<"wybrales: "<<odpowiedz[wybor - 1]<<endl;

            wybor_odpowiedzi[wybor - 1]=wybor_odpowiedzi[wybor - 1]+1;

3. cout<<"Odpowiedz 1 padla: "<<wybor_odpowiedzi[0]<<" razy.\n"; możesz zamienić na pętle wykonująca się do póki i + 1 < niż rozmiar tablicy, a w niej robić coś takiego: cout<<"Odpowiedz " + i + "padla" <<wybor_odpowiedzi[i - 1]<<" razy.\n"; Nie wiem czy dokładnie tak, ale chyba wiesz o co chodzi.

4. Co do treści zadania to nie zagłębiałem się.

Comments

int wybor_odpowiedzi[3]={}; możesz zmienić na po prostu int wybor_odpowiedzi[3]

nie możesz, bo wtedy wartości nie byłyby zainicjalizowane.

Answer 2

Również rozwiązuje to zadanie, jednak zastanawiam się, jak narysować wykres. Może jakaś wskazówka?