Talk:Meeskond "codebastards"

Mõned sõnad, siis "koodimölakate" vaimuvilja kohta.

Koodi oleks võinud natuke rohkem kommenteerida... noh, et loll ka aru saaks ;).

Samas, kui võtta ette graafiline pool, siis oli asi tehtud ilusti, mõistlikult ja suhteliselt intuitiivselt. Minut klõpsimist ja ruumide tegemise loogika oli selge. Edasine oleks olnud puhas omalooming. Isiklikult oleks tahtnud näha võimalust nt. kogu ruum katta ühe klikiga üht tüüpi muruga vms (template'd ?). Et kui tahan teha ühtlaselt muruga kaetud ruumi, siis on natuke tülikas kogu ruudustikku ükshaaval muruga täita.

Kiidaksin "mölakaid", et võtsid ette sellise ülesande (koos XNA projektiga) ja selle väljundiks võib tulla mõnus alternatiiv vanakooli mängurite nostalgitsemisele (originaalmäng vajab juba DOS boxi, et seda mängida). Loodan, et jätkub aega ja jaksu asi lõpuni viia.

Denis Veltson, DK21

---

• Rakendus

Mängu ennast testida ei saanud kuna, installeri versioon on HiDef'i peal. Tõmmates viimase Solutioni lähtekoodi, tekib kohe errorid teie fontide puudu olemisest. Pole wikis juhendeid ka kuidas talitada sellises olukorras. Editori saab tööle kui unloadida kõik peale Leveleditori. Editor tundub päris lahe ja pakub lihtsat võimalust uusi leveleid luua. Graafiline pool on kä väga ilus. Vahetades erinevate tile settide vahel, ei kuvata eelmiste tilesettide asju, tekitab palju segadust kui oled juba samale ruudule midagi muud pannud. Samuti on teil objektide loend tühi, ei tea kas sinna tulevad teil nn Itemid, kuigi puu vms tundub ka objekt ja mingi muru tundub olema tile pigem. Raske on arusaada mida te teha olete tahtnud kui pole seletusi ja kommentaare.

• Lähtekood

Programmi lähtekood on tundub töötavat, aga kui näiteks peale poole aastast pausi seda koodi peaksite modima siis võib raskeks minna kui teil ainsad kommentaarid on C# enda kirjutatud või paar üksikut enda poolt.

       /// <summary>
       /// Required designer variable.
       /// </summary>
       private System.ComponentModel.IContainer components = null;

       /// <summary>
       /// Clean up any resources being used.
       /// </summary>
       /// <param name="disposing">true if managed resources should be disposed; otherwise, false.</param>

Ainus kasulik kommentaar mille suutsin leida.

• Wiki

Tundub, et olete hea asja valmis teinud, aga hästi retsenseerida on väga raske kui kõrvalistel puudub korralik informatsioon tehtud tööst ja koodi kommentaare pole ka põhimõtteliselt olemas. Projekt on jätkusuutlik, aga nõuab veel kõvasti tööd.

• Jaan Kruusma, D22