tulir / lindeb

A simple link manager with powerful search
https://lindeb.mau.lu
GNU General Public License v3.0
3 stars 2 forks source link

Koodikatselmointi #19

Closed vvenla closed 6 years ago

vvenla commented 6 years ago

Projektia tarkasteltu 3.1. klo 11:30.

Mielenkiintoinen erilainen toteutustapa. Dokumentti on kattava. Luokkakaavio ja relaatiokaavio ovat selkeitä, ja tarvittava monesta moneen -suhde löytyy. Olisin kaivannut ReadMehen helposti löydettävät testikäyttäjätunnukset, sekä toisen viikon palautuksissa kysytyt linkit sovelluksen näkymiin. Nyt pääsin testaamaan sovellusta vain rekisteröitymällä, ja sen jälkeen itse lisäämällä uusia linkkejä tyhjään pohjaan. Näkymät, joissa olisi valmiiksi sisältöä, auttaisivat hahmottamaan sovelluksen toimintaa paremmin.

Koodi näyttää noudattavan Clean code -periaatteita, toki sen seuraaminen on haastavaa jos ei tunne Gota ym.

tulir commented 6 years ago

Olisin kaivannut ReadMehen helposti löydettävät testikäyttäjätunnukset

Harkitsin asiaa aiemmin, mutta rekisteröityminen on vielä tässä vaiheessa niin helppoa (ei vaadi sähköpostia yms), että päätin jättää testitunnukset pois. Jos lisään sähköpostivaatimuksen (#24) niin lisään varmaan myös demotilan, joka ei vaadi kirjautumista (#22)

sekä toisen viikon palautuksissa kysytyt linkit sovelluksen näkymiin

Lisäsin linkit sovelluksen näkymiin (commit 819184ae095e14fb07615c474ad37548b5423f94). Erillisiä näkymiä on tosin vain neljä. Lisäksi kirjautuneet käyttäjät eivät pääse kirjautumisnäkymään ja kirjautumattomat käyttäjät pääsevät vain kirjautumisnäkymään.

En tehnyt erikseen staattisia HTML-sivuja, koska niitä ei olisi voinut uudelleenkäyttää helposti näkymien varsinaisissa toteutuksissa.

Näkymät, joissa olisi valmiiksi sisältöä, auttaisivat hahmottamaan sovelluksen toimintaa paremmin.

Hyvä pointti, taidan lisätä uusille käyttäjille muutaman valmiin linkin ja tagin (#20).