В чём разница? Почему подчёркивает красным?

Привет! Давай чуток разберем на примерах эту непонятку. ) Integer.valueOf(5) Это статический фабричный метод, который возвращает объект Integer, представляющий указанное значение. Для значений от -128 до 127 (по умолчанию) Java использует пул объектов (IntegerCache). Если запрашиваемое значение находится в этом диапазоне, valueOf возвращает уже существующий объект из кэша, а не создает новый. Это экономит память и улучшает производительность. new Integer(5) Это конструктор, который всегда создает новый объект Integer в куче, независимо от значения. Тут нет кэширования. Даже если значение находится в диапазоне -128 до 127, каждый вызов new Integer(5) создает новый объект, что увеличивает потребление памяти. Создание такого нового объекта каждый раз менее оптимально с точки зрения производительности и памяти. По итогу: При использовании valueOf для значений в диапазоне -128 до 127, объекты будут одинаковыми (по ссылке), так как они берутся из кэша. Например:

Integer a = Integer.valueOf(5);
Integer b = Integer.valueOf(5);
System.out.println(a == b); // true (из кэша)
System.out.println(a.equals(b)); // true (значения равны)

При использовании new Integer, объекты всегда разные по ссылке, даже если значения одинаковы. Например:

Integer c = new Integer(5);
Integer d = new Integer(5);
System.out.println(c == d); // false (разные объекты)
System.out.println(c.equals(d)); // true (значения равны)

При автоупаковке (например, Integer a = 5;) компилятор автоматически использует Integer.valueOf(5), а не new Integer(5). Это означает, что автоупаковка также задействует кэш для значений от -128 до 127. Integer a = 5; // Эквивалентно Integer.valueOf(5) Integer b = 5; // Эквивалентно Integer.valueOf(5) System.out.println(a == b); // true (из кэша)

Этот конструктор считается устаревшим еще с 9й Java и помечен к удалению, если наведёшь на подчёркивание скорее всего сможешь это прочесть. Устаревшими методами пользоваться не рекомендуют. Ну и в самом javadoc к этому методу написано примерно такое:

/**
* @deprecated
* It is rarely appropriate to use this constructor.
* Use {@link #parseFloat(String)} to convert a string to a
* {@code float} primitive, or use {@link #valueOf(String)}
* to convert a string to a {@code Float} object.
*/

В условии просят создавать через метод valueOf(), а не через конструктор, может по этому? хороший плагин SonarQube для IDE, очень здорово подсказывает про устаревшие методы