PHP Junior Backend Developer állás - Code Review?
Sziasztok!
Kinéztem magamnak egy állást, amit hamarosan szeretnék megpályázni.
Az álláshírdetés követelményei: 1 év tapasztalat: PHP (OOP szemlélet), HTML, GIT technológiákban
A cég github repojában szerencsére találtam teszt feladatokat jelentkezők számára, ezekből választottam egy nekem szimpatikusat.
Úgy döntöttem ez egy remek lehetőség kipróbálni magam "élesben", így elkezdtem megcsinálni az eddigi tapasztalatom és tudásom alapján.
Symfony 6 keretrendszert 2 hete kezdtem el tanulni, így ezt használtam a feladatnál is.
Sajnos nincs tapasztalt (senior) webfejlesztő ismerősöm, aki megnézné amit csináltam, így gondoltam megosztom itt, hátha akad egy, aki szánna pár percet egy code reviewra.
Arra lennék kíváncsi, hogy a megoldásom alapján megfelelek-e egy junior szintnek vagy tanuljak még jelentkezés előtt?
Github Repo: [link]
README tartalmazza a feladat leírását, illetve fent van a Swagger fájl is
Köszönöm!
válasza:Nem nagyon néztem bele, mert ez nem társkereső, meg nem vagyok PHP expert.
Clean code szempontból nem rossz elsőre, de egy komoly hibát azért látok, amin én elkaszálnék egy ilyet:
Abszolút 0 kommentet írtál. Értem én hogy self-documenting a kód, és egészen átlátható, mert nem sok tartalomra sem, de azért a 0 komment az erősen problémás.
Nem tudom melyik cégnél mi a guide, de szerintem a legtöbb komoly helyen 30% minimum hasznos kommentet elvárnak.
Hasznos információ, köszönöm!
Mindig igyekszem úgy elnevezni a metódusokat / változókat, hogy még egy laikus is értse mi történik
Remélem egy PHP experttől is sikerül pár tanácsot kapnom :D
válasza:#1
"Nem tudom melyik cégnél mi a guide, de szerintem a legtöbb komoly helyen 30% minimum hasznos kommentet elvárnak."
Én nagy multiknál dolgoztam csak, de sehol nem találkoztam ilyen guide-dal.
Mi is csak akkor írunk, ha nagyon fontos, egyébként senki nem ír.
További kérdések:
Minden jog fenntartva © 2024, www.gyakorikerdesek.hu
GYIK | Szabályzat | Jogi nyilatkozat | Adatvédelem | Cookie beállítások | WebMinute Kft. | Facebook | Kapcsolat: info(kukac)gyakorikerdesek.hu
Ha kifogással szeretne élni valamely tartalommal kapcsolatban, kérjük jelezze e-mailes elérhetőségünkön!