Дз № 4 + исправления по дз № 3 + spring + ДЗ № 5 #6
Conversation
|
Изначально исправления по предыдущему дз были в прошлом PR (hw-3), перенесла их сюда. |
Ellena557
changed the title
Ellena557
changed the title
| import java.util.HashMap; | ||
| import java.util.Map; | ||
|
|
||
| public class SensorEventAdapter extends SensorEvent { |
There was a problem hiding this comment.
Нужно сделать через отдельный класс, который принимает на вход событие библиотечного типа и возвращает событие "нашего" типа.
Наследование здесь не нужно.
|
|
||
| for (SensorEventProcessor eventProcessor : eventProcessors) { | ||
| EventHandler handler = new EventProcessorAdapter(eventProcessor); | ||
| ((ArrayList<EventHandler>) handlers).add(handler); |
There was a problem hiding this comment.
зачем cast к ArrayList - в Collection есть метод add
There was a problem hiding this comment.
У меня там раньше немного другое было написано, каст идея сгенерировала, потом я поменяла строчку, а каст остался:(
| public void start() { | ||
| this.eventsManager.registerEventHandler( | ||
| stringEvent -> { | ||
| toString(stringEvent.getEventType(), stringEvent.getObjectId()); |
There was a problem hiding this comment.
Здесь не понял вообще что происходит. Зачем это?
There was a problem hiding this comment.
Моя ошибка, там в Sample приведена строчка, которую выводить надо, a я написала то, как это в SensorCommand было.
| public SensorEventsManagerAdapter(List<SensorEventProcessor> eventProcessors) { | ||
| this.eventsManager = new SensorEventsManager(); | ||
|
|
||
| Collection<EventHandler> handlers = new ArrayList<>(); |
There was a problem hiding this comment.
Почему вообще не сделать просто:
eventProcessors.each(processor -> registerEventProcessor(processor))
?
| public class RemoteControlApp implements RemoteControl { | ||
|
|
||
| private HashMap<String, Command> buttonsVocab; | ||
| private RemoteControlRegistry remoteControlRegistry; |
There was a problem hiding this comment.
Зачем нужно это поле в этом классе?
There was a problem hiding this comment.
Не нужно, не успела убрать.
No description provided.