ootee / ammutulos Goto Github PK

Repositorio ladattu 10.08.2017 klo 18:15

Projekti näyttää olevan hyvällä mallilla ja suunnitelmat vaikuttavat hyviltä. AIhealuen vaikuttaa sopivan mielenkiintoiselta eikä liian laajalta. Tiedoston readme.MD tiedoston "Rasti" linkki johtaa ainakin tällä hetkellä olemattomaan osoitteeseen.

Ensimmäisenä sovelluksen avattaessa huomaa että base.html tiedostoa ei olla muokattu. Ainakin kohdat

8    <title>
9        {% block title %}
10          Tietokantasovellus
11       {% endblock %}
12   </title>

ja

35    <!-- Sovelluksen nimi -->
36    <a class="navbar-brand" href="{{base_path}}">Tsoha Bootstrap</a>

olisi hyvä muokata oman sovelluksen mukaiseksi.

Muuten sovelluksen sivut näyttävät hyvältä, vaikkei tällä hetkellä toiminnallisuutta pääsekkään testaamaan.

Testidatassa voisi olla vähän kattavampaa ja sisältää useita rivejä tietoa per tietokantataulu.

Koodikatselmointi 22.08.2017 - 12:08

Katselmointi on tehty commitin 1c84428 perusteella.

• Projektin aihe näyttää kurssiin nähden sopivan laajalta ja mielenkiintoiselta
• Projektisi on edennyt hyvin viikkodeadlinien mukana
• Käytät projektissa kahta eri tapaa muutujien asettamiseen SQL kyselyihin. Mielestäni olisi selkeämpää valita toinen tapa ja käyttää sitä yhtenäisesti koko projektissa.
  Joko

$query->bindValue(':etunimi', $this->etunimi, PDO::PARAM_STR);

Tai

$query->execute(array('kilpailija_id' => $this->kilpailija_id));

• Kannattaa tutustua myös tietokantataulujen luomisosiossa CASCADE ominaisuuden käyttöön, jolla voit helpottaa esim. linkitetyn datan käsittelyä poistotilanteissa. Lisää infoa Postgresql docs - constraints
• Koodistasi on erittäin vaikea löytää juurikaan mitään huomautettavaa, sillä kaikki on tehty mallien mukaan oikein
• Jos haluat helpottaa muiden kehittäjien osallistumista, niin koodin kommentointi on aina hyvä ratkaisu.

Hyvältä näyttää ja jatka samaan malliin!

Good luck!