• У генератора должно быть достаточно один раз вызвать метод generate. Он вообще может быть статическим.

• Примитивные объекты передавать в интерфейс как final - ненужное ограничение на реализацию. Вызывающая сторона всё равно не увидит результата изменения переданной копии

https://github.com/DavidArutiunian/OOP/blob/c1e06a62fcb14035484fb57abd5cadf26f941d69/src/main/java/labs/lab2/mini_dictionary/Dictionary.java#L9-L22

• Интерфейс слишком широкий. Многие клиенты используют лишь его часть. Читайте про Interface Segregation Principle (Принцип разделения интерфейса)

• Зачем наружу выставять кишки в виде map?

• Это словарь. Никаких key и value

• Метод clear не используется нигде, кроме тестов. Выпилить.

Читайте принципы SOLID.

• Программа падает при следующем вводе:

let x0=1
let x1=2

• Сделать, чтобы компаратор принимал не-null-able элементы. Проверки реализовать внутри самой функции findMax

• Итератор нарушает инкапсуляцию. Он должен возвращать значения типа String, а не StringNode.

• Итератор не должен позволять классам вне своего пакет получать доступ к StringNode

• В методе setUp лучше создавать новый автомобиль каждый раз
• Текущая реализация метод setUp не гарантирует, что перед каждым тестом автомобиль будет в одном и том же состоянии

• Не реализованы операции вычитания, сложения, умножения и деления, в которых левым аргументом выступает double, а правым - complex