search

Cha0z / springBootHomeWork

0 stars 1 forks source link

Code review #7 Patterns #7

Open hjvfyfyfy opened 5 years ago

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/ab8b89c6eb4ab7e71589f8546f5117544bdcbb76/src/main/java/com/homework/home/pattern/builder/MessageBuilder.java#L7

Це не є реалізацією патерна Builder Переглянь https://softwareengineering.stackexchange.com/questions/305504/is-stringbuilder-an-application-of-the-builder-design-pattern

https://refactoring.guru/uk/design-patterns/builder

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/ab8b89c6eb4ab7e71589f8546f5117544bdcbb76/src/main/java/com/homework/home/pattern/factory/model/AlertMessage.java#L10

https://github.com/Cha0z/springBootHomeWork/blob/ab8b89c6eb4ab7e71589f8546f5117544bdcbb76/src/main/java/com/homework/home/pattern/factory/model/InfoMessage.java#L7

В одному випадку є extends Message в іншому немає, тут є якась логіка ?

hjvfyfyfy commented 5 years ago

https://github.com/Cha0z/springBootHomeWork/blob/ab8b89c6eb4ab7e71589f8546f5117544bdcbb76/src/main/java/com/homework/home/pattern/mediator/MainPage.java#L25-L29

ну думаю що ініціалізацію треба робити в медіаторі, тут її не місце сам клас медіатора за звичай і так великий

hjvfyfyfy commented 5 years ago

middleware зазвичай називають chain of responsibility фільтри по такому прикладу працюють