Pogovor:Izpitno vprašanje DIRI2005 6100

Iz MaFiRaWiki

(Razlika med različicami)
Različica od 15:21, 24 februar 2008
MatijaLokar (Pogovor | prispevki)

← Prejšnja različica
Trenutna različica
MatijaLokar (Pogovor | prispevki)

Vrstica 9: Vrstica 9:
--[[Uporabnik:MatijaLokar|Matija Lokar]] 16:21, 24 februar 2008 (CET) --[[Uporabnik:MatijaLokar|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());
 +--[[Uporabnik:MatijaLokar|Matija Lokar]] 12:06, 4 april 2008 (CEST)

Trenutna različica

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