Назначение последнего поля в блоке try-catch внутри конструктора

Question

Итак, я пытаюсь инициализировать DatagramSocket в конструкторе, и я хочу, чтобы это поле было final, но мой компилятор (т.е. Eclipse) выдает мне следующую ошибку:

Пустое конечное поле дейтаграммы возможно, не было инициализировано

Это понятно. Вот фрагмент кода:

    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?

EDIT:

Для тех из вас, кто заинтересован, вот решение, которое я придумал из ваших рекомендаций:

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() isn 'Вы не добавляете много ценности, не правда ли?

Answer 2

Возможная альтернатива - заставить ваш конструктор генерировать исключение SocketException. Это избавит от необходимости в блоке try-catch, который заставляет вас использовать временную переменную.