Alguém pode me dizer se existe algum problema com a classe abaixo? O FindBugs está apontando os seguintes problemas: Method may expose internal representation by returning reference to mutable object (getDataCadSenha) …

Problemas com métodos

1 de julho de 2004 2 respostas

JonhyBoy 1 de julho de 2004

Alguém pode me dizer se existe algum problema com a classe abaixo? O FindBugs está apontando os seguintes problemas:

Method may expose internal representation by returning reference to mutable object (getDataCadSenha)
Method may expose internal representation by incorporating reference to mutable object (setDataCadSenha)

public class Usuario implements Serializable &#123;
    ...
    private Date dataCadSenha;

    public Date getDataCadSenha&#40;&#41; &#123;
        return dataCadSenha;
    &#125;

    public void setDataCadSenha&#40;Date dataCadSenha&#41; &#123;
        this.dataCadSenha = dataCadSenha;
    &#125;
&#125;

2 Respostas

Rafael_Steil 1 de jul. de 2004

Nao eh bug, apenas um alerta de falta de imutabilidade do teu objeto.

Vc tem um membro privado do tipo Date… ao prover um getDate(), que retorna o membro privado diretamente, voce esta possibilitando que alguem altere a representacao interna do mesmo. Por exemplo.

obj.getDataCadSenha&#40;&#41;.setMont&#40;11&#41;;

ira alterar o mes de cadastro da senha.

Em relacao ao setXxx() eh o mesmo lance.

A ideia eh que, ao inves de retornar direto o membro date, voce retornasse uma copia “read-only” / nao-orignal dele ( crie uma nova chamando clone(), nesse caso, por exemplo ), impedindo assim que a data seja alterada por meios “indevidos”…

Mas isso nao significa que a tua classe esta necessariamente bugada / errada. Eh mais um conceito de “boas praticas”.

Rafael

JonhyBoy 1 de jul. de 2004

Obrigado pelas explicações.

Valeu mesmo.

Criado 1 de julho de 2004

Ultima resposta 1 de jul. de 2004

Respostas 2

Participantes 2

2 Respostas

Topicos relacionados