Как избежать предупреждений о генераторах при использовании «сырых типов» в качестве статического поля/типа возврата?

Question

Это в основном следующее: question со вчерашнего дня. Я попытался реализовать «самый общий способ» (я мог подумать), чтобы создать «действительно» un-modifiable коллекции. Код ниже работает нормально; но это требует подавления предупреждений («rawtypes» и «unchecked»).

Я пробовал всевозможные вещи, чтобы избежать этих предупреждений; но просто не мог найти альтернативы. Итак, мой вопрос: есть ли чистый способ избежать предупреждений, которые я подавляю?

(я видел это другое question, но я не уверен на 100%, если он действительно применим к моему коду, так как я не уверен, могу ли я записать интерфейс «Generator», не используя там сырой тип).

private interface Generator<T> { 
    T generateNewInstance(); 
} 

@SuppressWarnings("rawtypes") 
private final static Generator ListGenerator = new Generator<List>() { 
    @Override 
    public List generateNewInstance() { 
     return new ArrayList(); 
    } 
}; 

public static <T> List<T> unmodifiableListBasedOnCloneOf(List<T> elements) { 
    @SuppressWarnings("unchecked") 
    List<T> newInstance = makeNewInstanceOf(elements.getClass(), ListGenerator); 
    newInstance.addAll(elements); 
    return Collections.unmodifiableList(newInstance); 
} 

private static <T> T makeNewInstanceOf(Class<T> incomingClass, Generator<T> fallbackGenerator) { 
    try { 
     return (T) incomingClass.newInstance(); 
    } catch (IllegalAccessException | InstantiationException e) { 
     return fallbackGenerator.generateNewInstance(); 
    } 
} 

Answer 1

Если вы делаете копию List<T> в новую ArrayList<T>, и передать эту эталонную встроенный в Collections.unmodifiableList, ничего, кроме неизменяемо декоратора имеет доступ к копии на изменить его:

List<String> original = new ArrayList<>(Arrays.asList("Hello", "World")); 

    // This simply decorates original, so updates to original are 
    // reflected in this list. 
    List<String> unmodifiableDecorator = Collections.unmodifiableList(original); 

    // The reference to the new ArrayList is scoped to the call to 
    // Collections.unmodifiableList, so nothing else can have a 
    // reference to it. 
    List<String> unmodifiableCopy = 
     Collections.unmodifiableList(new ArrayList<>(original)); 

    System.out.println("Before clear:"); 
    System.out.println(original); 
    System.out.println(unmodifiableDecorator); 
    System.out.println(unmodifiableCopy); 

    original.clear(); 

    System.out.println("After clear:"); 
    System.out.println(original); 
    System.out.println(unmodifiableDecorator); 
    System.out.println(unmodifiableCopy); 

Выход:

Before clear: 
[Hello, World] 
[Hello, World] 
[Hello, World] 
After clear: 
[] 
[] 
[Hello, World] 

Так что unmodifiableCopy не изменен и не может быть изменен, если вы не можете каким-то образом убедить unmodifiableList, чтобы дать вам ссылку на делегата.

Поскольку вы не можете изменить его, вам не нужно делать копию того же типа конкретного списка, что и список ввода. Просто сделайте его ArrayList, так как он имеет в значительной степени лучшую производительность чтения, которая может быть достигнута с помощью реализации .

Так что ваш метод, без предупреждения и дополнительных классов, могут быть:

public static <T> List<T> unmodifiableListBasedOnCloneOf(List<T> elements) { 
    return Collections.unmodifiableList(new ArrayList<>(elements)); 
} 

Answer 2

Не всегда можно избежать аннотации @SuppressWarnings. Я пытаюсь ограничить область действия локальной переменной.

Я пришел с этим:

private interface Generator<T> { 

    T generateNewInstance(); 
} 

private static class ListGenerator<T> implements Generator<List<T>> { 
    @Override 
    public List<T> generateNewInstance() { 
     return new ArrayList<>(); 
    } 
}; 

public static <T> List<T> unmodifiableListBasedOnCloneOf(List<T> elements) { 
    @SuppressWarnings("unchecked") 
    Class<List<T>> clazz = (Class<List<T>>)elements.getClass(); 
    List<T> newInstance = makeNewInstanceOf(clazz , new ListGenerator<T>()); 
    newInstance.addAll(elements); 
    return Collections.unmodifiableList(newInstance); 
} 

private static <T> T makeNewInstanceOf(Class<T> incomingClass, Generator<T> fallbackGenerator) { 
    try { 
     return (T) incomingClass.newInstance(); 
    } catch (IllegalAccessException | InstantiationException e) { 
     return fallbackGenerator.generateNewInstance(); 
    } 
}