MyCity » Opšte programiranje » Petlja i bug

Petlja i bug

Napisano na dan: 27.9.2014
1

Petlja i bug
Sledeća strana Pagination

Idi na vrh

Poslao: 27 Sep 2014 22:13
Idi na vrh
offline
  • Pridružio: 14 Feb 2008
  • Poruke: 12405

Testiram neki kod i naleteo sam na bug koji se vratio ...

Da li ste ikada videli ovakvu petlju ?

  1. var QCnt : int = 0;
  2.  
  3. for (var i = 0; i <= linije.Length-1; i++){
  4.  
  5. test[QCnt].QID= parseInt(linije[i]);
  6. i++;
  7.  
  8. test[QCnt].QTXT= linije[i];
  9. i++;
  10.  
  11.  
  12. test[QCnt].A1TXT= linije[i];
  13. i++;
  14.  
  15. test[QCnt].A2TXT= linije[i];
  16. i++;
  17.  
  18. test[QCnt].A3TXT= linije[i];
  19. i++;
  20.  
  21. test[QCnt].A4TXT= linije[i];
  22. i++;
  23.  
  24. test[QCnt].C1= parseInt(linije[i]);
  25. i++;
  26.  
  27.  
  28. test[QCnt].C2= parseInt(linije[i]);
  29. i++;
  30.  
  31. test[QCnt].C3= parseInt(linije[i]);
  32. i++;
  33.  
  34. test[QCnt].C4= parseInt(linije[i]);
  35.  
  36. QCnt++ ;
  37.  
  38. }


Da li vidite neki potencijalni problem sa tim kodom ili neku nelogičnost ?


Šta još treba da znate o kodu :

test je niz klase sačinjene od različitih tipova promenljivih - koji se mogu videti u kodu iznad (QID, QTXT, ID... )

Dakle
  1. test = ImeKlase[20];
  2.  
  3. Imeklase{
  4. QID :int;
  5. QTXT : string;
  6. ...
  7. }

//
Test niz izgleda ovako ako sve bude ok :
test[0].QID = 23;
...
test[1].QID = 25;
...

//
Postoji jedan storage fajl sa istim formatom unosa :

  1. 23 [end]
  2. Test [end]
  3. Mycity
  4. forum [end]
  5. bla[end]
  6. ...

Ja delim taj storage fajl upotrebom .Split() u niz linije i zatim prolazim kroz ceo niz i brišem sve \n i \r charove.

Tako da taj niz na kraju dobije ovakav izgled :

linije[0] = 23
linije[1] = Test
linije[2] = MyCity forum
linije[3] = bla

I sada dolazimo do onog prvog koda gore, koji iz tog niza popunjava niz klase.

I šta je tu problem ?
Kod nekada perfektno radi, kod nekada ne radi uopšte, kod nekada radi ali dobijem grešku poput :
Array Index Out of Range ili da je unos u pogrešnom formatu kod parseInta ... Što nagoveštava da grešim negde sa i brojačem ali to nije istina jer i kada se pojavi greška sve radi perfektno... Dok ne dodam neki novi unos u storage fajlu.

Svi predlozi dobrodošli ... Very Happy



Registruj se da bi učestvovao u diskusiji. Registrovanim korisnicima se NE prikazuju reklame unutar poruka.
Poslao: 27 Sep 2014 22:49
Idi na vrh
offline
  • Milan
  • Pridružio: 17 Dec 2007
  • Poruke: 14825
  • Gde živiš: Niš

Da li je broj linija uvek jednak proizvodu broja atributa i broja objekata koji se čitaju?

Inače, ovakav način pisanja petlji je uvek rizičan za "Array Index Out of Range" izuzetke. Zašto? Zato što se promenljiva i menja i u zaglavlju petlje, ali i u telu petlje, a koristi se kao indeks unutar petlje. Uzmi za primer da ti je linije.Length = 3. Tada je početni uslov za petlju zadovoljen, ali si ti već nakon trećeg inkrementiranja van dužine niza linije.



Poslao: 28 Sep 2014 03:27
Idi na vrh
offline
  • Pridružio: 14 Feb 2008
  • Poruke: 12405

Ovo je fiksna stvar.

