ArrayList超出范围错误

Question

我正在为类构建程序，但在对数组列表进行排序时遇到问题。 我需要将arraylist的元素按代码的值排序。

public static ArrayList<MenuItem> orderByCode( ArrayList<MenuItem> items ){
  ArrayList<MenuItem> order=new ArrayList<MenuItem>(items.size());
  for (int i = 0; i < items.size(); i++) {
        for (int j = 1; j < (items.size() - i); j++) {
             if ( order.get(i).getCode()<order.get(i-1).getCode()){
                  order.add(items.get(j-1));
                  order.set(j-1, order.get(j));
                  order.set(j, order.get(order.size()-1));
             }
        }
  }
  return order;
 }

这是代码中as = n问题的一部分。 如果您需要我添加课程的其余部分或用于测试的内容，则可以这样做。

public static void main(String[] args){
  ArrayList<MenuItem> items = new ArrayList<MenuItem>();
  items.add( new MenuItem( "Big Bad Burger", 9.95, 2.15, 1, true ) );
  items.add( new MenuItem( "Cheeky Chicken", 5.95, 0.75, 1, true ) );
  items.add( new MenuItem( "Wild Wings", 5.95, 0.50, 0, true ) );
  items.add( new MenuItem( "Flying Fish", 15.95, 7.61, 1, false ) );
  items.add( new MenuItem( "Igloo Icecream", 1.95, 0.28, 2, true ) );

  ArrayList<MenuItem> ordered = orderByCode( items );

  // should list items in this order:
  // Wings, Burger, Chicken, Fish, Icecream
  for ( MenuItem item : ordered )
    System.out.println( item.menuString() );
}

有测试代码。

Answer 1

我想您在此行中遇到了错误，

if ( order.get(i).getCode()<order.get(i-1).getCode()){

上一行将使ArrayList超出i=0;的界限错误i=0;

Answer 2

学习使用调试器。 在调试器的帮助下，应该很容易发现这种问题，以便您知道引起问题的是哪条线，以及那一刻变量的值是多少。

无论如何，尽管我不知道您要做什么，但这行显然有问题：

if ( order.get(i).getCode()<order.get(i-1).getCode()){

对于i = 0，您会order.get(i-1)

Answer 3

检查项目的大小，因为我正在遍历项目arraylist，但是用于访问订单，您可能需要将其更改为

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

祝好运！

Answer 4

这条线需要固定

if (order.get(i).getCode() < order.get(i - 1).getCode())

Answer 5

当遍历ArrayList时，创建一个Iterator。

Iterator<MenuItem> i = order.iterator();
while (i.hasNext())
{
   MenuItem current = i.next();
   // perform operations on current
}

或者，将您的ArrayList转换为数组，并对其进行迭代

MenuItem[] orderArray = order.toArray(new MenuItem[order.size()]);
for (int i = 0; i < orderArray.length; i++)
{
   // perform operations on each order by using orderArray[i].relevantMethod()
}

这样清理代码将使您更好地了解正在发生的事情。

祝好运！