Как я должен обработать ожидаемые погрешности? например, “имя пользователя уже существует”

Я часто обрабатываю такие вещи, вставляя свою проверку в какой-то объект домена (например, пользователь).

Например:

<?PHP
$user->name = 'Joe';
$user->password = 'secret';

//try to save the user.
if (! $user->save()){
   $errors = $user->getMessages();
   // ... do something with error messages
}else{
   //the user object set it's own ID.
   $id = $user->id;
}

Теперь за пользователем абстрагируется много вещей. Там может быть целая вселенная объектов, но это от вас.

Более серьезная проблема здесь заключается в том, почему в мире у вас есть класс аутентификации создать пользователя? Создание пользовательских объектов, вероятно, выходит за рамки аутентификации.

Возможно, было бы разумно иметь какой-то метод типа authenticate($username,$password) return объект типа 'user' (представляющий пользователя, который только что аутентифицировался), но даже это немного беспорядочно.

Вместо этого рассмотрим что-то вроде:

<?PHP
$auth = new AuthService();
$u = new User();
$u->username='Joe';
$u->password='secret';

$authResult = $auth->authenticate($user);

if ($authResult->success){
  //$user has properties set as a side effect.
}else{
  //find out why it failed.
  $errors = $authResult->getErrors();
}

Конечно, для этого требуется определить какой-то результат.класс value для AuthService::authenticate() для заполнения и возврата. Но, по крайней мере, вы не получите методы, которые иногда возвращают логические значения, а иногда возвращают объекты и т. Д.

Если вы управляете типом возврата метода, вы также можете изменить его подпись, чтобы вернуть что-то вроде объекта OperationResult, который будет иметь свойство ErrorDescription и свойство объекта UserID или User.

Самый простой способ: вернуть NULL, если проверка прошло, и сообщение об ошибке, если это не удалось. Предотвращает необходимость создавать классы ошибок или исключения. Также лучший по производительности.

например.

function numeric($input) {
    if (!is_numeric($input)) return 'Must be numeric.';
    return NULL;
}

function usernameavailable($input) {
    //query database....
    if ($result) return 'Username already taken, please choose another one.';
    return NULL;
}

Вы также можете использовать функции с параметрами:

function length($input, $min, $max) {
    if (strlen($input) < $min) return "Must be longer than $min characters.";
    if (strlen($input) > $max) return "Must be shorter than $max characters.";
    return NULL;
}

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

Я бы категорически не одобрял предложения, сделанные одним из других ответов. Недействительный ввод пользователя не является исключительным и не требует исключения или ошибки. Это просто ожидаемый вариант использования, с которым нужно работать как можно проще.

С помощью этого метода вы можете довольно легко создать класс Form и просто создавать экземпляры элементов с аргументами, являющимися функциями проверки. Вы также можете расширить это, чтобы включить проверку AJAX, отправляющую асинхронные запросы к тем же функциям проверки PHP.

Пример из моей структуры:

$Form = new AjaxForm();
$Form->add(new TextBox('Username', 'usernameavailable|length[3,12]'));
$Form->add(new SubmitButton());

Всего с тремя строчками я создал полностью функциональную форму с проверкой как на стороне клиента, так и на стороне сервера. Конечно, все детали зависят от вас, но я по-прежнему считаю, что недопустимый ввод пользователя следует ожидать, а не быть исключительным.

Это должно быть сделано следующим образом -

Создайте объект ошибки с кодами ошибок в них, текстом ошибки и методом, который выбросил ошибку.

Передайте объект ошибки обратно.

Вы можете бросать исключения обратно, но исключения стоят дорого

EDIT - Основываясь на правке ОП. В этом случае вам придется бросать объект исключения обратно.

Создайте класс исключения с кодом ошибки, описанием ошибки, именем класса, именем метода. Бросьте исключение обратно.

Вы можете возвращать целые числа, которые определены в глобальном файле заголовка с такими именами, как DUPLICATE_USER или INVALID_PASSWORD, или любым другим, подходящим для вашего использования. Затем вызывающая функция может легко сверить эти коды возврата с объявлениями глобального заголовка, чтобы определить, какой тип ошибки произошел.

Вы должны проверить, существует ли имя пользователя ПЕРЕД вызовом метода createUser(). Если вы этого не делаете или что-то происходит между временем проверки и вызова createUser(), вы создаете исключение. Это далеко не так дорого, как это представляется.

Есть ли у этого стоимость? Да. Достаточно ли он велик, чтобы иметь значение? Скорее всего, нет.