java - 适当对待NullPointerException

我有一个带有以下方法的PurchaseOrderList类，该类返回列表中的对象数组。

public PurchaseOrder [] getAllOrders()
{
    PurchaseOrder [] ordersArray = new PurchaseOrder[listOfOrders.size()];

    if (listOfOrders.isEmpty())
        ordersArray = null;
    else
        for(int i = 0; i < listOfOrders.size(); i++)
        {
            ordersArray[i] = listOfOrders.get(i);
        }

    return ordersArray;
}

如果为空，则必须返回null！

我需要维护2个列表-一个用于当前，一个用于去年。如果是新的一年，则必须将当前年份的数据覆盖到lastYearList。并且必须清除本年度的清单。同时，折扣基于相关时段内客户的totalOrdersValue值，因此我需要从pastYearList中减去每个客户的过往订单值，然后才能忽略。

我有以下内容，但不知道如何对其进行优化。

public void startNewMonth()
{
    PurchaseOrder [] lastYearArray = lastYearList.getAllOrders();
    PurchaseOrder [] currentYearArray = currentYearList.getAllOrders();

    try
    {
        if(lastYearList != null)
        {
            for(int i = 0; i < lastYearArray.length; i++)
            {
                String foundID = lastYearArray[i].getCustomersID();
                int valueOfOrder = lastYearArray[i].getFullPriceValue();

                customersList.findCustomer(foundID).
                                      updateTotalOrdersValue(-valueOfOrder);
            }
        }

        System.arraycopy(currentYearArray, 0, lastYearArray, 0,
                                                   currentYearArray.length);

        for(int i = 0; i < lastYearArray.length; i++)
        {
            lastYearList.addPurchaseOrder(lastYearArray[i]);
        }

        currentYearList.setListToEmpty();
    }

    catch(Exception e)
    {
        System.out.println(e);
    }
}

似乎从第一行抛出了NullPointerException（如果没有数据），但是如果我将其放入“ try”中，代码​​将变得很冗长，并且我必须为不同的尝试键入很多相同的内容，以及捕获语句。我相信这样做有很短的路要走。

非常感谢！

最佳答案

是否有充分的理由拥有这些中间阵列？

如果可以将getOrders()方法添加到PurchaseOrderList类，例如

List<PurchaseOrder> getOrders() { retur listOfOrders; }

然后，您可以简化startNewMonth()

public void startNewMonth() {
    if(lastYearList != null)
    {
        for(PurchaseOrder po : lastYearList.getOrders()) {
            String foundID = po.getCustomerID();
            int valueOfOrder = po.getFullPriceValue();
            customersList.findCustomer(foundID).updateTotalOrdersValue(-valueOfOrder);
        }

        for (PurchaseOrder po : currentYearList.getOrders()) {
            lastYearList.addPurchaseOrder(po);
        }

        currentYearList.setListToEmpty();
    }
}

请注意，方法的逻辑很臭：名称为startNewMonth，并且您管理yearList ...