tsoha-cook-it's People
tsoha-cook-it's Issues
Vertaisarviointi
Sivusto toimii sulavasti ja on intuitiivinen käyttää. Reseptien lisääminen, poistaminen ja kommentointi toimii. Hakutoiminnasta pidin erityisesti. Whats buzzing kentän kanssa näytti olevan jotain ongelmaa, se näytti tyhjää vaikka kommentoituja reseptejä olikin. Pieninä muotoseikkoina Add new comment kentässä lukee Title of recipe, tulee tavallaan kahteen kertaan. Sitten, like nappulaa painamalla uudestaan tykkäys häviää joka sinänsä on mielestäni loogista, mutta ehkä painikkeen nimeä voisi muuttaa samalla unlike että toiminta olisi selkeämpi?
Koodi on loogista ja selkeää. Yksi asia mikä osui omaan silmään, recipes taulussa on like_count ja comment_count kentät joita päivitetään, tuleeko tässä päällekkäisyyttä koska myös likes taulukossa on periaatteessa sama tieto tallessa? Muuten sql kyselyt on helppo ymmärtää, ja koodin jako moduuleihin vaikuttaa toimivalta.
Code Review 1
Vertaisarviointi
- Projekti näyttää olevan jo hyvällä mallilla
Heroku
- Sivuston käyttö on helppoa
Search recipes
-näppäin ei toimi, joten se voisi olla hyvä piilottaa sivulta- Uuden reseptin luominen onnistuu helposti, käyttöliittymä on selkeä
- Myöskään etusivun
What's trending
ja profiilisivunRecipes liked
näppäimet eivät tee mitään, joten nekin voisi piilottaa kunnes niiden toiminnallisuus on toteutettu - Ulkoasu on miellyttävä
Koodi
- Koodi pääosin selkeää ja rakenteeltaan hyvää
- Jaettu järkevästi eri tiedostoihin
routes.py
- Rekisteröitymislomakkeen salasanatarkistuksen voisi tehdä JavaScriptillä, jolloin virheviestin saa näkyviin esimerkiksi heti seuraavaan kenttään siirryttäessä ja lomaketta ei tarvitse ladata kokonaan uudelleen, jos esim. salasana on liian lyhyt
- Pieniä tyylivirheitä välilyöntien käytössä muuttujien sijoituksessa funktion kutsun nimetyiksi parametreiksi, ks. materiaali
users.py
logout
-funktio ei tyhjennä kokosession
iacheck_csrf
-funktiossa käytettyäabort
-metodia ei ole määritelty (lisää flask importteihin)
recipes.py
- Tietokannasta haetaan tietoa hienosti suoraan pelkillä SQL-komennoilla
- Funktioille voisi kuitenkin luoda docstring-kommentteja niissä olevien SQL-komentojen palauttaman sisällön helpompaa ymmärtämistä varten
- timestampin luominen onnistuu myös suoraan PostgreSQL:llä (
NOW()
) get_recipes
-funktion viimeinen return on turha, sitä ei suoriteta koskaan
Muuta
- HTML-templatet omaan silmään hyvin rakennettuja
Profile page has issues
Author's name isn't shown correctly in liked and commented by user lists.
What's buzzing has issues
What's buzzing does not currently work for users that have not added any recipes yet.
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.