привет. Этот кусок можно как то оптимизировать? Java 8

На основании чего сделан вывод, что данный кусок кода неоптимален?

теоретически, оптимален. Но может я чего то не заметил

Кто мешает сразу сабстринг от исходной строки в буфер класть?

Точнее вместо буфера вообще возвращать сабстринг

тоже верно

думаю так и сделаю

это какой шрифт?

JB Mono

Прохожу курс по спрингу. В данном случае упражнение на ManyToOne/OneToMany.
И вот препод говорит: нам надо инициализировать books. И легче всего это сделать через new HashSet<>(). Кстати, на ManyToMany тоже также говорил делать. Так вот, дальше он и говорит: иначе у вас будет NullPointerException. Есть, говорит, и другие способы, но этот самый простой.

@Entity
public class Publisher {

...
   @OneToMany
   @JoinColumn(name="publisher_id")
   private Set<Book> books = new HashSet<>();

Я, честно говоря, вообще не понял, что тут имелось в виду. Во-первых, почему тут будет NPE, если не инициализировать?
Во-вторых, что делает этот new HashSet<>(). Что это такое - вообще непонятно. Был бы признателен за помощь.

В Java есть объявление переменной, а есть инициализация. Пока переменная не проинициализирована, обращаться к ней нельзя

А почему это все про джойны? И почему с одной стороны? Вот где many, там надо инициализировать. А где one - там не надо.

one2many ты джойнишь в один объект много строк, в сет. many2one - у тебя много объектов будет ссылаться на один. Т.е. в конкретном одном объекте будет не массив в поле, а единичное значение

Да просто, обычно, в коде к коллекциям обращение идёт в виде publisher.getBooks().add(someBook); Если ты коллекцию не инициализируешь, то везде в коде придётся вставлять проверки, что books вообще не null, прежде чем книжки добавлять / читать. Это можно, просто не очень удобно.

да, похоже на то. Возьму это себе на заметку.

Посоны, налетай😂

Некогда, надо жабу учить

@dmsol тут спам

Не очень подходящая группа для подобных заявлений -_-

Я лучше почитаю чего