Bir sınıftaki özel alanın değiştirilmesini nasıl önleyebilirim?

Bu sınıfa sahip olduğumu düşünün:

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

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

Şimdi, yukarıdaki sınıfı kullanan başka bir sınıf var:

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

İşte sorun şu: Dışarıdan bir sınıfın özel alanına eriştim! Bunu nasıl önleyebilirim? Yani bu diziyi değişmez hale nasıl getirebilirim? Bu, her alıcı yöntemiyle özel alana erişmek için çalışabileceğiniz anlamına mı geliyor? (Guava gibi herhangi bir kütüphane istemiyorum. Bunu yapmanın doğru yolunu bilmem yeterli).

Dizinizin bir kopyasını döndürmelisiniz .

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

Dizi yerine bir Liste kullanabiliyorsanız Koleksiyonlar değiştirilemez bir liste sağlar :

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

niteleyici private yalnızca alanın kendisine diğer sınıflardan erişilmesini engeller, ancak bu alanın nesne referanslarını korumaz. Başvurulan nesneyi korumanız gerekiyorsa, vermeyin. Değişiklik

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

için:

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

veya

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

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

Collections.unmodifiableListDaha önce de belirtildiği - Arrays.asList()garip değil! Benim çözüm de listeyi dışarıdan kullanmak ve diziyi aşağıdaki gibi sarmak olacaktır:

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

Diziyi kopyalamakla ilgili sorun şudur: koda her eriştiğinizde ve dizi büyük olduğunda yapıyorsanız, çöp toplayıcı için çok fazla iş yaratabilirsiniz. Yani kopya basit ama gerçekten kötü bir yaklaşım - "ucuz" diyebilirim, ama hafıza pahalı! Özellikle 2'den fazla elemana sahip olduğunuzda.

Eğer kaynak koduna bakarsanız Arrays.asListve Collections.unmodifiableListaslında çok fazla yaratılmamışsa. Birincisi diziyi kopyalamadan sarar, ikincisi listeyi sarar ve değişiklik yapamaz.

ImmutableListStandarttan daha iyi olanı da kullanabilirsiniz unmodifiableList. Sınıf Guava'nın bir parçasıdır Google tarafından oluşturulan kütüphanelerinin bir .

İşte açıklama:

Hala değişebilen ayrı bir koleksiyonun görünümü olan Collections.unmodifiableList'in (java.util.List) aksine, bir ImmutableList örneği kendi özel verilerini içerir ve asla değişmez

İşte nasıl kullanılacağına dair basit bir örnek:

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

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

bu bakışta sistem dizisi kopyasını kullanmalısınız:

public String[] getArr() {
   if (arr != null) {
      String[] arrcpy = new String[arr.length];
      System.arraycopy(arr, 0, arrcpy, 0, arr.length);
      return arrcpy;
   } else
      return null;
   }
}

Verilerin bir kopyasını döndürebilirsiniz. Verileri değiştirmeyi seçen arayan yalnızca kopyayı değiştirir

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

    public String[] getArr() {

        String[] b = new String[arr.length];

        System.arraycopy(arr, 0, b, 0, arr.length);

        return b;
    }
}

Sorunun özü, değiştirilebilir bir nesneye bir işaretçi döndürüyor olmasıdır. Hata. Ya nesneyi değişmez kılarsınız (değiştirilemez liste çözümü) ya da nesnenin bir kopyasını döndürürsünüz.

Genel bir konu olarak, nesnelerin kesinliği, nesneleri değiştirilebildiğinde değiştirilmekten korumaz. Bu iki sorun "kuzenleri öpmek" tir.

Değiştirilemez bir listenin geri gönderilmesi iyi bir fikirdir. Ancak, getter yöntemine yapılan çağrı sırasında değiştirilemeyen bir liste yine de sınıf veya sınıftan türetilen sınıflar tarafından değiştirilebilir.

Bunun yerine, sınıfı genişleten herkesin listenin değiştirilmemesi gerektiğini açıkça belirtmelisiniz.

Yani örneğinizde aşağıdaki koda yol açabilir:

import java.util.Arrays;
import java.util.Collections;
import java.util.List;

public class Test {
    public static final List<String> STRINGS =
        Collections.unmodifiableList(
            Arrays.asList("1", "2"));

    public final List<String> getStrings() {
        return STRINGS;
    }
}

Yukarıdaki örnekte STRINGS alanı herkese açık , prensip olarak, değerler zaten bilindiği için yöntem çağrısını ortadan kaldırabilirsiniz.

Ayrıca dizeleri bir private final List<String> , sınıf örneğinin oluşturulması sırasında değiştirilemez hale alana . Sabit veya örnekleme bağımsız değişkenlerinin kullanılması (yapıcıdan) sınıfın tasarımına bağlıdır.

import java.util.Arrays;
import java.util.Collections;
import java.util.List;

public class Test {
    private final List<String> strings;

    public Test(final String ... strings) {
        this.strings = Collections.unmodifiableList(Arrays
                .asList(strings));
    }

    public final List<String> getStrings() {
        return strings;
    }
}

Evet, dizinin bir kopyasını döndürmelisiniz:

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