Метод возвращает null

@Tundra444 · Регистрация: 03.01.2020

Студворк — интернет-сервис помощи студентам

Пытаюсь сделать простую регистрацию, написал класс, знаю что говнокод, но всё же

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
<?php
 
namespace Model\Registration;
 
require_once 'Db.php';
use Model\Database\Db;
 
class Registration {
 
    public $connection;
 
    public function __construct() {
        $this->connection = Db::Connect();
    }
 
    public function Registr() {
        if(isset($_POST['btn'])) {
            if(isset($_POST['email']) && $_POST['login'] && $_POST['password'] !== "") {
                $dbh = $this->connection;
                $sql = 'INSERT INTO users(email, login, password) VALUES (:email, :login, :password)';
                $sqlParams = [':email' => $_POST['email'], ':login' => $_POST['login'], ':password' => $_POST['password']];
                $sth = $dbh->prepare($sql);
                $result = $sth->execute($sqlParams);
                header('Location: /');
            } else {
                echo 'Заполните все поля <a href="../View/registration.php"><b>Registration</b></a>';
            }
        }
    }
 
}

Есть у меня так же, что то типа контроллера, из которого я пытаюсь дернуть метод Registr

PHP
1
2
3
4
5
6
7
8
9
<?php 
 
require 'Model/Registration.php';
use Model\Registration\Registration;
 
$registration = new Registration();
$a = $registration->Registr();
 
require 'View/index.php';

В итоге Registr выдаёт null, и соответственно ничего не работает, Но, если я создаю объект и вызываю метод из того файла, в котором написан класс, метод успешно обрабатывает POST данные и отправляет их в БД, хотя при попытке вар дампа метода так же null, но данные как я уже сказал в бд лежат. Подскажите пожалуйста где я накосячил, никак не пойму.

@sad67man · 04.01.2020, 00:03

Tundra444, Что бы метод что-либо возвращал нужно использовать return.

@Tundra444 · 04.01.2020, 08:31 **[ТС]**

Сообщение от sad67man

Что бы метод что-либо возвращал нужно использовать return.

А что мне тут возвращать? Там же просто по сути проверка на нажатие кнопки и заполненность полей.

@sad67man · 04.01.2020, 13:32

Tundra444, Тут не нужно ничего возвращать. Тут операция либо выполняется, либо кидает исключения. Так же не нужно обращаться напрямую к POST, делать редиректы и что-либо выводить через echo. Это задача контроллера.

@wwowa · 04.01.2020, 16:53

Сообщение от Tundra444

соответственно ничего не работает

У вас вся работа метода основана на проверке $_POST, значит класс не видит его. Попробуйте передать в конструктор класса вместе с базой данных еще и $_POST.

@sad67man · 04.01.2020, 17:57

Tundra444, Вот накидал такой примерчик.

PHP/HTML
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
50
<?php
 
$service = new SignUpService(new UserRepository());
 
$signUpType = new SignUpType();
$commonError = '';
 
if(!empty($_POST)) {
    $signUpType->load($_POST);
    if ($signUpType->validate()) {
        try {
            $user = $service->signUp($signUpType);
            header('Location: /user.php?id=' . $user->id); die();
        } catch (DomainException $e) {
            $commonError = $e->getMessage();
        }
    }
}
 
?>
 
<h1>Регистрация</h1>
<div style="color: red"><?= $commonError ?></div>
<form method="POST">
    <div>
        <label> Логин:
            <input name="login" value="<?= htmlspecialchars($signUpType->login) ?>">
            <span style="color:red"><?= $signUpType->getError('login') ?></span>
        </label>
    </div>
    <div>
        <label> Пароль:
            <input name="password" type="password" value="<?= htmlspecialchars($signUpType->password) ?>">
            <span style="color:red"><?= $signUpType->getError('password') ?></span>
        </label>
    </div>
    <div>
        <label> Повторите пароль:
            <input name="passwordRepeat" type="password" value="<?= htmlspecialchars($signUpType->passwordRepeat) ?>">
            <span style="color:red"><?= $signUpType->getError('passwordRepeat') ?></span>
        </label>
    </div>
    <div>
        <label> email:
            <input name="email" value="<?= htmlspecialchars($signUpType->email) ?>">
            <span style="color:red"><?= $signUpType->getError('email') ?></span>
        </label>
    </div>
    <input type="submit">
</form>

Первым шагом мы заполняем и валидируем вод данных (SignUpType). Это простая валидация, проверка на число, email, кол-ва символов и т.д.

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
50
51
52
53
54
55
56
57
58
59
60
61
<?php
 
class SignUpType {
 
    public $email;
    public $login;
    public $password;
    public $passwordRepeat;
 
    private $errors = [];
 
