Почему не работает метод?

Question

[Karpkarp]

Есть файл с регистрацией

namespace Registration;

require_once 'Db.php';
use Database\Db;

class Registration {

	public $email;
	public $login;
	public $password;
	public $connection;

	public function __construct() {
		$this->email = $_POST['email'];
		$this->login = $_POST['login'];
		$this->password = $_POST['password'];
		$this->connection = Db::Connect();
	}

	public function Registr() {
		if(isset($_POST['btn'])) {
			if($this->login !== '' && $this->email !== '' && $this->password !== '') {
				$dbh = $this->connection;
				$sth = $dbh->prepare("INSERT INTO users (email,login,password) VALUES ('$this->email','$this->login','$this->password')");
				$sth->execute();
				header('Location: /');
			} else {
				echo 'Заполните все поля <a href="../View/registration.php"><b>Registration</b></a>';
			}
		}
	}

}

И файл который вызывает метод регистрации

require_once 'Model/Registration.php';
use Registration\Registration;

$registration = new Registration();
$registration->Registr();

require_once 'View/index.php';

При вызове функции Registr() ничего не происходит, NULL. Работает только в том случае, если метод вызывается из того файла, где он написан.

Comments

[Karpkarp]

Лентюй, Если бы там писали почему не работает и способ решения, не спрашивал бы,а так вот

Notice: Undefined index: email in C:\PHP\OSPanel\domains\trvlblog\Model\Registration.php on line 20

Notice: Undefined index: login in C:\PHP\OSPanel\domains\trvlblog\Model\Registration.php on line 21

Notice: Undefined index: password in C:\PHP\OSPanel\domains\trvlblog\Model\Registration.php on line 22

Answer 1

[hecby_v]

По идее, раз других ошибок нет, у вас null возможен только если не пришел $_POST['btn']

Answer 2

[Евгений]

У вас ООП ради ООП. Код ужасен, вот и не работает. Смысла в классе таком вообще нет (глобально видимые переменные в конструкторе). А ваш db.prepare достоин отдельных слов: сама идея такого prepare имеет примерно столько же смысла, что и класс регистрации.

Comments

[Karpkarp]

А так?

class Registration {

	protected $connection;

	public function __construct() {
		$this->connection = Db::Connect();
	}

	public function Registr() {
		if(isset($_POST['btn'])) {
			if(isset($_POST['email']) !== '' && isset($_POST['login']) !== '' && isset($_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);
				$sth->execute($sqlParams);
				header('Location: /');
			} else {
				echo 'Заполните все поля <a href="../View/registration.php"><b>Registration</b></a>';
			}
		}
	}

}

[Евгений]

Karpkarp, уже лучше, по крайней мере с биндингом.
Но остались следующие проблемы:
1. зачем вам $this->connection в классе?
2. функция Registr плохо называется и делает аж три вещи (сохранение, редирект либо вывод хтмл)
она должна выглядеть так
function registration($db, $data) {}