forked from Tsoha/Tsoha-Bootstrap
-
Notifications
You must be signed in to change notification settings - Fork 1
Open
Description
Koodi ladattu 25.4 n. 21.30
Muutama huomio:
- Favor composition over inheritance: Minulle tuli tästä koodin haju, tuli semmoinen intuitio että BaseModel-luokan voisi kirjoittaa jollain muulla tavalla kuin että se perii kaikkia luokkia.
- Esim. Blog-luokan voisi mielestäni tehdä mieluummin siten että se sitoisi enemmänkin vain domain-logiikkaa. Nyt Blog-luokka hoitaa ACL:ää, Domain-logiikkaa ja tietokantaan siirtoa. Mielestäni varsinaisen tietokantayhteyden voisi abstraktoida omaan palveluun joka olisi vastuussa Blog-luokan luonnista, hausta tietokannasta jne.
Ohjelma näyttää toimivan ja näyttää hyvältä.
Kokonaisuutena hyvää työtä!
Metadata
Metadata
Assignees
Labels
No labels