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

    +155

    1. 1
    element.data('selected') ? element.show() : $.noop();

    Либо человек из ассемблера пришел, либо "IF" сильно ему по жизни насолил.

    Запостил: InnerJoin, 11 Июля 2012

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

    • ava shmaltorhbooks 11.07.2012 01:14 # −3

      тернарный оператор очень клёвый, если его с умом использовать. тут всё нормально.
      Ответить

      • ava sap1ens 11.07.2012 05:45 # +5

        Вы наверное не в курсе, что такое $.noop()? =) Здесь его применение на 100% не оправдано.

        if(element.data('selected')) {
        element.show()
        }

        Выглядит гораздо понятнее.
        Ответить

        • ava Lowezar 11.07.2012 07:27 # 0

          А где его применение вообще оправдывается? :)
          ...Ну, ASM, разумеется, не трогаем, я сам системщик. :) А в джаваскрипте зачем?
          Ответить

          • ava sap1ens 11.07.2012 07:31 # +1

            Например как mock для callback'ов
            Ответить

          • ava Fai 11.07.2012 08:15 # 0

            А в асм он зачем? Для выравнивания?
            Ответить

            • ava Lowezar 11.07.2012 10:00 # 0

              Для задержки при работе с периферией в основном.
              Ответить

              • ava Lowezar 11.07.2012 10:08 # +2

                Иногда ещё чтобы заткнуть дырку на месте проверок регистрации/активации/подлинности. :) Не в своём софте, разумеется. :-]
                Ну и выравнивание тоже, но компилер обычно сам этим занимается, в универе эту тиму благополучно упустили а на практике я по вебу пошёл, личного опыта нет...
                Ответить

                • ava Fai 11.07.2012 15:16 # 0

                  Просто я сам как-бы системщик, но nop не использовал никогда.
                  Ответить

        • ava oneguy 20.07.2012 02:30 # 0

          Можно ещё так:
          element.data('selected')&&element.show() ;
          Ответить

    • ava Fai 11.07.2012 05:18 # 0

      > $.noop()

      $noopy
      Ответить

    • ava WGH 11.07.2012 17:27 # +6 

      (element.data('selected') ? element.show : $.noop)();
      Ответить

      • ava bormand 11.07.2012 17:41 # +4

        Функциональное программирование...
        Ответить

      • ava anonimb84a2f6fd141 12.07.2012 00:11 # +2

        Может не покатить:
        var element = {
	show: function() {
		alert(this == element);
	}
};
(true ? element.show : noop)();
        Ответить

        • ava WGH 12.07.2012 00:16 # 0

          Офигеть.
          Ответить

          • ava wvxvw 12.07.2012 13:49 # 0

            Ну так это по-сути: element.show.call() только в профиль. Можно было бы:
            (element.data('selected') ? element.show : $.noop).apply(element);

            тогда бы все работало как задумано.
            Ответить

    • ava rsod 13.07.2012 05:08 # 0

      позвольте, зачем jquery, есть же void(0), чем он хуже
      Ответить

      • ava wvxvw 13.07.2012 10:09 # 0

        Это вообще не функция. void это оператор который убирает значение вычисленное в предыдущей операции. В таком варианте как у вас - он вообще ошибочен / бесполезен.
        В принципе, его не нужно использовать никогда, но в теории, что-то типа такого могло бы работать (только не понятно зачем):
        var a = 0, b = 1;
a = void ++b;
console.log("a = " + a + " b = " + b); // a = undefined b = 2
        Ответить

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