Форум программистов, компьютерный форум, киберфорум
PHP: ООП
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.89/9: Рейтинг темы: голосов - 9, средняя оценка - 4.89
276 / 201 / 143
Регистрация: 22.01.2016
Сообщений: 637

Реализация простой регистрации с ООП

23.06.2016, 22:37. Показов 1836. Ответов 11
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Доброго времени суток. Начал вникать в ООП и написал простенькую регистрацию. Если не тяжело, намекните на ошибки и недостатки в моём коде. Может рекомендации какие есть. Хочется писать красивый, и, что самое главное, правильный код.

Класс registration.php:
PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
<?php
 
class Registration {
    private $host;
    private $db_user;
    private $db_pass;
    private $db;
    
    private $login;
    private $pass;
 
    public function __construct($hostDB, $userDB, $passDB, $nameDB, $newUserLogin, $newUserPass) {
        $this->host = $hostDB;
        $this->db_user = $userDB;
        $this->db_pass = $passDB;
        $this->db = $nameDB;
        
        $this->login = $newUserLogin;
        $this->pass = $newUserPass;
    }
    public function connection() {
        $connect = mysqli_connect($this->host, $this->db_user, $this->db_pass, $this->db);
        return $connect;
    }
    public function checkData() {
        if (isset($this->login, $this->pass) && !empty($this->login)) {
            $this->login = htmlspecialchars($this->login);
            $this->pass = htmlspecialchars($this->pass);
            return true;
        }
        else {
            return false;
        }
    }
    public function createNewUser() {
        if ($this->connection() && $this->checkData()) {
            mysqli_query($this->connection(), "INSERT INTO `users` VALUES('', NOW(), '$this->login', '$this->pass')");
            return true;
        }
        else {
            return false;
        }
    }
    public function returnLogin() {
        return $this->login;
    }
}
 
?>
Файл обработчик action.php:
PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
<?php
 
require_once('registration.php');
 
define('HOST', 'localhost');
define('USER', 'admin');
define('PASS', 'password123');
define('DB', 'nameDatabase');
 
$registrationObject = new Registration(HOST, USER, PASS, DB, $_POST['login'], $_POST['password']); //передаём данные для подключения к БД и логин/пароль с формы
if ($registrationObject->createNewUser()) {
    echo $registrationObject->returnLogin(), ', Вы успешно зарегистрировались!';
}
else {
    echo 'Произошла ошибка при регистрации.';
}
?>
0
cpp_developer
Эксперт
20123 / 5690 / 1417
Регистрация: 09.04.2010
Сообщений: 22,546
Блог
23.06.2016, 22:37
Ответы с готовыми решениями:

Простой ООП калькулятор с наследованием
Я сделал второй класс, который наследует всё у первого. Он должен вывести результат метода sum() в квадрате, но этого не происходит,...

Реализация моделей ООП
Здравствуйте, в данный момент ищу себе работу и имею проблемы с работой над тестовым заданием. Принципы ооп знакомы, но опыта работы с...

Грамотная реализация MVC ООП
Доброе утро всем, уже вторые сутки лазию в просторах интернета и не могу найти нормальную реализацию MVC ООП, без ООП нашел, но все жи...

