jaggi
2
  • Najlepsze
  • Wszystkie komentarze
Szab
if (field3.GetState() != State.state.Occupied) ;

    else pawn_alone = false;


@Golomp: lol co
3
bambosze_babuni
Treść przeznaczona dla osób powyżej 18 roku życia...
2
westsajd
westsajd
@jaggi: co to za kody na nieśmiertelność?

if ((i == 0 || i == 1 || i == 5 || i == 6) && (j == 0 || j == 1 || j == 5 || j == 6))
1
G.....p
G.....p
@jaggi: ;_______;

try
{
    State field1 = (State)tableLayoutPanel1.GetControlFromPosition(i + 1, j);
    if (field1.GetState()
1
ogrod87
@jaggi: Uncle Bob by się załamał widząc te "rzygi" programistyczne. W moim zespole życia byś nie miał ;)
1
G.....p
G.....p
@jaggi: Dlatego warto przejrzeć kod przed upublicznieniem ;)
Swoją drogą takie puste trykacze to zła praktyka :P
1
markaron
@jaggi: Wyjątki (Exception) są to jak sama nazwa mówi sytuacje wyjątkowe, nie do przewidzenia. W fragmencie kodu, który wrzucił @Golomp wszystkie sytuacje są do przewidzenia więc są tam zbędne.

@ogrod87: u mnie w jednej (dużej) aplikacji wciąż można znaleźć takie potworki (stary kod), ale na szczęście udało mi się przemówić chłopakom do rozumu ( ͡° ͜ʖ ͡°)
1
jaggi
@bambosze_babuni: wiem, że z------m co poniektóre linie tego kodu po całości. ale w momencie pisania tego postu nie widziałem tego. jak człowiek coś napisze i czyta to, po niezbyt długiej przerwie, to czyta to "z pamięci" tak jak wydaje mu się że to wygląda, tak jak chce żeby to wyglądało.
upubliczniam bo czasami ktoś napisze mi coś mądrego zamiast wytykać że brzydkie nazwy. Ja rozumiem że jak się pracuje w
1
westsajd
westsajd
a co samego kodu, to serio, nie obrażę się, jak ktoś mi powie co jest złego w tym ifie, o którym pisał @westsajd.


@jaggi: w tym ifie złe jest to, że musiałem spytać się Ciebie co on robi. Zobacz o ile czytelniej byłoby napisać coś w stylu

if(!fieldIsAvailableToPlay(i,j){...}
1
jaggi
@westsajd: plansza do gry to zwykła macierz/tablica dwuwymiarowa więc pozbywam się tych zbędnych pól w rogach
0
jaggi
@Golomp: wiem :P mi też się to nie podoba ale nie mogłem dać tego w jednym bloku bo nie chciało działać tak jak chciałem
0
G.....p
G.....p
@jaggi: Ale po co puste catche i w ogóle takie warunki w ifach?
0
G.....p
G.....p
@Szab: tak, ja też pierwszy raz w życiu widzę taka konstrukcję :D
0
jaggi
@Golomp: warunki są mi po prostu potrzebne :D jak to po co?
a te puste catche bo krzyczał że unhandled exception, a jak występują te wyjątki to w tym wypadku nie chce nic z nimi robić
0
G.....p
G.....p
a te puste catche bo krzyczał że unhandled exception


@jaggi: ale tak bez powodu krzyczał?

if (field4.GetState() != State.state.Occupied) ;
else pawn_alone
0
jaggi
@Golomp: lepiej, fakt.
coś takiego mi zostało po milionie zmian i po prostu nie zwróciłem uwagi na to
0
jaggi
@ogrod87: domyślam się. coś takiego pisze trochę na odwal. jak mam jakiś projekt na uczelnię napisać to już się staram żeby ten kod w miarę wyglądał. a tu to piszę, poprawiam milion razy i powstaje mi coś takiego

if (field4.GetState() != State.state.Occupied) ;
else pawn_alone = false;

jak dotarlo do mnie co ja napisałem to sam się smiać zacząłem
0
bambosze_babuni
@jaggi Czyli chciałbyś, żeby pod tagiem #codereview znaleźli się dobrzy ludzie, którym zależy i którzy mogliby dać Ci pomocne tipsy? Dobrze zrozumiałem? A dlaczego komukolwiek ma zależeć na tym kodzie skoro Tobie na nim nie zależy ("pisze trochę na odwal")? Zrozum, że problem nie tkwi w tym, że czegoś nie wiesz, tylko w Twoim podejściu do tematu.

To trochę tak, jakbyś był początkującym pisarzem i wysyłał swoje próby do
0