janil/tulospalvelu

Tietokantasovellus-harjoitustyö

Hiihtokisojen tulospalvelu

Huom.

Kieli vaihtunut Rubysta PHP:hen.

http://jjluukko.users.cs.helsinki.fi/tulospalvelu/

Tietokantasovellus-harjoitustyö

Aiheen kuvaus: http://advancedkittenry.github.io/suunnittelu_ja_tyoymparisto/aiheet/Hiihtokisojen_tulospalvelu.html

Dokumentaatio: https://github.com/JaniL/tulospalvelu/raw/master/doc/dokumentaatio.pdf

Kisojen listaus / etusivu: http://jjluukko.users.cs.helsinki.fi/tulospalvelu/#/kilpailut

Kisatulokset: http://jjluukko.users.cs.helsinki.fi/tulospalvelu/#/kisa/1

Kilpailijoiden listaus: http://jjluukko.users.cs.helsinki.fi/tulospalvelu/#/kilpailijat

tulospalvelu's Issues

Koodikatselmointi #2

Projekti ladattu 16.4. klo 18:25

Koodikatselmointi

Positiivista

• Koodi on selkeää, muuttujat, luokat ja muut tiedostot on nimetty selkeästi, jonka ansiosta koodia on helppo lukea. Siis noudattaa hyvin clean-code periaatteita.
• Aihe vaikuttaa sopivan haastavalta.

Parannettavaa

• Kontrollerissa ei ole vielä juurikaan sisältöä. Kannattaa lisätä sinne varmaan aluksi store() ja create() metodit, jotta kilpailijoita ja kisoja voi lisätä tietokantaan. Samalla saat käyttöön noi jo toteutetut Kilpailija-luokan metodit save() jne.
• Vaikuttaa siltä, että tällä hetkellä kaikki toimiva on tuolla sinun herokuapp-sivulla. Kannattaa varmaan joko poistaa readme:sta kokonaan toi http://jjluukko.users.cs.helsinki.fi/tulospalvelu/, tai sitten vaihtaa toiminnallisuus sinne 😄