C# Iks-Oks igrica

Svaka cast

Evo rijesio sam se power pack-a i sad zahtjeva samo .NET framework... Nisam ni znao za dropbox evo sad sam se registrovao na njegi i vidim da je super i dosta bolje rješenje od bilo kojeg druge stranice i jednostavnije za rad...

evo igrica

Igricu sam radio u windows 7 gdje je pozadina iste boje kao pozadina iksa i oksa a nisam znao za xp a isto tako i za onaj gumb kreni odmah... slike krizica i kruzica sam imao napravite prvi puta dobro posto sam koristio onaj power pack ali prob mi je bio eto to da se nemoze na drugim kompjuterima pokrenuti bez njega i onda sam morao to nekako zamjeniti pa nisam niti pazio kako sam crtao... ali sve u svemu mislim da sam to jos dobro napravio s obzirom da sam totalni početnik sto se tice c#... mozda je jos malo i doradim kada nadjem vremena....

Meni se baš sviđa igrica, za početnika nije loše i ničiji rezultati nisu bili savršeni u početku. OK za kritike (dobronamjerne nadam se), ali ohrabrenje i poticaj s moje strane definitvno ne izostaju.

Pozz

"Nadas" se da su kritike dobronamjerne, ali svejedno si mi udijelila thumb-down. Svasta. Upoznaj se s izrazom "konstruktivna kritika". Koliko sam shvatio, slobo14 je ovdje dosao nesto nauciti - upravo tome sluze moji komentari.

Pokusaj promijeniti pozadinu forme/prozora da vidis o cem je rijec. Ili jednostavno eksplicitno specificiraj boje (a ne kao "boja prozora" ili kako je vec to u VS-u).

Za pocetnika - dobro, svakako i naravno. Nadam se da ces je doraditi. Ako govorimo o slikama, one ionako nisu stvar C# i programiranja, nego uredjivanja slika/cranja.

Inace i sam se i sam zabavljao s tom igrom - nekad davno u DOS-u, u Turbo Pascalu, a sad nedavno kad sam jedan zadatak vezan uz validaciju podataka unesenih u HTML formu obogatio i igrom krizic-kruzic napravljenoj u JavaScriptu (kao i u tvom slucaju, i ovdje je rijec o pocetnickome pokusaju jer sam tada JavaScript jedva nesto znao) - mozes je vidjeti ovdje pri dnu stranice (slike nisam crtao, uzeo sam ih odnekud s weba).

Rekao sam nesto pozitivno i u prosloj kritici - da je skuzio logiku igre. Druga je stvar sto nisi razumjela sto to znaci.

Prvo mu u jednom postu savjetujem kako da ubuduce najlakse podijeli s drugima ono sto napravi, a u sljedecem mu objasnim sto mora promijeniti da bi mu igra zadovoljavala barem neku minimalnu kvalitetu. Kontruktivna kritika je po definiciji "podrskodavna".

Ne ocekujes valjda da cu mu ja reci kakvu je super igru napravio? Jer nije - igra takva kakva je ne zadovoljava ni nekakvu minimumalnu kvalitetu. S druge strane, pohvalno je za njega sto je uopce isao nesto programirati. I ako nastavi s programiranjem, uvazavajuci konstruktivnu kritiku, njegova ce igra doseci neki minimum kvalitete i moci ce biti zadovoljan s tim sto je napravio. Dotle se moze pohvaliti samo njegov trud, ali ne i rezultat tog truda. Nema smisla hvaliti nesto sto ne valja.

Kad se ne slazes s necijom porukom zbog toga sto si je pogresno protumacila i das TD, ne mozes ocekivati da druga strana ostane ravnodusna, jer si napravila nesto sto nije fer, nesto sto nije pravedno. Ne znam kakva je situacija s drugim forumasima, no mene osobno jako smetaju stvari koje nisu fer.

