Sort function issue...

Question

     public void sortDate() {
          ArrayList<Round> temp = new ArrayList<Round>();
          ArrayList<Round> list = new ArrayList<Round>();
//rounds is and arrayList defined in the class
          rounds.trimToSize();
          temp = rounds;
          int oldestIndex = 0, oldestRound = rounds.get(0).getDate();
          // Sort date from oldest to newest based on yymmdd format
          for (int i = 0; i < temp.size(); i++) {
               oldestIndex = temp.get(0).getDate();
               for (int j = 0; j < temp.size(); j++) {
                    if (temp.get(j).getDate() < oldestRound) {
                         oldestRound = temp.get(j).getDate();
                         oldestIndex = j;
               // Add oldest in the pass to list
               list.add(temp.get(oldestIndex));
               temp.remove(oldestIndex);
          // After all passes to sort dates list is set to this.round
          this.rounds = list;
     }What is wrong with this sort function?

Answer 1

wiebs2334 wrote:
          for (int i = 0; i < temp.size(); i++) {
               temp.remove(oldestIndex);
          }You are iterating over the temp arraylist from 0-temp.size(), during that time you are removing elements. This is most likely the cause of your index out of bounds exception.
A solution is to use an iterator and the corresponding remove methods.
Mel