<?php
$connect = @mysql_connect('localhost','root','') or die('ERROR Conection!');
@mysql_select_db('pepsoman',$connect) or die ('DataBase ERROR');
$success = "";
if(isset($_POST['reg'])){
$name = $_POST['name'];
$surname = $_POST['surname'];
$username = $_POST['username'];
$password = $_POST['password'];
$con_password = $_POST['con_password'];
$age = $_POST['age'];
$gender = $_POST['gender'];
$insert = mysql_query("insert into users (name,surname,username,password,age,gender) values('$name','$surname','$username','$password','$age','$gender')");
if($con_password != $password){
$success = '
<div class="panel panel-danger">
<div class="panel-heading">ERROR ConfirmPassword</div>
</div>
';
}
else{
$success = '
<div class="panel panel-success ">
<div class="panel-heading">Registration</div>
</div>
';
}
}
?>
-
вам не стоит использовать интрефейс mysql тк он является устаревшим– MaximPro24 сен 2016 в 12:36
-
Если вы решили свою проблему ответами ниже то отметьте тот который вам помог– MaximPro25 сен 2016 в 15:48
2 ответа
`$insert = mysql_query("insert into users (name,surname,username,password,age,gender) values('$name','$surname','$username','$password','$age','$gender')");`
- Проверьте приходят ли
$_POST
данные. - Вообще не плохо было бы проверять перед записью в
mysql
есть ли они вообще черезisset($_POST['name'])&&isset($_POST['surname'])
и т.д. В данном конкретном случае sql запрос должен выглядеть так:
$insert = mysql_query("insert into users (name,surname,username,password,age,gender) values('{$name}','{$surname}','{$username}','{$password}','{$age}','{$gender}')");
или
$insert = mysql_query("insert into users (name,surname,username,password,age,gender) values('".$name."','".$surname."' и т.д.
Так же если планируете выкладывать этот сайт в общий доступ советую почитать статьи про базовую защиту от sql-иньекций, salt и методологию хранение паролей.
-
1
-
-
1кстати второй вариант более читабельный на мой взгляд, и да раз пошла пьянка, то лучше свой query билдер написать с плейсхолдерами как у функции
sprintf
Для практики самое-то!– MaximPro24 сен 2016 в 12:44
На улице уже 2016, самое время освоить mysqli
или pdo
. И так, немного изменю твой код:
$link = mysqli_connect("host", "user", "password", "pepsoman")
or die(mysqli_error($link));
if(isset($_POST['reg'])){
$name = $_POST['name'];
$surname = $_POST['surname'];
$username = $_POST['username'];
$password = $_POST['password'];
$con_password = $_POST['con_password'];
$age = $_POST['age'];
$gender = $_POST['gender'];
$result = mysqli_query($link, "INSERT INTO users (name, surname, username, password, age, gender)
VALUES ('$name','$surname','$username','$password','$age','$gender')");
if(mysqli_affected_rows($result)>0){
echo 'Добавлено';
}
}
- Данные которые ты получаешь POST методом надо чистить.
strip_tags
,mysqli_real_escape_string
и тд, это как минимум.
Знак @ перед ф-цией отключает вывод ошибки, это не хорошо
-
Процедурный стиль...ммм проще тогда использовать просто mysql и не париться =)– MaximPro24 сен 2016 в 13:32
-
я добавил @ потому что у меня выводится ошибка есть новая база установить новую базу или что-то вроде того я работаю с помощью WAMPSERVER 3.0.6 24 сен 2016 в 14:51
-
-
@HarutKostanyan незнаю что у вас за логика, но если есть ошибка, её надо исправить, а не скрыть. Без разницы какой у вас сервер, а ошибка выводилась наверное из за неправильного соединения с базой. В любом случае потратьте 30-40 минуть, чтобы разобраться с
mysqli
как минимум. Плюс, еще надо очищать данные при получении. При желание, гугл вас очень обрадует. 25 сен 2016 в 4:04