Początki programowania w C++, ocena kodu, uproszczenie go.

Question

Witam, jestem uczniem 3 klasy gimnazjum i dopiero zaczynam swoją przygodę z programowaniem. Jestem po drugiej lekcji podstaw programowania C++, uczyłem się z filmików PasjaInformatyki. Aby lepiej poznać ify, chciałem napisać małą grę, coś w stylu milionerów. Napisałem trochę kodu i chciałem dać go komuś doświadczonemu do wglądu, czy dobrze rozumuje, czy da się go uprościć i jak i czy jest sens pisania tego dalej, czy lepiej przejść do kolejnych lekcji. W dalszej części chciałbym wprowadzić pytania dwuetapowe typu Prawda/Fałsz, obie odpowiedzi na pytanie musiałyby być poprawne, oba warunki spełnione, by można było przejść do następnego pytania. Program jak na razie spełnia swoje zadanie. Oto kod:

 
 

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

using namespace std;

string x;

int main()
{
    cout << "Witaj w drugiej czesci gry Milionerzy, stworzonej przez Tomek, enjoy!"<<endl;
    Sleep(1000);
    cout<<"Oto pierwsze pytanie..."<<endl;
    Sleep(1000);
    cout<<"W ktorych latach istniala konfederacja barska?"<<endl;
    cout<<"A. 1769-1775"<<endl;
    Sleep(1000);
    cout<<"B. 1768-1772"<<endl;
    Sleep(1000);
    cout<<"C. 1670-1672"<<endl;
    Sleep(1000);
    cout<<"D. 1795-1798"<<endl;
    Sleep(500);
    cout<<"Twoja odpowiedz: ";
    cin>>x;

    if((x=="B")||(x=="b"))
    {
        cout<<"Poprawna odpowiedz, gratuluje! Przechodzisz dalej!"<<endl;

        Sleep(1000);
        cout<<"Oto nastepne pytanie... dziedzina wiedzy: Geografia..."<<endl;
        Sleep(1000);
        cout<<"Rzeka Wieprz to..."<<endl;
        Sleep(1000);
        cout<<"A. lewy doplyw Wisly"<<endl;
        Sleep(1000);
        cout<<"B. lewy doplyw Odry"<<endl;
        Sleep(1000);
         cout<<"C. prawy doplyw Wisly"<<endl;
    Sleep(1000);
    cout<<"D. prawy doplyw Odry"<<endl;
    Sleep(500);
    cout<<"Twoja odpowiedz: ";
    cin>>x;

        if ((x=="c")||(x=="C"))
        {
            cout<<"Poprawna odpowiedz, przechodzisz do nastepnej rundy!"<<endl;

               Sleep(1000);
        cout<<"Nastepne pytanie... dziedzina wiedzy: Historia..."<<endl;
        Sleep(1000);
        cout<<"Stanislaw Antoni Poniatowski objal tron Polski, jednoczesnie bedac ostatnim krolem w wolnej elekcji w:"<<endl;
        Sleep(1000);
        cout<<"A. 1764r."<<endl;
        Sleep(1000);
        cout<<"B. 1777r."<<endl;
        Sleep(1000);
         cout<<"C. 1853r."<<endl;
    Sleep(1000);
    cout<<"D. 1790r."<<endl;
    Sleep(500);
    cout<<"Twoja odpowiedz: ";
    cin>>x;

    if ((x=="a")||(x=="A"))
    {
        cout<<"Swietnie! nastepna runda!"<<endl;

    }
    else
    {
        cout<<"Bledna odpowiedz, sprobuj jeszcze raz!";
    }

        }
        else
        {
            cout<<"Niepoprawna odpowiedz, dziekuje za zagranie! The game was created by Tomek.";
        }
    }
else
{
    cout<<"Zla odpowiedz, try again!"<<endl;
    cout<<"Gra stworzona przez Tomek, thank you for your time!";
}

    return 0;
}

 

Answer 1

Generalnie tak:

