<?php ob_start(); session_start(); $conn = mysql_connect('localhost','root',''); mysql_select_db('mydb',$conn); if(isset($_POST['login'])){ $email = $_POST['email']; $password = $_POST['password']; if(!empty($email)&&!empty($password)) { $query = "SELECT id FROM users WHERE email = '$email' AND password = '$password'"; if($run = mysql_query($query)) { $numrows = mysql_num_rows($run); if($numrows ==0){ echo "<fieldset>Incorrect username or password</fieldset>"; }else if ($numrows==1){ $_SESSION['name'] = $_POST['email']; header('Location: vavas.php'); echo "Logged in"; } } } } ?><style type="text/css"> <!-- a:link { color: #006; } --> </style> <br /><br /><br /><br /><br /> <form action="pap.php" method="POST"> <center> <body bgcolor="#E7E7E7"> <br /><br /> <div id="bobi"> Email:<br /><input type="text" placeholder="Email here" name="email" class="email" size="31" required/><br /> Password:<br /><input type="text" placeholder="Password here" name="password" class="password" size="31" required/><br /> <input type="submit" class="bobi" name="login" value="Login" /> </center> <style type="text/css"> fieldset{ box-shadow: 0px 1px 2px gray; background-color:#FF2D2D; border:2px #C00 solid; height:56px; text-align:left; } input.bobi:hover{ cursorointer; box-shadow: 1px 1px 2px gray; border-radius:2px; border:0px; border-top:2px white; background: -moz-linear-gradient( white,#e5e5e5); height:32; width:70; } input.bobi{ border:#999 1px solid; position:relative; bottom:-7px; right:-66px; box-shadow: 1px 1px 2px gray; border-radius:2px; border:0px; border-top:2px white; background: -moz-linear-gradient( white,#e5e5e5); height:32; width:70; } input.bobi:hover{ box-shadow: 1px 1px 2px gray; border-radius:2px; border:0px; border-top:2px white; background: -moz-linear-gradient( #e5e5e5,white); height:32; width:70; } #bobi{ font-family:"adobe gothic std"; font-size:14px; border:#333 1px solid; -moz-transform: rotate(7deg); width:250px; background: -moz-linear-gradient(#FF0,#FC0); height:148px; box-shadow:0px 0px 7px gray; } input.email:focus{ box-shadow:0px 0px 5px #4A8EE1; } input.password:focus{ box-shadow:0px 0px 5px #4A8EE1; } input.email{ border:#06F 1px solid; } input.password{ border:#06F 1px solid; } #mite{ #bobi font-family:"adobe gothic std"; font-size:14px; border:#333 1px solid; -moz-transform: rotate(7deg); width:250px; background: -moz-linear-gradient(#FF0,#FC0); height:50px; box-shadow:0px 0px 7px gray; } href{ position:relative; bottom:-7px; } </style>
1) Вот это категорически не правильно PHP: $query = "SELECT id FROM users WHERE email = '$email' AND password = '$password'"; Во первых приучитесь оформлять названия поле и таблиц в `, а во вторых у вас тут sql inj, в третих при запросе не надо указывать сразу 2 параметра логин и пароль, и только в том случае если логин у нас есть тогда сравниваем переданный пароль с тем что в базе. Правильнее будет так. Да и раз на то пошло надо бы выше добавить проверку на то что введен именно email а не левые какие то данные. PHP: $query = "SELECT `id`,`password` FROM `users` WHERE `email` = '".mysql_real_escape_string($email)."'"; $run = mysql_query($query) or die(mysql_error()); if(isset($run["id"])) { if($run["password"]==$password) echo "Логин осуществлен"; else echo "Не вошли"; } 2) Зачем if($run = mysql_query($query)) если проще и правильнее это все не в if.. PHP: mysql_query($query) or die("Не могу выполнить запрос"); или PHP: mysql_query($query) or die(mysql_error()); 3) Зачем $numrows = mysql_num_rows($run); if($numrows ==0) Правильнее так. PHP: if(isset($run["id"])) { } 4) Зачем echo "Logged in"; если у вас перед этим происходит редирект и пользователь это не увидит ) А вообще тут много еще чего не так ) Дальше лень писать.
