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

    +167

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    var pic_jpg = /(\.jpg)$/ig;
var pic_jpeg = /(\.jpeg)$/ig;
var pic_png = /(\.png)$/ig;
var pic_bmp = /(\.bmp)$/ig;
 var a = $("a");
$.each(a, function (i, val) {
		var src = $(val).attr("href");
		if (src !== undefined) {
				if (src.match(pic_jpg)) {
						AppendImage(i, src, 'a');
				}
				if (src.match(pic_jpeg)) {
						AppendImage(i, src, 'a');
				}
				if (src.match(pic_png)) {
						AppendImage(i, src, 'a');
				}
				if (src.match(pic_bmp)) {
						AppendImage(i, src, 'a');
				}
		}
});

    Запостил: DarkThinker, 06 Июня 2012

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

    • ava ikenfin 06.06.2012 17:13 # +2

      ну да, зачем писать альтернативы в регулярке? Только хардкор!=)
      Ответить

      • ava Vindicar 06.06.2012 20:27 # +1

        массивы? Не, не слышал.
        Ответить

        • ava bormand 06.06.2012 20:39 # +3

          '|' ?, не, не слышал.

          var pics = /(\.jpg|\.jpeg|\.png|\.bmp)$/i;

if (src.match(pics)) {
    AppendImage(i, src, 'a');
}

          Зачем тут массивы ?)
          Ответить

          • ava HottDogg 06.06.2012 20:43 # +4

            RegExp.test тоже не слышал :)
            Ответить

          • ava wvxvw 06.06.2012 22:44 # +2

            А еще можно было бы строку развернуть и искать в обратную сторону, а то регулярки умеют только слева направо, что в данном случае непрактично (poker face).
            Ответить

            • ava Lowezar 06.06.2012 23:27 # +1

              Обрати внимание на $ в конце - это служит регуляркам достаточной подсказкой.
              ...Хотя честно говоря не знаю, какая реализация регулярок в JS, может и правда не умеет. :)
              Ответить

              • ava wvxvw 07.06.2012 00:12 # −1 

                function extensionMatches(s){
  	   for (var i = s.length - 1, r = "";
	       i >= 0 && r.length < 4 && r[i] != ".";
	       r += s[i], i--);
	   return /^(ge?pj|gnp|pmb)/i.test(r);
}
                :3
                Ответить

          • ava Vindicar 07.06.2012 18:42 # +1

            Ну просто массивы обычно проходят ДО регулярок. Я поверю что автор не знал про |, но не знать это... "JS за 24 минуты" детектед.
            Ответить

          • ava Connor 14.06.2012 21:07 # 0

            var pics = /\.(jpe?g|png|bmp)$/i;
            Ответить

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