Oceńcie moją prace ;)

Question

Witam, po obejrzeniu poradnika od Pana Mirosława odnośnie liczb pseudolosowych postanowiłem, że rozbuduję troszkę ten program lotto. Pierwsza wersja kodu była bez tablic gdyż nie wiedziałem jeszcze o nich, dopiero wczoraj obejrzałem kolejny odcinek z tablicami i postanowiłem że je dodam przez co zmniejszył mi się kod chociaż o troszkę. Proszę was o ocenę napisanie jakie błędy popełniłem co mógłbym zmienić na lepsze i jak można skrócić ten kod.

Pozdrawiam ;)

http://wklej.org/id/1888941/

Answer 1

Najbardziej rzucają mi się w oczy te wielke ify z decyzją. Najlepiej zamień małe litery na duże lub na odwrót i sprawdzaj tylko jedną możliwość. Zamiast:

if((decyzja=="Tak")||(decyzja=="tAk")||(decyzja=="taK")||(decyzja=="TAK")||(decyzja=="TAk")||(decyzja=="TaK")||(decyzja=="tak"))

Napisz:

    for(int i = 0; i < decyzja.size(); i++)
    {
        decyzja[i] = toupper(decyzja[i]);
    }
    if(decyzja == "TAK")

{//Dalszy kod}

Podobnie zrób z decyzja == "NIE" i innymi.

P.S: Jeśli nie zadziała dodaj biblioteke cctype.

 

Comments

Wiedziałem że da się to jakoś skrócić ale nie wiedziałem jak dzięki wielkie ;)

Nie wiem jednak co oznacza:  toupper

---

toupper to funkcja która zamienia małe litery na duże.

---

Pomógł byś mi zamienić tą linijkę na krótszą ?

if((liczby[0]>49)||(liczby[1]>49)||(liczby[2]>49)||(liczby[3]>49)||(liczby[4]>49)||(liczby[5]>49))

Próbowalem to przez pentle for ale coś mi nie działało i się w końcu poddałem

---

bool czy_wieksza = false;
                for(int i = 0; i < 5; i++)
                {
                    if(liczby[i] > 49)
                        czy_wieksza = true;
                }
                    if(czy_wieksza){//dalszy kod}

Nie wiem czy będzie działać bo nie kompilowałem.

 

 

---

Działa, ale i tak ten wpis wykracza za moją wiedze :D

Dzięki za pomoc pozdrawiam ;)

---

Może najpierw naucz się podstaw, potem zacznij pisać programy?

---

"Może najpierw naucz się podstaw, potem zacznij pisać programy?"

Jak nie będzie pisał programów z tego co się nauczył to zapomni.

---

Tego czego się nauczyłem do tej pory czyli po obejrzeniu 5 pierwszych odcinków z c++ starałem się zawrzeć w tym programie. A podstaw uczę się cały czas

Answer 2

Bardzo dobra aplikacja, prosta wiedza, a jak efektywnie użyta. Pozdrawiam.

Answer 3

Bardzo fajna aplikacja!

Mała wskazówka ode mnie:

Zamiast: 

if((liczby[0]>49)||(liczby[1]>49)||(liczby[2]>49)||(liczby[3]>49)||(liczby[4]>49)||(liczby[5]>49))

Ja bym zastosował:

while((liczby[0]>49)||(liczby[1]>49)||(liczby[2]>49)||(liczby[3]>49)||(liczby[4]>49)||(liczby[5]>49))

Wtedy program nie będzie działał dopóki ktoś nie wpisze prawidłowych cyfr :).

 

Pozdrawiam! :)

 

Comments

Dzięki działa, z for mi sprawdzało tylko jeden raz. Nie potrafiłem na to wpaść a wczoraj troszkę główkowałem nad tym :)

Pozdrawiam ;)

---

Możesz dać łapkę w górę :D.

---

jak pisałem kom nie mogłem dać :)

Answer 4

Błąd mało istotny, ale może zapamiętasz. zdarzyć/zdarzenie  oraz zdążyć/zdążenie. Gratuluję pierwszych sukcesów! Super kod i bądź tylko wytrwały, a na pewno będziesz sam z każdym odcinkiem ulepszał swoje kody.

Comments

Dzięki zapamiętam ;)

Answer 5

Jest kilka błędów "logistycznych".
1. Możesz "sprawdzanie kuponu" wyrzucić przed warunkami logicznymi i w warunkach logicznych po prostu wpisać daną instrukcję. Napisałeś 2 lub 3 razy Sleep(5000)

2. Tam przy wypisywaniu:

for(int i=0; i<5;i++)
                            {
                               liczby[i];
                                cout<<liczby[i]<<",";
                            }
                                cout<<liczby[5];

 

Możesz to zastąpić kodem

for(int i=0; i<6;i++)
                            {
                                cout<<liczby[i];
                                if (i<5) cout <<  ", ";
                            }
       

Co to jest za instrukcja?

// coś tam 
Liczby[i];
// coś tam

Kolejną sprawą jest to, że na końcu:
 

cout<<"Sa trzy mozliwosci dlaczego pojawil sie ten komunikat:"<<endl;
            cout<<"1. Zle wpisales Tak"<<endl;
            cout<<"2. Zle wpisales Nie"<<endl;
            cout<<"3. Wpisales cos innego niz Tak lub Nie"<<endl;

 

Wypisałeś wszystkie przypadki "TAK" oraz "NIE", więc nie ma możliwości wpisania źle "tak" lub 'nie" zważając na małe lub duże litery.  Jedyną możliwością jest wpisanie coś innego, niż "tak" lub "nie". Doczytaj coś o kontroli błędów, poczytaj o warunkach while, ucz się i rozwijaj. Też niedawno pisałem podobne programy ;)

P.S: Doucz się z tablic, na pewno to pomoże ;)

Comments

Komendę Sleep użyłem żeby to jakoś pozytywnie się wyświetlało dla oka bo gdyby to się na raz wyświetlało według mnie to by brzydko wyglądało .

Co do zastąpienia kodu na Twój to skorzystam chociaż w mojej wersji niepotrzebnie napisałem linie : liczba[i];

Nie wiem o co chodziło Ci z

// coś tam

Liczby[i];

// coś tam

i do ostatniego jakiś użytkownik może przecież napisać Tak w sposób " taak" bo np. klawiatura się przycięła i wypisała 2x"a"

Dzięki za rady na pewno skorzystam :)

Pozdrawiam ;)

Answer 6

Nie sprawdzałem, ale na oko dwukrotne złe wypełnienie kuponu przepuści drugie (złe) wypełnienie, do tego nie sprawdzasz, czy liczby mogą się powtarzać, oraz duplikujesz komendy w kilku miejscach. System powinien także pokazywać liczbę trafionych liczb i na ich postawie wyliczać wygraną.

Comments

Już poprawiłem to wypełnianie kuponu teraz dopiero jak kupon zostanie dobrze wypełniony to go przyjmie co do powtarzalności liczb zrobie to na czasie a co do obliczania wygranej i ilośći trafionych numerków to może wkrótce bo na razie nie mam pomysłu jak to zrobić:)