toxpuro / tsoha Goto Github PK

Kunnianhimoinen ja laaja aihe! 💯

Kommentit sovelluksesta Herokussa

• Toimii hyvin: ryhmän lisääminen, threadin lisääminen ja haku
• Ei toimi vielä: threadin äänestäminen, kommentin lisääminen threadiin tai ryhmään liittyminen (yhdessä threadissa oli kommentteja, mutta en itse onnistunut niitä lisäämään)
• Syötteiden pituudet tarkistetaan 👍🏻
• Hakukentän ohje-teksti englanniksi vaikka sovellus muuten suomeksi
• En päässyt urleja manipuloimalla näkemään mitään ei-sallittua 👍🏻
• Huomio: jos lisää saman nimisen ryhmän ei tule virheilmoitusta, mutta ryhmä ei myöskään näy tuplana - ehkä näin tarkoituskin?
• Jos aikaa jää, virhesivut voisi lisätä esim. osoitteille joita ei löydy

Huomiona seuraaviin kommentteihin: tämä on ensimmäinen kurssi, jolla käytän Pythonia, joten sen osalta en osaa kovin syvällisiä kommentoida..

Sovelluksen koodi ja rakenne

• Venv, pycache yms. hakemistot kannattaa varmaan deletoida githubista, olivat ehtineet uida mukaan alussa. Nyt gitignore kunnossa.
• Sovelluksen rakenne vaikutti selkeältä - jaettu järkevästi nimettyihin hakemistoihin 👍🏻
• Lomakkeissa huomioitu syötteiden maksimipituudet - hyvä! 👍🏻
• Csrf huomioitu lomakkeissa - hyvä! 👍🏻 (Lukuunottamatta new.html, mutta se ei tainnut olla käytössä)
• Pitkät SQL-kyselyt kannattaisi ehkä rivittää luettavuuden helpottamiseksi (oliko projektissa pylint käytössä?)

Muuta

• flash() oli minulle uusi tuttavuus- kiitos opin uutta!

Testattu 12.10.2021 klo 0:30

Sovellusta on helppoa ja intuitiivista käyttää, ja sillä on miellyttävä ulkonäkö.

Muutama ongelma kuitenkin löytyy:

• Jos et ole liittynyt mihinkään ryhmään ja yrität luoda ketjun, serveri palauttaa Bad Requestin
• Rekisteröintisivulla painaessa "Kirjaudu"-painiketta selain ohjautuu "/login" sivulle, joka ei salli GET-pyyntöjä.
• Yrittäessä poistaa ja muokata ketjua sain virheen. Ketju oli minun tekemä ja sijaitsi ryhmässä, josta olin poistunut.
• Kaikilla sivuilla olisi hyvä olla linkki mennä yksi taaksepäin, esim. ketjusta takaisin ryhmään. Nyt käyttämättä selaimen toimintoja pääsee vain etusivulle.
• Olisi intuitiivista, jos dropdown-valikot sulkeutuisivat klikatessa muualle tai kun avataan toinen valikko.
• Ketjujen kommenttien pisteiden olisi varmaan parempi olla upvote/downvote -painikkeiden vasemmalla puolella, jottei niitä sekottaisi kommentin osaksi.

Kurkkailin hieman koodiasi ja voi hyvinkin olla että moni näistä bugeista joutuu Pythonin dictionaryistä (joita esim. request.form ja session ovat). Jos koitat hakasuluilla hakea dictionarystä jotain, esim. session["username"] ja avainta username ei ole olemassa, se heittää KeyErrorin. Kun avain ei mahdollisesti ole olemassa, kannattaa käyttää tässä tapauksessa session.get("username"), joka antaa Nonen jos avainta ei ole olemassa. Vaihtoehtoisesti getin toiseksi parametriksi voit myös antaa arvon, jonka haluat Nonen sijaan.

Nämä kaikki ovat onneksi kuitenkin aika pieniä ongelmia, ja projekti on muutoin hyvällä mallilla. Hieno homma ja onnea jatkoon!

Hei!

Yritin luoda käyttäjätunnusta sovellukseen, jotta pääsisin testaamaan, mutta saan aina virheilmoituksen, että "Käyttäjänimi on jo otettu". Voisitko laittaa minulle testitunnukset joko telegramiin tai spostilla.

-- Pia