布尔方法不检查是否声明

Question

我正在尝试编写布尔方法，但它不起作用

   public  boolean addPresent(Present present)
   {
      if(totWeight+present.getWeight()<=maxWeight)
      {
          presents.add(present);
          presents1.add(present);
          return true;

      }
          for(Present pres : presents1)
          {
          totWeight+=pres.getWeight();
          }return false;
   }

它的工作，如果我写这个方法：

 public  void addPresent(Present present)
  {
       if(totWeight+present.getWeight()<=maxWeight)
      {
          presents.add(present);
          presents1.add(present);
         System.out.println("true");

      }else
      {             System.out.println("false");
      }
          for(Present pres : presents1)
          {
          totWeight+=pres.getWeight();
           }
 }

我应该如何为它制作布尔方法？

Answer 1

在清理了格式并比较了您描述的两种方法后，我注意到一个主要的逻辑错误。

在第一种方法中，您执行以下操作：

如果true，返回true并停止运行该方法的其余部分。-如果false，添加所有当前权重并返回false

在第二种方法中，您执行以下操作：

如果true，返回true并继续运行该方法的其余部分。-如果false，添加所有当前权重并返回false

如果第一种方法是您想要的方法，请在方法的开头声明一个布尔值，并根据测试结果将其设置为true或false，然后返回该布尔值。这样，您就可以推迟实际的返回语句，直到完成计算。

public  boolean addPresent(Present present)
{
    boolean temp = false;
    if(totWeight+present.getWeight()<=maxWeight)
    {
        presents.add(present);
        presents1.add(present);
        temp = true;
    }
    for(Present pres : presents1)
    {
        totWeight+=pres.getWeight();
    }
    return temp;

}

Answer 2

我注意到的主要区别是，当返回true时，它不会向总重量添加任何内容。我还认为您想用类似的方法重置总计

public boolean addPresent(Present present) {
    boolean r = false;
    if (totWeight + present.getWeight() <= maxWeight) {
        presents.add(present);
        presents1.add(present);
        r = true;
    }
    totWeight = 0; // <-- reset total.
    for (Present pres : presents1) {
        totWeight += pres.getWeight();
    }
    return r;
}

Answer 3

您需要在for循环之前返回false，或者在检查权重小于maxWeight的条件中包含for循环。或者你可以只添加当前的重量到总重量，不需要循环。

public  boolean addPresent(Present present){

    if(totWeight+present.getWeight()<=maxWeight){
        presents.add(present);
        presents1.add(present);
        totWeight += present.getWeight();
        return true;
    }
    return false;
}