iorena / beerpong-stats Goto Github PK

• Ladattu klo 17 (ti 17.6), testattu firefoxissa

Ohjelmasta:

• toimii kivasti, joitain pikkubugeja:
• Lisää peli: jotkut kentät meni oletusarvoisiksi jos oli alle 10 pistettä kun yritti tallettaa
• Lisää peli: kommenttitekstissä kursori menee aluksi kummaan paikkaan
• Lisää peli: voi olla sama henkilö useassa eri paikassa
• Omat tilastot: päiväys ei toimi
• Joukkueet: voittoprosentti ei toimi
• En havainnut admin kirjautuneena mitään eroa tavalliseen kirjautujaan
  (dokumentin mukainen admin salasana 'iorena') että
• dokumentaatio mainio ja selkeä

koodi:

• views: php koodi on aika kökköistä ainakin viewseistä on vaikea saada tolkkua
• libs/models: tämä on paremmin tajuttavaa koodia, mutta kommentteja ei ole
• juuri: onko paras paikka laittaa controllerit juureen?
• controllerit omat mukavan lyhyitä mutta edelleen kommentit puuttuvat

Koodi ladattu 16.41

Aihe vaikuttaa mielenkiintoiselta ja työ näyttää edistyvän. Yritin laittaa tähän joitakin asioita, jotka pistivät minun silmääni.

• Modellit
  • Users-modelliin kannattaisi lisätä konstruktori, jonka avulla luodaan uusia luokan ilmentymiä.
  • findUser metodi palauttaa käyttäjän id:n user-olion siasta. Järkevämpää olisi luoda funktiossa uusi user-olio tietokannasta saatujen tietojen perusteella ja palauttaa se.
  • Olisi myös kiva, jos modellit vastaisivat tietokannan tauluja. Esim. sinulla on modelli stats, mutta ei taulua stats. Funktiot getPlayerName(), gamesPlayed(), averagePoints(), jne. voisi hyvin siirtää user-modelliin, koska ne palauttavat tietoa joka liittyy johonkin käyttäjään. Se autttaisi myös luettavuudessa, sillä esimerkiksi kun sinulla on luokassa stats metodi averagePoints(), on vähäsen hankala päätellä mitä metodi tekee. Nimen perusteella se voisi palauttaa kaikkien pelaajien keskiarvon, tietyn ottelun keskiarvon, tai ties vaikka mitä. Siirtämällä tämän user-malliin selkiintyy, että tämän metodin tarkoituksena on palauttaa yhden pelaajan keskiarvo.
• Tietokantayhteys saadaan tiedostosta yhteys.php. Koska tiedosto ei ole kontrolleri, vaan yleiskäyttöinen apuluokka, siirtäisin sen projektin juuresta libs-kansioon.
• Näkymien ongelmia

Näkymissä on tägejä vähäsen oudosti. Esim views/login.php ja views/index.php lopussa on body-tägi. Muissa vieweissä, kuten vaikka views/mystats.php kyseistä tägiä ei ole. Koska body:n sulkeminen kaikilla sivuilla on hyvä idea, suosittelisin tägin siirtämistä sivupohjaan.

• Tarkkuutta sisennyksiin

Koodi on sisennetty, mutta sisennykset ovat sekoitus tabeja ja välilyöntejä. Tämä tekee sisennyksistä sekavaa ainakin minun koneella. Kannattaa valita yksi sisennystyyli ja pysyä siinä. Esimerkkejä:

• Sisennys tab tab väli: https://github.com/iorena/beerpong-stats/blob/37fa6b87fcbfb205e4874f71fdcccdbdb4c0b9f0/views/common.php#L10-13
• Sisennys tab tab väli tab:

  beerpong-stats/views/login.php

  Line 16 in 37fa6b8

  <input type="password" class="form-control" id="inputPassword" name="password" placeholder="Salasana">

Ongelma näyttäisi keskittyvän vieweihin.

• Salasanat ovat tietokannassa selväkielisenä. Salasanat olisi luonnollisesti hyvä tiivistää salasanoille suunnitellulla tiivistefunktiolla (kuten bcrypt tai scrypt). Muistaakseni kurssilla tätä ei kuitenkaan vaadita.