Почему не работает метод .replace()?

Question

[Olushon]

Помогите найти ошибку в коде. В строке каждую английскую букву надо заменить на номер ее позиции в алфавите. Остальные символы игнорировать и не возвращать.

var str = '';
function alphabetPosition(text) {
     str = text.toUpperCase();
     for (var i = 0; i < str.length; i++) {
          if ( (parseFloat(str.charCodeAt(i)) > 64) && parseFloat((str.charCodeAt(i) < 91))) {
               var letter = text.charAt(i);
               var sym = new RegExp(letter, 'g');
               str = str.replace(sym, str.charCodeAt(i) - 64);
          }
     }
 return str;
};

Срабатывает только .toUpperCase(), помогите, пожалуйста!

Comments

[OVK2015]

1. Зачем там parseFloat вообще?
2. var letter = text.charAt(i); а сама строка к этому времени уже в верхнем регистре
var sym = new RegExp(letter, 'gi');

[Olushon]

OVK2015:
1. Без parseFloat цикл зависал почему-то.
2. В самом деле, спасибо. Но все равно не работает

Answer 1

[Rsa97]

Жуть какая.

function alphabetPosition(text) {
  return text.toUpperCase().replace(/([A-Z])/g, function(char) { 
    return char.charCodeAt(0) - 64;
  });
}

Comments

[Olushon]

Да, у Вас симпатичнее выглядит :)
А что с моим не так?

[Aves]

Olushon: parseFloat((str.charCodeAt(i) < 91)) это NaN

[Rsa97]

Olushon:
1. str объявляется глобально, а используется в функции, это неправильно, само по себе ошибкой не является, но может привести к ошибке, если переменная str будет задействована в другой функции.
2. Перебор циклом по строке, если она модифицируется внутри цикла - моветон.
3. charCodeAt() возвращает целое число. Нет никакого смысла переводить его в вещественное через parseFloat.
4. Вы запутались в скобках внутри условия if, в результате вместо parseFloat(str.charCodeAt(i)) < 91 получили parseFloat(str.charCodeAt(i) < 91).
5. Перебор в цикле идёт по строке str, замена тоже делается в ней, а регулярное выражение составляется по символу из строки text.

[Rsa97]

Olushon: И, кстати, с учётом того, что в строке могут быть и символы других алфавитов, а их переводить в верхний регистр не надо, правильно будет так:

function alphabetPosition(text) {
  return text.replace(/([A-Z])/gi, function(char) { 
    return (char > 'Z' ? char.charCodeAt(0) - 96 : char.charCodeAt(0) - 64);
  });
}

[Olushon]

Rsa97: Спасибо за развернутый ответ!

Answer 2

[Дмитрий Беляев]

function alphabetPosition(text) {
     var str = text.toUpperCase();
     var result = '';
     for (var i = 0; i < str.length; i++) {
         var code = str.charCodeAt(i);
         var letter = text.charAt(i);
         if (64 < code && code < 91) {
             result += code - 64;
         } else {
             result += letter;
         }
     }
     return result;
}