Redelijk goed beveiligd?
Ik ben sins kort weer beetje bezig met php nu heb ik een script gemaakt een simpele gewoon maar vroeg me af af hij wel goed is ik heb me tijwels er over
hier stukje code
<code>
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
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
<?php
mysql_connect("localhost","root","wachtwoord");
mysql_select_db("test");
if($_SERVER['REQUEST_METHOD']=="POST"){
if(!$_POST['naam'] || (!$_POST['achternaam'])){
$message="u moet uw naam en achternaam in vullen";
echo $message;
}
else{
$naam=mysql_real_escape_string($_POST['naam']);
$achternaam=mysql_real_escape_string($_POST['achternaam']);
$query="INSERT INTO aanmelden(id,naam,achternaam)
VALUES('','$naam','$achternaam')";
$result=mysql_query($query);
if(mysql_affected_rows()==1){
echo"succes";
}
}
}
?>
mysql_connect("localhost","root","wachtwoord");
mysql_select_db("test");
if($_SERVER['REQUEST_METHOD']=="POST"){
if(!$_POST['naam'] || (!$_POST['achternaam'])){
$message="u moet uw naam en achternaam in vullen";
echo $message;
}
else{
$naam=mysql_real_escape_string($_POST['naam']);
$achternaam=mysql_real_escape_string($_POST['achternaam']);
$query="INSERT INTO aanmelden(id,naam,achternaam)
VALUES('','$naam','$achternaam')";
$result=mysql_query($query);
if(mysql_affected_rows()==1){
echo"succes";
}
}
}
?>
<html>
<head>
<title>test database insert into</title>
</head>
<body>
<form method="POST" action="">
Naam:<input type="text" name="naam">
achternaam:<input type="text" name="achternaam">
<input type="submit" value="verzenden">
</form>
</body>
</html>
</code>
graag zeggen wat ik beter kan doen
mmvg nick
Gewijzigd op 26/07/2011 21:51:13 door Nick van der heijden
- Geen variabelen overschrijven (lijn 12/13)
- Foutafhandeling
En gebruik code-tags om je code heen.
kan je je code tussen [code] en [/code]-tags plaatsen?
Gewijzigd op 26/07/2011 22:49:37 door - Ariën -
Code (php)
Waarom niet gewoon dit:
Nick van der heijden op 26/07/2011 23:02:38:
jaa weet niet het kan alle twee bovenste vond ik leuker:PxD
En zo verspil je nou resources, je doet daarna niks meer wat $message, dus gewoon een hoop verspilde bytes.
Verder is de code van PHP geen kunstobject, maar hoort het gewoon logisch in elkaar te steken. Een opmerking zoals 'vind ik leuk', hoort niet bij het programmeren. ;-)
Gewijzigd op 26/07/2011 23:18:52 door - Ariën -