java - ForLoop在覆盖方法Android中不断循环

Question

我有一个接口(interface)，它的方法在其他类中实现。在该方法中，我使用 forLoop 检查项目是否已存在于特定列表中，但 ForLoop 不断调用。
这是我的代码:

@Override
public void saveResultList(MyOrderResultPOJO myOrderResultPOJO) {

    myOrderResultList.add(myOrderResultPOJO);


    checkItemId(myOrderResultPOJO);

    //myOrderResultHashMap.put(inventoryClassificationId, myOrderResultList);
    setMyOrderResultListNew(myOrderResultList);
}

private boolean checkItemId(MyOrderResultPOJO myOrderResultPOJO){
    boolean v=false;

    for (int i=0;i<myOrderResultList.size();i++) {

        if (myOrderResultList.get(i).getItemId().equals(myOrderResultPOJO.getItemId())) {
            myOrderResultList.add(i,myOrderResultPOJO);
            Log.d("myOrderResultList","entering");
            v=true;
        }
    }

    return v;

}

接口(interface):

public interface MyOrderSubList {
    void addMtPcsSpinner(Context ctx, Spinner spinner);

    void showMessages(String message);

    void hideKeyBoards();

    void saveResultList(MyOrderResultPOJO myOrderResultPOJO);
}

Answer 1

你应该只返回 true如果你找到匹配项。您绝对不想将现有元素添加到 List再次( myOrderResultList.add(i,myOrderResultPOJO); 是无限循环的原因)。

private boolean checkItemId(MyOrderResultPOJO myOrderResultPOJO)
{
    for (int i=0;i<myOrderResultList.size();i++) {
        if (myOrderResultList.get(i).getItemId().equals(myOrderResultPOJO.getItemId())) {
            Log.d("myOrderResultList","entering");
            return true;
        }
    }

    return false;    
}

也就是说，调用方方法 ( saveResultList ) 的逻辑很奇怪。首先将元素添加到 List , 然后你调用 checkItemId(myOrderResultPOJO) , 检查它是否存在于 List 中(并且您忽略了该方法返回的值)。它可能应该是相反的:

public void saveResultList(MyOrderResultPOJO myOrderResultPOJO) 
{
    if (!checkItemId(myOrderResultPOJO)) {
        myOrderResultList.add(myOrderResultPOJO);
    }

    setMyOrderResultListNew(myOrderResultList);
}