 Proszę o ocenę - GitHub |
  |
| Proszę o ocenę - GitHub |
Post
#1
|
|
|
Grupa: Zarejestrowani Postów: 38 Pomógł: 1 Dołączył: 20.04.2011 Ostrzeżenie: (0%) 
 |
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 |
 |
 
|
|
Post
#2
|
|
|
Grupa: Zarejestrowani Postów: 6 381 Pomógł: 1116 Dołączył: 30.08.2006 Ostrzeżenie: (0%)
|
|
|
|
|
|
Post
#3
|
|
|
Grupa: Moderatorzy Postów: 36 559 Pomógł: 6315 Dołączył: 27.12.2004 |
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!"; (IMG:style_emoticons/default/questionmark.gif) 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!" |
|
|
|
|
Post
#4
|
|
|
Grupa: Zarejestrowani Postów: 38 Pomógł: 1 Dołączył: 20.04.2011 Ostrzeżenie: (0%)
|
|
|
|
|
|
Post
#5
|
|
|
Grupa: Zarejestrowani Postów: 8 068 Pomógł: 1414 Dołączył: 26.10.2005 Ostrzeżenie: (0%)
|
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 (IMG:style_emoticons/default/wink.gif) |
|
|
|
|
|
Aktualny czas: 9.10.2025 - 19:14 |
