Unescaped '^' с помощью jslint

Question

Это мой код:

/**********************************************************
 * remove non-standard characters to give a valid html id *
 **********************************************************/
function htmlid(s) {
 return s.gsub(/[^A-Z^a-z^0-9^\-^_^:^\.]/, ".");
}

Почему jslint выдает эту ошибку?

Lint at line 5 character 25: Unescaped '^'.
return s.gsub(/[^A-Z^a-z^0-9^\-^_^:^\.]/, ".");

Answer 1

Помимо очевидного изменения в регулярном выражении, я рекомендую следующее изменение самой функции:

function htmlid(s) {
  // prevents duplicate IDs by remembering all IDs created (+ a counter)
  var self = arguments.callee;
  if (!self.cache) self.cache = {};

  var id = s.replace(/[^A-Za-z0-9_:.-]/, "."); // note the dash is at the end!
  if (id in self.cache) id += self.cache[id]++;
  self.cache[id] = 0;

  return id;
}

Answer 2

Не голосуйте за это ... проголосуйте за ответ Томалака, если вам это нравится (он такой же, как и его, но без использования arguments.callee плюс кэширование самого регулярного выражения).

var htmlid = (function(){
    var cache = {},
        reg = /[^A-Za-z0-9_:.-]/;
    return function(s){
        var id = s.replace(reg, ".");
        if (id in cache){ id += cache[id]++;}
        cache[id] = 0;

        return id;
    };
}());

Answer 3

Если то, что вы намереваетесь иметь, это отрицательный класс символов, то это то, что вам нужно:

return s.gsub(/[^A-Za-z0-9_:.-]/, ".");

Answer 4

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

htmlid("foo bar");  // -> "foo.bar"
htmlid("foo bar");  // -> "foo.bar"
htmlid("foo.bar");  // -> "foo.bar0"
htmlid("foo.bar0"); // -> "foo.bar00"
htmlid("foo.bar");  // -> "foo.bar0"

Однако, я принял ваши функции:

var htmlid = (function () {
    var cache = {},
        ncache = {},
        reg = /[^A-Za-z0-9_:.-]/;
    return function (s) {
        var id;
        if (s in cache) {
            id = cache[s];
        } else {
            id = s.replace(reg,".");
            if (id in ncache) {
                id += ncache[id]++;
            }
            ncache[id] = 0;
            cache[s] = id;
        }
        return id;
    };
}());