[C#] Szwaknujące kółko i krzyżyk...

[Zdziwiony 0]

Witam. Dziś po raz pierwszy spróbowałem napisać w C# w WPF kółko-krzyżyk, oto kod:

http://wklej.org/hash/a2424fbcc74/

 

A tu gra do pobrania:

Klik

 

Mój problem polega na tym, że gra ma dziwne bugi..., które możecie zobaczyć po włączeniu gry (nie da się nie zauważyć ).

Polegają one na tym, że np. funkcja odpowiadająca za sprawdzanie kto wygrał, nie reaguje na to, że komputer wygrał... (o dziwno, jeżeli wygra użytkownik to działa ) Do tego mój sposób zaznaczania pól kółkami pozostawia wiele do życzenia ... Tu ciężko przegrać - nie wygrać .

 

Tu mam do Was prośbę - jeśli ma ktoś ktoś z Was chęci, aby sprawdził ten kod i powiedział mi jakie błędy popełniłem , i jak powinienem tą grę zakodzić... (domyślam się, że większość kodu jest źle napisana )

 

Proszę Was o pomoc

[misiutychy 4]

Szkoda że nie mam kompilatora zainstalowanego, ale tak oglądając kod zastanawiałem się o co chodzi w tym fragmencie:

...
wygrana();
przegrana();
...

Przecież te funkcje zwracają jakąś wartość także powinieneś użyć ich od razu w 'if' bez wcześniejszego ich używania.

Wydaje mi się że sprawdzanie wygranej/przegranej dla komputera jest trochę za późno ;]

Tak samo nie wiem czy kończenie gry od razu po zastawieniu 9 pola jest OK skoro też może się okazać że ktoś wtedy wygrał/przegrał.

Nie sprawdzasz pionowych warunków (1,4,7; 2,5,8; 3,6,9)

Ogólnie popracuj nad momentem w którym sprawdza czy ktoś wygrał/przegrał, mi się udało doprowadzić że dostałem info o tym że przegrałem(3 kółka na przekątnej), ale za dużo warunków i mi po kilka razy wyświetliło że przegrałem i koniec gry.

[fajny Rafałek 0]

Dołącz jeszcze xamla z designem okna.

 

W każdym razie po co aż 9 handlerów różniących się dosłownie jedną linijką?

 

Do każdego pola (Button?) podepnij ten sam handler, a różnica będzie tylko

 

(sender as Button).Content = "X"; //jeśli kliknięto ustaw X

 

Zamiast ID_1, ID_2, ID_3...

 

I dlaczego funkcje wygrana/przegrana zwracają floata zamiast zwykłego boola? Wtedy miałbyś:

 

if(wygrana())
 MessageBox.Show("Wygrałeś!");
else if(przegrana())
 MessageBox.Show("Przegrałeś!");
else if(remis())
 MessageBox.Show("Remis!");

 

Bez sensu jest wyświetlać MessageBox w funkcji w x miejscach, jak można tylko w jednym.

 

Poza tym warunki zwycięstwa i porażki nie mają sprawdzania linii pionowych (1, 4, 7), (2, 5, 8), (3, 6, 9).

[januzi 24]

nie ma sensu robić osobnych funkcji sprawdzających, wystarczy jedna, która będzie odsyłać liczbę:

0 - nie ma jeszcze wyniku

1 - wygrana

2 - przegrana

3 - remis

i tyle

 

odpalasz ją po każdym wstawieniu znaku

[Zdziwiony 0]

Kod XAML:

