在构造函数中的try-catch块中分配final字段

Question

所以,我正在尝试初始化一个DatagramSocket构造函数,我想要这个字段final,但我的编译器(即Eclipse)给我以下错误:

空白的最终字段datagramSocket可能尚未初始化

这是可以理解的.这是一段代码片段:

    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在运行时异常捕获它并重新抛出它之后.由于您的变量是final并且在对象初始化期间遇到错误,因此您的对象可能处于不正确的状态,并且您可以保证它将保持不变.

记录异常对于异常处理和隐藏来说可能不足以SocketException隐藏对象无效并允许您继续,冒险NullPointerException或其他事件的事实.

如果你真的想创建这样一个有缺陷的对象,你的建议很好,只需使用另一种方法:

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...
        }
 }

至于返回null:考虑子类化DatagramSocket和创建:

• NoOpDatagramSocket

• NullDatagramSocket

• BrokenDatagramSocket

• MemoryDatagramSocket

• ......你明白了:-)

PS:为什么synchronized？

PS2:// Log error之前的评论logger.error()并没有增加太多价值,你不觉得吗？

Answer 2

一个可能的替代方法是让你的构造函数抛出SocketException.这将消除强制您使用临时变量的try-catch块的需要.