Cześć,
Prosiłbym o ocenę funkcji służącej do generowania określonych typów haseł. Nie chodzi mi o to, że skrypt nie będzie przydatny. Chciałbym wiedzieć czy kod jest "lekki", czy jego sposób umieszczenia na gitHubie jest prawidłowy?
Link: github.com/wudokacom/passGen
https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-2-coding-style-guide.md
Procz tego co podal viking to jeszcze:
nie ma za bardzo co oceniac. Ot zwykly for bioracy losowo cos z tablicy...
W dobie php7 pisanie kodow nie w php7 to juz nieteges.
wypadaloby poznac roznice miedzy == a === i stosowac sie do tego drugiego
return $error = "Not supported type!";
Pomijajac bezsensowne tworzenie zmiennej $error to jeszcze ten return.... funkcja ma zwracac wyjatek gdy cos jest nie tak a nie error jako return gdzie mialo byc haslo. Toz teraz jako haslo dostane "Not supported type!"
Właśnie o to mi chodziło!
Super - dziękuję
Polecam http://phppackagechecklist.com/#
- Zrób z tego paczkę composera (tzn najpierw tylko na githubie)
- Napisz sobie Unit Testy (bo masz błędy)
- Uzywaj PHP7 !
- Do tego co @viking podał możesz używać albo edytora ze wsparciem albo https://github.com/FriendsOfPHP/PHP-CS-Fixer
- Zrób z tego ładny kod. Nawet mała funkcjonalnośc może być ładnie napisana.
Ucz się dalej a będą z Ciebie ludzie
Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)