Git Product home page Git Product logo

tsoha-cook-it's People

Contributors

ruusukivi avatar

Watchers

 avatar

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 profiilisivun Recipes 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ä koko sessionia
  • check_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

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo 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.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.