Строки сравнения Java - PullRequest
       5

Строки сравнения Java

1 голос
/ 18 июля 2011
 /**
 * A method to compare Strings
 * @param arg1
 * @param arg2
 * @return 
 */
public boolean myQuickCompare(String arg1, String arg2) {
    boolean a = arg1.length() == arg2.length();
    if (a) {
        for (int b = 0; b > arg1.length(); b++) {
            if (arg1.charAt(b) != arg2.charAt(b)) {
                a = false;
            }
        }
    }
    return a;
}

Я понимаю, что цикл for неправильный, b никогда не будет больше длины строки.Как бы вы исправили эту проблему?

Какие разумные имена переменных вы бы дали для a и b?

Ответы [ 5 ]

6 голосов
/ 18 июля 2011

Используйте arg1.equals(arg2). Нет необходимости в пользовательских функциях. Не пытайтесь перехитрить разработчиков Java. Большую часть времени они выигрывают.

2 голосов
/ 18 июля 2011

Я всегда использую StringUtils.compare ( Apache Commons ).Это также обрабатывает нулевой регистр для любого аргумента String.

2 голосов
/ 18 июля 2011

Я понимаю, что цикл for неправильный, b никогда не будет больше, чем> длина строки.Как бы вы исправили эту проблему?

использовать equals() из String напрямую

Какие разумные имена переменных вы бы дали для a и b?

a может быть result

b может быть index


Вот реализация equals() из open jdk7

 public boolean equals(Object anObject) {
 1014           if (this == anObject) {
 1015               return true;
 1016           }
 1017           if (anObject instanceof String) {
 1018               String anotherString = (String)anObject;
 1019               int n = count;
 1020               if (n == anotherString.count) {
 1021                   char v1[] = value;
 1022                   char v2[] = anotherString.value;
 1023                   int i = offset;
 1024                   int j = anotherString.offset;
 1025                   while (n-- != 0) {
 1026                       if (v1[i++] != v2[j++])
 1027                           return false;
 1028                   }
 1029                   return true;
 1030               }
 1031           }
 1032           return false;
 1033       }
0 голосов
/ 18 июля 2011

Пара вещей:

  1. Когда вы сравниваете строки на равенство, используйте метод .equals. == используется для сравнения ссылок на объекты и определения, ссылаются ли они на один и тот же экземпляр. .equals фактически сравнивает символы внутри объекта String.
  2. Даже если вы все делаете по-своему (что неверно), цикл for должен выглядеть следующим образом

    для (int b = 0; b

0 голосов
/ 18 июля 2011

a => результат b => текущий

Было бы полезно проверить, является ли один из аргументов null.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...