Usprawnienie kodu na poziomie podstawowym

Question

Cześć dostałem na zadanie żeby zrobić program, który:

"Wyobraź sobie, że przychodzi do Ciebie gość. Musisz się go zapytać czy chce kawe czy herbatę oraz do tego ciastko albo nic. Musi wybrac jedną z dwóch rzeczy"

Oto mój kod. Jakbyście wykonali zadanie ? Dodam, że JESZCZE nie programuje, ale uczę się i robię kursy.

#include <iostream>

using namespace std;

string kawa,ciastko;

int main()
{
    cout<<"Czesc kolego!"<<endl;
    cout<<"Kawa, herbata ?"<<endl;
    cin>> kawa;
    if (kawa=="kawa")
    {
        cout<<"Masz kawe"<<endl;
    }
    else
    {
        cout<<"Masz herbate"<<endl;
    }
    cout<<"Ciastko do tego ?"<<endl;
    cout<<"tak czy nie?"<<endl;
    cin>>ciastko;
    if (ciastko=="tak")
    {
        cout<<"Masz ciastko";
    }
    else
    {
        cout<<"Nie to nie :)";
    }


    return 0;
}

 

Answer 1

A co się stanie jeśli na pytanie kawa czy herbata? "Gość" odpowie sok pomarańczowy albo jaszczurka :)?

Edit: Przypomniał mi się żart:
Do baru wchodzi Tester i zamawia:
1 piwo
999999999 piw
-1 piwo
Jaszczurkę

 

Edit2: Nie używaj zmiennych globalnych, Twoje std::string na spokojnie mogą być w ciele int main()

Comments

Żartownisia można skasować, sprawdzając dane które są wprowadzane do programu.

---

https://youtu.be/I0NGCKOeT1E

Nawet ostatnio prowadziłem krótką rozmowę, bo myślałem, że testowanie oprogramowania polega na deassemblacji itd. coś jak w reverse engineeringu,

A tu się okazuje, że siedzi sobie pani Krysia przed programem i wpisuje literki w polu na numer telefonu.

Answer 2

#include <iostream>

//Parametry funkcji zostają przekazane przez wartość 
bool GetUserInput(std::string msg,std::string err_msg, std::string option_1, std::string option_2){
    //Wybierz
    std::cout << msg << "  options : [" << option_1 << ',' << option_2 << "]\n";

    //tzw nieskończony while, będzie powtarzać się
    while(true){
        //Zmienna lokalna na wejście użytkownika
        std::string user_input="";

        //Pobierz wejście od użytkownika
        std::cin >> user_input;

        //Pierwsza opcja
        //Podczas porównywania wykorzystywane jest przeciązenie operatora == dla std::string
        if(user_input == option_1){
            //Return wychodzi z funkcji i zwraca odp. wartość.
            return true;
        }//Lub jeśli druga opcja
        else if(user_input == option_2){
            return false;
        }//Wszystko inne
        else
        {
            //Wypisz wiadomość o błędzie
            std::cout << err_msg << '\n';
        }
    }
    //Nigdy nie zostanie osiągnięty
    return false;
}

int main()
{
    //Wypisujemy powitalną wiadomość
    std::cout << "Czesc kolego!\n";

    //Używamy tej samej wiadomości o błędzie w obu przypadkach, może zostać zadeklarowana jako stała.
    const std::string error_msg = "Podano zla wartosc, podaj ponownie!";

    //Sprawdzanie wejścia użytkownika wyrzucone do zewnętrznej funkcji
    bool coffee_tea = GetUserInput("Kawa czy herbata?", error_msg, "kawa", "herbata");

    //Używamy wartości zwróconej z funkcji do zainicjalizowania zmiennej lokalnej.
    bool cookie = GetUserInput("Czy chcesz jeszcze ciastko?", error_msg, "tak", "nie");

    //Operator ternarny sprawdza warunek (w tym przypadku jest to zmienna logiczna) i zwraca jedną z 2 wartości(muszą być tego samego typu).
    //W tym przypadku wynikiem operatora ternarnego jest stała w postaci cioągu znaków.
    //Warto zwrócić uwagę na nawiasy, wynikiem wyrażenia w nawiasach jest jeden z 2 stałych ciągów znaków
    std::cout << "Masz " << (coffee_tea ? "kawe" : "herbate") << (cookie ? " i ciastko" : " i to wszystko") << "!\n";

    //Wynik dla systemu operacyjnego. 0 oznacza brak błędów w wykonaniu.
    return 0;
}

Warto się zastanowić nad problemem. W obu przypadkach wybór ma tylko 2 opcje, do tego dochodzi sprawdzenie czy wejście użytkownika jest poprawne. 

Comments

Super wytłumaczyłeś i to wykonałeś, dzięki :))))

---

@afferu,  Mam jeszcze kilka pytań. Czemu wszystko pisałeś z std ?

std::string
std::cout 

itp.

Czemu w niektórych miejscach na końcach wierszy jest {

bool GetUserInput(std::string msg,std::string err_msg, std::string option_1, std::string option_2){

while(true){

itp.

Co oznacza '\n';

std::cout<<err_msg<<'\n';

 

---

1. std::string oznacza, że odwołujesz sie do obiektu string w przestrzeni nazw(namespace) std czyli po prostu bibliotece standardowej. using namespace std; oznacza tyle, że używasz tej przestrzeni nazw w zasiągu gdzie znajduje sie to polecenie.

2. To gdzie stawiasz nawiasy zasięgów (scope) to sprawa stylu pisania.

3. '\n' to znak nowej linii tzw escape character, powoduje, oznacza przejście do nowej linii.

---

Stało by się coś gdybym napisał string zamiast std::string ?

Answer 3

Nie rób czegoś takiego, jak:  kawa == "kawa" :)  [*] Załóż zmiennną "wybór" (choice) i używaj angielskich nazw:

np:

byte choice;

Wystarczy na 256 "wyborów" (napoi/drinków).

[*]  - zamiast zmiennej kawa, lepsza byłaby boolean kawa_czy_nie_kawa;   :D  <- ale tak też nie rób.

 

Możesz zrobić małe menu wyboru:

1) kawa

2) mleko

3) piwo

i zapytać o liczbę od 1 do 3.....Oczywiście warto sprawdzić poprawność wprowadzonych danych.

Comments

byte choice;

jeśli mówisz o https://en.cppreference.com/w/cpp/types/byte to nie zgadzam się, że to dobry pomysł. Jeśli zrobiłeś kalke z Javy i miałeś na myśli 8-bit signed integer to prędzej :P

---

@NIMuser, Super, że odpowiedziałeś. Jutro spróbuję zrobić to z tym menu, jest to świetny pomysł, a zarazem podejście z innej perspektywy. W komentarzu wkleję kod jak mi poszło ;p

---

@RafalS, Masz rację, 100 lat nie używałem tego typu.