Hur? — webForum

FoJjMedlem sedan maj 200178 inlägg

11 maj 2001 11:49#1

<body>
<?php
require("mydb.inc");
mysql_connect(localhost,$user,$password);

@mysql_select_db($db) or die( "Unable to select database");

$query="insert into contact values('','$name','$email','$tel')";
if ($name == "") {
echo "Du glvmde fylla i namn ";
mysql_close();
echo "<a href='index.php'>Tillbax</a>";
}
else if ($email== ""){
echo "Du glvmde fylla i email ";
mysql_close();
echo "<a href='index.php'>Tillbax</a>";
}
if(ereg("^[\w\.]+@(\w+\.)+\w$", $email) != $email) { <-- FUNKAR INTE
echo "$email dr inte en korekt email adress";
mysql_close();
echo "<a href='index.php'>Tillbax</a>";
}
else if ($tel == "") {
echo "du glvmde fylla i tel ";
mysql_close();
echo "<a href='index.php'>Tillbax</a>";
}
else {
$result=mysql_query($query);
mysql_close();
echo "Klart Databasen dr uppdaterad ";
echo "<a href='index.php'>Tillbax</a>";
}
?>
</body>
</html>

Hur ska man få det o funka?

LarsGMedlem sedan dec. 200012 464 inlägg

11 maj 2001 14:49#2

Hejsan.

Funktionen ereg returnerar true eller false vilket inte är så lämpligt att jämföra med strängen email

if(!ereg("^[\w\.]+@(\w+\.)+\w$", $email))

En annan sak, det är bättre att öppna databasen efter det att du har kontrollerat parametrarna. Då slipper du också att ha med mysql_close i de olika if-statements som du har.

------------------
essentitia preter non sans multiplicandum