Форум сайта MyRusakov.ru

Михаил, вот посмотрите, примерно так выглядит класс регистрации ?


<?php
<?php
class registration {

function Check_data() {

if (isset($_POST['login'])){
$login = $_POST['login'];
}

if (isset($_POST['name'])) {
$name = $_POST['name'];
}

if (isset($_POST['password'])) {
$password=md5($_POST['password']);
}

if (empty($login) or empty($name) or empty($password)) {
exit ("Вы ввели не всю информацию, вернитесь назад и заполните все поля!";
}
}



function Data_processing() {
$login = stripslashes($login);
$login = htmlspecialchars($login);
$name = stripslashes($name);
$name = htmlspecialchars($name);
$password = stripslashes($password);
$password = htmlspecialchars($password);

$login = trim($login);
$name = trim($name);
$password = trim($password);
}


function Check_existence_login() {
$result = mysql_query("SELECT id FROM users WHERE login='$login'",$db);
$myrow = mysql_fetch_array($result);
if (!empty($myrow['id'])) {
exit ("Извините, введённый вами логин уже зарегистрирован. Введите другой логин.";
}
}


function Db_insert() {
$result2 = mysql_query ("INSERT INTO users (id, login, name, password) VALUES('','$login','$name','$password')";
if ($result2=='TRUE')
{
echo "Вы успешно зарегистрированы! Теперь вы можете зайти на сайт. <a href='/index.php'>Главная страница</a>";
}
else {
echo "Ошибка! Вы не зарегистрированы.";
}
}
}
?>



В целом, да, но здесь огромное количество логических ошибок, которые показывают Ваш начинающий уровень в ООП. Самое первое, что бросается в глаза - это названия методов. Примерный вид должен быть такой: словоСловоСлово(). То есть разбить на набор слова и соединить их все вместе, причём первое слово должно идти с маленькой буквы.
Также каждый объект должен отвечать только за себя. У Вас же помимо регистрации, почему-то ещё выводятся какие-то информационные сообщения. Эти сообщения должны выводиться тем объектом, который за это отвечает и который вызывает эти методы. В общем, здесь много нюансов, но пока вроде неплохо. Я бы Вам порекомендовал почитать учебники по ООП, чтобы посмотреть, как должен выглядеть грамотный код.

Спасибо))) Намотал на ус про названия методов))) Теперь зато понял, как именно строится класс))) Учебник уже читаю, правда незнаю, тот ли. Читаю книгу Гради Буча "Объектно ориентированный анализ и проектирование". Может конечно не то, но книга весьма понятная, с её помощью понял, что ооп из себя представляет))
Если стоит читаль что то другое, подскажите пожалуйста, что именно)))
Заранее спасибо))
С уважением, BrusSENS)))

Хорошая книга, продолжайте читать.

Спасибо))))

Михаил, вот класс вроде создан, но вот есть проблемма.
Я создаю reg.php, куда отправляются вобщем то данные из формы, затем делаю структуру файла
<?php
require_once 'classes/dbconnect.php';
require_once 'classes/registration.php';
$registr = new registration();
echo $registr->checkData();
echo $registr->dataProcessing();
$sql = new mysql();
echo $sql->dataBaseConnect();
echo $registr->checkExistenceLogin();
echo $registr->dataBaseInsert();
?>
Сама структура файла dbconnect.php
<?php
class mysql {
function dataBaseConnect() {
$host = "localhost";
$usname = "root";
$pass = "";
$dbname = "myproject";
$db = mysql_connect ("$host","$usname","$pass";
mysql_select_db ("$dbname",$db);
}
}
?>
При попытке регистрации выдаёт:
Warning: mysql_query(): supplied argument is not a valid MySQL-Link resource in L:\home\test33.ru\www\classes\registration.php on line 40

Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in L:\home\test33.ru\www\classes\registration.php on line 41
Вы успешно зарегистрированы! Теперь вы можете зайти на сайт. Главная страница
В чём может быть ошибка?
Код самого класса:
<?php
class registration {

function checkData() {

if (isset($_POST['login'])){
$login = $_POST['login'];
}

if (isset($_POST['name'])) {
$name = $_POST['name'];
}

if (isset($_POST['password'])) {
$password=md5($_POST['password']);
}

if (empty($login) or empty($name) or empty($password)) {
exit ("Вы ввели не всю информацию, вернитесь назад и заполните все поля!";
}
}



function dataProcessing() {
$login = stripslashes($login);
$login = htmlspecialchars($login);
$name = stripslashes($name);
$name = htmlspecialchars($name);
$password = stripslashes($password);
$password = htmlspecialchars($password);

$login = trim($login);
$name = trim($name);
$password = trim($password);
}


function checkExistenceLogin() {
$result = mysql_query("SELECT id FROM users WHERE login='$login'",$db);
$myrow = mysql_fetch_array($result);
if (!empty($myrow['id'])) {
exit ("Извините, введённый вами логин уже зарегистрирован. Введите другой логин.";
}
}


function dataBaseInsert() {
$result2 = mysql_query ("INSERT INTO users (id, login, name, password) VALUES('','$login','$name','$password')";
if ($result2=='TRUE')
{
echo "Вы успешно зарегистрированы! Теперь вы можете зайти на сайт. <a href='/index.php'>Главная страница</a>";
}
else {
echo "Ошибка! Вы не зарегистрированы.";
}
}
}
?>

Ошибок много и все они одни и те же. Например, здесь:
$result = mysql_query("SELECT id FROM users WHERE login='$login'",$db);

Переменной $db не существует. Чтобы она существовала, её нужно передавать извне конструктору. Далее эта переменная должна записываться в поле класса, а после должно быть обращение такое: $this->db

Михаил, почему то сделал вроде всё, но в базе создаются пустые поля, хотя регистрация прошла, в чём дело может быть?

INSERT INTO users (id, login, name, password) VALUES('','$login','$name','$password')"
id должно идти auto_increment, следовательно, его вообще не нужно указывать, и тем более, пустой строкой.

Спасибо, понял, сейчас попробую)))А id у меня в базе идёт как раз AUTO-INCREMENT)))