<Window x:Class="Kółko_Krzyżyk.MainWindow"
       xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
       xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
       Title="Kółko-Krzyżyk" Height="178.262" Width="172.827" ResizeMode="NoResize">
   <Grid Margin="0,0,-8,-7">
       <Button x:Name="ID_1" Content="" HorizontalAlignment="Left" Height="47" VerticalAlignment="Top" Width="52" Click="ID_1_Click" FontSize="18" FontWeight="Bold"/>
       <Button x:Name="ID_2" Content="" HorizontalAlignment="Left" Height="47" VerticalAlignment="Top" Width="52" Click="ID_2_Click" FontSize="18" FontWeight="Bold" Margin="52,0,0,0"/>
       <Button x:Name="ID_3" Content="" HorizontalAlignment="Left" Height="47" VerticalAlignment="Top" Width="52" Click="ID_3_Click" FontSize="18" FontWeight="Bold" RenderTransformOrigin="2.745,0.696" Margin="104,0,0,0"/>
       <Button x:Name="ID_4" Content="" HorizontalAlignment="Left" Height="47" VerticalAlignment="Top" Width="52" Click="ID_4_Click" FontSize="18" FontWeight="Bold" Margin="0,47,0,0"/>
       <Button x:Name="ID_5" Content="" HorizontalAlignment="Left" Height="47" VerticalAlignment="Top" Width="52" Click="ID_5_Click" FontSize="18" FontWeight="Bold" Margin="52,47,0,0"/>
       <Button x:Name="ID_6" Content="" HorizontalAlignment="Left" Height="47" VerticalAlignment="Top" Width="52" Click="ID_6_Click" FontSize="18" FontWeight="Bold" Margin="104,47,0,0"/>
       <Button x:Name="ID_7" Content="" HorizontalAlignment="Left" Height="47" VerticalAlignment="Top" Width="52" Click="ID_7_Click" FontSize="18" FontWeight="Bold" Margin="0,94,0,0"/>
       <Button x:Name="ID_8" Content="" HorizontalAlignment="Left" Height="47" VerticalAlignment="Top" Width="52" Click="ID_8_Click" FontSize="18" FontWeight="Bold" Margin="52,94,0,0"/>
       <Button x:Name="ID_9" Content="" HorizontalAlignment="Left" Height="47" VerticalAlignment="Top" Width="52" Click="ID_9_Click" FontSize="18" FontWeight="Bold" Margin="104,94,0,0"/>


   </Grid>
</Window>

 

Poza tym warunki zwycięstwa i porażki nie mają sprawdzania linii pionowych (1, 4, 7), (2, 5, 8), (3, 6, 9).

 

Oups... o tym zapomniałem

 

Bez sensu jest wyświetlać MessageBox w funkcji w x miejscach, jak można tylko w jednym.

 

Wiem, i tak zrobiłem przy sprawdzaniu playera. Jednak przy kompie nie chciało mi to zadziałać, więc dopisałem jeszcze parę MessageBox'ów .

 

Przecież te funkcje zwracają jakąś wartość także powinieneś użyć ich od razu w 'if' bez wcześniejszego ich używania.

Ale chyba muszę je wywołać, żeby przybrały wartość ?

 

Ogólnie popracuj nad momentem w którym sprawdza czy ktoś wygrał/przegrał, mi się udało doprowadzić że dostałem info o tym że przegrałem(3 kółka na przekątnej), ale za dużo warunków i mi po kilka razy wyświetliło że przegrałem i koniec gry.

 

Hmm... tego nie przewidziałem

 

nie ma sensu robić osobnych funkcji sprawdzających, wystarczy jedna, która będzie odsyłać liczbę:

Zrobiłem jedną, jednak sprawdzanie czy komputer wygrał nie działało, ponad to czasem program zawieszał się przy losowaniu liczby... (nad tym, żeby losowanie działało poprawnie jeszcze dłużej się męczyłem, bo program zawieszał się czasami nieoczekiwanie, okazało się, że przy skoczeniu gry muszę zawsze zerować wartość zmiennej koniec, bo inaczej program idzie się 'rypać', a funkcja komputer(); wykonuje się w nieskończoność, bo wszystkie pola są zajęte, więc jakoś z funkcji muszę wyjść ).

 

No nic postaram się to dziś wieczorem poprawić, a jak nie dziś to w przyszłym tygodniu, bo muszę dziś gdzieś jechać.

 

Dzięki za uwagi