Почему Class.newInstance () "злой"?

Question

Райан Делукки спросил здесь в комментарии № 3 к Tom Hawtin ответ:

почему Class.newInstance () "злой"?

это в ответ на пример кода:

// Avoid Class.newInstance, for it is evil.
Constructor<? extends Runnable> ctor = runClass.getConstructor();
Runnable doRun = ctor.newInstance();

итак, почему это зло?

Answer 1

Документация по API Java объясняет, почему (http://java.sun.com/javase/6/docs/api/java/lang/Class.html#newInstance()):

Обратите внимание, что этот метод распространяет любое исключение, выданное нулевым конструктором, включая проверенное исключение. Использование этого метода эффективно обходит проверку исключений во время компиляции, которая в противном случае выполнялась бы компилятором. Метод Constructor.newInstance позволяет избежать этой проблемы, помещая любое исключение, выброшенное конструктором, в (проверено) InvocationTargetException.

Другими словами, он может победить систему проверенных исключений.

Answer 2

Еще одна причина:

Современные IDE позволяют вам найти использование классов - это помогает во время рефакторинга, если вы и ваша IDE знаете, какой код использует класс, который вы планируете изменить.

Если вы не используете явное использование конструктора, а используете вместо него Class.newInstance (), вы рискуете не обнаружить это использование во время рефакторинга, и эта проблема не проявится при компиляции.

Answer 3

Я не знаю, почему никто не предоставил простое объяснение, основанное на примере, по сравнению с, например, Constructor::newInstance, так как окончательно Class::newInstance устарело с java-9.

Предположим, у вас есть очень простой класс (не имеет значения, что он сломан):

static class Foo {
    public Foo() throws IOException {
        throw new IOException();
    }
}

И вы пытаетесь создать его экземпляр с помощью отражения. Первый Class::newInstance:

    Class<Foo> clazz = ...

    try {
        clazz.newInstance();
    } catch (InstantiationException e) {
        // handle 1
    } catch (IllegalAccessException e) {
        // handle 2
    }

Вызов этого приведет к выбросу IOException - проблема в том, что ваш код не обрабатывает его, ни handle 1, ни handle 2 не поймают его.

В отличие от этого при использовании Constructor:

    Constructor<Foo> constructor = null;
    try {
        constructor = clazz.getConstructor();
    } catch (NoSuchMethodException e) {
        e.printStackTrace();
    }

    try {
        Foo foo = constructor.newInstance();
    } catch (InstantiationException e) {
        e.printStackTrace();
    } catch (IllegalAccessException e) {
        e.printStackTrace();
    } catch (InvocationTargetException e) {
        System.out.println("handle 3 called");
        e.printStackTrace();
    }

будет вызван дескриптор 3. Таким образом, вы будете обрабатывать его.

Фактически, Class::newInstance обходит обработку исключений - что вам действительно не нужно.