De PHPhulp kroeg
Rommelige php code, kom er niet uit
Zou iemand graag de code overzichtelijk kunnen zetten en in goede volgorde? Het liefst met uitleg zodat ik er ook van leer!
Alvast bedankt voor de moeite
Ps: die(""); ben ik nog mee bezig om daar een goede foutafhandeling van te maken.
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
<?php
// 'send me an email' script
$to = "[email protected]";
$subject = "Email From ......";
if($_POST['submit'])
{
//get data from form
$name = $_POST['name'];
$message = $_POST['message'];
if($name&&$message) //check existance
{
if (strlen($name)<=20&&strlen($message<=300) //length check
{
//everything is ok
//SET smtp in php.ini
ini_SET("SMTP", "mailhost.shef.ac.uk");
//setup varaibles
$to = "[email protected]";
$subject = "Email From ......";
$headers = "FROM: [email protected]";
$body ="Dit is een mail van $name\n\n$message";
mail($to, $subject, $body);
die("Mail sended");
}
else
die("Max lentgh for name is 20, and max lenght for message is 300")
}
else
die("You must enter a name <u>and</u> message");
}
?>
// 'send me an email' script
$to = "[email protected]";
$subject = "Email From ......";
if($_POST['submit'])
{
//get data from form
$name = $_POST['name'];
$message = $_POST['message'];
if($name&&$message) //check existance
{
if (strlen($name)<=20&&strlen($message<=300) //length check
{
//everything is ok
//SET smtp in php.ini
ini_SET("SMTP", "mailhost.shef.ac.uk");
//setup varaibles
$to = "[email protected]";
$subject = "Email From ......";
$headers = "FROM: [email protected]";
$body ="Dit is een mail van $name\n\n$message";
mail($to, $subject, $body);
die("Mail sended");
}
else
die("Max lentgh for name is 20, and max lenght for message is 300")
}
else
die("You must enter a name <u>and</u> message");
}
?>
Code (php)
1
2
3
4
5
6
7
8
9
10
2
3
4
5
6
7
8
9
10
<html>
<head>
<body>
<form action='sendmeanemail.php' method='POST'>
Name: <input type='text' name='name' maxlength='20'><br>
Message: <textarea name='message'></textarea><p>
<input type='submit' name='submit' value='stuur'
</form>
</body>
</html>
<head>
<body>
<form action='sendmeanemail.php' method='POST'>
Name: <input type='text' name='name' maxlength='20'><br>
Message: <textarea name='message'></textarea><p>
<input type='submit' name='submit' value='stuur'
</form>
</body>
</html>
Checken of het formulier is verzonden doe je met
if($_SERVER['REQUEST_METHOD'] == 'POST')
Niet nutteloos $vars copiëren.
Dit: if($name&&$message) is geen fatsoenlijke check.
De juiste mailheaders.
Hou $vars buiten de quotes.
Je mailt, maar zonder de headers.
Er wordt niet gecontroleerd of het mailen is gelukt.
Gebruik geen die().
Gewijzigd op 14/06/2011 19:39:00 door - SanThe -
Thanks