par
AB » 09 janv. 2008, 23:31
Et puis ton code est mal organisé, faut éviter de mettre du code php dans ton code html quand ce n'est pas nécessaire. En reprenant les éléments de ton premier post ça devrait ressembler à quelque chose comme :
<?php session_start();
if(isset($_POST['validation']))
{
function stripgpc($value) {
if(get_magic_quotes_gpc()) $value = stripslashes($value);
return $value;
}
$mdp = isset($_POST['mdp']) ? md5(stripgpc($_POST['mdp'])) : '';
$login = isset($_POST['login']) ? stripgpc($_POST['login']) : '';
if(!empty($mdp) && !empty($login))
{
$server="*******";
$user="****";
$pass="*****";
$db="pdr";
mysql_connect($server,$user,$pass) or die('erreur de connexion');
mysql_select_db($db) or die ('impossible de se connecter a la base');
$selection = sprintf("SELECT COUNT(*) as compte FROM user WHERE mdp_user ='%s' AND login_user ='%s'",
mysql_real_escape_string($mdp),
mysql_real_escape_string($login));
$query = mysql_query($selection) or die(mysql_error());
$totalReponse = mysql_fetch_assoc($query);
if ($totalReponse['compte'] == 1 )
{
$_SESSION['login'] = $login;
header("Location: accueil.php"); exit();
}
else
{
$message = 'login ou mot de passe incorrect';
}
}
else
{
$message = 'login ou mot de passe manquant';
}
}
?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<title>Document sans nom</title>
<style type="text/css" title="mes_styles" media="all">
<!--
#login {
border: medium solid #FF9900;
border-collapse: collapse;
position: absolute;
top: 20px;
left: 350px;
}
#ok{
font-family: Georgia, "Times New Roman", Times, serif;
color:#FF9900;
margin-top: 120px;
margin-left: 350px;
}
-->
</style>
</head>
<body>
<div>
<form action="<?php echo($_SERVER['PHP_SELF']); ?>" method='post'>
<table id="login">
<tr>
<td>Login :</td>
<td><input type="text" name="login" maxlength="250"></td>
</tr>
<tr>
<td>Password</td>
<td><input type="password"name="mdp" maxlength="10"></td>
</tr>
<tr>
<td colspan="2" align="center"><input type="submit" name="validation" id="validation" value="log in"></td>
</tr>
</table>
</form>
</div>
<div><?php if (isset($message)) echo $message ?></div>
</body>
</html>
Voilà quelque chose de plus "propre". Encore que tu aurais pu te passer de tableau et utiliser simplement des balises div ou p pour faire ton formulaire
Et puis ton code est mal organisé, faut éviter de mettre du code php dans ton code html quand ce n'est pas nécessaire. En reprenant les éléments de ton premier post ça devrait ressembler à quelque chose comme :
[php]<?php session_start();
if(isset($_POST['validation']))
{
function stripgpc($value) {
if(get_magic_quotes_gpc()) $value = stripslashes($value);
return $value;
}
$mdp = isset($_POST['mdp']) ? md5(stripgpc($_POST['mdp'])) : '';
$login = isset($_POST['login']) ? stripgpc($_POST['login']) : '';
if(!empty($mdp) && !empty($login))
{
$server="*******";
$user="****";
$pass="*****";
$db="pdr";
mysql_connect($server,$user,$pass) or die('erreur de connexion');
mysql_select_db($db) or die ('impossible de se connecter a la base');
$selection = sprintf("SELECT COUNT(*) as compte FROM user WHERE mdp_user ='%s' AND login_user ='%s'",
mysql_real_escape_string($mdp),
mysql_real_escape_string($login));
$query = mysql_query($selection) or die(mysql_error());
$totalReponse = mysql_fetch_assoc($query);
if ($totalReponse['compte'] == 1 )
{
$_SESSION['login'] = $login;
header("Location: accueil.php"); exit();
}
else
{
$message = 'login ou mot de passe incorrect';
}
}
else
{
$message = 'login ou mot de passe manquant';
}
}
?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<title>Document sans nom</title>
<style type="text/css" title="mes_styles" media="all">
<!--
#login {
border: medium solid #FF9900;
border-collapse: collapse;
position: absolute;
top: 20px;
left: 350px;
}
#ok{
font-family: Georgia, "Times New Roman", Times, serif;
color:#FF9900;
margin-top: 120px;
margin-left: 350px;
}
-->
</style>
</head>
<body>
<div>
<form action="<?php echo($_SERVER['PHP_SELF']); ?>" method='post'>
<table id="login">
<tr>
<td>Login :</td>
<td><input type="text" name="login" maxlength="250"></td>
</tr>
<tr>
<td>Password</td>
<td><input type="password"name="mdp" maxlength="10"></td>
</tr>
<tr>
<td colspan="2" align="center"><input type="submit" name="validation" id="validation" value="log in"></td>
</tr>
</table>
</form>
</div>
<div><?php if (isset($message)) echo $message ?></div>
</body>
</html>[/php]
Voilà quelque chose de plus "propre". Encore que tu aurais pu te passer de tableau et utiliser simplement des balises div ou p pour faire ton formulaire :wink:
