PHP || Vendégkönyv dolgok ahogy én csinálom. Van rá szebb módszer? Ti mit csinálnátok?
Van egy form; név, email, üzenet
Van egy script amely ellenőrizgeti a dolgokat pl.:
--Bannolva vagy-e
--Helyes-e a captcha
--Eltelt-e harminc másodperc az utolsó posttól
--Volt-e input ellenőrzésnél valami error (hibás e-mail ...blabla)
--Lefutott-e a query
Ehhez rengetek if függvény van használva, amelyek miatt én úgy látom, hogy a kód csúnya nagyon. Milyen megoldások vannak amelyekkel szebbé lehet tenni? (Nem OOP, de lehet OOP a válasz)
Ez összesen 6 } jelet jelent.
Tehát van ennél szebb megoldás?
válasza:Annyi if kell, amennyi hibaüzenetet akarsz megjeleníteni. De ha ellenőrzöl mindent egyszerre a végén meg csak elég ha kiírod, hogy sikerült elküldeni, vagy nem sikerült, akkor elég csak egy if.
Az egymásba ágyazást úgy lehet így lehet elkerülni:
$errors = array();
if(empty($_POST['nev'])) {
$error[] = 'Üres a név.';
}
if(!isEmail($_POST['email'])) {
$error[] = 'Nem jól írtad be az email címet.';
}
if(empty($_POST['msg'])) {
$error[] = 'Üres az üzenet.';
}
if(empty($errors)) {
// Hozzászólás küldése.
}
else {
echo join('<br />', $errors);
}
válasza:Szívesen megmutatnám a kódot, de ha ide kiposztolom elég olvashatatlan lesz.
Hova lehet posztolni ahol formatálja is a kódot?
válasza:} else { if...
helyett
} elseif {
és máris átláthatóbb lesz.
De így mindig leáll az első hibánál. Miért nem ellenőrzöd mindet, és kiírsz egy listát az összes hibáról?
válasza:Kapcsolódó kérdések:
Minden jog fenntartva © 2024, www.gyakorikerdesek.hu
GYIK | Szabályzat | Jogi nyilatkozat | Adatvédelem | Cookie beállítások | WebMinute Kft. | Facebook | Kapcsolat: info(kukac)gyakorikerdesek.hu
Ha kifogással szeretne élni valamely tartalommal kapcsolatban, kérjük jelezze e-mailes elérhetőségünkön!