3

Estou estudando para o exame de certificação Java Programmer SE 7 I.

Utilizo o SelfTest Kaplan (recomendado pela própria Oracle).

Me deparei com a seguinte pergunta: (vou deixar tudo em inglês propositalmente. Para não traduzir e acabar omitindo informação).

Given:

public class Pedometer {
     private double stride;
     private double[] measurements;
}

Which code fragment is a method that meets good encapsulation principles?

a) public void getStride(double stride) {
    stride = this.stride;
} 

b) public void getStride(double stride) {
    this.stride = stride;
} 

c) public double[] getMeasurements() {
    return this.measurements;
} 

d) public double[] getMeasurements() {
    return measurements.clone();
}

Repsondi a opção c, embora tenha achado a opção d bem interessante.

Quando fui ver o resultado vi que de fato, a opção correta era a d.

Por que?

Não estou questionando o funcionamento do método clone() e nem questionando que clonar um objeto seja "melhor", mas questionando: por que essa implementação não é comum?

Já trabalhei em diversos projetos Java e nunca vi um getter assim.

O Eclipse, por exemplo, não gera os getter s dessa forma (ggas).

Nem mesmo o lombok faz essa implementação.


Explicação da resposta pelo Kaplan:

Explanation: The following code fragment is a method that meets good encapsulation principles:

public double[] getMeasurements() {
    return measurements.clone(); 
}

In this code fragment, the accessor method uses the clone method to return a copy of the measurements field, rather than a reference to the field itself. Accessor methods should return copies of field objects. A copy of a field will prevent other classes from modifying fields directly and will require going through mutator methods.

The two versions of the accessor method getStride do not meet good encapsulation principles. Neither method returns the value of the stride field as expected, but instead modifies the stride field like a mutator method. Also, the statement stride = this.stride; overwrites the stride parameter, rather than assigning the stride parameter to the stride field as expected.

The accessor method getMeasurements that returns a direct reference to the measurements field object does not meet good encapsulation principles. Accessor methods should not return direct references to field objects. Direct access will allow other classes to modify fields directly without going through mutator methods.

12
  • 2
    Qual a sua dúvida? Acho que na resposta já está bem explicado o porquê de fazer dessa forma. Apesar de nunca ter visto também acredito que a explicação faz muito sentido
    – Sorack
    Commented 27/02/2017 às 16:42
  • 1
    @Sorack editei a pergunta e deixei a dúvida em negrito. Se essa é a forma correta de implementar, por que não implementamos assim? Commented 27/02/2017 às 16:46
  • 1
    Acho que este é um dos exemplos do que diferencia alguém que programa em Java e alguém que programa e é certificado. Tem coisas que a gente só aprende estudando para a certificação. Eu acho que faz muito sentido essa implementação, tanto que normalmente quando você importa classes de um WS, elas já vem com um setter e getter diferenciado para List, por exemplo.
    – Sorack
    Commented 27/02/2017 às 16:48
  • 1
    Entendo. É que por outro lado, se utilizarmos o clone() para todos os getter s de todas nossas classes, será criada uma quantidade enorme de objetos. Imagine que para cada getAtributo() seja criado um novo objeto? Quantas vezes fazemos isso em nossos sistemas? Várias! Commented 27/02/2017 às 16:52
  • 1
    Mas veja bem, isso não inclui tipos primitivos. Portanto não é pra tudo que isso é utilizado, entende? Eu acho que é um pouco questão de arquitetura também...
    – Sorack
    Commented 27/02/2017 às 16:53

2 Respostas 2

2

Me baseando nos comentários feitos na pergunta e em algumas pequisas, tomo como resposta que:

Fazer getters com clone() é de fato a melhor/correta implementação. Com tudo, é preciso analisar caso a caso pois pode-se acarretar na criação de vários objetos, podendo até sobrecarregar o Garbage Collector.

Fazer getters retornando a referência direta do atributo, segundo a literatura, não é uma boa prática de encapsulamento porque ao retornar a referência direta você permite que sejam feitas modificações no objeto através da variável atribuída com o retorno do getter.

2
  • O .Net também funciona dessa mesma forma? Commented 7/06/2018 às 21:22
  • @JedaiasRodrigues é uma boa pergunta. Eu acho que sim Commented 7/06/2018 às 21:37
-4

No caso é pq funciona como se fosse um backup. Não interfere no objeto principal e vc não enfrentará uma classe que no meio da interposição possa modificar o objeto principal.


Por exemplo, vc pode ter um objeto private String nome; O java não se importa que vc faça public String getNome(){

return this.nome +"outro nome"; } Ele vai deixar concatenar e de fato quebrará o princípio do encapsulamento.

É a mesma idéia do Object. Assing do JavaScript.

1
  • Como está escrito atualmente, sua resposta não é clara. Por favor, pode editar para adicionar mais detalhes, que ajudarão outras pessoas a entender como isso resolve a pergunta feita. Você pode encontrar mais informações sobre como escrever boas respostas na Central de Ajuda.
    – Comunidade Bot
    Commented 2/04/2022 às 17:43

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .