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

    +170

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    if (!opts.matchCase){ 
	var regx = new RegExp("(?![^&;]+;)(?!<[^<>]*)(" + query + ")(?![^<>]*>)(?![^&;]+;)", "gi");
} else {
	var regx = new RegExp("(?![^&;]+;)(?!<[^<>]*)(" + query + ")(?![^<>]*>)(?![^&;]+;)", "g");
}

    http://code.drewwilson.com/entry/autosuggest-jquery-plugin

    Запостил: sapegin, 26 Апреля 2011

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

    • ava Lure Of Chaos 26.04.2011 18:37 # 0

      офигели обьявлять во внутреннем блоке, да еще и по два раза
      Ответить

    • ava FelikZ 26.04.2011 22:12 # −4

      var regx = null;
      if (!opts.matchCase)
      regx = /(?![^&;]+;)(?!<[^<>]*)(" + query + ")(?![^<>]*>)(?![^&;]+;)/gi;
      else
      regx = /(?![^&;]+;)(?!<[^<>]*)(" + query + ")(?![^<>]*>)(?![^&;]+;)/g;
      Ответить

      • ava sapegin 26.04.2011 22:13 # +2

        Что это за кошмар?! Да к тому же неработающий.
        Ответить

        • ava FelikZ 26.04.2011 22:40 # −1

          Что в моем примере работает иначе исходного варианта?
          Ответить

          • ava sapegin 26.04.2011 22:58 # +1

            1. У вас «" + query + "» в пустоте, так что пример просто не работает.
            2. Он по сути идентичен опубликованному мной варианту, т. е. ничего не улучшено.
            Ответить

            • ava FelikZ 26.04.2011 23:06 # 0

              1. ой печаль :( заработался :(
              2. вынес определение regx за иф, так корректнее имхо, но исходя из (1) беда :)
              Ответить

              • ava guest 26.04.2011 23:22 # +3

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

      • ava FelikZ 26.04.2011 22:15 # +2

        Или вообще:
        var regx = new RegExp("(?![^&;]+;)(?!<[^<>]*)(" + query + ")(?![^<>]*>)(?![^&;]+;)", (!opts.matchCase)?"gi":"g");
        Ответить

        • ava sapegin 26.04.2011 22:17 # +2

          Остаётся только убрать лишние скобки и обратить условие.
          Ответить

      • ava guest 27.04.2011 17:21 # −1

        regx = /(?![^&;]+;)(?!<[^<>]*)(%query)(?![^<>]*>)(?![^&;]+;)/gi;
        Ответить

        • ava guest 28.04.2011 07:40 # −1

          сходите отлейте
          Ответить

          • ava guest 28.04.2011 15:28 # −1

            хосни суйпов, цидор
            Ответить

            • ava guest 28.04.2011 15:38 # −1

              масая кака кимутует нои поссы?
              Ответить

              • ava guest 28.04.2011 15:54 # −2 

                function shuffle(str) {
	str = str.split("");
	var consPos = [];
	for (var i = 0; i < str.length; i++)
		if (/[бвгджзйклмнпрстфхцчшщ]/i.test(str[i]))
			consPos.push(i);
	for (var i = 0; i < consPos.length; i++) {
		if (Math.random() > 0.3) continue;
		var temp = str[consPos[i]], rnd = ~~(Math.random() * consPos.length);
		str[consPos[i]] = str[consPos[rnd]];
		str[consPos[rnd]] = temp;
	}
	return str.join("");
}


                текежь вы морете писать кап я
                Ответить

                • ava guest 28.04.2011 15:58 # −2

                  слыш ты жаллий гик, с хуя ни мы метя блять кинутуеть, сене к жизши бавой скрипл не натисапь бтя
                  Ответить

                  • ava bugmenot 28.04.2011 17:10 # 0

                    херовато работает это ваше шуффле
                    Ответить

                    • ava guest 28.04.2011 17:14 # +2

                      надо вот такую хуету добавить:

                      var l = consPos.length;
...
rnd = ~~(Math.random() * (l - i < 10 ? l - i : 10))


                      тогда будет заебца
                      Ответить

    • ava wvxvw 26.04.2011 22:57 # +3

      У меня прям даже голова от этого начала болеть... что автор хотел сказать, когда начал с negative lookahead?
      Ответить

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