11
Эксперт PHP
4925 / 3920 / 1620
Регистрация: 24.04.2014
Сообщений: 11,441
23.06.2016, 22:59
Цитата Сообщение от mrNTF Посмотреть сообщение
Если не тяжело, намекните на ошибки и недостатки в моём коде.
Это что угодно, но не объектно ориентированный код. Все ошибки и как от них избавиться не уместится в 1 сообщение. Сначала нужно разобраться в основных понятиях ООП, изучить синтаксис. Когда все вопросы с основами будут поняты переходить к изучению объектно ориентированного проектирования, чтобы понимать как ООП использовать в реальных задачах.
И про уязвимости не забывать, в частности sql инъекции
0
276 / 201 / 143
Регистрация: 22.01.2016
Сообщений: 637
24.06.2016, 00:28  [ТС]
Я ожидал такого ответа( Так как естественно банально знания, как создать класс или методы недостаточно. Хоть и пересмотрел множество уроков. Можете написать основные моменты, пожалуйста?
0
25 / 25 / 20
Регистрация: 12.05.2016
Сообщений: 181
26.06.2016, 11:21
Вот пример простенькой регистрации на ООП:

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
define('HOST','localhost');
define('USER','root');
define('PASS','');
define('DB','tutorials');
class Registration{
    public $db;
    
    public function __construct($host,$user,$pass,$db){
        $this->db = mysqli_connect($host,$user,$pass,$db);
        if(!$this->db){
            exit('There is not connect to database');
        }
        mysqli_query($this->db,"SET NAMES 'utf8'");
        
        return $this->db;
    }
    public function createNewUser($name,$login,$pass){
        $sql = "INSERT INTO `users` VALUES('','$name','$login','$pass')";
        $res = mysqli_query($this->db,$sql);
    if(!$res){
        echo "FALSE";
            return FALSE;
        }
    }
}
$reg = new Registration(HOST,USER,PASS,DB);
$reg->createNewUser($_POST['Name'],$_POST['Login'],md5($_POST['Pass']));
0
Software Engineer
 Аватар для Custos
332 / 335 / 55
Регистрация: 23.09.2014
Сообщений: 988
26.06.2016, 22:41
Цитата Сообщение от codding_hanter Посмотреть сообщение
public function createNewUser($name,$login,$pass){
* * * * $sql = "INSERT INTO `users` VALUES('','$name','$login','$pass')";
Цитата Сообщение от codding_hanter Посмотреть сообщение
$sql = "INSERT INTO `users` VALUES('','$name','$login','$pass')";
У меня для тебя плохие новости
0
0 / 0 / 1
Регистрация: 17.12.2013
Сообщений: 66
29.07.2016, 20:33
Кстати тоже недавно начал по не воле изучать ООП, ткине носом где есть нормальные мануалы, которые действительно учат проэктировать на ООП, ну или в том духе, потому что я наверно бы сделал точно так же как и ТС, потому что сразу видно разницу... как написал ТС и как привели пример чуть ниже.. Логика то понятна, но где такой научится, ? потому что простые ООП туториалы учат именно таким вещам как написал ТС, ну а пример что ниже видно что сделать короче, и без лишних движений т.е направление логики понятно..
0
 Аватар для fanatikus
1932 / 1523 / 703
Регистрация: 17.11.2012
Сообщений: 6,585
29.07.2016, 21:00
Цитата Сообщение от codding_hanter Посмотреть сообщение
return $this->db;
return в конструкторе???
0
29.07.2016, 21:50

Не по теме:

Что вы прицепились, ребята учатся только
Самому понравилось

PHP
1
2
 echo "FALSE";
            return FALSE;

0
Hello Kitty
 Аватар для WhiteMind
690 / 562 / 402
Регистрация: 12.02.2016
Сообщений: 1,436
Записей в блоге: 1
30.07.2016, 00:42
Цитата Сообщение от fanatikus Посмотреть сообщение
return в конструкторе???
конструктор такой же метод как и другие.
и return там вполне уместен(другое дело что new его не выдаст)
0
 Аватар для php10
146 / 105 / 44
Регистрация: 30.04.2016
Сообщений: 550
30.07.2016, 12:24
codding_hanter, Ваш код ничем не лучше кода ТС. Зачем каждый раз подключаться к базе данных в конструкторе? Что за мода такая, если Connection можно внедрить в конструкторе или в сеттере, коннектор к бд должен быть один на все App. Форматирование кода отсутствует, вот тут createNewUser у вас SQL-Injection,

ну а вот это:
PHP
1
2
3
4
5
if(!$res){
        echo "FALSE";
            return FALSE;
        }
    }
финиш...

mrNTF, вот так должна выглядеть крутая регистрация пользователя, в стиле AR:
PHP
1
2
3
4
$user = new User();
$user->login = "php10";
$user->password = "password"; // пароль шифруется в сеттере или при сохранении
$user->save();
Да и отказаться от mysqli лучше и использовать PDO. Как минимум удобнее работать с биндингами, забудете про SQL-Inj.

Добавлено через 1 минуту
phpbb, Скачиваете исходники Laravel 5 или Yii2 и читаете. Там все есть! Нормальных книг по ООП я не видел, по паттернам книги есть.
0
276 / 201 / 143
Регистрация: 22.01.2016
Сообщений: 637
30.07.2016, 13:39  [ТС]
Ааа, как мне стыдно за этот код, тут только только начинал изучать ООП и почти не шарил, что к чему. Ну вот почему нельзя закрыть или вообще удалить тему на этом форуме. -_-
0
 Аватар для php10
146 / 105 / 44
Регистрация: 30.04.2016
Сообщений: 550
30.07.2016, 16:36
mrNTF, любой человек "перлы" пишет, особенно когда нужно сделать что-то по-быстрому. Ничего страшного нет, главное учиться и больше смотреть исходники других приложений на PHP, реально помогает.
1
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
raxper
Эксперт
30234 / 6612 / 1498
Регистрация: 28.12.2010
Сообщений: 21,154
Блог
30.07.2016, 16:36
Помогаю со студенческими работами здесь

Реализация простой регистрации на сайте
Я создаю электронный учебник, в html. Понадобилось сделать регистрацию. Подскажите пожалуйста как можно сделать простинькую регистрацию для...

Создания простой регистрации
Здраствуйте, знаю есть подобные темы но найти свою проблему так и не могу. У меня есть простая регистрация, сначала создаёш данные,...

Простой модуль регистрации
Ищу простейший модуль регистрации из двух полей. Чтоб данные из полей падали сразу в админку сайта. Не попадался такой? Добавлено...

Реализация регистрации!
Доброго времени суток господа форумчане! Меня гложет ,уже несколько месяцев , один и тот же вопрос...Хочу реализовать регистрацию в своем...

Реализация регистрации
Такой вопрос. Условно я делаю регистрацию, т.е если по простому, человек вбивает данные в форму, скрипт подхватывает их и вносит в БД. ...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
12
Ответ Создать тему
Новые блоги и статьи
SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве
8Observer8 02.02.2026
Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .
Философия технологии
iceja 01.02.2026
На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .
SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf
8Observer8 01.02.2026
Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .
SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли
8Observer8 30.01.2026
Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .
SDL3 для Web (WebAssembly): Установка Emscripten SDK (emsdk) и CMake для сборки C и C++ приложений в Wasm
8Observer8 30.01.2026
Содержание блога Для того чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. . . .
SDL3 для Android: Подключение Box2D v3, физика и отрисовка коллайдеров
8Observer8 29.01.2026
Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .
Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT
bedvit 28.01.2026
Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .
SDL3 для Android: Загрузка PNG с альфа-каналом с помощью SDL_LoadPNG (без SDL3_image)
8Observer8 28.01.2026
Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru