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 themeasurements
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 thestride
field as expected, but instead modifies thestride
field like a mutator method. Also, the statementstride = this.stride;
overwrites thestride
parameter, rather than assigning thestride
parameter to thestride
field as expected.The accessor method
getMeasurements
that returns a direct reference to themeasurements
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.
setter
egetter
diferenciado paraList
, por exemplo.clone()
para todos os getter s de todas nossas classes, será criada uma quantidade enorme de objetos. Imagine que para cadagetAtributo()
seja criado um novo objeto? Quantas vezes fazemos isso em nossos sistemas? Várias!