Ответ 1
В показанном коде проблемы проверки не возвращаются к коду, который создает этот экземпляр объекта. Вероятно, это НЕ ХОРОШЕЕ.
Вариант 1:
Если вы поймаете исключение внутри метода/конструктора, обязательно передайте что-то обратно вызывающему. Вы можете поместить поле isValid, которое получает значение true, если все работает. Это будет выглядеть так:
private boolean isValid = false;
public FileDataValidator(String[] lineData){
this.lineData = lineData;
removeLeadingAndTrailingQuotes();
try
{
validateName();
validateAge();
validateTown();
isValid = true;
}
catch(InvalidFormatException e)
{
isValid = false;
}
}
public boolean isValid() {
return isValid;
}
Вариант 2:
Или вы можете разрешить исключение или какое-либо другое исключение для вызывающего. Я показал это как исключение, не прошедшее проверку, но делаю все, что работает в соответствии с вашей религией обработки исключений:
public FileDataValidator(String[] lineData){
this.lineData = lineData;
removeLeadingAndTrailingQuotes();
try
{
validateName();
validateAge();
validateTown();
}
catch(InvalidFormatException e)
{
throw new com.myco.myapp.errors.InvalidDataException(e.getMessage());
}
}
Вариант 3:
Третий метод, который я хочу упомянуть, имеет такой код. В вызывающем коде вы должны вызвать конструктор, а затем вызвать функцию build(), которая либо будет работать, либо нет.
String[] lineData = readLineData();
FileDataValidator onePerson = new FileDataValidator();
try {
onePerson.build(lineData);
} catch (InvalidDataException e) {
// What to do it its bad?
}
Вот код класса:
public FileDataValidator() {
// maybe you need some code in here, maybe not
}
public void build(String[] lineData){
this.lineData = lineData;
removeLeadingAndTrailingQuotes();
try
{
validateName();
validateAge();
validateTown();
}
catch(InvalidFormatException e)
{
throw new com.myco.myapp.errors.InvalidDataException(e.getMessage());
}
}
Конечно, функция build() может использовать метод isValid(), который вы вызываете, чтобы увидеть, правильно ли это право, но исключение, для функции сборки.
Вариант 4:
Четвертый метод, который я хочу упомянуть, - это то, что мне больше всего нравится. У этого есть такой код. В вызывающем коде вы должны вызвать конструктор, а затем вызвать функцию build(), которая либо будет работать, либо нет.
Этот вид следует за тем, как работают JaxB и JaxRS, что похоже на то, что у вас есть.
- Внешний источник данных - у вас есть файл, у них есть входящее сообщение в формате XML или JSON.
- Код для создания объектов - у вас есть код, у них есть свои библиотеки кода, работающие по спецификациям в различных JSR.
- Валидация не привязана к построению объектов.
Вызывающий код:
String[] lineData = readLineData();
Person onePerson = new Person();
FileDataUtilities util = new FileDataUtilities();
try {
util.build(onePerson, lineData);
util.validate(onePerson);
} catch (InvalidDataException e) {
// What to do it its bad?
}
Вот код класса, в котором живут данные:
public class Person {
private Name name;
private Age age;
private Town town;
... lots more stuff here ...
}
И код утилиты для сборки и проверки:
public FileDataValidator() {
// maybe you need some code in here, maybe not
}
public void build(Person person, String[] lineData){
this.lineData = lineData;
removeLeadingAndTrailingQuotes();
setNameFromData(person);
setAgeFromData(person);
setTownFromData(person);
}
public boolean validate(Person person) {
try
{
validateName(person);
validateAge(person);
validateTown(person);
return true;
}
catch(InvalidFormatException e)
{
throw new com.myco.myapp.errors.InvalidDataException(e.getMessage());
}
}