Странные методы кодирования В java

Question

Я столкнулся с двумя образцами кода из унаследованной системы, в которых я не понимаю, почему кто-то будет писать код таким образом. Приложение находится на Java и составляет около 10-15 лет.

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

if(condition) { 
    String[] hdtTmp = { "Range (Brand):", "Build Region:", "Design:", "Plan (Size):", "Facade:", "Region:" , "Internal Colour", "External Colour"}; 
    hdt = hdtTmp; 

    String[] hddTmp = { p.RangeName, brName, p.HomeName, p.Name, f.Name, "North", "Red", "Blue"};       
    hdd = hddTmp; 

    hddTmp = null; 
    hdtTmp = null; 
} 

Я не понимаю, почему вы не просто присвоили его атрибуту в первую очередь? А так как hdtTmp и hddTmp находятся внутри блока, почему они делают null?

max = hdt.length -1; 

for(int i=0; ; i++) { 
    // do some stuff here 
    if(i == max) 
     break; 
} 

Опять же, кажется, что первоначальный программист не знал, как работать с петлями?

Они никогда не учили этому, когда я делал свою степень, поэтому мой вопрос: зачем кому-то писать такой код?

Answer 1

Установка локальных переменных на нуль ... Ну, еще в 90-х годах в какой-то старой документации Sun говорилось, что это поможет, но это было так долго и эта информация, вероятно, больше не доступна, поскольку она больше не верна. Я столкнулся с этим в большом количестве старого кода, и в этот момент он ничего не делает, поскольку локальные переменные теряют ссылку на объект, как только метод выходит, и GC достаточно умен, чтобы понять это.

Вопрос, связанный с циклом for(), связан с тем, что кто-то строит цикл с критерием выхода внутри фактического цикла. Это просто неудачный стиль кодирования и, вероятно, написанный относительно младшим разработчиком.

Этот код определенно кажется кому-то, кодирующим java после изучения C/C++. Я видел достаточно этого, люди C/C++ были обучены очистке после выделения, а присвоение нулевому - что-то, что сделало их счастливыми (по крайней мере, те, с которыми я работал в тот же день). Еще одна вещь, которую они сделали, - это переопределить метод finalize, который является плохой формой в java, но был самым близким к деструкторам.

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

for (;;) { 
    // Do stuff 
    if (something_happened) 
     break; 
} 

Старые (плохие) привычки умирают с трудом.

Answer 2

Существует узкий прецедент для обнуления переменных в общем случае.

Если код между назначением null и окончанием области переменной (часто конец метода) длительный, то обнуление дает более раннюю возможность GC объекта, на который ранее ссылались.

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

Эти «преимущества» довольно слабы.

Код в вопросе, однако, совершенно бесполезен.