Drukowana wersja tematu

Kliknij tu, aby zobaczyć temat w orginalnym formacie

Forum PHP.pl _ Algorytmy, klasy, funkcje _ Proszę o ocenę - GitHub

Napisany przez: RazoR2011 18.12.2017, 13:15:41

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

Napisany przez: viking 18.12.2017, 13:19:35

https://github.com/php-fig/fig-standards/blob/master/accepted/PSR-2-coding-style-guide.md

Napisany przez: nospor 18.12.2017, 13:33:30

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!"

Napisany przez: RazoR2011 18.12.2017, 13:55:17

Właśnie o to mi chodziło!

Super - dziękuję

Napisany przez: Pyton_000 18.12.2017, 21:12:19

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