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 - >現時点

エティム - >終了時間

LIM - >リミット

Answer 1

の上から物事のカップル私の頭：

1. 可変名より表現力豊かでなければならない。省略しないでください。特に1,2文字を保存しないようにしてください（例：ptim、chk()）
2. 括弧を閉じる前、括弧を閉じる前、演算子の前後にスペースを入れてください。それはうんざりしているように聞こえるが、あなたのコードがどれだけ読みやすくなるかは驚かれるだろう。
3. 例えばif (bool == false)がif (!bool)
4. が適切{}ブロックを使用し、ワンライナーif Sまたはwhileのを避けてください。
5. あなたがする必要がない場合はありません事前に宣言した変数を実行しなければなりません（不要なブール値の比較を避けてください。
ラベルの代わりに、あなたは、ネストされたループを持っているダブルブレイク構造の
6. 、break
7. そして、すべてのほとんど：要因これらのループのいくつかの実施方法に

Answer 2

まず、入力を省くために変数を省略しないでください。これは、そのような不可能なコードを難読化することにつながります。

第2に、既にjava.util.Mapについて聞いたことがありますか？