Koodikatselmointi

[stadibo]

Haettu 30.11.2018 klo. 17.50

Positiivisia kommentteja

• Koodin luettavuus on hyvällä tasolla
• Hyvät kommentit niissä kohdissa missä niitä oli. Selkeyttää koodiin tutustumista merkittävästi.
• Yleisesti muuttujilla ja metodeilla kuvaavat nimet.
• Näkymien kansiorakenne on järkevä.
• Erittäin siisti toteutus monen kategorian bookmarkiin liittämiselle.
• Hyvä, että kategoriaa poistettaessa ei poisteta bookmark.

Parannusehdotuksia

Pienet:

• Create nappi voisi olla hieman näkyvämpi, ehkä toisessa reunassa.
• Vaikka on aika tehokasta ja selkeää mitä yhden kirjaimen muuttujat b (bookmark) ja c (category), niin olisi hyvä pitää yhdenmukaisuus niin, että kaikki muuttujat ovat kuvaavasti nimetty. Esim. “user” muuttuja ei ole nimetty “u”...

Vähän suuremmat:

• Uuden käyttäjän rekisteröidyttyä olisi mukavaa kirjata sisään tämä käyttäjä, jotta sovelluksen käyttöönotto olisi sulavampaa.

• Olisiko käyttötarkoituksen perusteella hyvä tarkastaa että duplikaatti bookmark ei luoda samalle käyttäjälle saman kategorian alle? (Jos on tarkoitus tehdä bookmark ylläpito selkeämmäksi)

• Kun luo uuden bookmark niin on tällä hetkellä mahdollista luoda se ilman “text” kentän arvoa, joka voisi myös nimetä kuvaavammin, esim. title (koska on jo olemassa description). Tämä johtaa yhden kategorian bookmark listauksessa siihen, että ei ole olemassa tekstiä mihin kiinnittää linkki bookmarkiin.

• Jos bookmark määrä kasvaa ja poistaa joihinkin liittyvät kategoriat, voi olla vaikeaa löytää nämä bookmarkit ilman kategoriaa. Voisi siis lisätä filtteröinti että on kategoria “ilman kategoriaa” joka näkyy kategorioissa aina kun on olemassa bookmark ilman kategoriaa.

[ikanher]

Moi,

Kiitoksia palautteesta ihan miljoonasti! :)

Hyvä komentti toi u/c/b (user/category/bookmark) nimeämisestä. En ollut itse tajunnutkaan, että noissakin olisi hyvä suosia yhtenäistä nimeämiskäytäntöä..!

Ton voisi kanssa, että kirjaudutaan heti sisään rekisteröimisen jälkeen. Ollut itsellä mielessä myös, mutta jotenkin jäänyt toteuttamatta..

Duplikaatteja en ole edes yhtään kerennyt vielä miettiä, hyvä pointti taas.. URL varmaan voisi aika hyvin identifoida onko Bookmark jo olemassa. Varmaan pitää kanssa korjailla, ettei voi olla samannimisiä kategorioita.

Ja nimeämisessä myös varsin oikeassa! Link voisi olla URL (ehkä, tietääkö kaikki mikä se on?) ja text tosiaan title.

Ja edelleen. Oon kanssa miettinyt tuota. Voisikohan siinä 'Bookmarks' listalla olla jotkut valinnat 'List all', 'List uncategorized' ja 'Create' tms... Mutta en ole tätä vielä kerenny pohtimaan, mikä olisi paras käyttökokemus.

Kiitos tosi paljon. :) Toivottavasti saan yhtä hyvän katselmoijan viimeiseen koodikatselmointiin!