Review - Frod Package

Acum doua luni am fost atat de entuziasmat de un proiect incat sunt posibilitati sa aveti multe de comentat pe seama lui, dar cu siguranta discutiile pe tema asta nu pot aduce decat beneficii…

Cod: https://github.com/IonutBajescu/frod/tree/master/src/Ionut/Frod
Aici m-am jucat putin si am separat partea de CSS de cea de JS folosind trait-uri, aruncandu-le pe fiecare in propriul lui fisier. Pentru mine e mult mai bine, dar sunt curios ce parerea au altii de asta.

Teste: https://github.com/IonutBajescu/frod/tree/master/tests (aici sunt operatii I/O pe fisiere, natura proiectului m-a obligat sa fac asta)

2 Likes

ce-ar trebui sa faca?

Vezi aici :slight_smile: https://github.com/IonutBajescu/frod/blob/master/readme.md
Apropo, interesant!

M-am uitat pe cod, apreciez ca ai depus destul de mult efort, dar nu cred ca ai folosit o abordare simpla a problemei.

Nu-mi este foarte clara valoarea daca structurez un proiect dupa conventiile “Frod”. NPM, Bower, Component iti sunt cunoscute?

Da, imi sunt cunoscute. Dar sunt chestii diferite.

Frod-ul este ceva mai diferit, nu necesita manevrarea prin consola.

Este “live”, pachetele nu sunt instalate, frod actioneaza direct din codul php, totul se petrece in cod - nu in consola.

M-am tot gandit daca nu am complicat lucrurile. Se puteau face scurtaturi la partea de storage, dar am dorit ca in v2 sa poti uploada asset-urile locale catre CDN-uri.

mie-mi place ideea. nu l-am incercat da pare misto.

Wow, atâtea comentarii și nici un review :slight_smile:

Na, că sunt eu primul care aruncă piatra: nu fac eu foarte mult unit testing, dar chiar este ok să ai logică în teste? Mă gândesc că dacă ai logică în teste vei avea nevoie de un alt set teste să testezi primul set de teste. Greșesc?

Nu este logica acolo, sunt doar teste automate create automat(ca sa nu pun assert-urile alea manual, m-am folosit de foreach si preg match).
Pentru ca link-urile alea se presupune ca sunt generate corect, iar daca nu functioneaza, testele esueaza.

Dar ai dreptate, am trisat la unit testing, de multe ori am testat chestii care nu prea aveau legatura cu unit, ci doar cu testing.
Dar asta ca sa ma asigur ca nu stric niciodata nimic, iar daca stric, travis este cu ochii pe commit-urile mele.

Sa mai asteptam o parere ca nici eu nu stiu cat de bun e foreach-ul ala, poate necesita o alta abordare.

Păi ce poate conține mai multă logică decât condițiile if ? :smiley:

1 Like

Ai dreptate.

https://github.com/IonutBajescu/frod/commit/594110f0702632c9eb8125e26c2fa911bac7dcd5