JSPの配列リストの問題

Question

私は値/サイズが20以上の場合は項目を追加doesntの配列を作成しようとしています。 この解決策は、1つのアイテムを追加し、値が20未満であっても停止します。 それだけで20件までの最大値を受け入れるように、どのように私はそれを変更します。

package business;
import java.io.Serializable;
import java.util.ArrayList;

public class Cart implements Serializable
{
private ArrayList<LineItem> items;

    public Cart()
{
    items = new ArrayList<LineItem>();
}

    public ArrayList<LineItem> getItems()
{
    return items;
}

    public int getCount()
{
    return items.size();
}

    public void addItem(LineItem item)
{
    String code = item.getProduct().getCode();
    int quantity = item.getQuantity();
    double credit = item.getProduct().getCHours();
    String credit2 = Double.toString(item.getProduct().getCHours());
    int isize = items.size();


    for (int i = 0; i < items.size(); i++)
    {
        if(isize <= 20)
        {
        LineItem lineItem = items.get(i);
            lineItem.setQuantityCredit(credit);
            return;
       }
    }
    items.add(item);
    }

    public void addItemCredit(LineItem item)
{
        double credit = item.getProduct().getCHours();
        String credit2 = Double.toString(item.getProduct().getCHours());
        String code = item.getProduct().getCode();

    for (int i = 0; i < 20; i++)
    {
        LineItem lineItem2 = items.get(i);
            lineItem2.setQuantityCredit(credit);
            return;

        }
    items.add(item);

    }

public void removeItem(LineItem item)
{
        String code = item.getProduct().getCode();
        for (int i = 0; i < items.size(); i++)
        {
            LineItem lineItem = items.get(i);
            if (lineItem.getProduct().getCode().equals(code))
            {
                items.remove(i);
                return;
            }
        }
    }
}

私はuが右ので、私はこれを試していると述べワットと思います。

if(isize <= 20) 
{ 
LineItem lineItem = items.get(i); lineItem.setQuantityCredit(credit); 
} 
return; 
} 
items.add(item); 
} 


and 


if(isize <= 20) 
{ 
LineItem lineItem = items.get(i); lineItem.setQuantityCredit(credit); 
} 
} 
items.add(item); 
return; 
} 

が、どちらも働きました。そうWHR私は、d return文を置くのですか？

Answer 1

あなたはリストに項目を追加する前にaddItemメソッドから戻っています。

if(isize <= 20)
{
     LineItem lineItem = items.get(i);
     lineItem.setQuantityCredit(credit);
     return; // This is the problem
}

Answer 2

私はちょうど私が答えることを試みる前に、あなたのコードを明確にしたい。

// get the total number of items in the cart
int isize = items.size();

// loop through the items
   // if there are more than 20 items then set the credit quantity and
     // return without adding
   // otherwise do nothing 
for (int i = 0; i < items.size(); i++)
{
    if(isize <= 20)
    {
    LineItem lineItem = items.get(i);
        lineItem.setQuantityCredit(credit);
        return;
   }
}
// if there are more than 20 items add another
items.add(item);

次のように

それだけで一つの項目を追加し、もはやされることが理由です。 この方法は「ISIZE」と呼ばれる最初の時間がゼロに等しく、従ってループのためスキップされます。この方法は「ISIZE」と呼ばれる第二の時間はゼロに等しく、したがって、ループが添加前に一度戻る進む。

私はあなたがそれは本当に意味がないとして、forループの内部で行うようにしようとしているものかどうか分かりません。あなただけのアイテムの配列が20以下に制限したい場合は、単に追加します。

if (items.size() < 20) {
  items.add(item);
}

あなたのコードに。あなたがしようとしているいくつかのより複雑なロジックがある場合は、あなたの目標を説明します。