Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
Reply to this topicStart new topic
> Proszę o ocenę - GitHub
RazoR2011
post 18.12.2017, 13:15:41
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
Go to the top of the page
+Quote Post
viking
post 18.12.2017, 13:19:35
Post #2





Grupa: Zarejestrowani
Postów: 6 365
Pomógł: 1114
Dołączył: 30.08.2006

Ostrzeżenie: (0%)
-----


https://github.com/php-fig/fig-standards/bl...-style-guide.md


--------------------
Go to the top of the page
+Quote Post
nospor
post 18.12.2017, 13:33:30
Post #3





Grupa: Moderatorzy
Postów: 36 440
Pomógł: 6290
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!";
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!"


--------------------

"Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista
"Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer

Go to the top of the page
+Quote Post
RazoR2011
post 18.12.2017, 13:55:17
Post #4





Grupa: Zarejestrowani
Postów: 38
Pomógł: 1
Dołączył: 20.04.2011

Ostrzeżenie: (0%)
-----


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

Super - dziękuję biggrin.gif
Go to the top of the page
+Quote Post
Pyton_000
post 18.12.2017, 21:12:19
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 wink.gif
Go to the top of the page
+Quote Post

Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Wersja Lo-Fi Aktualny czas: 28.03.2024 - 16:34