条件にある要素のみリストに追加したいのですが、すべて追加されます。何が間違っているのでしょうか。

Question

別のクラスの変数をこちらのクラスのリストに追加していくのですが、指定の重さに達するとリストへの追加しないようにしたいのですが、pu、blic void addTavaraメソッドで間違ってますでしょうか。今の状態ですと、重さを超えてもすべて追加されます。

import java.util.ArrayList;
public class Matkalaukku {
    private final int maksimipaino;
    private final ArrayList<Tavara> tavarat;
    public Matkalaukku(int maksimipaino){
        this.maksimipaino = maksimipaino;
        this.tavarat = new ArrayList<>();
    }
    public void addTavara(Tavara tavara){
        while(true){
            int omosa = 0;
            omosa = omosa + tavara.getPaino();
            if(omosa <= this.maksimipaino){//ここで追加されるのはthis.maksimipaino
                 this.tavarat.add(tavara);//以下にしたつもりですが、すべて追加されます。
                break<
            }
            
        }
    }
    @Override
    public String toString(){
        if(this.tavarat.isEmpty()){
                return "ei tavaroita (0 kg)";
            }
        
        int count = 0;
        int omosa = 0;
        for (Tavara tavara: tavarat) {
            count++;
            omosa = omosa + tavara.getPaino();                  
        }
       return count +" tavaraa ("+omosa+"kg)";
        
        
    }
}
---
public class Tavara {
    private String nimi;
    private int paino;

    public Tavara(String nimi, int paino){
        this.nimi = nimi;
        this.paino = paino;
    }
    public String getNimi(){
        return this.nimi;
    }
    public int getPaino(){
        return this.paino;
    }
    public String toString(){
        return this.nimi+ "("+this.paino+"kg)";
    }
}
---
public class Main {

    public static void main(String[] args) {
        Tavara kirja = new Tavara("Aapiskukko", 2);
        Tavara puhelin = new Tavara("Nokia 3210", 1);
        Tavara tiiliskivi = new Tavara("tiiliskivi", 4);

        Matkalaukku matkalaukku = new Matkalaukku(5);
        System.out.println(matkalaukku);

        matkalaukku.lisaaTavara(kirja);
        System.out.println(matkalaukku);

        matkalaukku.lisaaTavara(puhelin);
        System.out.println(matkalaukku);

        matkalaukku.lisaaTavara(tiiliskivi);
        System.out.println(matkalaukku);
    }

}

Comments

[swordone]

まずコンパイルが通るコードを書いてください。

Answer 1

変数名やメソッド名が意味不明なので、質問で使っている「重さ」で表現を統一します。
クラス名やメソッド名、変数名は意味が分かるようにしてください。

    public void addTavara(Tavara tavara){
        while(true){
            int omosa = 0;
            omosa = omosa + tavara.getPaino();
            if(omosa <= this.maksimipaino){//ここで追加されるのはthis.maksimipaino
                 this.tavarat.add(tavara);//以下にしたつもりですが、すべて追加されます。
                break<
            }

        }

ifで判定する時点の重さは、必ず引数の重さになります。
結局比較するのは、引数の重さと制限の重さとなり、あなたがやろうとしている「重さの合計の制限」ができません。
これをやるためには、追加しようとするたびに現在の合計の重さを計算するか、現在の重さの合計を保存しておくフィールド変数が必要です。

public class Matkalaukku {
    private final int maksimipaino;
    private final ArrayList<Tavara> tavarat;
+   private int totalWeight = 0;  //「現在の重さの合計」を管理するフィールド変数
    public Matkalaukku(int maksimipaino){
        this.maksimipaino = maksimipaino;
        this.tavarat = new ArrayList<>();
    }
    public void addTavara(Tavara tavara){
-        while(true){
-           int omosa = 0;
-           omosa = omosa + tavara.getPaino();
-           if(omosa <= this.maksimipaino){//ここで追加されるのはthis.maksimipaino
+           if(totalWeight + tavara.getPaino() <= this.maksimipaino) {
                 this.tavarat.add(tavara);
+                totalWeight += tavara.getPaino()
-                break<
            }

-       }
    }

Comments

[Chibichan]

ありがとうございました！ようやく謎が解けました！

Answer 2

      if(omosa <= this.maksimipaino){//ここで追加されるのはthis.maksimipaino
        break;  //以下にしたつもりですが、すべて追加されます。
        this.tavarat.add(tavara);
      }

this.tavarat.add(tavara);には到達しない、というコンパイルエラーがでているのではありませんか。

ま、break文をみなかったことにすれば、tavara.getPaino()が、0か0以下の値を返してるのではなかろうか、と。

Tavaraクラスの内容も含めたちゃんとコンパイルできて動くソースを提示してみてください。

---

---

Tavaraクラスを下に追加しました。ご指摘お願いします。

ああ。while(true),breakが意味不明だったのですが、omosaを定義する場所がおかしいでしょう。
whileの中で定義したら、というか、addTavaraメソッドの中で定義したら、omosaは毎回0から開始。tavara.getPaino()がどんな値を返すか想像できませんが、if文はたいてい真になるのでしょうね。だから。

diff --git a/src/main/java/Matkalaukku.java b/src/main/java/Matkalaukku.java
index f2f792d..4789dc6 100644
--- a/src/main/java/Matkalaukku.java
+++ b/src/main/java/Matkalaukku.java
@@ -4,6 +4,7 @@ import java.util.ArrayList;
 public class Matkalaukku {
   private final int maksimipaino;
   private final ArrayList<Tavara> tavarat;
+  private int omosa = 0;
 
   public Matkalaukku(int maksimipaino) {
     this.maksimipaino = maksimipaino;
@@ -11,13 +12,9 @@ public class Matkalaukku {
   }
 
   public void addTavara(Tavara tavara) {
-    while (true) {
-      int omosa = 0;
-      omosa = omosa + tavara.getPaino();
-      if (omosa <= this.maksimipaino) {//ここで追加されるのはthis.maksimipaino
-        this.tavarat.add(tavara);//以下にしたつもりですが、すべて追加されます。
-        break;
-      }
+    omosa = omosa + tavara.getPaino();
+    if (omosa <= this.maksimipaino) {//ここで追加されるのはthis.maksimipaino
+      this.tavarat.add(tavara);//以下にしたつもりですが、すべて追加されます。
     }
   }

Comments

[Chibichan]

Tavaraクラスを下に追加しました。ご指摘お願いします。

[Chibichan]

Mainクラスも追加しました。Matkalaukkuクラスをご指摘の通り変更しましたが、ダメでした。Main クラスのMatkalaukku matkalaukku = new Matkalaukku(5);が、5なので、Tavara tiiliskivi = new Tavara("tiiliskivi", 4);の4を追加するとオーバーになるので、追加できないのですが、追加されます。
public void addTavara(Tavara tavara)メソッドで、条件文でMaksimpainoの指定の数を超えないようにしたいのですが、それができません。