Строковый указатель за пределами границ? (Java, петля подстрока)

Question

Эта программа, которую я делаю для COSC конечно не компиляции правильно, я получаю ошибку:

Исключение в потоке «основной» java.lang.StringIndexOutOfBoundsException: индекс строки из диапазон: 2

в java.lang.String.substring (String.java:1765) в VowelCount.main (VowelCount.java:13)

Вот мой код:

import java.util.Scanner; 

public class VowelCount { 
public static void main(String[] args) { 
    int a = 0, e = 0, i = 0, o = 0, u = 0, count = 0; 
    String input, letter; 
    Scanner scan = new Scanner (System.in); 

    System.out.println ("Please enter a string: "); 
    input = scan.nextLine(); 

    while (count <= input.length()) { 
    letter = input.substring(count, (count + 1)); 

    if (letter == "a") { 
    a++; } 
    if (letter == "e") { 
    e++; } 
    if (letter == "i") { 
    i++; } 
    if (letter == "o") { 
    o++; } 
    if (letter == "u") { 
    u++; } 

    count++; 

    } 
    System.out.println ("There are " + a + " a's."); 
    System.out.println ("There are " + e + " e's."); 
    System.out.println ("There are " + i + " i's."); 
    System.out.println ("There are " + o + " o's."); 
    System.out.println ("There are " + u + " u's."); 
} 
} 

Для мои знания это должно сработать, но почему? Любая помощь будет большой. Спасибо!

Answer 1

Вы, возможно, потребуется вынимать = в строке

while (count <= input.length()) { 

и сделать его

while (count < input.length()) { 

, потому что он вызывает подстроку читать дальше длины строки.

=============== Но я добавлю несколько дополнительных битов советов, даже если его не просили:

не использовать == для сравнения строк , использование

letter.equals("a") 

вместо этого. Или еще лучше, попробуйте использовать

char c = input.charAt(count); 

, чтобы получить текущий символ затем сравнить, как это:

c == 'a' 

Answer 2

Удаление знака равенства следует исправить.

while (count < input.length()) {

и так как вы хотите получить один символ, вы должны сделать это:

substr(count,1)

, потому что второй параметр является на самом деле длина, не показатель.

Answer 3

Я думаю, что ваше состояние петли должно быть count < input.length. Прямо сейчас, последняя итерация выполняется с count == length, поэтому вашему звонку substring присваивается начальный индекс после последнего символа в строке, что является незаконным. Эти типы ошибок границ очень распространены при написании таких циклов, поэтому всегда полезно дважды и трижды проверять условия цикла, когда вы сталкиваетесь с такой ошибкой.

Кроме того, сравнение строк с оператором == обычно не будет делать то, что вы хотите. Это сравнивает, ссылаются ли две переменные на один и тот же объект. Вместо этого вы хотите протестировать string1.equals(string2), который сравнивает содержимое двух строк.

Answer 4

Исправлено с помощью всех, и особенно Винсента. Спасибо! Бегает чудесно.

import java.util.Scanner; 

public class VowelCount { 
    public static void main(String[] args) { 
     int a = 0, e = 0, i = 0, o = 0, u = 0, count = 0; 
     String input; 
     char letter; 

     Scanner scan = new Scanner (System.in); 

     System.out.print ("Please enter a string: "); 
     input = scan.nextLine(); 

     while (count < input.length()) { 
      letter = input.charAt (count); 

      if (letter == 'a') 
       a++; 
      if (letter == 'e') 
       e++; 
      if (letter == 'i') 
       i++; 
      if (letter == 'o') 
       o++; 
      if (letter == 'u') 
       u++; 

      count++; 

     } 
     System.out.println ("There are " + a + " a's."); 
     System.out.println ("There are " + e + " e's."); 
     System.out.println ("There are " + i + " i's."); 
     System.out.println ("There are " + o + " o's."); 
     System.out.println ("There are " + u + " u's."); 
    } 
} 

Answer 5

Перед цикла, попробуйте ниже

if(input.length()>0){ 
//you code 
}