Tror det var meningen att detta skulle till listan och inte bara till mig?
---------- Vidarebefordrat meddelande ----------
Från: Fredrik Präntare <frepr183@student.liu.se>
Datum: 10 maj 2014 10:53
Ämne: Kodcheck 3
Till: Jon Dybeck <jondy276@student.liu.se>
Hej! Här kommer kodcheck 3.
- Dokumentationen för EditorModel.cs säger att den klassen är en
“Model for a jump point”. Verkar konstigt.
- Variabeln Scenario i EditorModel.cs har konstig/oseriös
dokumentation: “The Scenario to be Editored™.”
- Jag tycker inte att det är helt klart vad GetId i EditorModel.cs gör
(det är en getter-funktion men den gör lite mer). Lägg till lite
dokumentation där.
- I ServerState.cs saknar metoder under “NewModelAPI” dokumentation.
- ServerHandler.cs i ServerSoftware-mappen saknar dokumentation.
- Konstruktor i PlayableScenario.cs saknar dokumentation.
Mvh
Fredrik
_______________________________________________
Tddd77-group5 mailing list
Tddd77-group5@lists.lysator.liu.se
http://lists.lysator.liu.se/mailman/listinfo/tddd77-group5