2

This is my code:

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

Why does jslint throw this error?

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

4 に答える 4

5

これに投票しないでください...これが好きなら、Tomalakの答えに投票してください(これは彼と同じですが、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;
    };
}());
于 2010-05-03T17:13:35.887 に答える
5

正規表現への明らかな変更とは別に、関数自体に次の変更をお勧めします。

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;
}
于 2010-05-03T16:47:59.803 に答える
2

あなたが持っているつもりのものが否定された文字クラスであるなら、これはあなたが望むものです:

return s.gsub(/[^A-Za-z0-9_:.-]/, ".");
于 2010-05-03T16:37:02.207 に答える
1

まず第一に、答えてくれてありがとう。同じ文字列を 2 回クエリすると同じ ID を返す必要があるため、関数のセマンティックに小さなエラーが発生しました。例えば:

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;
    };
}());
于 2010-05-05T13:29:00.093 に答える