Как обрабатывают ошибки в классах?

Question

[Алексей]

Всем привет! Пытаюсь овладеть классами и вот задался вопросом... есть такой вот простенький класс:

class Ports
{
	private $db;
	
	public function __construct($db)
	{
		$this->db = $db;
	}
	
	private function AddPort($port) // записываем название порта в базу
	{
		$port = $this->db->real_escape_string($port);
		$res = $this->db->query("INSERT INTO `ports` (`name`) VALUES('{$port}')");
		return $this->db->insert_id;
	}
	
	public function Port($port) // Проверяем существование порта
	{
		$port = $this->db->real_escape_string(trim($port));
		$res = $this->db->query("SELECT `id` FROM `ports` WHERE `name` = '{$port}' LIMIT 1");
		
		if($res->num_rows == 1){
			$row = $res->fetch_assoc();
			return $row['id'];
		}else{
			$port_id = $this->AddPort($port);
			return $port_id;	
		}
	}
}

Из вне, я вызываю метод Port которому передаю в параметрах название порта (морской порт к примеру). Метод проверяет наличие такого названия в базе и возвращает его id в случае удачи. Если такого порта не существует, вызываем метод AddPort и передаем ему название порта, там порт добавляется в базу и метод AddPort возвращает id добавленной записи. И тут вопрос: "Как должна работать логика, если вдруг, по каким-то причинам AddPort не вернет id добавленной записи? Это простой пример, но ведь цепочка может быть большая, то как вывести ошибку пользователю где именно код сработал не верно?"

Другими словами, нужно вставить проверку сюда:

if($this->db->query("INSERT INTO `ports` (`name`) VALUES('{$port}')")){
//continue
}else{
 "Не удалось добавить запись"; // не понятно, куда добавить эту информацию, чтобы она аккуратно вывелась пользователю и приостановила весь скрипт.
}

Comments

[Алексей]

Обратите внимание, при добавлении порта в базу, его название будет дважды пропущено через real_escape_string

Answer 1

[Сергей Протько]

1) ошибки происходят не в классах. Разберитесь в чем разница между объектами и классами, что есть что и что для чего.

2) исключения.

Answer 2

[IceJOKER]

во-первых - начните название методов с маленькой буквы и пишите в стиле camelCase.
во-вторых - дайте методам говорящие названия, а не Port - что за метод? лучше бы назвали getPortId или getPort, или checkPort и т.д.

а насчет вопроса - если не вернул, значит не смог добавить порт, если не смог, значит возвращаем ошибку и останавливаем работу. + вы можете написать отправку письма на почту, чтоб в случае проблем вас оповещали.

[sarcasm]ИЛИ ЖЕ запускаете в цикле до тех пор пока не добавится :D будьте настойчивы![/sarcasm]

Comments

[Алексей]

Что нужно остановить работу скрипта и вывести ошибку - это понятно, но как её вывести пользователю? Обновил вопрос.

[IceJOKER]

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

[AlikDex]

Евгений: пользователю не нужно знать что за ошибки в скрипте случаются(но их можно логировать). Максимум не прошедшие валидацию данные. А для девелопмента хватит вывода стандартных ошибок.

[Алексей]

AlikDex: Ну я этот класс написал просто для примера, но бывают же случаи, когда пользователю нужно знать ошибку, например если это регистрационная форма, а метод AddPort не добавляет данные а проверяет наличие такого логина в базе и должен вывести ошибку пользователю "такой логин существует"

[AlikDex]

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

[Алексей]

AlikDex: т.е. в моем методе Port в самом конце проверять существует ли в массиве какая-то ошибка, и если существует, то вместо return true делать return $this->errors. ?

[AlikDex]

Евгений: не совсем. Ошибки подключения к бд, кеш серверам, отсутствие каких-нибудь файлов или проблемы с записью\созданием и прочие подобные обрабатывать исключениями но НЕ выводить их на экран обычным пользователям. Только на этапе разработки. Если приложение работает в штатном режиме что называется "production" то глушить их. Сами посудите, какой толк пользователю от таких ошибок? Разве он сможет повлиять на работу скрипта? Нет конечно. Но вот какому-нибудь злоумышленнику это может сыграть на руку. И еще нужно разработчику на этапе отладки. Все.
Вот, а ошибки, допущенные пользователем уже складировать куда-то. Например неправильно ввел пароль. Или загрузил не картинку, а текстовик какой-нибудь. Ну вот в таком духе. То есть , то что касается обычного пользователя напрямую.

Answer 3

[danforth]

Я конечно не гуру в PHP, но сделал бы так:
- написал бы класс DBException который наследовал бы PDOException
- все операции с БД проводил через PDO
- во время отрабатывания метода query делал бы try - catch.
- если проскакивает ошибка, собрал бы лог и куда-то бы сохранил

Но в вашем случае можно просто try catch воспользоватся