Динамическое регулярное выражение для десятичной точности не работает
У меня есть следующее жестко закодированное выражение RegEx для десятичной точности и масштаба, которое работает (в другом проекте):
// This would be a Decimal(2,3)
var regex = /\d{0,2}(\.\d{1,3})?$/;
var result = regex.test(text);
Однако я не хочу жестко кодировать несколько вариантов. И, что интересно, следующее не работает ... но я не знаю почему.
Я «думаю», что конкатенация может (каким-то образом) влиять на «тест»
- Что я здесь делаю не так?
ОБРАЗЕЦ:
var validationRules = {
decimal: {
hasPrecision: function (precision, scale, text) {
var regex = new RegExp('\d{0,' + precision + '}(\.\d{1,' + scale + '})?$');
var result = regex.test(text);
// result is ALWAYS true ????
alert(result);
alert(regex);
}
}
};
ОТКАЗ ОТ ОБРАЗЦА-СНИПЕТ:
$(document).ready(function () {
var validationRules = {
decimal: {
hasPrecision: function (precision, scale, text) {
var regex = new RegExp('\d{0,' + precision + '}(\.\d{1,' + scale + '})?$');
var result = regex.test(text);
alert(result);
alert(regex);
}
}
};
var masks = {
decimal: function (e) {
// TODO: get Kendo MaskedTextBox to run RegEx
var regex = new RegExp("^([0-9\.])$");
var key = String.fromCharCode(!event.charCode ? event.which : event.charCode);
if (!regex.test(key)) {
event.preventDefault();
return false;
}
}
};
var button = $('.btn');
var textbox = $('.txt');
textbox.on('keypress', masks.decimal);
button.on('click', function () {
var text = textbox.val();
validationRules.decimal.hasPrecision(2, 3, text);
});
});<script src = "https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>
<input type = "text" value = "1111" class = "txt">
<input type = "button" class = "btn" value = "Run it">@revo Нет ... не сработало. Но спасибо.
После исправления этой проблемы, я думаю, вам следует рассмотреть возможность использования якоря ^ или, по крайней мере, \b, чтобы избежать разрешения таких значений, как 123.123, при вызове .hasPrecision(2, 3, text).
Попробуйте new RegExp('^\\d{0,' + precision + '}(\\.\\d{1,' + scale + '})?$');
![Безумие обратных вызовов в javascript [JS]](https://i.imgur.com/WsjO6zJb.png)
Ответы 1
Всегда посмотрите на результат при построении динамических строк. В вашем случае вы создаете его и просто предполагаете, что он превращается в шаблон REGEX, который вы хотите.
На самом деле вы строите, например:
d{0,2}(.d{1,3})?$
Почему? Поскольку шаблоны REGEX, построенные с помощью конструктора (в отличие от литералов), строятся как строки, а в строках \ интерпретируется как escape-символ.
Однако вам нужны эти обратные косые черты, чтобы они сохранялись в вашем шаблоне, поэтому вам нужно двойное экранирование. По сути, вам нужно избежать побега, чтобы сохранить последний.
var regex = new RegExp('\\d{0,' + precision + '}(\\.\\d{1,' + scale + '})?$');
В результате получится эквивалент вашего жестко запрограммированного шаблона предполагаяprecision и scale будут содержать промежуточные числа, которые, как вы думаете, они содержат. Проверьте это тоже. (Например, если они содержат числа с плавающей запятой, это испортит ваш шаблон.)
Что касается ваших ложных срабатываний, вероятно, это связано с отсутствующей инструкцией по привязке запуска, то есть ^.
/\d{0,2}(\.\d{1,3})?$/.test("1234"); //true
/^\d{0,2}(\.\d{1,3})?$/.test("1234"); //false, note ^
Вы должны дважды экранировать RegExp:
\\d. Это строка типа.