Koodikatselmointi

[rescawen]

Koodikatsemoinnin aika: 11:17 AM, 08.04.2019. Projektia ei ole viimeisen kuuden päivänä muokattu.

Projekti on tällä hetkellä periaatteessa todosovelluksesta vähän muokattu versio. Task:ien tilalla on pokemonit.

Nyt tämä tietokanta kaaviossa nimetty pokeboard ja koodissa kansio arena, jotka ovat varmaan sama asia on vasta kehityksen alkuvaiheessa. Minun piti kommentoida koodit pois kaikkien arena kansion tiedostoista koska tuli monta eri module not found virheilmoitusta kun yritin suorittaa projektia. Kyseisessä kuvassa on vain yksi monesta module not found virheistä. Tämmöistä koodia ei kuuluisi committaa ollenkaan.

Käyttäjän ja pokemonin yksi moneen suhde on ongelmallista koska teoriassa monet käyttäjät joutuvat lisätä samoja pokemoneja. Teoriassa olisi parasta jos suhde on monesta moneen, niin että sovelluksessa on valmiiksi pokemoneja joita voi ottaa haltuun. Toisin sanoen käyttäjät voisivat ottaa haltuun toisten käyttäjien luomia pokemoneja.

Tietokanta kaaviossa tyyppi on erillinen taulukko, mutta todellisuudessa se on vain toinen attribuutti pokemon taulukossa. Käyttäjä on vain oma taulukko ja pokeuser taulukkoa ei ole vielä implementoitu.

Käyttäjät pitää manuaalisesti lisätä tietokantaan, eli rekisteröityminen puuttuu täysin.

Validointi puuttuu kaikkialta ja pokemonien poistaminen puuttuu myös. Nyt voi luoda ja editoida tyhjiä pokemoneja tietokantaan.

Oikeita HTML-elementtejä on käytetty ja isoja ongelmia ei ole tältä puolelta. Projektiin ei kuitenkaan ole otettu bootstrap tai vastaavaa käyttöön. Frontend:ssä ainut ongelma on kirjautumatta List Pokemon ja New Pokemon linkit ovat näkyvillä vaikka ne pitäisi piilottaa kokonaan jos niitä ei voida näyttää kirjautumattomalle käyttäjälle.

Heroku linkki on rikki.

[vikke1234]

ei näköjään pushannu mun viimeisimpiä commit:eja ja nytten laturi hukassa

[vikke1234]

ne ovat nytten korjattuja, luulin että pushasin ne sinne