Unosi su u istom formatu.
Jednu celinu čini 10 unosa/linija.
Svejedno prepravio sam kod malo i sada izgleda ovako :

  1. var repeat: int = linije.Length - 1 / 10;
  2. Debug.Log("Repeat" + repeat);
  3. var i: int = 0;
  4. //for (var i = 0; i <= linije.Length-1; i++){
  5. for (var b = 0; b <= repeat; b++) {
  6.     //Debug.Log ("cnt" + i);
  7.     if (i >= linije.Length - 1) {
  8.         break;
  9.     }
  10.     test[Qcnt].QID = parseInt(linije[i]);
  11.     i++;
  12.     //Debug.Log ("cnt" + i);
  13.  
  14.     test[Qcnt].QTXT = linije[i];
  15.     i++;
  16.     //Debug.Log ("cnt" + i);
  17.  
  18.     test[Qcnt].A1 = linije[i];
  19.     i++;
  20.     //Debug.Log ("cnt" + i);
  21.  
  22.     test[Qcnt].A2 = linije[i];
  23.     i++;
  24.     //Debug.Log ("cnt" + i);
  25.  
  26.     test[Qcnt].A3 = linije[i];
  27.     i++;
  28.     //Debug.Log ("cnt" + i);
  29.  
  30.     test[Qcnt].A4 = linije[i];
  31.     i++;
  32.     //Debug.Log ("cnt" + i);
  33.  
  34.     test[Qcnt].C1 = parseInt(linije[i]);
  35.     i++;
  36.     //Debug.Log ("cnt" + i);
  37.  
  38.     test[Qcnt].C2 = parseInt(linije[i]);
  39.     i++;
  40.     //Debug.Log ("cnt" + i);
  41.  
  42.     test[Qcnt].C3 = parseInt(linije[i]);
  43.     i++;
  44.     //Debug.Log ("cnt" + i);
  45.  
  46.     test[Qcnt].C4 = parseInt(linije[i]);
  47.     i++;
  48.     Qcnt++;


Dakle sada prvo delim ceo niz sa 10, dobijam tačan broj celina.
Ponavljam petlju samo toliko puta a ona upisuje redom liniju po liniju u odgovarajući niz klase.
-1 zato što streamwriter ostavlja jednu liniju praznu na kraju fajla pa je lakše tako to rešiti nego čistiti tu jednu liniju.

Takođe sam prepravio kod koji upisuje u fajl i uklonio razmak ispred " [end]" jer sam imao čudne probleme sa formatiranjem.

I sada to lepo radi, testirao sam sa 6 celina tj. 60 unosa...



Ali izgleda da ovde nije kraj i da nema mnogo veze sa ovom petljom...

Čas isti kod radi, čas ne radi. Verujem da dolazi do drugih problema ispod haube samog endžina :/

Baš me izluđuje.

Ova izmena bi trebala da bude sigurnija od prethodne u svakom slučaju ...
U suštini ne interesuju me problemi ispod haube, bitno je da na kraju radi kada se iskompajlira.

No, moguće je da će i ovo bacati probleme sa novim unosima, probaću sutra da napravim ogromnu bazu i da testiram.

Ziveli

Poslao: 28 Sep 2014 07:52
Idi na vrh
offline
  • PHP developer
  • Pridružio: 22 Mar 2006
  • Poruke: 3760
  • Gde živiš: 127.0.0.1

vasa.93 ::ovakav način pisanja petlji je uvek rizičan za "Array Index Out of Range" izuzetke

U ovom konkretnom slucaju "Array Index Out of Range" nije rizican, vec skoro siguran, osim ako duzina niza "linije" nije deljiva sa 10 bez ostatka.

Ja bih pre uradio ovako nesto:

  1. //uvek izbaci kalkulacije van petlje
  2. var limit = Math.ceil(linije.length / 10);
  3.  
  4. for (var i = 0; i < limit; i++){
  5.    var idx = (i - 1) * 10;
  6.    var tmp = {
  7.       QID     : typeof linije[idx]   !== "undefined" ? linije[idx]   : "n/a",
  8.       QTXT    : typeof linije[idx+1] !== "undefined" ? linije[idx+1] : "n/a",
  9.       A1TXT   : typeof linije[idx+2] !== "undefined" ? linije[idx+2] : "n/a",
  10.       A2TXT   : typeof linije[idx+3] !== "undefined" ? linije[idx+3] : "n/a",
  11.       A3TXT   : typeof linije[idx+4] !== "undefined" ? linije[idx+4] : "n/a",
  12.       A4TXT   : typeof linije[idx+5] !== "undefined" ? linije[idx+5] : "n/a",
  13.       C1      : typeof linije[idx+6] !== "undefined" ? linije[idx+6] : "n/a",
  14.       C2      : typeof linije[idx+7] !== "undefined" ? linije[idx+7] : "n/a",
  15.       C3      : typeof linije[idx+8] !== "undefined" ? linije[idx+8] : "n/a",
  16.       C4      : typeof linije[idx+9] !== "undefined" ? linije[idx+9] : "n/a",
  17.    };
  18.    test.push(tmp);
  19. }


uz napomenu da nemam pojma o kom se jeziku radi, ali da jako lici na javascript, pa sam u njemu napisao kod iznad.

Poslao: 28 Sep 2014 10:51
Idi na vrh
offline
  • Pridružio: 19 Maj 2011
  • Poruke: 297

  1. var repeat: int = linije.Length - 1 / 10;

Pazi ovde, deljenje ima prednost!

Da bi ti kod bio citljiviji i sigurniji prvo sto upises u fajl bi trebalo da bude sam broj unosa.
  1. var count : int = parseInt(linije[0]);
  2. var test : ImeKlase [count];
  3.  
  4. for (var iUnos = 0; iUnos < count; ++iUnos) {
  5.     test[iUnos].QID = parseInt(linije[iUnos * 10 + 1 + 0]); // +1 zato sto preskacemo prvu liniju (broj unosa)
  6.     test[iUnos].QTXT = linije[iUnos * 10 + 1 + 1];
  7.     ...
  8.     test[iUnos].C4 = parseInt(linije[iUnos * 10 + 1 + 9]);
  9. }

Kad si vec korisito Debug.Log trebao si ispisivati sadrzaj instanci a ne kauntera (i) da bi bolje video sta se dogadja.

Mozda sam pogresio u sintaksi jer ne poznajem Unity javascript-like skript jezik.

Poslao: 28 Sep 2014 11:14
Idi na vrh
offline
  • Milan
  • Pridružio: 17 Dec 2007
  • Poruke: 14825
  • Gde živiš: Niš

Napisano: 28 Sep 2014 11:12

Morando, zar umesto

  1. test[iUnos].QID = parseInt(linije[iUnos + 1 + 0]);
ne bi trebalo da ide
  1. test[iUnos].QID = parseInt(linije[iUnos*10 + 1 + 0]);
 Question

U prvom prolazu iUnos je 0, pa se čitaju linije od 1 do 10, u drugom prolazu iUnos je 1, pa se čitaju linije od 2 do 11. Morao bi da se iUnos množi sa brojem atributa/linija koje se čitaju.

Dopuna: 28 Sep 2014 11:14

Rastafarii ::U ovom konkretnom slucaju "Array Index Out of Range" nije rizican, vec skoro siguran, osim ako duzina niza "linije" nije deljiva sa 10 bez ostatka.Upravo sam na to mislio kada sam upitao da li je broj linija jednak proizvodu broja atributa i broja objekata koji se čitaju. Obzirom da je broj atributa 10, broj linija mora da bude celobrojni umnožak broja 10, a to je broj koji je deljiv sa 10. Very Happy

Poslao: 28 Sep 2014 11:50
Idi na vrh
offline
  • Pridružio: 19 Maj 2011
  • Poruke: 297

Da, puta deset, to sam zaboravio. Very Happy
Dodje mi komsija i dok smo pili kafu seti se toga ali ne stigoh da ispravim.

Poslao: 28 Sep 2014 19:33
Idi na vrh
offline
  • Pridružio: 14 Feb 2008
  • Poruke: 12405

Izvinite što kasnim sa odgovorom, nisam mogao pre da se dočepam kompa.

Javiću kasnije šta se izdešavalo sa mojim kodom i predlozima ovde.

Kao što rekoh, uvek je fiksno sve kada su unosi u pitanju tako da ima bar jedna celina od 10 linija...
Tako će uvek biti osim u slučaju kada neko čačne storage fajl nepravilno, pri čemu je sam kriv.

Morando, ne znam koliko bi meni bilo praktično da brišem i pišem tu liniju svaki put (broj unosa) zbog ograničenja Unityja sa editorom. Morao bih onda stalno da osvežavam inspektor ... Ovako sigurno znam koliko unosa ima kada podelim sve linije sa 10. Kontrolisano ih dodajem u storage fajl.

U pravu si naravno za deljenje, zaboravio zagrade jutros Very Happy

Takođe nije bitno u kom jeziku pišete, shvatiću poentu dok god nema u kodu nešto specifično samo za taj jezik.

Javim se večeras, tokom jutra ili sutra sa rezultatima Ziveli

Poslao: 28 Sep 2014 20:02
Idi na vrh
offline
  • PHP developer
  • Pridružio: 22 Mar 2006
  • Poruke: 3760
  • Gde živiš: 127.0.0.1

Srki, a jesi li razmisljao da umesto plain-text storage fajla koristis JSON? Tako ti ne treba ovaj kod uopste Wink

Poslao: 29 Sep 2014 03:27
Idi na vrh
offline
  • Pridružio: 14 Feb 2008
  • Poruke: 12405

Rastafarii ::Srki, a jesi li razmisljao da umesto plain-text storage fajla koristis JSON? Tako ti ne treba ovaj kod uopste Wink

Nisam, niti sam siguran da li bi radilo to tako, Plugins folder se obično koristi u pro verziji Unityja.

Ovaj kod je OK za moje potrebe. Izvrši se samo jednom kada se učitava prvi put igra i traje tokom cele sesije. Nema nikakvih izmena dok radi igra već je sve fiksno i podesi se pre kompajliranja, uključujući redosled izvršavanja skripti, jer se ovo sastoji od bar 3 skripte. Ako sve radi pre kompajliranja u editoru radiće i u igri kasnije bez problema.

Inače, testirao sam sa 60 i 80 unosa kod koji sam drugi put postovao.
Sve radi savršeno i bio sam u pravu - postoji jedan problem ispod haube.

Kada se upišu nove vrednosti u storage fajl Unity ih ne pokupi uopšte i onda skripta nema iz čega da čita tj. kada pokuša da nađe ono što traži toga nema...

To je ipak drugi problem i nema veze sa ovom temom niti sa opštim programiranjem, moraću da pitam na Unity forumu zašto je to tako.

Sve u svemu izgleda da smo rešili problem Very Happy
Nadam se... tako sam mislio i prošli put pa se vratio posle mesec dana Smile

Hvala društvo, imate pivo na okupljanju ili šta god, sok od zove i to Mr. Green Zagrljaj

MyCity » Opšte programiranje » Petlja i bug

Ko je trenutno na forumu
 

Ukupno su 1149 korisnika na forumu :: 59 registrovanih, 10 sakrivenih i 1080 gosta   ::   [ Administrator ] [ Supermoderator ] [ Moderator ] :: Detaljnije

Najviše korisnika na forumu ikad bilo je 3466 - dana 01 Jun 2021 17:07

Korisnici koji su trenutno na forumu:
Korisnici trenutno na forumu: acov34, airsuba, Alexagarett, babaroga, bbelic, Bobrock1, Bojan198527, bolenbgd, Boris BM, Borski1977, calvi, cenejac111, Chainsaw, cinoeye, csipetcsapat, darios, DavidA, Djokislav, DM1994, Dzoni2412, Electron, Ezbuck, Hitri, interesujeme, InzenjerBL, kn19, Lucije Kvint, Marko Marković, MB120mm, mdp92, mercedesamg, milenko crazy north, Miletić Zoran, milutin134, Musklfiber, Mzee, NMNJ, nobutado, nsharambasa, nuki1234, Oscar2, Povratak1912, PrincipL, Pv123, raketaš, rikirubio, S2M, skvara, Sr.Stat., Stanlio, Tas011, tuja, VanZan, Vaske8990, vathra, vensla, vuksa72, Webb, Čivi