Java - модификация локальных переменных

Question

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

class A{ 
    void doSth(int[] array){  
     int index1, index2, index3; 
     int value1, value2, value3;  

     if(array[index1] > 10){  
      //Long code modifies value1, value2, value3 
     } 

     if(array[index3] > 100){  
      //Same long code modifies value1, value2, value3 
     }  

     if(array[index2] > 20){  
      //Same long code modifies value1, value2, value3 
     }  
    } 

пренебрегая что это пытается достичь, я хотел бы, чтобы каким-то образом сделать это избыточность исчезнет. обычно, я передавал значения методу hlper, но я не могу, так как блок модифицирует локальные переменные. Любая идея, как это упростить?

Answer 1

Это звучит, как вы value1, value2 и value3, вероятно, имеют какое-то значение в сочетании. Поэтому инкапсулируйте их в отдельный класс, и в этот момент вы можете вызвать метод, который либо изменяет существующий экземпляр, либо возвращает новый экземпляр этого класса. В любом случае, с одной локальной переменной вы будете в порядке.

Answer 2

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

частное ИНТ значение1, значение2, value3;

private void doIt(int index, int threshold) { 
    if (array[index] <= threshold) 
     return; 
    ... //Same long code modifies value1, value2, value3 
} 

затем заменить основной код на это:

void doSth(int[] array, int index1, int index2, int index3) { 
    doIt(index1, 10); 
    doIt(index3, 100); 
    doIt(index2, 20); 
} 

И вы сделали.

Answer 3

Если коды в случае утверждения все точно так же, почему вы не просто делать это так:

class A{ 
    void doSth(int[] array){ 
     int value1, value2, value3; 
     int index[][] = { 
     {val1, 10}, 
     {val2, 100}, 
     {val3, 20} 
     }; 

     // ... 

     for(int i = 0; i < index.length; i++){ 
     if(array[index[i][0]] > index[i][1]){ 
      // ... Long code modifies value1, value2, value3 
     } 
     } 
    } 
}