Есть ли смысл инкапсулировать обработку исключительных ситуаций, вынося код из контроллера в метод сущности?

Question

[mindgrow]

Добрый день!

Подскажите, как вы считаете, как лучше реализовать проверку наличия какой-то сущности при изменении

public ActionResult SomeController(SomeModel model)
{
	
	var entityDT = m_repository.Get(model);
	
	if (entityDT == null)
	{
		HttpNotFound();
	}
	else
	{
		Automapper.Map<SomeModel,SomeEntity>(model, entityDT);
		m_repository.Save(entityDT);
		
		return View();
	}
	
}

или

public ActionResult SomeController(SomeModel model)
{
	
	var entityDT = m_repository.GetOrNotFound(model);
	
	Automapper.Map<SomeModel,SomeEntity>(model, entityDT);
	m_repository.Save(entityDT);
	
	return View();	
}

или Ваш вариант

Answer 1

[MrDywar Pichugin]

1) Контроллер - связывает UI и логику, но это не сама логика, максимум тут подготовка модели к дальнейшей передаче. GRASP - Контроллер.
2) Логику по получению, маппингу и т.п. переносим в сервис. SomeModelService например.
3) Repository - просто коллекция, никаких not_found там нет.
4) Данный сценарий как я понял обновляет существующую сущность. Значит вероятно следует добавить проверку на конкуретный доступ, оптимистический, т.к. это веб. Т.е. пусть в SomeModel будет значение версии, дата или int или что вам удобно. И хендлинг всех возможных ситуаций делайте не в контроллере а в сервисе. Т.к. UI может добавится другой.
5) Вместо конкретных репозиториев и сервисов используйте интерфейсы. SOLID, буква D. Это позволит легко тестировать приложение, и разрабатывать модули не ожидая все другие модули, которые вы используете.

Но если приложение простое, или учебное, то можно просто забить на все это, т.к. оверхед большой, и пилить как удобно. В таком случае разницы между 1 и 2 способ не много, но 1-ый чище.
Примеры всяких GetOrXXX хорошо подсмотреть у C# LINQ. FirstOrDefault и т.п.

Comments

[mindgrow]

вызов HttpNotFound(); откуда предлагаешь делать? Из сервиса?

[MrDywar Pichugin]

Viktor Ilukhin, сервисом кидать not_found_exception если это является исключительной ситуацией, т.е. ошибкой.
Все exceptions ловит фильтр исключений который висит на все API, и преобразует их в формат удобный для API клиентов.

Севису все равно кто его вызвал, тест или API клиент, или еще кто. Он работает так как работает, и описывает это в своем интерфейсе.

Если API клиент хочет HTTP ответ с ошибкой и статус кодом 404, он это получит, фильтр это сделает.
Если API клиент хочет что то другое, он это получит, т.к. контроллер для таких клиентов другой (desktop client, test client).

[mindgrow]

MrDywar Pichugin, я таким не понял в каком контексте ты бы вызвал бы метод HttpNotFound() который перенаправил на соответствующую страницу. Речь идёт только о проекте Asp Mvc web клиент.

Я вижу ты говоришь о контекстах сервиса. То есть в сервисе создавать исключение, а обработка этого исключения где должна происходить ? В контроллере или может в контексте HttpApplication?

[MrDywar Pichugin]

Viktor Ilukhin, mvc 5 Фильтры исключений - https://metanit.com/sharp/mvc5/8.4.php, или возвращать null и в контроллере. Это по желанию.
Но таких мест могут быть десятки, везде проверять на null не очень удобно будет.
И не всегда сервис может вернуть null, может там десятая по вложенности сущность отсутствует или еще что-либо.

[mindgrow]

MrDywar Pichugin, какой же ты не конкретный. Ладно, так и не получил от тебя ответа на свой вопрос