whileループ実行中のIndexOutOfBoundsException - Java

Question

package com.cp.javapractice; 

import java.util.ArrayList; 
import java.util.Scanner; 

public class Cp { 
public static void main(String args[]) { 
    ArrayList al = new ArrayList(); 
    Scanner s = new Scanner(System.in); 
    String str = null; 
    str = new String(); 
    System.out.println("Enter the string which you want to remove the  duplicates"); 
    str = s.nextLine(); 
    String arr[] = str.split(" "); 
    for (int k = 0; k < arr.length; k++) { 
     al.add(arr[k]); 
    } 
    try { 
     for (int i = 0; i < arr.length; i++) { 

      for (int j = i + 1; j < arr.length; j++) { 

       if (arr[i].equalsIgnoreCase(arr[j])) { 
        al.remove(j); 
       } 
      } 
     } 
     System.out.println(al); 
     } 
    catch (Exception e) { 
     System.out.println(e); 
    } 
} 
} 

私は特に指定された文字列をユーザーから置き換えようとしています。だから、splitメソッドを使って与えられた文字列をスペースで分割し、配列とarraylistに入れます。

配列を繰り返して条件が同じであることを確認した後、ArrayListでそれを削除しました。しかし、それを削除すると、インデックスの境界外の例外が表示されます。

このコードは小さな配列サイズで動作しますが、多数の配列サイズを指定すると例外を示します。 配列サイズが13ワードの文字列を与えている間に問題が発生しています。

ここに私のフルコードです。

Answer 1

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

    for (int j = i + 1; j < al.size(); j++) { 

     if (al.get(i).equals(al.get(j)) { 
      al.remove(j); 
     } 
    } 
} 

Answer 2

例外は、al.size()の代わりにarr.lengthを使用していることが原因です。すべての取り外しのために、arraylistのサイズは小さくなりますal。したがって、配列のサイズの代わりにarraylistのサイズを使用することを検討する必要があります。

for (int i = 0; i < al.size(); i++) { // change arr.length to al.size() 

      for (int j = i + 1; j < al.size(); j++) { // change arr.length to al.size() 

       if (arr[i].equalsIgnoreCase(arr[j])) { 
        al.remove(j); 

       } 
      } 
     } 

私は重複を除去するあなたの努力を減らしHashSetとTreeSetをチェックアウトして、あなたをお勧めします。

---

HashSetに実装：

import java.util.Arrays; 
import java.util.HashSet; 
import java.util.Scanner; 
import java.util.Set; 


public class Cp { 
public static void main(String args[]) { 

    Scanner s = new Scanner(System.in); 
    String str = null; 
    str = new String(); 
    System.out.println("Enter the string which you want to remove the  duplicates"); 
    str = s.nextLine(); 
    String arr[] = str.split(" "); 

    Set<String> ts = new HashSet<String>(Arrays.asList(arr)); // -> added only this line 

     System.out.println(ts); 

} 
} 

Answer 3

問題は、あなたの第二のループです。スタートはi + 1です。しかし、私は0から長さ-1です。したがって、最後のeinはj = length-1 + 1となり、配列の長さを超えます。

そうにループのための第一の変更：これは、デバッガを使用する方法を学ぶのに最適な時間です

for(int i=0;i < arr.length-2;i++)