Упрощенный процесс для условной упаковки бина в Java

Question

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

Ниже вы можете найти разработанный Java-код. Существует ли значительно упрощенный альтернативный поток для этого кода?

void binpack() {
boolean set=false, cmap=false;
        while(set==false) {
            set=true;
            cmap=false;
            pack[k.cnt]=new bins(k.lim, k.ptim);
            for(int i=0;i<tot;i++) {           
                int s1,s2;
                if(jm[i][1]==0) {
                    s1=jm[i][0]; 
                    set=false;

                    else if(pack[k.cnt].pushk<k.lim) {
                        for(int j=0;j<pack[k.cnt].pushk;j++) {
                            s2=pack[k.cnt].binjm[j][0];
                            for (Iterator<Integer> g= list[s1].iterator(); g.hasNext();) {

    }
}

Заметки:

ptim -> настоящее время

Этим-> Время окончания

Нт -> предел

Answer 1

Пара вещей с моей головы:

1. Имена переменных должны быть более выразительными. Не сокращайте, особенно, чтобы не сохранять один или два символа (например, ptim, chk())
2. Поставьте пробелы после открытия и перед закрытием скобок, после и перед операторами и так далее. Звучит глупо, но вы будете удивлены, насколько легче будет читать ваш код.
3. Избегайте ненужных логических сравнений (например, if ( bool == false ) должно быть if ( !bool )
4. Избегайте однострочников ifс или whiles, используйте правильное {} блоки.
5. Не объявляйте заранее переменные, если это не нужно.
6. Вместо структуры двойного разрыва, которую вы имеете в своем вложенном цикле, break на этикетку.
7. И самое главное: включите некоторые из этих циклов в методы.

Answer 2

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

Во-вторых, вы уже слышали о java.util.Map?