![]() |
![]() |
![]()
Post
#1
|
|
Grupa: Zarejestrowani Postów: 577 Pomógł: 6 Dołączył: 21.07.2008 Ostrzeżenie: (0%) ![]() ![]() |
Usiłuję wykonać generator numeru PESEL KOD TUTAJ
Mam prośbę o sprawdzenie i poprawienie błędów. Ten post edytował Malinaa 29.01.2020, 13:50:55 |
|
|
![]() |
![]()
Post
#2
|
|
Grupa: Moderatorzy Postów: 36 559 Pomógł: 6315 Dołączył: 27.12.2004 ![]() |
Nie bede ocenial czy prawidlowo to generujesz, ale pare poprawek odnosnie kodu
1) Zacznij uzywac PSR bo tragicznie sie czyta twoj kod, albo uzywaj php-cs-fixer co ci juz mowiono. A najlepiej i to ito 2) uzywaj === - tez zdaje ci sie juz mowilem. Po co pytasz o poprawy kodu skoro potem i tak sie nie stosujesz do wskazowek 3) brak typowania 4) Po co losujesz 2 liczby skoro i tak uzywasz zawsze jednej? array_rand($arrMale, 2); 5) if ($pRadioOptions == 'male') { $arand = array_rand($arrMale, 2); $sex = $arrMale[$arand[0]]; } else { $arand = array_rand($arrFemale, 2); $sex = $arrFemale[$arand[0]]; } Strasznie duza i bezsensowna duplikacja. Pomysl jak to poprawic 6) Jak juz uzywasz [] a nie array() to badz konsekwentny a nie raz to a raz tamto |
|
|
![]() ![]() |
![]() |
Aktualny czas: 15.10.2025 - 15:17 |