Bagaimana saya mencegah modifikasi bidang pribadi di kelas?

Bayangkan saya memiliki kelas ini:

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

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

Sekarang, saya memiliki kelas lain yang menggunakan kelas di atas:

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

Jadi ini masalahnya: Saya telah mengakses bidang pribadi kelas dari luar! Bagaimana saya bisa mencegah ini? Maksud saya, bagaimana saya bisa membuat array ini tidak berubah? Apakah ini berarti bahwa dengan setiap metode pengambil Anda dapat bekerja dengan cara Anda untuk mengakses bidang pribadi? (Saya tidak ingin ada perpustakaan seperti Guava. Saya hanya perlu tahu cara yang tepat untuk melakukan ini).

Anda harus mengembalikan salinan array Anda.

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

Jika Anda dapat menggunakan Daftar alih-alih array, Koleksi menyediakan daftar yang tidak dapat dimodifikasi :

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

Pengubah privatehanya melindungi bidang itu sendiri dari akses dari kelas lain, tetapi bukan referensi objek oleh bidang ini. Jika Anda perlu melindungi objek yang dirujuk, tapi jangan berikan. Perubahan

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

untuk:

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

atau untuk

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

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

The Collections.unmodifiableListtelah disebutkan - yang Arrays.asList()anehnya tidak! Solusi saya juga menggunakan daftar dari luar dan membungkus array sebagai berikut:

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

Masalah dengan menyalin array adalah: jika Anda melakukannya setiap kali Anda mengakses kode dan array besar, Anda pasti akan membuat banyak pekerjaan untuk pemulung. Jadi salinannya adalah pendekatan yang sederhana tetapi sangat buruk - saya akan mengatakan "murah", tetapi memori mahal! Terutama ketika Anda memiliki lebih dari 2 elemen.

Jika Anda melihat kode sumber Arrays.asListdan Collections.unmodifiableListsebenarnya tidak banyak yang dibuat. Yang pertama hanya membungkus array tanpa menyalinnya, yang kedua hanya membungkus daftar, membuat perubahan itu tidak tersedia.

Anda juga bisa menggunakan ImmutableListmana yang harus lebih baik dari standar unmodifiableList. Kelas adalah bagian dari perpustakaan Guava yang dibuat oleh Google.

Berikut uraiannya:

Tidak seperti Collections.unmodifiableList (java.util.List), yang merupakan pandangan dari koleksi terpisah yang masih dapat berubah, turunan ImmutableList berisi data pribadi sendiri dan tidak akan pernah berubah

Berikut ini adalah contoh sederhana cara menggunakannya:

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

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

pada titik pandang ini Anda harus menggunakan salinan array sistem:

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;
   }
}

Anda dapat mengembalikan salinan data. Penelepon yang memilih untuk mengubah data hanya akan mengubah salinan

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;
    }
}

Inti masalahnya adalah Anda mengembalikan pointer ke objek yang bisa diubah. Ups. Entah Anda membuat objek tidak berubah (solusi daftar tidak dapat dimodifikasi) atau Anda mengembalikan salinan objek.

Secara umum, finalitas objek tidak melindungi objek dari perubahan jika mereka bisa berubah. Dua masalah ini adalah "mencium sepupu."

Mengembalikan daftar yang tidak dapat dimodifikasi adalah ide yang bagus. Tetapi daftar yang dibuat tidak dapat dimodifikasi selama panggilan ke metode pengambil masih dapat diubah oleh kelas, atau kelas yang diturunkan dari kelas.

Alih-alih, Anda harus menjelaskan kepada siapa pun yang memperluas kelas bahwa daftar tidak boleh dimodifikasi.

Jadi, dalam contoh Anda ini dapat mengarah ke kode berikut:

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;
    }
}

Dalam contoh di atas saya telah membuat STRINGSbidang publik, pada prinsipnya Anda bisa menghilangkan panggilan metode, karena nilai sudah diketahui.

Anda juga bisa menetapkan string ke private final List<String>bidang yang tidak dapat dimodifikasi selama konstruksi instance kelas. Menggunakan argumen konstanta atau instantiasi (dari konstruktor) tergantung pada desain kelas.

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;
    }
}

Ya, Anda harus mengembalikan salinan array:

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