1. JavaScript / Говнокод #9950

    +154

    1. 1
    2. 2
    3. 3
    escape = function(string) {
    return (''+string).replace(/&/g, '&amp;').replace(/</g, '&lt;').replace(/>/g, '&gt;').replace(/"/g, '&quot;').replace(/'/g, '&#x27;').replace(/\//g,'&#x2F;');
  };

    Ваше мнение?

    Запостил: __proto__, 14 Апреля 2012

    Комментарии (12) RSS

    • ava guest 14.04.2012 21:09 # −8

      показать все, что скрытоГде здесь C++, __proto__?
      Ответить

    • ava Lure Of Chaos 15.04.2012 09:58 # +1

      антихакир?
      Ответить

    • ava Steve_Brown 16.04.2012 10:18 # 0

      Почему не &apos; ? И чем мешает косая черта?
      Ответить

    • ava victor-homyakov 16.04.2012 17:17 # 0

      Нормальное экранирование. Про апостроф автор был не в курсе. Только всё-таки зачем экранировать слэш?
      Ответить

      • ava JavaGovno 16.04.2012 17:43 # 0

        к конце с огоньком
        Ответить

      • ava wvxvw 16.04.2012 18:22 # 0

        Чем нормальное? функция escape - уже существует. Да и вместо того, чтобы писать много раз одно и то же - почему не циклом? Или даже рекурсивно и то было бы лучше (т.как короче).
        Но самое ужасное конечно в том, что каждая замена проходит всю строку целиком. Т.е. вместо одного цикла с заменой получилось 6.

        function multiReplace(input){
    return input.replace(
	    /[&"'\/<>]/g,
	function(x){
	    return "&" +
		{ "&" : "amp",
		  "\"" : "quot",
		  "'" : "apos",
		  "/" : "#x2F",
		  "<" : "lt",
		  ">" : "gt" }[x] + ";"; }); }

        Делает то же самое, но в 6 раз быстрее.
        Ответить

    • ava guest 20.04.2012 08:26 # 0

      Достаточно неочевидно, но указанный в сообщении вариант быстрее, чем вариант с одной заменой.
      Ответить

    Добавить комментарий