search

heidilees / tudeng14-kodutood

0 stars 0 forks source link

Töö 8 #2

Closed kuuba closed 9 years ago

kuuba commented 9 years ago

Vaatasin koodi ja nimetaks seda päris heaks alguseks - omajagu on muidugi veel parandamist: os ja argv sys teegist on imporditud aga kuskil neid ei kasutata; funktsiooni ruutjuur() oleks mõttekas defineerida main() funktsioonist väljapool, vastasel juhul ei saa me teda kuigi mugavalt moodulina kasutada ja ka kogu lokaalse konteksti mõte kaob ära; docstringe ega kommentaare pole üldse kasutatud; "executable" (x) bit pole failil dragonfishis ära märgitud; lõppeks - mingi väike üksik print on faili lõppu ära eksinud.

heidilees commented 9 years ago

Võtsin arvesse teie märkusi ja lisasin kodutöö 8st uue versiooni.

kuuba commented 9 years ago

Tundub et ongi korras. Õnnitlen - see on esimene töö 8 mille kohta ma tabelisse ok saan panna.

kuuba commented 9 years ago

Näib kena - rida 22-l on miskipärast funktsiooni lihtsalt niisama välja kutsutud - see on täiesti üleliigne.

Parimat, Toomas

On Tue, Apr 28, 2015 at 1:28 PM, heidilees notifications@github.com wrote:

Võtsin arvesse teie märkusi ja lisasin kodutöö 8st uue versiooni.

Reply to this email directly or view it on GitHub https://github.com/heidilees/tudeng14-kodutood/issues/2#issuecomment-97006634 .