luksfarris / MC851-Quack

Projeto final da disciplina de Projeto de Sistema de Informação da Unicamp
GNU General Public License v3.0
0 stars 0 forks source link

Bugs em MessageImpl.java #21

Closed JorgeStolfi closed 9 years ago

JorgeStolfi commented 9 years ago

O campo {body} de uma repostagem deveria ser {null}, acho. Afinal {getText} já segue recursivamente os links {parent} Talvez mais tarde esse campo seja usado para um comentário.

A {initialize} de repostagem tem uma recursão espúria. Não tem que inicializar o {parent}, ele já deve estar inicializado quando o método é chamado.

Em vez de ter um método {setId}, o parâmetro {id} ficaria melhor nos métodos {initialize}.

augustomorgan commented 9 years ago

A recursão de initialize é para que repostagens de outras repostagens sejam na verdade uma repostagens da mensagem original.

Note que não estamos re-inicializando o parent, mas inicializando a mensagem com o parent do parent.

Retiramos a recursão para melhor entendimento.