Блог

Рассмотрим следующие два класса (один из них — метод Main with main() ):

Класс VO:

 public class TheVO {
    private String[] theValues = null;
    /**
     * 
     */
    public TheVO(String[] theParam) {
        this.theValues = theParam;
    }

    /**
     * 
     * @return
     */
    public String[] getValues(){
        return this.theValues;
    }

    @Override
    public String toString() {
        StringBuffer buf = new StringBuffer("");
        if(this.theValues == null){
            return buf.toString();
        }
        for(String read:this.theValues){
            buf.append(read);
            buf.append(" ");
        }
        return buf.toString().trim();
    }
}
  

Основной класс:

 public class Main {
    /**
     * 
     */
    public Main() {
        super();
    }

    /**
     * @param args
     */
    public static void main(String[] args) {
        TheVO theV = new TheVO(new String[]{"Hello","World!!"});
        String[] vale = theV.getValues();
        vale[0] = "Goodbye";
        System.out.println(theV);
    }
}
  

Результат после выполнения:

Прощай, мир!!

Вопрос:

Я знаю, что vale переменная массива ссылается на ту же переменную, которая анализируется в конструкторе, и если я изменяю один из индексов в массиве, он изменяет то же самое String[] в VO.

Как мне «исправить» или изменить класс TheVO, чтобы мой результат был?:

Привет, мир!!

Вам нужно использовать защитное копирование: скопируйте String[] в свой конструктор. Это гарантирует, что параметр не будет изменен после того, как он был передан вашему классу VO. Затем скопируйте String[] в средство получения, чтобы вызывающий средство получения не модифицировал ваш внутренний String[] . Самый простой способ скопировать arrray — вызвать clone :

 this.theValues = theParam.clone();
  

Если бы вы использовали коллекции вместо массивов, вы могли бы избавиться от защитной копии в получателе, обернув свою коллекцию с помощью Collections.unmodifiableList() вместо этого (что является намного более дешевой операцией):

 private List<String> theValues;

public List<String> getValues(){
        return Collections.unmodifiableList(this.theValues);
}
  

Однако вам все равно понадобится защитная копия в конструкторе ‘.

Вы можете попробовать изменить метод getValues();, чтобы возвращать копию массива tevo.theValues вместо ссылки на исходный массив.

Вы можете дублировать (или клонировать) свою строку[] в методе getValues().

Таким образом, создавая новый массив, вы теряете связь между массивами строк.

Я бы предложил выполнить копирование внутреннего массива при возврате из TheVO.getValues() . Если вы работаете на Java 1.6, вы можете использовать Arrays.copyOf() метод.

 public String[] getValues() {
    return Arrays.copyOf(theValues, theValues.length);
}
  

Обратите внимание, что здесь нет необходимости использовать this для доступа к полю экземпляра.

Создайте копию массива в вашем конструкторе VO

измените метод getValues(), чтобы клонировать массив…

Что-то вроде этого…

 public String[] getValues(){
    return this.theValues.clone();
}