search

Prof-Calebe / substituicao

Sistema de controle de Substituição de Professores
7 stars 18 forks source link

Aula.bateCom() #115

Open williamhjcho opened 8 years ago

williamhjcho commented 8 years ago 
if(this.diaDaSemana != outraAula.getDiaDaSemana()){
    return false;
}else if (this.bateHorarioCom(outraAula.getPeriodo())){
    return true;
}

return false;

reduzível para 

return this.diaDaSemana == outraAula.getDiaDaSemana()
        && this.bateHorarioCom(outraAula.getPeriodo());
calebepb commented 7 years ago

Foi confirmado que os testes existentes validam essa refatoração? Se não, construir testes para garantir o antes/depois.

ErikAoki commented 7 years ago

Existe um teste dentro dos pacotes de teste que testa esse método com um objeto teste, e ao reduzir o código do jeito que foi sugerido, o teste continua aprovando.

calebepb commented 7 years ago

Perfeito. Falta apenas fazer o Pull-Request #185 ser aprovado.

ErikAoki commented 7 years ago

Fiquei meio confuso, então eu faria uma branch issue-185 e faria o pull-request? E seria #185 ao invés de #115 mesmo?