Poker c++ 24 karty

Question

Witam. Jestem dość mocno początkujący i napisałem sobie grę w pokera :) Jakby ktoś rzucił okiem i podpowiedział co powinienem poprawić, co jest bezsensu, byłbym wdzięczny.

http://4programmers.net/Pastebin/4493

Pozdrawiam

Answer 1

Zmienne globalne to zło. Niejasne nazwy zmiennych (np. a, albo dupa) utrudniają zrozumienie kodu. Dużo fragmentów też się powtarza i można by je skrócić. Tyle tak na pierwszy rzut oka.

Answer 2

Zmienna dupa ? Za pare tyg nie będziesz pamiętał nic w swoim kodzie z takimi nazwami.

Mogłeś spróbować napisać algorytm wykrywania figur zamiast wszystko na ifach :)

Answer 3

Przede wszystkim - cały kod w mainie to nie jest dobra technika. Większe zadania, np potasuj, rozdaj, itd. zamknij w funkcje, a zbiory tych funkcji i różnych zmiennych - w klasę, np. Game. Dzięki temu będziesz wykonywać funkcje na konkretych obiektach.

Karty też możesz jako tako owrappować w klasę lub strukturę (wartość, nazwa, ilość - zmienne obiektu klasy lub struktury Karta). Klas używaj, jeśli chcesz korzystać z czegoś jako obiektu, np. zegarek (poprzez metodę Zegarek.odczytaj() uzyskujesz dostęp do godziny), a struktur, jeżeli coś ma być zlepkiem zmiennych (np. PunktUkladuWspolrzednych - składa się tylko ze zmiennych x, y, ewentualnie z).

Answer 4

Przepraszam najmocniej za to "dupa", zapomniałem poprawić :\ A ma ktoś pomysł jak w miarę prosto stworzyć algorytm tasujący taką talie kart, która nie opiera się na tablicach ? Bo widziałem dużo ciekawszą impelemtacje mojego pomysłu, na 54 karty, z tym że tamto rozwiązanie ma przeszło 2000 lini kodu :/

Comments

Polecam nauczyć się biblioteki vector. Taka ulepszona tablica, można dodawać elementy, zmieniać rozmiar, pobierać ten rozmiar, itd.

Answer 5

A jakieś pomysły dlaczego to:

for(int a=0; a<5; a++)
{
    tutaj:
    int i=(rand() % 6 ) + 0;
    int j=(rand() % 4 ) + 0;
    reka1[a]=karty[i][j];
    for(int i=0; i<5; i++)
    {
        for (int j=0; j<5; j++)
        {
            if(reka1[i]==reka1[j])
            {
                goto tutaj;
            }
            else
            {
                goto koniec;
            }

        }
    }

    koniec:
    cout <<  reka1[a] << " ";

Nie działa tak samo jak to:

 

    if(reka1[0]==reka1[1] || reka1[0]==reka1[2] || reka1[0]==reka1[3] || reka1[0]==reka1[4]
    || reka1[1]==reka1[0] || reka1[1]==reka1[2] || reka1[1]==reka1[3] || reka1[1]==reka1[4]
    || reka1[2]==reka1[0] || reka1[2]==reka1[1] || reka1[2]==reka1[3] || reka1[2]==reka1[4]
    || reka1[3]==reka1[0] || reka1[3]==reka1[1] || reka1[3]==reka1[2] || reka1[3]==reka1[4]
    || reka1[4]==reka1[0] || reka1[4]==reka1[1] || reka1[4]==reka1[2] || reka1[4]==reka1[3])
    {
        goto tutaj;
    }

cout <<  reka1[a] << " ";

? Konkretnie to pierwsza wersja wcale nie działa.

Comments

Używaj proszę tagu code:

int sampleFunc() {
  int someVar = 9;
  return ++someVar;
}