(empty($navn) || empty($besked) || empty($mail || empty($emne)) // Hvis felter ikke er udfyldt { echo "Alle felter skal udfyldes <br><b><a href='html.html' title='Tilbage'>Jeg vil tilbage!<a></b>"; }
else // Ellers send { if(ereg("@", $mail) // Er mailen gyldig? { $sendto = "Mail@mmmmm.com"; //Den mail der skal have beskeden $subject = $emne; $besked = "Besked: \n Navn: " . $_POST['navn']. " \n E-mail: " . $_POST['mail'] . " \n Besked: " . $_POST['besked'];
} else { echo "Din mail var ikke korrekt, prøv igen <br><b><a href='html.html' title='Tilbage'>Jeg vil tilbage!<a></b>"; } if(mail($sendto, $subject, $besked, "Mailen er fra: $mail\\r\\n" . "Besvar: $mail\\r\\n")); { echo "Emailen er nu sendt!"; } else { echo "Der opstod et problem, prøv igen"; } } } ?>
if (!isset($_POST['navn']) || !isset($_POST['besked'] || !isset($_POST['mail']) || !isset($_POST['emne']) Kan også kombineres til isset($var1,$var2,[...]). Dog er det lidt irrelevant, da de altid vil være sat, når man submitter en form, så der er kun behov for at kontrollere et enkelt felt.
if (empty($navn) || empty($besked) || empty($mail || empty($emne)) // Hvis felter ikke er udfyldt { echo "Alle felter skal udfyldes <br><b><a href='html.html' title='Tilbage'>Jeg vil tilbage!<a></b>"; }
else // Ellers send { if(ereg("@", $mail) // Er mailen gyldig? { $sendto = "Mail@hotmail.com"; //Den mail der skal have beskeden $subject = $emne; $besked = "Besked: \n Navn: " . $_POST['navn']. " \n E-mail: " . $_POST['mail'] . " \n Besked: " . $_POST['besked'];
} else { echo "Din mail var ikke korrekt, prøv igen <br><b><a href='html.html' title='Tilbage'>Jeg vil tilbage!<a></b>"; } if(mail($sendto, $subject, $besked, "Mailen er fra: $mail\\r\\n" . "Besvar: $mail\\r\\n")); { echo "Emailen er nu sendt!"; } else { echo "Der opstod et problem, prøv igen"; } } } ?>
Tror du ikke selv du lige kunne bruge et par minutter til at gennemgå din egen kode, dkeigel? Det kræver ikke særlig mange PHP-kompetencer at se, at du har 2 felter med name="mail", hvor det ene burde være "emne". :)
splazz: Det der med at udskrive fejl, når folk bevidst forsøger at "hacke" siden eller gøre andre ting, der ikke er intentionelle, går jeg ikke så meget ind for. Det giver blot brugeren et større indblik i koden. Hvis folk bevidst forsøger at gøre noget siden ikke er lavet til, så plejer jeg gerne bare at sende dem tilbage til siden de kom fra uden at fortælle dem noget som helst. De ved jo ganske udmærket godt, hvordan de skal bruge siden - og de ved også, at de ikke gør det rigtigt.
Her bunder det dog også lidt i, at han overhovedet ikke validerer sit indhold nogen steder.
Tilladte BB-code-tags: [b]fed[/b] [i]kursiv[/i] [u]understreget[/u] Web- og emailadresser omdannes automatisk til links. Der sættes "nofollow" på alle links.