Геттеры и сеттеры для массивов

У меня есть несколько вопросов о геттерах и сеттерах для массивов. Предположим, что у нас есть такой класс, который делает частную копию массива в его конструкторе:

import java.util.Arrays;

public class Foo
{
    private int[] array;

    public Foo(int[] array) {
        this.array = Arrays.copyOf(array, array.length);
    }
}

Мы хотим, чтобы массив был доступен или мутировал только через getters и seters. Если у нас есть геттер, который выглядит так:

public int[] getArray() {
    return array;
}

он побеждает цель геттера, когда мы возвращаем ссылку, которая позволяет пользователю напрямую изменять элементы массива. например.

Foo foo = new Foo(someArray);
...
int[] bar = foo.getArray();
bar[0] = 123; // Now foo.array[0] = 123 and we haven't used a setter!

Итак, предположим, нам нужно что-то вроде этого:

public int getElement(int index) {
    return array[index];
}

Аналогично для сеттеров. Но если мы делаем вещи на основе каждого элемента, нам также нужно будет предоставить средства для получения длины:

public int getArrayLength() {
    return array.length;
}

Это уже немного беспорядочно для 1-мерного массива, но, скажем, у нас есть многомерный массив:

import java.util.Arrays;

public class Foo
{
    private int[][][] array;

    public Foo(int[][][] array) {
        // Code for making a deep copy here
    }

    public int getElement(int i, int j, int k) {
        return array[i][j][k];
    }

    public void setElement(int i, int j, int k, int value) {
        array[i][j][k] = value;
    }

    public int getArrayLength() {
        return array.length;
    }

    public int getArrayLength(int i) {
        return array[i].length;
    }

    public int getArrayLength(int i, int j) {
        return array[i][j].length;
    }
}

Это много кода для такой тривиальной задачи, и, что более важно, это беспорядок на самом деле. Неужели нам действительно нужно что-то в этом роде, или есть лучший способ сделать это? Я смотрел повсюду, и для этого не существует "стандартной практики".

Ответы

Ответ 1

Многомерный массив также является одномерным массивом: int[a][b][c] действительно просто int[a*b*c], поэтому проблема сводится к тому, как вы обеспечиваете доступ безопасно? Просто так:

public class Foo {
    private int[] array;

    public Foo(int[] array) {
        this.array = Arrays.copyOf(array, array.length);
    }

    /** @return a copy of the array */
    public int[] getArray() {
        return Arrays.copyOf(array, array.length);
    }
}

Что это.

У вызывающих абонентов есть безопасная копия массива и может использоваться в полном режиме. Нет необходимости в методах делегирования.

Ответ 2

Как вы думаете, ArrayList делает, или Vector перед этим?

Я думаю, что лучший вопрос заключается в том, почему на данный момент вы показываете, что Foo поддерживается массивом вообще? Если вы пытаетесь инкапсулировать его, вам не нужно иметь аксессоров и сеттеров повсюду. Если вы просто пытаетесь создать оболочку класса вокруг массива, тогда я предлагаю вам иметь интерфейс, называть его IntList или что-то еще и сделать Foo конкретную реализацию, поддерживаемую списком.

Ответ 3

В отношении первой части ваш получатель не может выглядеть как конструктор?

public int[] getArray() {
    return Arrays.copyOf(array, array.length);
}

Ответ 4

Я написал небольшой API для многомерных общих массивов. Там у вас есть геттеры, сеттеры для каждого элемента, независимо от ваших размеров.

MDDAJ на github

Вот пример: создание 5-мерного массива строк:

MDDAPseudo<String> c = new MDDAPseudo<String>(10,20,5,8,15);
c.set("xyz", 0,0,2,1,0); // setter indices: [0][0][2][1][0]
String s = c.get(0,0,0,0,0); // getter indices [0][0][0][0][0]

Bohemian уже написал, что вы можете использовать только одно измерение. В этом случае класс PDDAPSeudo также имеет одно измерение. Но API предоставляет вам доступ, как многомерный массив