Comment empêcher la modification d'un champ privé dans une classe ?

Question

Imaginez que j'ai cette classe :

public class Test
{
  private String[] arr = new String[]{"1","2"};    

  public String[] getArr() 
  {
    return arr;
  }
}

Maintenant, j'ai une autre classe qui utilise la classe ci-dessus :

Test test = new Test();
test.getArr()[0] ="some value!"; //!!!

Voici donc le problème : j'ai accédé à un champ privé d'une classe depuis l'extérieur ! Comment puis-je empêcher cela ? Je veux dire comment puis-je rendre ce tableau immuable ? Cela signifie-t-il qu'avec chaque méthode getter, vous pouvez vous frayer un chemin jusqu'à l'accès au champ privé ? (Je ne veux pas de bibliothèques telles que Guava. J'ai juste besoin de connaître la bonne façon de faire).

Answer 1

Si vous pouvez utiliser une liste au lieu d'un tableau, Collections fournit une liste non modifiable :

public List<String> getList() {
    return Collections.unmodifiableList(list);
}

Answer 2

Vous devez renvoyer un copie de votre tableau.

public String[] getArr() {
  return arr == null ? null : Arrays.copyOf(arr, arr.length);
}

Answer 3

Modificateur private protège uniquement le champ lui-même contre l'accès à partir d'autres classes, mais pas les objets référencés par ce champ. Si vous avez besoin de protéger l'objet référencé, il suffit de ne pas le donner. Modifier

public String [] getArr ()
{
    return arr;
}

à :

public String [] getArr ()
{
    return arr.clone ();
}

ou à

public int getArrLength ()
{
    return arr.length;
}

public String getArrElementAt (int index)
{
    return arr [index];
}

Answer 4

En Collections.unmodifiableList a déjà été mentionné - le Arrays.asList() étrangement non ! Ma solution serait également d'utiliser la liste de l'extérieur et d'envelopper le tableau comme suit :

String[] arr = new String[]{"1", "2"}; 
public List<String> getList() {
    return Collections.unmodifiableList(Arrays.asList(arr));
}

Le problème avec la copie du tableau est le suivant : si vous le faites à chaque fois que vous accédez au code et que le tableau est grand, vous allez certainement créer beaucoup de travail pour le ramasseur de déchets. La copie est donc une approche simple mais vraiment mauvaise - je dirais "bon marché", mais coûteuse en mémoire ! Surtout lorsque vous avez plus de deux éléments.

Si vous regardez le code source de Arrays.asList y Collections.unmodifiableList il n'y a en fait pas beaucoup de création. La première ne fait qu'envelopper le tableau sans le copier, la seconde n'enveloppe que la liste, rendant les modifications de celle-ci indisponibles.

Answer 5

Vous pouvez également utiliser ImmutableList ce qui devrait être meilleur que la norme unmodifiableList . Le cours fait partie de Goyave des bibliothèques qui ont été créées par Google.

Voici la description :

Contrairement à Collections.unmodifiableList(java.util.List), qui est une vue d'une collection séparée qui peut toujours changer, une instance de ImmutableList contient ses propres données privées et ne changera jamais.

Voici un exemple simple de son utilisation :

public class Test
{
  private String[] arr = new String[]{"1","2"};    

  public ImmutableList<String> getArr() 
  {
    return ImmutableList.copyOf(arr);
  }
}