Часто интервьюеры отписывают мне, что в тестовых заданиях я не делаю валидацию.
Ранее для меня концепции юнит-тестирование
, валидация
, try...catch
как-то было одно и тоже. И только за последний год я "освоил" юнит-тестирование и теперь не представляю свой код без тестов и кодирую через TDD (без фанатизма).
try...catch для меня тоже немного темный лес, а именно: я так и не понял когда именно его использовать.
Но сегодня я хочу чтобы вы мне помогли разобраться с валидацией.
Вот прямо сейчас пишу код, реализуя паттерн фабричный метод, где создается объект TResultDTO для передачи его AngularJS во View. Объект TResultDTO формируется на основе EF-сущности TestResult. И я пытаюсь организовать валидацию:
public TResultDTO FactoryMethod(TestResult entity)
{
//Валидация
if(entity != null)
{
var dto = new TResultDTO
{
ExamCode = entity.TestPlan.Test.NumberCode,
ExamName = entity.TestPlan.Test.Name,
ExamDate = entity.TestDate.ToShortDateString(),
ParticipCode = entity.ParticipCode
};
return dto;
}
}
Вопрос
- Ну во первых VS сейчас пишет:
и что мне с этим делать? Я не вижу выхода, разве что в конце метода указать return null
. Ну тогда вообще что-то непонятное выходит.
- Достаточно ли проверить
entity
на null? Или в идеале необходимо проверить его свойства (entity.TestPlan.Test.NumberCode
,entity.TestPlan.Test.Name
...) , непосредственно к значениям которых обращается метод?
Я также понимаю, что наверника нет каких-то готовых формул и много (возможно) будет зависит от предметной области. Единственно, что я смог найти на этот счет в сети: коллега на форуме пишет, что входящие параметры публичных методов необходимо обязательно подвергать процессу валидации.
Обновление
Прислушавшийся к ответам и комментариям делаю валидацию следующим образом:
namespace Monit95App.Models
{
public class TResultDTOcreator : ITResultDTOcreator
{
public TResultDTO FactoryMethod(TestResult entity)
{
if (String.IsNullOrEmpty(entity.TestPlan.Test.NumberCode))
throw new ArgumentException("Property cannot be null or empty", nameof(entity.TestPlan.Test.NumberCode));
if (String.IsNullOrEmpty(entity.TestPlan.Test.Name))
throw new ArgumentException("Property cannot be null or empty", nameof(entity.TestPlan.Test.Name));
if (entity.TestPlan.TestDate == null || entity.TestPlan.TestDate == DateTime.MinValue)
throw new ArgumentException("Property cannot be null or empty", nameof(entity.TestPlan.TestDate));
if (String.IsNullOrEmpty(entity.ParticipCode))
throw new ArgumentException("Property cannot be null or empty", nameof(entity.ParticipCode));
var dto = new TResultDTO
{
ExamCode = entity.TestPlan.Test.NumberCode,
ExamName = entity.TestPlan.Test.Name,
ExamDate = entity.TestDate.ToShortDateString(),
ParticipCode = entity.ParticipCode
};
return dto;
}
}
}
Что теперь можете сказать коллеги?
Ранее забыл отметить, что в проекте используется подход database first
.