• Nazwy zmiennych zawsze definiuj możliwie najbardziej zrozumiale. W tym przypadku może nie jest to takie szkodliwe, ponieważ zdefiniowałeś tylko jedną zmienną, lecz złe nawyki zostają, a za pół roku będziesz się zastanawiał o co chodziło ze zmienną abcd w twoim kodzie :|
• Pamiętaj aby zachowywać poprawne wcięcia w kodzie (tabulacje) bardzo poprawia to czytanie kodu.
• Jeżeli kod po if'ie ma tylko jedną linijkę, to nie musisz pisać klamer:

  if(zmienna == 10)
      cout << "Zmienna wynosi 10" << endl;

   

• Myślę że jak na drugi odcinek to nie jest źle, ale jeszcze duuużo się musisz nauczyć :)

Pozdrawiam :)

Comments

I oczywiście w warunku operator porównania == nie przypisania = :)

---

Co do punktu 3 - sam prawdę mówiąc zazwyczaj tak robię, choć nie każdy zgodzi się z tą konwencją. Robert C. Martin pisał, że nie poleca opuszczania nawiasów klamrowych.

---

Cholera, tak podstawowy błąd ;_;
Dzięki za upomnienie.

---

Nie uzywaj przestrzeni nazw i unikaj zmiennych globalnych

Answer 2

1) Postaraj się powywalać jak najwięcej kodu z main do osobnych funkcji poniżej - zwłaszcza z bloków if i else.

2) Formatuj wcięcia - nie wiem czy masz taki układ czy ci się rozsypało przy wklejaniu na forum, jednak ilość tabów/spacji w lewym marginesie powinna ułatwiać czytanie kodu, nie utrudniać. 

Źle:

        if ((x=="c")||(x=="C"))
        {
            cout<<"Poprawna odpowiedz, przechodzisz do nastepnej rundy!"<<endl;
 
               Sleep(1000);
        cout<<"Nastepne pytanie... dziedzina wiedzy: Historia..."<<endl;
        Sleep(1000);
        cout<<"Stanislaw Antoni Poniatowski objal tron Polski, jednoczesnie bedac ostatnim krolem w wolnej elekcji w:"<<endl;
        Sleep(1000);
        cout<<"A. 1764r."<<endl;
        Sleep(1000);
        cout<<"B. 1777r."<<endl;
        Sleep(1000);
         cout<<"C. 1853r."<<endl;
    Sleep(1000);
    cout<<"D. 1790r."<<endl;
    Sleep(500);
    cout<<"Twoja odpowiedz: ";
    cin>>x;
 
    if ((x=="a")||(x=="A"))
    {
        cout<<"Swietnie! nastepna runda!"<<endl;
 
    }
    else
    {
        cout<<"Bledna odpowiedz, sprobuj jeszcze raz!";
    }
 
        }
        else
        {
            cout<<"Niepoprawna odpowiedz, dziekuje za zagranie! The game was created by Tomek.";
        }

Lepiej:

if ((x=="c")||(x=="C"))
{
    cout<<"Poprawna odpowiedz, przechodzisz do nastepnej rundy!"<<endl;
    Sleep(1000);
    cout<<"Nastepne pytanie... dziedzina wiedzy: Historia..."<<endl;
    Sleep(1000);
    cout<<"Stanislaw Antoni Poniatowski objal tron Polski, jednoczesnie bedac ostatnim krolem w wolnej elekcji w:"<<endl;
    Sleep(1000);
    cout<<"A. 1764r."<<endl;
    Sleep(1000);
    cout<<"B. 1777r."<<endl;
    Sleep(1000);
    cout<<"C. 1853r."<<endl;
    Sleep(1000);
    cout<<"D. 1790r."<<endl;
    Sleep(500);
    cout<<"Twoja odpowiedz: ";
    cin>>x;
 
    if ((x=="a")||(x=="A"))
    {
        cout<<"Swietnie! nastepna runda!"<<endl;
    }
    else
    {
        cout<<"Bledna odpowiedz, sprobuj jeszcze raz!";
    }
}
else
{
    cout<<"Niepoprawna odpowiedz, dziekuje za zagranie! The game was created by Tomek.";
}

3) Poza drobnymi wyjątkami (jak np. iterator w pętli for) unikaj nazw zmiennych typu "x", "dupa", "zmienna". Lepiej ją nazwać np. "userAnswer" lub wybrać inną nazwę zgodną z twoją konwencją, ale zawierającą informację czym jest zmienna.