Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Koodikatselmointi 1 #1

Open
oraisa opened this issue Nov 15, 2015 · 0 comments
Open

Koodikatselmointi 1 #1

oraisa opened this issue Nov 15, 2015 · 0 comments

Comments

@oraisa
Copy link

oraisa commented Nov 15, 2015

Repositorio kloonattu 14.11.2015 kello 17.04
Kaikki näkymät ovat täysin staattisia, eikä repositoriossa näytä olevan niiden lisäksi muuta koodia kun sql-tiedostot ja Lumen-frameworkin käyttämien tiedostojen runkoja. Katselmoitavaa koodia on siis hyvin vähän.

  • Näkymissä käytetään uudelleenkäytettäviä templateja oikein hyvin, mikä poistaa copy-pasten niistä kokonaan.
  • 255 merkin maksimipituus tulee todennäköisesti olemaan liian lyhyt ohjeiden ja niiden vaiheiden kuvauksiin. Esimerkkisivun lorem ipsumissa on 1354 merkkiä. Vaikka kuvaukset eivät olisi läheskään niin pitkiä, niistä tulee silti huomattavasti pidempiä kuin 255 merkkiä.
  • Drop tables-tiedostossa kannattaa käyttää CASCADE-avainsanaa, sillä silloin myös kaikki tauluihin viittaavat näkymät poistetaan ja foreign key-tarkistuksia ei tarvitse laittaa pois päältä, ne rajoitukset poistetaan taulujen poiston yhteydessä.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant