Conversation
ElyonaySouza
requested review from
eduardodsr,
erickrabello57,
gabrielvf64 and
gardmaster
| this.booksIds = booksIds; | ||
| this.loanDate = LocalDate.now(); | ||
| this.expectedReturnDate = loanDate.plusDays(3); | ||
| this.expectedReturnDate = this.loanDate.plusDays(3); |
There was a problem hiding this comment.
Que diferença fez o this e o Builder do topo? Tentou eliminar esse construtor pra fazer que nem fez com livro?
There was a problem hiding this comment.
É tipo uma ênfase pra evitar ambiguidade. As vezes eu gosto de colocar, pois o this vai driblar algum problema com variáveis ou parâmetros com o mesmo nome. Como tenho visto muitos conflitos aqui no Git, fiquei com medo de algo desse tipo.
There was a problem hiding this comment.
Isso é mais em javascript. É uma alteração única sendo que em todo projeto não tá nesse padrão e os objetos estão sendo construidos sem problema.
Pra embasar mais ainda, você está com medo de conflitos no git, essa alteração do this vai justamente causar conflito em todas as branchs já abertas porque está alterando algo que já existe.
Reverte pra ser coisa a menos pra ver.
E quanto ao Builder e tentar replicar o que fez no book?
| @@ -39,11 +39,4 @@ public class Book { | |||
| @Enumerated(EnumType.STRING) | |||
| private BookStatus status = BookStatus.AVAILABLE; | |||
There was a problem hiding this comment.
Fala jovem, estou levando uma exception ao criar o livro, aparentemente está tentando colocar null no status.
Acredito que esteja faltando a anotação para dizer que o status é aquele padrão available.
Se eu não me engano a annotation é "@Builder.Default". Depois dá um confere ae pfv.
3 participants
Foi necessário dar uma mexida no BookService e eu aproveitei e optei por essa forma maior de dispor dos argumentos. Acredito que fique mais legível e fluido no entendimento.