Optymalizacja kodu, Totolotek. Niechlujne porównywanie wartości w dwóch tablicach.

pytanie zadane 10 sierpnia 2016 w C i C++ przez Zekkeyo Nowicjusz (190 p.)

Napisałem sobie totolotka. Niby okej, ale postanowiłem zapisać liczby użytkownika jak te wylosowane przez komputer do tablic. Problem pojawił się, gdy trzeba było wymyślić pętle do porównanie tych tablic. Łącznie 36 porównań = 6 liczb użytkownika oraz 6 wylosowanych. Wykminiłem, ale mam wrażenie, że to jest mega słabe. Macie jakiś pomysł? Ostatnia pętla.

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


using namespace std;

int main()
{
    cout << "Witaj w losowaniu Duzego Lotka! Wybierz swoje 6 liczb z przedzialu 1-49: " << endl << endl;

    srand(time(NULL));
    int tablica_liczb_uzytkownika[6];
    int tablica_liczb_wylosowanych[6];

    for (int i=0; i<= 5; i++)
    {
        tablica_liczb_wylosowanych[i] = rand()%49+1; // losowanie
        cout << "Prosze podac " << i+1 << " liczbe: ";
        cin >> tablica_liczb_uzytkownika[i];

    }

    cout << "\n\nLosowanie nastapi za "; // no dobra, torche oszukane

    for (int i=3; i>=1; i--)
    {
        cout << i;
        Sleep(1000);
        cout << '\b'; // usun znak poprzedzajacy kursor
    }

    cout << "\r                      " << "\rWylosowane liczby to: " << endl; // ustaw kursor na poczatku, wstaw spacje, zastap

    for (int i=0; i<= 5; i++)
    {
        cout << "\nWylosowana liczba" << i+1 <<  ": " << tablica_liczb_wylosowanych[i];
    }

    int trafienia = 0;

    for (int i=0, j=0; i<6;)
    {
        if(tablica_liczb_uzytkownika[i] == tablica_liczb_wylosowanych[j])
        {
           trafienia++;
        }
        j++;

        if(j == 6)
        {
            i++;
            j=0;
        }
    }

    cout << "\n\nIlosc twoich trafien: " << trafienia;

    return 0;
}

Dziękuję za pomoc!

2 odpowiedzi

odpowiedź 10 sierpnia 2016 przez MichuDev Pasjonat (20,300 p.)

Podziel program na funkcje.
rand() nie jest dobrym pomysłem jeśli chodzi o taką losowość jaka ma być krytyczną w aplikacji. Lepiej użyć silników losowości i rozkładów.

Ostatnią pętlę zamieniłbym na taką:

for(int i=0;i<6;i++){
  if(tablicz_liczb_uzytkownika[i] == tablica_liczb_wylosowanych[i]) trafienia++;
}

komentarz 10 sierpnia 2016 przez Zekkeyo Nowicjusz (190 p.)

Dzięki za rady. Z funkcjami jak się zaprzyjaźnię to spróbuję. Z silnikami losowości chyba sobie dam siana, chyba przerzucę się na Javę.

Co do pętli - to było pierwsze co przyszło mi na myśl. Ale ta pętla porównuje tylko jeden do jednego, a chyba może być tak, że wylosowano 1 3 5 9 11 13 a ja zaznaczyłem 13 9 11 42 43 44. Wtedy tez mam trójkę, o ile się nie mylę.

komentarz 10 sierpnia 2016 przez adas94 Nałogowiec (29,200 p.)


for(int i=0;i<6;i++){
    for(int j=0;j<6; j++) {
    if(tablicz_liczb_uzytkownika[i] == tablica_liczb_wylosowanych[j])
        trafienia++;
     }
}

Teraz już masz każdego z każdym ;)