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
Nej, alla får var sin lista
Den 10 maj 2014 13:39 skrev Jon Dybeck jondy276@student.liu.se:
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
tddd77-group5@lists.lysator.liu.se