刪除數組元素時出錯

Question

我有新的數組列表，1個數組列表，其中有10個客戶已插入。我正在運行一個循環，從arraylist中挑選一個隨機客戶，並將其添加到第二個數組列表中。但是，當我將客戶插入第二個數組列表時，我會得到重複的數據。所以當循環運行後，將客戶添加到第二個數組列表中時，它將從第一個數組列表中移除它。

但是，當它運行時我得到一個錯誤：Intervals error: java.lang.IndexOutOfBoundsException: Index: 7, Size: 7

ArrayList<String> customer = new ArrayList<String>(Arrays.asList(list)); 

int customerlist = customer.size(); 

while (line.isEmpty()) 
     { 
      for (int x = 0; x < customerlist; x++) 
      { 
       try 
       { 
        Thread.sleep(intervals * 1000); //Sleep method to hold the arrival time by 1-2 seconds. 
        int cus = (int) (Math.random() * customerlist); //Random customer is picked here. 
        String new_cus = customer.get(cus); //New customer object is created ere. 
        line.add(new_cus); //Customer objects are added to the empty LinkedList queue. 
        customer.remove(cus); 

        //For loop statement to outputting the queue. 
        for (String s : line) 
        { 
         System.out.print("[" + s.toString() + " " + "]"); //Outputting each customer and using the ".name" method so customers are readable. 
        } 
        //Outputting the whole queue and stating who has joined the queue. 
        System.out.println("\n" + "The queue has " + line.size() + " customers so far" + "\n" + 
        new_cus.toString() + " Has Joined the Queue " + " <=== WAITING" + "\n"); 
       } 
       catch(Exception e) //ERROR handler for sleep method. 
       { 
        System.out.println("Intervals error: " + e); //Outputting the ERROR message. 
        System.exit(0); //If ERROR found exit system. 
       } 

      } 
     } 

Answer 1

你從數組刪除您可以有效地迭代，尚未相應地更新狀態。

變化：

for (int x = 0; x < customerlist; x++) 

到

for (int x = 0; x < customer.size(); x++) 

（或者更好的是，使用迭代器在基本ArrayList，這樣就可以使用Iterator.remove()功能安全地刪除。）

而且換行：

int cus = (int) (Math.random() * customerlist); 

到

int cus = (int) (Math.random() * customer.size()); 

Answer 2

這就是問題所在：

int cus = (int) (Math.random() * customerlist); 

這很好（雖然不是乾淨呼籲Random.nextInt）對於第一次迭代 - 但事後，customer.size()已經改變（如元素具有已被刪除）但customerlist仍然是一樣的。因此，在下一次迭代中，您正在挑選錯誤範圍內的元素。

說實話，你最好用Collections.shuffle()來改變原來的清單 - 這就是你想要的結果，對吧？

Answer 3

添加

customerlist--; 

後

customer.remove(cus); 

也，你可以改變

for (int x = 0; x < customerlist; x++) 

通過

for (int x = 0; x < customer.size(); x++) 

但我認爲在每個循環中調用.size函數會使用比局部變量更多的資源。