Java Android-索引1无效，大小为1-超出范围

因此，我加入了一个正在编写android的团队，因此有一个问题导致应用程序崩溃。这是由于arraylist越界。下面是代码，Failes是代码的最后一位（类在此之后继续）。我希望有人能指出确切的问题及其解决方案，这不会破坏其他任何东西：

public class OneClickEssentialExtrasFragment extends BaseOneClickFragment {

    public static final String TAG = "PhoneOneClickEssentialExtrasFragment";

    private View myView;

    Map<Integer, Integer> stockProductMap;

    Product mProducts;

    private List<Stock> mStockCollection;

    private List<Stock> mStockDelivery;

    private List<Integer> mStockProductId;

    ArrayList<String> myEssenatialList;

    Map<String, Product> myProduct;

    private String deliveryZipCode = "EC3M3BD";

    WebServiceFacade mWebManager;

    private int ESSENTIAL_EXTRA_PRODUCT_ONE = 0;

    private int ESSENTIAL_EXTRA_PRODUCT_TWO = 1;

    private int ESSENTIAL_EXTRA_PRODUCT_THREE = 2;

    private int currentState = 0;

    private static final int STATE_LOADING = 0;

    private static final int STATE_DISPLAY_ESSENTIAL_EXTRAS = 1;

    public static  String mreservedProduct="";


    public View onCreateView(final LayoutInflater inflater,
            final ViewGroup container, final Bundle savedInstanceState) {
        return inflater.inflate(
                R.layout.fragment_one_click_essential_extras_list, container,
                false);
    }

    public void onViewCreated(final View view, final Bundle savedInstanceState) {
        super.onViewCreated(view, savedInstanceState);
        myView = view;
        currentState = STATE_LOADING;
        Bundle b = getActivity().getIntent().getExtras();
        Product mProduct = b.getParcelable("product");
        mreservedProduct =mProduct.getReference();
        mWebManager = WebServiceFacade.getInstance();
        myProduct = new HashMap<String, Product>();
        Bundle bundle = getActivity().getIntent().getExtras();
        mProducts = bundle.getParcelable("product");

        mStockCollection = new ArrayList<Stock>();
        mStockDelivery = new ArrayList<Stock>();
        mStockProductId = new ArrayList<Integer>();
        mStockProductId.add(Integer.parseInt(mProduct.getReference()));
        // To-do
        myEssenatialList = new ArrayList<String>();
        myEssenatialList.add(mProduct.getReference());
        renderEssentialScreen();
        if (myEssenatialList == null || myEssenatialList.size() == 0) {
            // myView.findViewById(R.id.essential_extras_item_icon_container)
            // .setVisibility(View.GONE);
            resetEssentialViews();
            return;
        }

        if (mProducts != null) {
            for (int i = 0; i < myEssenatialList.size(); i++) {
                mWebManager.getProduct(mProductDetailsCallback,
                        myEssenatialList.get(i));
            }
        } else {

        }
    }

    private void checkStockCollection(Product mProduct, int quantitiesselected) {

        int storeId = 0;
        if(mArgosApplication.getCurrentStore()!=null)
            storeId = mArgosApplication.getCurrentStore().getStoreNumber();

        List<String> products = new ArrayList<String>();
        //TO DO: Get the products list from Order API and add here
        products.add(mProduct.getReference());
        mWebManager.checkStock(mStockCheckCallbackCollection,quantitiesselected, storeId, products,
                deliveryZipCode);
    }

    private final UICallback<Map<Integer, Map<Integer, Stock>>> mStockCheckCallbackCollection = new BaseUICallback<Map<Integer, Map<Integer, Stock>>>() {

        @Override
        protected void handlePlannedMaintenance(final int errorCode,
                final String errorMessage) {
            QLog.d("ignored stock lookup 503 error: " + errorMessage);
        }


        @Override
        public void onSuccess(final Map<Integer, Map<Integer, Stock>> data) {
            if (data != null && mProducts != null) {
                Map<Integer, Stock> productStockMap = null;
                if(mArgosApplication.getCurrentStore()!=null)
                    productStockMap = data.get(mArgosApplication.getCurrentStore().getStoreNumber());
                else // Global stock, see NewStockHandler.java ->convertStockToMap is constructing the map with 1000, in case store URI is null
                    productStockMap = data.get(1000);
                if (productStockMap != null) {
                    for (int index = 0; index < productStockMap.size(); index++) {
                        if (mStockProductId != null) {
                            mStockCollection.add(productStockMap
                                    .get(mStockProductId.get(index)));
                            currentState = STATE_DISPLAY_ESSENTIAL_EXTRAS;
                            renderEssentialScreen();
                        }
                    }
                }
            } else {
                QLog.d("No Response for Product");
            }
        }


        @Override
        public boolean handleFailure(int errorCode, String errorMessage) {
            QLog.d("503 Stock Check not Available");
            return false;
        }
    };

