Prośba o CR, formularz kontaktowy

pytanie zadane 3 października 2018 w PHP przez sebeks Mądrala (5,300 p.)

Proszę o code review formularza kontaktowego w PHP. Potrzebowałem takiego i otrzymałem taki kod https://pastebin.com/NjCkFmm9 Nie jestem jego autorem, a PHP znam tyle co z kursów Mirka, więc potrzebuję by ktoś bardziej doświadczony ocenił i poprawił część w PHP. Tym bardziej, że autor skryptu też nie jest zawodowym programistą. Wasze odpowiedzi poprawiające kod też mu przekażę. Część skryptu zawierająca HTML na pewno będzie przepisana na HTML 5. Ja wiem, że na sieci jest tego trochę, ale jeśli ktoś ma i może udostępnić kod swojego formularza kontaktowego w PHP to też chętnie się z tym kodem zapoznam.

1 odpowiedź

odpowiedź 4 października 2018 przez OdsetekGlupoty Pasjonat (15,360 p.)
wybrane 4 października 2018 przez sebeks

Najlepsza

Witam,

Będę leciał po kolei:

Po co tworzysz 2 zmienne $flagAllOK i $failFlag? Wystarczy, że utworzysz $failFlag a później sprawdzisz czy ta tablica jest pusta czy nie.
Nie powinno się mieszać kodu HTML z PHP. Tam gdzie poprzez PHP wypisujesz parametr value pół tekstowych, nie powinno być żadnego warunku. Warunek sprawdź w kodzie u góry. Najpierw ustaw zmienną $mail na pusty ciąg, a później po sprawdzeniu czy użytkownik wysłał formularz, ta zmienna automatycznie zmieni się na prawidłowy mail. A w polu value wystarczy value="<?php $mail ?>".
Według standardu PSR-2, klamra otwierająca w ifie powinna być w tej samej linii co rozpoczęcie ifa a nie w następnej.
Do sprawdzenia czy użytkownik wysłał formularz, nie powinno używać się tego śmiesznego ifa, którego masz na początku, tylko
```
if ($_SERVER['REQUEST_METHOD] == 'POST'
```

komentarz 4 października 2018 przez sebeks Mądrala (5,300 p.)

Ok, wielkie dzięki :) Przekaże uwagi autorowi kodu. Z Twojego komentarza wnioskuję, że kod jest w miarę Ok (za wyjątkiem pkt.4) i wymaga tylko niewielkich poprawek.