생성자에서 시도/캐치 - 권장 사례?

Question

뭔가 항상

public class FileDataValidator { 

private String[] lineData; 

public FileDataValidator(String[] lineData){ 

    this.lineData = lineData; 
    removeLeadingAndTrailingQuotes(); 

    try 
    { 
     validateName(); 
     validateAge(); 
     validateTown(); 
    } 
    catch(InvalidFormatException e) 
    { 
     e.printStackTrace(); 
    } 

} 

//validation methods below all throwing InvalidFormatException 

인가의 호기심을 봤는데 내 생성자 내에서 try/catch 블록을 포함하지 않는 것이 좋습니다? 생성자가 예외를 호출자에게 던져 줄 수 있음을 알고 있습니다. 당신은 내가 Constructor에서했던 것처럼 메소드를 호출 할 때 무엇을 선호합니까? 호출 클래스에서 FileDataValidator의 인스턴스를 만들고 해당 인스턴스에서 메서드를 호출하는 것이 더 좋을까요? 다만 약간 의견을 듣기 위하여 흥미있다!

Answer 1

표시하는 코드에서 유효성 검사 문제는이 개체 인스턴스를 만드는 코드와 다시 통신하지 않습니다. 그것은 아마 좋지 않습니다.

변화 1 : 메소드/생성자 내에서 예외를 잡을 경우

, 호출자에게 다시 뭔가를 통과해야합니다. 모든 것이 작동하면 true로 설정된 필드 isValid을 입력 할 수 있습니다.

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() 함수를 호출해야합니다. 오른쪽은하지만 예외가 나에게 올바른 방법을 보이는 경우

물론

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() 방법을 사용할 수 있습니다 여기에

String[] lineData = readLineData(); 
FileDataValidator onePerson = new FileDataValidator(); 
try { 
    onePerson.build(lineData); 
} catch (InvalidDataException e) { 
    // What to do it its bad? 
} 

클래스 코드 빌드 기능.

4 변형

내가 언급 할 네 번째 방법은 내가 가장 좋아하는 것입니다. 이런 코드가 있습니다. 호출 코드에서 생성자를 호출 한 다음 작동하거나 작동하지 않는 build() 함수를 호출해야합니다.

JaxB와 JaxRS의 작동 방식은 다음과 같습니다. 이는 사용자가 가지고있는 것과 비슷한 상황입니다.

1. 외부 데이터 원본 - 파일이 있으며 XML 또는 JSON 형식의 수신 메시지가 있습니다.
2. 오브젝트를 빌드하는 코드 - 코드를 가지고 있기 때문에 코드의 라이브러리는 다양한 JSR의 스펙에 따라 작동합니다.
3. 유효성 검사는 개체 건물과 관련이 없습니다.

호출 코드 :

public class Person { 
    private Name name; 
    private Age age; 
    private Town town; 
... lots more stuff here ... 
} 

그리고 유틸리티 코드를 구축하고 검증하기 : 여기

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 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()); 
    } 

} 

Answer 2

내가 선호하는 예외는 코드가 알고있는 코드에 의해 다루어진다. 어떻게 그들을 다루는가. 이 경우 FileDataValidator를 만드는 약간의 코드는 파일 데이터가 유효하지 않은 경우 어떤 일이 발생하는지 알고 있으며 예외는 처리되어야한다고 가정합니다 (호출자에게 전파하는 것을지지합니다).

모범 사례를 논의하는 동안 클래스 이름 FileDataValidator가 나에게 냄새를 풍깁니다. 생성하는 객체가 파일 데이터를 저장하는 경우 FileData를 호출 할 것입니다. 아마도 validate 메서드를 사용했을 것입니다. 파일 데이터의 유효성을 검사하려는 경우 정적 메서드로 충분합니다.

Answer 3

정적 팩터 리 패턴을 고려해야합니다. 모든 인수 생성자를 private으로 만듭니다. 정적 FileDataValidator (args ...) 메소드를 제공하십시오. 이것은 모든 인수를 받아들이고 유효성을 검사합니다. 다 괜찮 으면 private 생성자를 호출하고 새로 생성 된 객체를 반환 할 수 있습니다. 실패한 경우 Exception을 throw하여 호출자에게 잘못된 값을 제공했음을 알립니다.

이 또한 언급해야합니다 : catch (Exception e) { printSomeThing (e); }

예외로 할 수있는 가장 치명적인 반 패턴입니까? 예, 명령 줄에서 몇 가지 오류 값을 읽을 수 있습니까? 호출자 (나쁜 ​​값을 제공 한 사람)가 잘못된 값에 대해 알지 못하면 프로그램 실행이 계속됩니다.