    private void renderEssentialScreen() {
        if (myEssenatialList == null || myEssenatialList.size() == 0) {
            resetEssentialViews();
        }
        switch (currentState) {
        case STATE_LOADING:
            renderProgressBar();
            break;
        case STATE_DISPLAY_ESSENTIAL_EXTRAS:
            renderEssentialData();
            break;
        }
    }

    private void renderProgressBar() {
        resetEssentialViews();
        myView.findViewById(R.id.progressbar).setVisibility(View.VISIBLE);
    }

    private void renderEssentialData() {
        myView.findViewById(R.id.progressbar).setVisibility(View.GONE);
        if (myEssenatialList == null || myEssenatialList.size() == 0) {
            resetEssentialViews();
            return;
        }
        resetEssentialViews();
        if (myEssenatialList.size() == ESSENTIAL_EXTRA_PRODUCT_ONE) {
            fillProductFirst();
        } else if (myEssenatialList.size() == ESSENTIAL_EXTRA_PRODUCT_TWO) {
            fillProductFirst();
            fillProductMiddle();
        } else {
            fillProductFirst();
            fillProductMiddle();
            fillProductLast();
        }
    }

    private void resetEssentialViews() {
        myView.findViewById(R.id.essential_extras_one).setVisibility(View.GONE);
        myView.findViewById(R.id.essential_extras_two).setVisibility(View.GONE);
        myView.findViewById(R.id.essential_extras_three).setVisibility(
                View.GONE);
    }

    private void fillProductMiddle() {

        if (isAdded() == false) {
            return;
        }


        System.out.println("size of array = " + myEssenatialList.size());
        for(int i =0; i <myEssenatialList.size(); i++){
            System.out.println("elements of array are: " + i);
        }

        myView.findViewById(R.id.essential_extras_two).setVisibility(View.VISIBLE);
        TextView productDescription = (TextView)myView.findViewById(R.id.product_name_two);
        productDescription.setText(myProduct.get(myEssenatialList.get(ESSENTIAL_EXTRA_PRODUCT_TWO))
                .getShortDescription());

---

这是一个简单的错误

您正在这样做：

myProduct.get(myEssenatialList.get(ESSENTIAL_EXTRA_PRODUCT_TWO))

其中：

private int ESSENTIAL_EXTRA_PRODUCT_TWO = 1;

---

如果您的列表只有一个元素怎么办？在这种情况下，您需要第0项。尝试访问第一个元素将抛出越界异常。

根据崩溃堆栈跟踪

在java.util.ArrayList.get（ArrayList.java:308）10-16 08:19:11.280： E/AndroidRuntime（23119）：

我相信是这条线

productDescription.setText(myProduct.get(myEssenatialList.get(ESSENTIAL_EXTRA_PRODUCT_TWO))
                .getShortDescription());

---

您应该调试此行，并将其拆分为单独的数组访问。。由于要链接两个get-inside-single语句，所以调试越界访问哪个数组是困难的。拆分后，您可以调试哪些被越界访问及其原因。

发布日志猫堆栈崩溃跟踪请发布日志猫，并告知引发异常的代码行。谢谢。这是一个直截了当的边界排列。行号是已知的。为什么你不能打印索引的值，然后自己计算出来呢。旧团队或新团队，您的代码或其他人，这是一组边界。只需查看行号，并记录索引。算算吧。。

android.fragment.pdp.OneClickEssentialExtrasFragment.fillProductMiddle(OneClickEssentialExtrasFragment.java:352)

productDescription.setText(myProduct.get(myEssenatialList.get(ESSENTIAL_EXTRA_PRODUCT_TWO))
                .getShortDescription());