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

    +169

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    function paramEncode(str){
	var text = escape(str);
	while(text.indexOf("%u") !=-1)
		text = text.replace("%u","!u");
	while(text.indexOf("%") !=-1)
		text = text.replace("%","!u00");
        while(text.indexOf(".") !=-1)                                                                                        
                text = text.replace(".","!u002E");   
        while(text.indexOf("/") !=-1)                                                                                        
                text = text.replace("/","!u002F");   
	return text;
}

    альтернатива if:)

    Запостил: moonie, 18 Января 2011

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

    • ava Lure Of Chaos 18.01.2011 15:05 # 0

      заменять пока заменяется!
      Ответить

    • ava dwinner 18.01.2011 15:09 # 0

      Блин, тоже самое хотел написать... ))) Опередили.
      Ответить

    • ava New_Folder 18.01.2011 15:17 # 0

      вообще-то в JS replace заменяет только первое вхождение искомого выражения.

      Поэтому или регекспом, или while
      Ответить

      • ava Lure Of Chaos 18.01.2011 15:20 # 0

        тогда лучше регекспом наверное
        Ответить

        • ava New_Folder 18.01.2011 15:24 # 0

          Давно хотел сравнить по производительности
          text.replace(a/g,'A'));
          и
          while(text.indexOf('a')!=-1) text.replace('a','A');
          да пока руки не доходят
          Ответить

          • ava moonie 18.01.2011 15:25 # 0

            ща проверим:)
            Ответить

          • ava Lure Of Chaos 18.01.2011 15:26 # 0

            да в чем проблема. написать оба выражение в цикле на охрененное колво итераций, замерить время, и поделить )

            это дело 5-10 мин )
            Ответить

            • ava New_Folder 18.01.2011 15:28 # 0

              Проблемы нету. Вспоминаешь про этот тест, только когда заходит разговор про while/regexp. И забываешь про этот тест через минуту.

              Сам обычно реплейсаю через регекспы:)
              Ответить

              • ava moonie 18.01.2011 15:40 # 0

                чето браузер валится от while(text.indexOf('a')!=-1) text.replace('a','A') :)
                Ответить

                • ava New_Folder 18.01.2011 15:46 # 0

                  да. забыл переприсвоить. так он в бесконечный цикл уходит
                  text = text.replace('a','A')
                  Ответить

              • ava Lure Of Chaos 18.01.2011 15:41 # +1 

                <html>
<script>
var cnt=100000;
var text='aaaaaaaa';
var t1=new Date();
for(var i=0;i<cnt;i++) {
 text.replace(/a/g,'A');
}
var t2=new Date();
document.write('re:'+(t2-t1)/cnt+' ');

var t1=new Date();
for(var i=0;i<cnt;i++) {
 while(text.indexOf('a')!=-1) text=text.replace('a','A');
}
var t2=new Date();
document.write('while:'+(t2-t1)/cnt+' ');
</script>
</html>


                на последнем файрфоксе под хрюшей дал 
                re:0.00209 while:0.00007
                Ответить

                • ava moonie 18.01.2011 15:43 # 0

                  while быстрее! так вот где оптимизация:)
                  Ответить

                  • ava Lure Of Chaos 18.01.2011 15:48 # 0

                    впринципе, это понятно, во втором случае мы организуем тупо цикл на замещение, а не более универсальный конечный автомат, пусть и "нативный"
                    Ответить

                • ava New_Folder 18.01.2011 15:44 # 0

                  Только тогда еще

                  for(var i=0;i<cnt;i++) {
                  text = text.replace(/a/g,'A');
                  }

                  Хотя и без присваивания разница заметна невооруженным глазом
                  Ответить

                  • ava Lure Of Chaos 18.01.2011 15:46 # 0

                    зачем? не надо.
                    мы заменяем одной инструкцией, но текст остается исходным для второго теста
                    во втором же присвоение нужно, потому что в цикле
                    Ответить

                    • ava New_Folder 18.01.2011 15:49 # 0

                      replace не меняет исходную строку вроде бы.
                      Поэтому чтобы увидеть результат, то надо переприсвоить.

                      при
                      var text='aaaaaaaa';
                      text.replace(/a/g,'A') ;

                      text будет равен 'aaaaaaaa'
                      Ответить

                      • ava Lure Of Chaos 18.01.2011 15:53 # 0

                        text = text.replace(/a/g,'A');

                        text будет равен 'АААААААА', а потом тогда второй тест сфейлит, ему нечего будет заменять. Либо перед вторым тестом тоже написать var text='aaaaaaaa';
                        Ответить

                        • ava New_Folder 18.01.2011 16:09 # 0

                          извини, не сразу понял смысла фразы

                          "текст остается исходным для второго теста"

                          Тогда во втором тесте замена внутри while у тебя отработает только 1 раз.

                          Значит, подсчитанное время будет неверным.

                          "Либо перед вторым тестом тоже написать var text='aaaaaaaa';" - внутри цикла по i
                          Ответить

                          • ava Lure Of Chaos 18.01.2011 16:44 # +3

                            точно, недоглядел
                            <html>
<script>
var cnt=100000;

var t1=new Date();
for(var i=0;i<cnt;i++) {
 var text='aaaaaaaa';
 text=text.replace(/a/g,'A');
}
var t2=new Date();
document.write(text+' re:'+(t2-t1)/cnt+' ');

var t1=new Date();
for(var i=0;i<cnt;i++) {
 var text='aaaaaaaa';
 while(text.indexOf('a')!=-1) text=text.replace('a','A');
}
var t2=new Date();
document.write(text+' while:'+(t2-t1)/cnt+' ');
</script>
</html>

                            AAAAAAAA re:0.00204 AAAAAAAA while:0.00443
                            Ответить

                            • ava New_Folder 18.01.2011 16:50 # 0

                              Спасибо за тест.

                              Теперь видно, что не так все хорошо с while, как казалось сразу.
                              Ответить

                            • ava gegMOPO4 18.01.2011 17:16 # 0

                              А если ещё взять 'aaaaaaaaaaaaaaaa' или хотя бы 'bbbbbbbbaaaaaaaa'...
                              Ответить

                            • ava babaika 18.01.2011 17:40 # 0

                              Для чистоты эксперимента можно добавить блок со split join:
                              t1=new Date();
for(var i=0;i<cnt;i++) {
 var text='aaaaaaaa';
 text=text.split('a').join('A');
}
t2=new Date();
document.write(text+' split:'+(t2-t1)/cnt+' ');

                              Опера пишет, что это немного медленнее, чем регулярка, но в 2 раза быстрее while.
                              Я так думаю, что на большом тексте разница будет намного заметнее. ГК защитан.
                              Ответить

    • ava qixKopY 25.08.2021 09:32 # 0

      vanished
      Ответить

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