Безопасная регистрация

require_once "../my-function/my-function.php";
require_once "../my-class/my-class.php";
 
//Получили данные от пользователя. Где проверка/фильтрация? Наверно далее....
$email = $_GET["email"];
$password = $_GET["password"];
 
 
//Если есть мыло и пароль то Authorization? Ок.... там проверка?
if(isset($email) && isset($password)) {
try{
Authorization::searchUser(array("email" => $email, "password" => $password));
.....
 
//Ваш класс из другого файла
class Authorization {
    static function searchUser($data) {
//Прекрасно. Подготовили sql запрос с непонятно каким email. Проверки нет. Далее вы выполните его. (
        $query_email = 'SELECT * FROM `users` WHERE email = :email';

        if(!is_array($data)) {
            throw new Exception("Не корректный тип входных данных!");
        }

        if(!preg_match(DefaultInfo::$p_login, $data['login'])){
            throw new Exception("Логин может состоять только из букв английского алфавита и цифр");
        }

        if (!filter_var($data["email"], FILTER_VALIDATE_EMAIL)) {
            throw new Exception("Не корректный email");
        }

        $q_add_user = "INSERT INTO `users` (login, email, password) VALUES (:login, :email, :password)";
 
        $db = new DB(DefaultInfo::$defaultDataDB);
        $pdo = $db->connect();
 
        $data = array(
            'email'     =>  $user->email,
            'login'     =>  $user->login,
            'password'  =>  $user->password
        );
 
        $user_data = $pdo->prepare($q_add_user); // 1
        $user_data->execute($data); // 2

<?
// почему все статичное?
class Registration {
    // зачем выносить sql сюда, почему бы не создать полноценные методы
    static private $query_table = 'SELECT * FROM `users`';
    static private $query_login = 'SELECT * FROM `users` WHERE login = :login';
    static private $query_email = 'SELECT * FROM `users` WHERE email = :email';
 
    // функциональность не совпадает с именем метода
    static function init($data) {
        // жесткая зависимость сразу от двух классов
        $db = new DB(DefaultInfo::$defaultDataDB);
        $pdo = $db->connect();
 
        // с какого перепуга Exception на банальную проверку данных.
        if(!is_array($data)) {
            throw new Exception("Не корректный тип входных данных!");
        }
 
        // почему регулярка берется из совершенно левого класса?
        if(!preg_match(DefaultInfo::$p_login, $data['login'])){
            throw new Exception("Логин может состоять только из букв английского алфавита и цифр");
        }
 
        // каша с названиями
        $login_test = $pdo->prepare(self::$query_login);
        $login_test->execute(array('login' => $data["login"]));
 
        // зачем цикл? для проверки достаточно найти хотя бы одного пользователя с таким логином
        foreach($login_test as $login_db){
            // то что это массив, не гарантирует что пользователь уже существует
            if(is_array($login_db)) {
                // грамматическая ошибка
                throw new Exception("Пользователь с таким логином уже суещствует!");
            }
        }
 
 
        if (!filter_var($data["email"], FILTER_VALIDATE_EMAIL)) {
            throw new Exception("Не корректный email");
        }
 
        $email_test = $pdo->prepare(self::$query_email);
        $email_test->execute(array('email' => $data["email"]));
        foreach($email_test as $email_db){
            if(is_array($email_db)) {
                // перемешан html и php.
                throw new Exception("Указанный e-mail(" . $data["email"] . ") уже занят!<br>");
            }
        }
 
        // еще две жесткие зависимости. 
        // переход в контроллер должен быть только из роутера
        DBController::addUser(new CreateUser($data));
 
        header("Location: http://" . $_SERVER["SERVER_NAME"] . "/registration/info.php?email=" . $data["email"]);
    }
}

    static function searchUser($data) {
        $query_email = 'SELECT * FROM `users` WHERE email = :email';
 
        $db = new DB(DefaultInfo::$defaultDataDB);
        $pdo = $db->connect();
 
        if(!is_array($data)) {
            throw new Exception("Не корректный тип входных данных!");
        }
 
        $user = $pdo->prepare($query_email);
        $user->execute(array('email' => $data["email"]));
 
        if($user->rowCount() == 0) {
            throw new Exception("Указанный вами e-mail(" . $data["email"] . ") не найден!");
        }
 
        foreach($user as $info){
            if($data["password"] == $info["password"]) {
                self::authorUser($info["id"]);
                header("Location: http://" . $_SERVER["SERVER_NAME"]);
            }
            else {
                throw new Exception("Вы ввели не верный пароль");
            }
        }
    }

require_once "../my-function/my-function.php"; //Подключили файл
require_once "../my-class/my-class.php"; //Подключили файл
 
$email = $_GET["email"]; //Приняли неизвестно что методом GET
$password = $_GET["password"]; //Приняли неизвестно что методом GET
 
//Если есть 2 неизвестно что
if(isset($email) && isset($password)) {
    try{
//Это неизвестно что формируем в массив. И этот массив с неизвестно чем передаем в ф-цию searchUser класса Authorization
Authorization::searchUser(array("email" => $email, "password" => $password));
//Далее ещё код
?>

//Класс и функция
class Authorization {
    static function searchUser($data) {
 
//Подготовили переменную с ниизвестно чем
        $query_email = 'SELECT * FROM `users` WHERE email = :email';
//Работаем с БД
        $db = new DB(DefaultInfo::$defaultDataDB);
        $pdo = $db->connect();
 
//Неизвестно что массив? Если нет - давай досвиданья (и тоже не совсем корректно, но не суть)
        if(!is_array($data)) {
            throw new Exception("Не корректный тип входных данных!");
        }
//Надежда на pdo? 
 $user = $pdo->prepare($query_email);
//Прекрасно, только что мы выполнили sql запрос с неисвестно чем!
        $user->execute(array('email' => $data["email"]));

$db = new DB(DefaultInfo::$defaultDataDB);
        $pdo = $db->connect();