Pogovor:Izpitno vprašanje DIRI2005 6100

Iz MaFiRaWiki

Načeloma ne bi bilo potrebno pisati uvodnega dela, ampak le odgovoriti na vprašanje. Ni pa nič narobe, če še enkrat obnovimo, kaj je sklad.

Pri odgovoru pa morda moti le:

  • imena spremenljivk v javi praviloma začenjamo z malo črko (tudi npr. Sklad_nov)
  • zadeva bi bila bolj nazorna, če bi pripravili metodo, ki "opravi delo" in bi bilo takop bolj jasno, kaj je pravzaprav odgovor in kaj "tisto okoli", ki le kaže, da je naš aloritem pravilen.
  • Metoda ne dela pravilno, če je v skladu le en element!
  • Ker pripravite cel kup pomožnih skaldov, pa skladov za izpis, pa ... je bralec na koncu izgubljen! Načeloma mooramo na začetku imeti npr. sklad z imenom s, po končani "operaciji" pa moramo še vedno imeti ISTI sklad s (če torej še vedno delamo z s, deloamo z istim skladom), le da nima več n-tega elementa.

Vem da je "zoprno", ker npr. izpis sklada le tega "pokvari", ampak če boste zadeve zložili v metode, bo stvar precej bolj jasna!

--Matija Lokar 16:21, 24 februar 2008 (CET)


  • v kodi bi bil lahko kak komentar. Enega sem dopisal ;-)
  • mislim, da bi bilo bolj smiselno, da metoda ne bi vračala ničesar, le spremenila obstoječi sklad (kar tako ali tako naredi). Sem popravil kar jaz ;-))
  • ker (velja tudi za ostale vaše odgovore) uporabljate Stack, bi bilo potrebno povedati, da se pop obnaša malo drugače kot naš odstrani - namreč poleg tega, da odstrani vrhnji element, ga še vrne kot rezultat. To ste tu izrabili v:
 sklad_pomozen.push(sklad.pop());

--Matija Lokar 12:06, 4 april 2008 (CEST)

Osebna orodja