    public function load(array $data): void
    {
        $this->email = trim($data['email'] ?? '');
        $this->login = trim($data['login'] ?? '');
        $this->password = $data['password'] ?? '';
        $this->passwordRepeat = $data['passwordRepeat'] ?? '';
    }
 
    public function validate(): bool
    {
        if (empty($this->email)) {
            $this->addError('email', 'Введите email');
        } elseif (!filter_var($this->email, FILTER_VALIDATE_EMAIL)) {
            $this->addError('email', 'Некорректный email');
        }
 
        if (empty($this->login)) {
            $this->addError('login', 'Введите логин');
        } elseif (strlen($this->login) < 3) {
            $this->addError('login', 'Минимум 3 символа');
        }
 
        if (empty($this->password)) {
            $this->addError('password', 'Введите пароль');
        } elseif (strlen($this->password) < 6) {
            $this->addError('password', 'Минимум 6 символов');
        } elseif (empty($this->passwordRepeat)) {
            $this->addError('passwordRepeat', 'Введите пароль повторно');
        } elseif ($this->password !== $this->passwordRepeat) {
            $this->addError('passwordRepeat', 'Пароли не совпадают');
        }
 
        return empty($this->getErrors());
    }
 
    public function getErrors(): array
    {
        return $this->errors;
    }
 
    public function getError($name): string
    {
        return $this->errors[$name] ?? '';
    }
 
    private function addError($name, $message): void
    {
        $this->errors[$name] = $message;
    }
}

А такие проверки типа уникальности логина, которые относятся в вариантам использования мы выносим в сам сервис. Который в случае чего выкидывает исключения.

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
<?php
 
class SignUpService
{
    private $users;
 
    public function __construct(UserRepositoryInterface $users)
    {
        $this->users = $users;
    }
 
    public function signUp(SignUpType $dto): User
    {
        if ($this->users->isLoginExist($dto->login)) {
            throw new DomainException('Пользователь с таким логином уже зарегистрирован');
        }
 
        $user = User::create($dto->login, $dto->password, $dto->email);
        $this->users->insert($user);
 
        return $user;
    }
 
}

Новые блоги и статьи Все статьи Все блоги /
Отчёт о затраченных материалах за определенный период с макетом печатной формы Maks 21.04.2026 Отчёт из решения ниже размещён в конфигурации КА2. Задача: разработка отчёта по затраченным материалам за определённый период, с возможностью вывода печатной формы отчёта с шапкой и подвалом. В. . .	Отчёт о спецтехнике находящейся в ремонте Maks 20.04.2026 Отчёт из решения ниже размещен в конфигурации КА2. Задача: отобразить спецтехнику, которая на данный момент находится в ремонте. Есть нетиповой документ "Заявка на ремонт спецтехники" который. . .	Памятка для бота и "визитка" для читателей "Semantic Universe Layer (Слой семантической вселенной)" Hrethgir 19.04.2026 Сгенерировано для краткого описания по случаю сборки и компиляции скелета серверного приложения. И пусть после этого скажут, что статьи сгенерированные AI - туфта и не интересно. И это не реклама -. . .	Запрет удаления строк ТЧ документа при определённом условии Maks 19.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "Аккумуляторы", разработанного в конфигурации КА2. У данного документа есть ТЧ, в которой в зависимости от прав доступа. . .
Модель заражения группы наркоманов alhaos 17.04.2026 Условия задачи сформулированы тут Суть: - Группа наркоманов из 10 человек. - Только один инфицирован ВИЧ. - Колются одной иглой. - Колются раз в день. - Колются последовательно через. . .	Мысли в слух. Про "навсегда". kumehtar 16.04.2026 Подумалось тут, что наверное очень глупо использовать во всяких своих установках понятие "навсегда". Это очень сильное понятие, и я только начинаю понимать край его смысла, не смотря на то что давно. . .	My Business CRM MaGz GoLd 16.04.2026 Всем привет, недавно возникла потребность создать CRM, для личных нужд. Собственно программа предоставляет из себя базу данных клиентов, в которой можно фиксировать звонки, стадии сделки, а также. . .	Знаешь почему 90% людей редко бывают счастливыми? kumehtar 14.04.2026 Потому что они ждут. Ждут выходных, ждут отпуска, ждут удачного момента. . . а удачный момент так и не приходит.

@sad67man 2604 / 1508 / 689 Регистрация: 23.08.2015 Сообщений: 3,839
	04.01.2020, 00:03
	Tundra444, Что бы метод что-либо возвращал нужно использовать return. 1

@sad67man 2604 / 1508 / 689 Регистрация: 23.08.2015 Сообщений: 3,839
	04.01.2020, 13:32
	Tundra444, Тут не нужно ничего возвращать. Тут операция либо выполняется, либо кидает исключения. Так же не нужно обращаться напрямую к POST, делать редиректы и что-либо выводить через echo. Это задача контроллера. 1

Опции темы