Forum PHP.pl

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

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

1) PSR jest tu taka ilość materiału, że nie wiem jak się za to zabrać (a kod na dzisiaj).
2) === jak mam stosować jak nie znam zastosowania (w PHP 5 nie stosowałem, a 7 dopiero zacząłem używać)
3) typowanie tzn. try i catch - myślałem, że to tylko przy zapytaniach SQL
5) dobra byłaby tu jedna tablica
$arrSex = array (
'male' => array (1, 3, 5, 7, 9),
'female' => array (0, 2, 4, 6, 8)
);
ale jak tu wylosować liczbę dla wartości male lub female?
6) jak rozumiem nowa składnia tablicy, powinna wyglądać tak:
$arrSex = [
'male' => [1, 3, 5, 7, 9],
'female' => [0, 2, 4, 6, 8]
];
Czy to masz na myśli?

ad2) No naprawde nie potrafisz zajrzec do manuala?
https://www.php.net/manual/en/language.oper....comparison.php

ad3) Typowanie a nie lapanie wyjatkow. Typowanie czyli okreslanie typow dla zmiennych oraz typow dla zwrotek funkcji

ad5) Nie, nie kazalem ci robic z tego tablicy wielowymiarowej, tylko usunac duplikacje Choc przepisanie tego na wielowymiarowke tez mozna. Usuniecie duplikacje w tym przypadku bedzie podobne

Ad6 No tak, zmiana array() na []

ad1) Na poczatek zacznij od psr-1 oraz psr-12.

2. Wiem co oznacza operator
=== identyczne
== równe
ale jeszcze nie czuję różnicy podobnie jak z array a [] .
Tablice już poczułem

3. zacząłem w kodzie dodawać przed zmiennymi (int), (string)...
chociaż tak sobie mi to jeszcze wychodzi.

5. Jeden jaki przyszedł mi pomysł na tę duplikacje to napisanie funkcji


ale kiedy usunę te 2:
$arrRand = array_rand($arrValues);
nie działa?

Ten post edytował Malinaa 28.01.2020, 18:25:04

Bo nie czytasz manuala....



No i widzisz jak ladnie. Jak chcesz to potrafisz Ja bym to jednak jeszcze troszke przerobil:

[PHP] pobierz, plaintext 
function randomSex($arrValues) { //USUWAM = [] - jest zbedne
return $arrValues[array_rand($arrValues)];  // DAJE OD RAZU return. Nie ma potrzeby generowac zbednej zmiennej sex jak mozesz wynik od razu zwrocic
}
 
$sex = $pRadioOptions === 'male' ? randomSex($arrMale) : randomSex($arrFemale);
[PHP] pobierz, plaintext 

Czytałem wymienione zdanie, ale nie skumałem co z tym zrobić.

Piękna jest ta funkcja. Jestem pełen podziwu: piękna, czysta i wspaniała jak kwiat doskonała
Myślałem, że o takich rzeczach tylko się marzy, a tu proszę.

Opowiedz jeśli można o tym:
$sex = $pRadioOptions === 'male' ? randomSex($arrMale) : randomSex($arrFemale);
kiedyś zdarzyło mi się pisać taki kod, ale obecnie średnio kumam
jak to działa === ? : i gotowe (proste, bez cudowania z ifami - super)

Teraz jest tak: https://dbm.org.pl/

Pozostała kosmetyka, kłopot jest z inputem.
Jak dam input hidden nie kopiuje. Jak ukryć input, ale aby kopiowało numer?

Ten post edytował Malinaa 29.01.2020, 13:52:28

Tam bylo napisane, ze jak dasz 1 to zwroci ci od razu wylosowany element. Jak dasz wiecej niz 1, to zwroci ci tablic elementow.
A na przyszlosc jak nie wiesz co pisza, to rob var_dump() i patrz co dostajesz. No banalniejszej metody na sprawdzanie juz nie ma.

Odnosnie ?: znowu odsylam do manuala
https://www.php.net/manual/en/language.oper...parison.ternary

Co do pola ukrytego:
widocznie nie dziala z polami ukrytymi.
Tu masz obejscie
https://stackoverflow.com/questions/3159329...xt-to-clipboard

Dziękuję bardzo. I kosmetyka została poprawiona.

Jest jeszcze fragment kodu... nie najlepszy.
Czy możemy ten fragment poprawić?

Nie musisz zmieniać na hidden. Wystarczy, że przesuniesz input poza ekran.

Pomijajac PSR, === oraz zbedne rzutowanie na INT w jednym miejscu to

substr(strstr($count/10, '.'), 1); - czyli chcesz pobrac reszte z dzielenia przez 10? Do tego sluzy MODULO czyli % w php

Poprawiłem, teraz jest tak



Pomińmy PSR (w tym kodzie), gdzie tutaj można użyć === ? INT - czy o ten chodziło.
MODULO czy teraz jest dobrze?

Zobaczyłem jeszcze jedno



W większości przypadków liczba (dla miesięcy) jest dwu cyfrowa, ale
dla lat 1900-1999 są liczby jedno cyfrowe i brakuje 0 z przodu liczby
przez co PESEL ma 10 nie 11 cyfr. Jak dodać 0 przed liczbą: 1, 2, 3... (01, 02, 03)

Również poprawione, jest dobrze.

[PHP] pobierz, plaintext 
$month_number = str_pad($arrMonthsMore[(int)$year_first][(int)$month], 2, '0', STR_PAD_LEFT);
[PHP] pobierz, plaintext 

Jeszcze tylko PSR i będzie super.

Ten post edytował Malinaa 29.01.2020, 13:28:15

Tak na totalnym marginesie:

nie: nie prawidłowa
a: nieprawidłowa

"nie " z przymiotnikami piszemy razem

Totalnie nie zwróciłem uwagi w skrypcie na pisownie
i machnąłem orta (późno, ale poprawione).

Zapytałbym jeszcze o inny temat, na który nie znalazłem jeszcze sposobu:
http://forum.php.pl/index.php?showtopic=267491
Może ktoś wie więcej, jak zrobić taką ocenę dobrze?