Nitko ti nista ne bi rekao da je u pitanju puko neslaganje sa sadrzajem poruke - u mom slucaju, da smatras da sam neku tocku neopravdano kritizirao - iako bih te tada trazio da argumentiras neslaganje, jer, naravno, postoji i mogucnost da sam pogrijesio u necem. Ovako iz moje perspektive ispada da si mi neopravdano dala TD. Jedno su rijeci, drugo su djela - covjeku doslovce korisnim savjetima pokusavam pomoci da poboljsa to sto je napravio (drugim rijecima - dajem mu aktivnu podrsku, da nastavi s tim s cim je zapoceo), a ti me optuzis da mu ne dajem podrsku. Davati nekome podrsku ne znaci (samo) reci "podrzavam te". Razmisli malo o tome.

Slobodno mozes nastaviti uciti C# uz C++ - stvari koje sam naucis u C# mozda ces moci iskoristiti za programiranje u C++-u (pogotovo algoritme), a isto tako stvari koje naucic u C++-u omogucit ce ti bolje razumijevanje C#-a. C# ti je inace fora - ima jednostavniji objektni model i ako se malo s tim poigras, lakse ces shvatiti klase u C++-u.

Nije nuzno da prolazis knjigu od A-Z - slobodno i sam eksperimentiras, dapace. Najbolje ce ti u sjecanju ostati situacije kad sam dokucis kako rijesiti neki problem.

Evo nešto što sam nabrzaka složio. Ima i source code. Pisano prije pol sata pa ako ima grešaka shvatite da je jako rano (kasno) za nekakvo ozbiljno testiranje :)

http://www.box.net/shared/qb52mtdcja

Par komentara... Zasto u eventhandleru ne iskoristiti Sendera? Mogao si iskoristiti svojstvo buttona Tag  - znaci, svakom gumbu odrediti Tag od 0 do 8 pa iz toga onda izvuces indeks za pozivanje funkcije postavi. Dakle, onda bi imao jedinstvenu funkciju za klik na gumb (umjesto ovako gdje imas zasebnu funkciju za svaki pojedini gumb!), a ta funkcija bi izgledala ovako (mozda ima negdje neka greskica - nisam nikad radio u C++Builderu):

void __fastcall TForm1::ButtonXClick(TObject *Sender)
{
    TButton *gumb = dynamic_cast<TButton*>(Sender);
    postavi(&kliknuto[gumb->Tag], &polje[gumb->Tag], gumb);
}

Ili je zapravo funkcija postavi mogla biti integrirana u event handler za gumb pa se na taj nacin jos pojednostavni kod:

void __fastcall TForm1::ButtonXClick(TObject *Sender)
{
  TButton *gumb = dynamic_cast<TButton*>(Sender);
  if(!(kliknuto[gumb->Tag])) {
    kliknuto[gumb->Tag] = true;
    if(potez == 1) {
      polje[gumb->Tag] = 1; // x
      potez = 2;
      gumb->Caption = "X";
    }
    else {
      polje[gumb->Tag] = 2; // o
      potez = 1;
      gumb->Caption = "O";
    }
    provjera();
  }
}

Primijetite da smo se tu usput rijesili logicki nepotrebnog returna  odmah na pocetku funkcije postavi - dakle umjesto if (*gdje) return; napisali smo ekvivalent if (!(*gdje)) {... napravi sve sto treba napraviti...} - funkcija postavi, koju smo integrirali u ButtonXClick u originalu izgleda ovako:
void postavi(bool* gdje, int* polje, TButton* gumb){
  if(*gdje) return;
  *gdje = true;
  if(potez == 1) {
    *polje = 1; // x
    potez = 2;
    gumb->Caption = "X";
    }
  else {
    *polje = 2; // o
    potez = 1;
    gumb->Caption = "O";
  }
  provjera();
}

Kod mozemo jos skratiti - npr. ako potez koristimo kao bool varijablu (jer ionako imamo samo dvije vrijednosti - dva igraca), kod se jos malo pojednostavljuje (mada bi varijablu potez bilo bolje preimenovati u naPotezuX da stvar bude jasnija - radi preglednosti, to sam stavio u komad koda iza ovog):

void __fastcall TForm1::ButtonXClick(TObject *Sender)
{
  TButton *gumb = dynamic_cast<TButton*>(Sender);
  if(!(kliknuto[gumb->Tag])) {
      kliknuto[gumb->Tag] = true;
      if(potez) {
        polje[gumb->Tag] = 1; // x
        gumb->Caption = "X";
    }
    else {
      polje[gumb->Tag] = 2; // o
      gumb->Caption = "O";
    }

    potez != potez;

    provjera();
  }
}

