Мой метод работает дважды, когда он вызывается только один раз

Question

Так что я вызываю метод makeCard (String info) из другого класса. Он работает нормально при первом запуске, но во второй раз я вызываю метод, который, кажется, запускается дважды, что создает ошибку выполнения StringIndexOutOfBoundsException, и я не могу понять, почему. Я немного новичок в Java, поэтому я могу пропустить что-то очевидное, но логика в моей голове говорит, что он должен запускаться только один раз, если вызван один раз. Надеюсь, кто-то может указать на мою ошибку.

Вот метод:

public void makeCard(String info){ 
    cInfo = new StringBuffer(info); 
    int i = 0; 

    while(cInfo.charAt(i)== ' '){ 
     cInfo.deleteCharAt(i); 
    } 
    while(cInfo.charAt(cInfo.length()-1)== ' '){ 
     cInfo.deleteCharAt(cInfo.length()-1); 
     i--; 
    } 
    seperateValues(); 
    makeObject(); 
} 

и здесь, где она называется:

@Override 
public void actionPerformed(ActionEvent e) { 
    MainWindow mw = new MainWindow(); 
    CardBreakdown cb = new CardBreakdown(); 
    if("submit".equals(e.getActionCommand())){ 
     cb.makeCard(cardInfo.getText()); 
     mw.removeAddPanel(); 
     cardInfo.setText(""); 
    } 
} 

Заранее спасибо за любую помощь вы можете предоставить

это ошибка: Исключение в потоке «AWT-EventQueue-0» java.lang.StringIndexOutOfBoundsException: Индекс строки за пределами допустимого диапазона: 0

Answer 1

Ваш код излишне свернут для простого удаления всех вхождений символа пробела из строки. Вместо этого:

cInfo = info.trim(); 

Answer 2

первый, что я буду делать замену этих двух while петель в makeCard() с простым вызовом String.trim() - эта функция будет удалить начальные и конечные пробелы для вас.

Вы должны, как правило всегда предпочитают вызов библиотеки в разработке собственных функций, особенно функции, которые могут иметь довольно фатальную ошибку, когда, например, обработка пустых строк или строк, которые состоят только из пространств :-)

Нет никакой проверки ни в одном из ваших циклов для случая, когда строка является или становится пустой, что означает, что charAt будет жаловаться горько.

Answer 3

Что делать, если информация была пуста или нулевым? Затем cInfo.charAt (0) вернет ошибку, которую вы получаете. Вы должны указать нулевую или пустую проверку в начале вашего метода makeCard.