Feedback proiect

[anrei0000]

Iti las mai jos cateva puncte pe care le poti imbunatati. Sunt intr-o ordine aleatorie si de dificultati diferite.

• gandeste-te cum sa separi php de html (sa nu fie in acelasi fisier)
• foloseste require_once
• scoate datele de conexiune din git
• foloseste composer: iti lipseste composer.json
• foloseste OOP: functiile din DBconnect
• corecteaza erorile din verifyinput.php: in php nu se declara o variabila asa $result;
• separa codul care face query-uri de codul care foloseste rezultatele: in verifyinput.php functia loginUser ar trebui sa apartina unui controller iar functia usernameExist ar trebui sa apartina unui model.
• verify naming conventions: ThisIsACorrectClassName, ThisIsACorrectClassFilename, $thisIsAVariableName
• implementeaza autoload

Cele mai importante ar fi:

• foloseste OOP
• foloseste composer
• fa un demo la proiect

[tudor-nicholas]

-gandeste-te cum sa separi php de html (sa nu fie in acelasi fisier) {notat} -foloseste require_once { am folosit } -scoate datele de conexiune din git {notat} -foloseste composer: iti lipseste composer.json { am folosit doar ca nu pot da upload cu tot cu fisierul de vendor pentru ca e prea mare https://prnt.sc/iB962w-fpzVw } -foloseste OOP: functiile din DBconnect {notat} -corecteaza erorile din verifyinput.php: in php nu se declara o variabila asa $result; {aceelea sunt errori decat in phpstorm, dar site-ul merge la perfectie, ceea ce inseamna ca detectorul de errori din phpstorm nu functioneaza cum trebuie} -separa codul care face query-uri de codul care foloseste rezultatele: in verifyinput.php functia loginUser ar trebui sa apartina unui controller iar functia usernameExist ar trebui sa apartina unui model. { exact ce ai scris mai sus de chestia cu oop-ul o sa o fac pe viitor, doar ca a fost mai usor sa fac proiectul cu functii simple} -verify naming conventions: ThisIsACorrectClassName, ThisIsACorrectClassFilename, $thisIsAVariableName {????} -implementeaza autoload {este implementat} Cele mai importante ar fi:

-foloseste OOP {notat} -foloseste composer {este implementat plus inca vreo 2 librarii fiind cea de la mpdf si stripeAPI} -fa un demo la proiect {??????}

[anrei0000]

-foloseste require_once { am folosit }

in app/DBconnect nu ai folosit

am folosit doar ca nu pot da upload cu tot cu fisierul de vendor pentru ca e prea mare https://prnt.sc/iB962w-fpzVw

nu e nevoie sa dai upload la tot, ci doar sa adaugi fisierul composer.json si composer.lock in git. de acolo, cand altcineva iti cloneaza proiectul, pentru instalare poate folosi composer install si composer instaleaza toate dependintele

{aceelea sunt errori decat in phpstorm, dar site-ul merge la perfectie, ceea ce inseamna ca detectorul de errori din phpstorm nu functioneaza cum trebuie}

ai feedback-ul meu, take it or leave it

-verify naming conventions: ThisIsACorrectClassName, ThisIsACorrectClassFilename, $thisIsAVariableName {????}

In scris, e suficient sa scrii ? in loc de ????.

Ex1:

Ex2:

Ex3:

-fa un demo la proiect {??????}

In scris, e suficient sa scrii ? in loc de ??????.

Clipul pe care mi l-ai trimis aici https://youtu.be/p6ajFsWtiHY e un exemplu super OK pentru site (desi are alt scop). Ti-am lasat comment-ul in ideea in care in timp vei uita ce ai facut intr-un proiect. Si, e o idee buna sa ai ceva care sa-ti aduca aminte cum ai facut lucrurile.