Sad, kad dalje gledam... Koji je uopce smisao vektora kliknuto, kad znas je li kliknuto na gumb ili ne prema tome nalazi li se u polje[x] vrijednost razlicita od nule. Dakle, kod se moze reducirati na:

void __fastcall TForm1::ButtonXClick(TObject *Sender)
{
  TButton *gumb = dynamic_cast<TButton*>(Sender);
  if((polje[gumb->Tag] == 0)) {    
      if(naPotezuX) {
        polje[gumb->Tag] = 1; // x
        gumb->Caption = "X";
    }
    else {
      polje[gumb->Tag] = 2; // o
      gumb->Caption = "O";
    }

    naPotezuX != naPotezuX;

    provjera();
  }
}

Ili ako idemo u perverzije (ali takve stvari ne bih preporucio):

void __fastcall TForm1::ButtonXClick(TObject *Sender)
{
    TButton *gumb = dynamic_cast<TButton*>(Sender);
    if(!(kliknuto[gumb->Tag])) {

      kliknuto[gumb->Tag] = true;
      const char znak2[] = {'O', 'X', 2, 1};
      gumb->Caption = znak2[naPotezuX % 2];   // uzima O ili X
      polje[gumb->Tag] = (int)znak2[naPotezuX % 2 + 2];   // uzima 2 ili 1

      naPotezuX != naPotezuX;

      provjera();
    }
}

Takodjer, u event handleru za gumb "Nova igra" ovo:

Button1->Caption = "";
Button2->Caption = "";
Button3->Caption = "";
Button4->Caption = "";
Button5->Caption = "";
Button6->Caption = "";
Button7->Caption = "";
Button8->Caption = "";
Button9->Caption = "";

Mogao si zamijeniti for petljom - do svakog pojedinog gumba bi dosao preko njegovog naziva ("Button" + indeks), posredstvom funkcije FindComponent. Ujedno bi im na taj nacin mogao definirati svojstvo Tag.

Nisi to bas pokazao. Ali nema veze, prihvatit cu tvoje objasnjenje o radjenju programa nabrzaka u 5 ujutro.

Živjeli bogomdani na ovom forumu. I ne nisam pogrešno shvatila tvoju poruku i unatoč tvom objašnjenju ponovno bih ti dala TD jer se tvoje izrazito"poštovanje" vlastitog mišljenja ne može shvatiti kao nešto pozitivno.

@autor teme

voljela bih vidjeti još tvojih radova i pratit tvoj napredak

Evo i ja nisam imao pametnijeg posla pa sam napravio svoju verziju.

Igra je rađena u C# i WPF-u i zahtijeva .net 3.5!!!

Link: http://www.speedyshare.com/files/24062441/Kri_i_-Kru_i_.rar

Uz igru je priložen i source!

Recimo ovako :)

Neces ti to ispraviti? Zar nisi krenuo to raditi da naucis nesto?

Ma ne moras ti to ispravljati radi mene. Ali ako hoces, mogu te uputiti kako da to napravis kako treba. Prva uputa - varijable:

        bool prvaPopunjena = false;
        bool drugaPopunjena = false;
        bool trećaPopunjena = false;
        bool četvrtaPopunjena = false;
        bool petaPopunjena = false;
        bool šestaPopunjena = false;
        bool sedmaPopunjena = false;
        bool osmaPopunjena = false;
        bool devetaPopunjena = false;

Su suvisne. Je li polje popunjeno ili ne znas prema tome je li polje.Content = "". Dakle, vrijedi relacija prvaPopunjena == !(prva.Content == "").

Druga uputa: Funkcija TkoJePobjednik je nevjerojatna - imas deset puta vise (identicnog) koda koji ti ne treba. Identicni kod se ili stavi u odvojenu funkciju pa onda samo pozoves tu funkciju ili kombiniras uvjete operatorom | (pipe, znaci ili, odnosno or).

Ako hoces jos uputa - reci. Imam ih.