생성자 내의 try-catch 블록에서 최종 필드 할당하기

Question

그래서 생성자에서 DatagramSocket을 초기화하려고하는데이 필드가 final이 되길 원하지만 내 컴파일러 (예 : Eclipse)는 나에게 다음 오류 :

The blank final field datagramSocket may not have been initialized

이것은 이해할 수 있습니다. 코드 스 니펫은 다음과 같습니다.

public class Foo 
    { 
     private final int DEFAULT_UDPLISTENPORT = 49400; 
     private final DatagramSocket datagramSocket; 

     public Foo() 
     { 
      synchronized(this) 
      { 
       try 
       { 
        datagramSocket = new DatagramSocket(DEFAULT_UDPLISTENPORT); 
       } 
       catch (SocketException e) 
       { 
        // Log error 
        logger.error("Trouble opening UDP port: ", e); 
       } 
      } 
     } 
    } 

이제 이것을 우회하는 방법이 있지만 임시 변수를 만들어야합니다.

public class Foo 
    { 
     private final int DEFAULT_UDPLISTENPORT = 49400; 
     private final DatagramSocket datagramSocket; 

     public Foo() 
     { 
      synchronized(this) 
      { 
       DatagramSocket tempSocket = null; 
       try 
       { 
        tempSocket = new DatagramSocket(DEFAULT_UDPLISTENPORT); 
       } 
       catch (SocketException e) 
       { 
        // Log error 
        logger.error("Trouble opening UDP port: ", e); 
       } 

       datagramSocket = tempSocket; 
      } 
     } 
    } 

그래서, 내 질문은 가정 :이 일을 더 우아한 방법이 있나요, 아니면 그냥 내가 원하는 경우 살 해야이 뭔가 여기에 코드입니다 그 필드는 final이 될까요?

편집 : 관심이있는 분들을 위해

, 여기에 내가 당신의 권고에서 함께했다 솔루션입니다 :

public class Foo 
{ 
    private static final Foo INSTANCE; 
    static 
    { 
     try 
     { 
      INSTANCE = new Foo(); 
     } 
     catch (SocketException e) 
     { 
      throw new ExceptionInInitializerError(e); 
     } 
    } 
    private final int DEFAULT_UDPLISTENPORT = 49400; 
    private final DatagramSocket datagramSocket; 

    public Foo() throws SocketException 
    { 
     synchronized (this) 
     { 
      datagramSocket = new DatagramSocket(DEFAULT_UDPLISTENPORT); 
     } 
    } 

    public static Foo getInstance() 
    { 
     return INSTANCE; 
    } 
} 

날이 올 알고, 또는 경우 주시기 바랍니다 다른 제안이 있습니다. 도움에 감사드립니다!

Answer 1

예, SocketException을 catch 한 후 런타임 예외로 감싸서 다시 던집니다. 변수가 final이고 개체 초기화 중 오류가 발생했기 때문에 개체의 상태가 잘못되었을 수 있으며 개체가 그대로 유지됩니다.

예외 처리에 대한 로깅은 예외 처리 및 숨기기로 충분하지 않습니다. SocketException은 개체가 유효하지 않다는 사실을 숨기고 계속해서 NullPointerException 또는 다른 위험을 감수 할 수 있도록합니다. DatagramSocket를 서브 클래 싱하고 만드는 것이 좋습니다 : null를 반환으로

public Foo() 
    { 
     synchronized(this) 
     { 
      datagramSocket = createSocket(); 
     } 
    } 

private DatagramSocket createSocket() { 
     try 
     { 
      return new DatagramSocket(DEFAULT_UDPLISTENPORT); 
     } 
     catch (SocketException e) 
     { 
      logger.error("Trouble opening UDP port: ", e); 
      return null; //I beg you, don't return null here... 
     } 
} 

:

당신이 정말로 이러한 잘못된 개체를 만들려면, 당신의 제안은 괜찮지 만 다른 방법을 사용

• NoOpDatagramSocket

• NullDatagramSocket

• BrokenDatagramSocket

• MemoryDatagramSocket

• ... 당신이

P.S : 왜 synchronized :-) 생각을?

P.S.2 : 바로 logger.error() 앞에 덧글 // Log error이 많은 가치를 더하고 있지 않습니까?

Answer 2

가능한 대안은 생성자가 SocketException을 throw하는 것입니다. 이렇게하면 임시 변수를 사용해야하는 try-catch 블록의 필요성이 제거됩니다.