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

    +159

    1. 1
    2. 2
    3. 3
    if (($("#menu-main").find("li span").is(".active")) && ($("#nav-wrapper").find("nav").is("#children_for_" + $("#menu-main").find("li span.active").parent("li").attr("id")))) {
    ...
}

    ...

    Запостил: tir, 02 Июля 2012

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

    • ava bormand 02.07.2012 14:41 # 0

      LINQ!
      Ответить

    • ava mr.The 02.07.2012 20:41 # +3

      Ну я не раз видел ситуацию типа:
      $('#something').parent().parent().parent().parent().parent().last();
      Ответить

      • ava 3.14159265 02.07.2012 20:49 # +1

        Да она и на гкоде много раз была.
        Кто-то тут даже предлагал: нужен parent(N), оно конечно полезно, но думаю children и has на них нету.
        Ответить

        • ava mr.The 02.07.2012 20:55 # 0

          Ну да, ещё и child(N).
          А вообще, подавляющее большинство такого кода заменяется чем-то вроде closest() или find().
          Ответить

      • ava sayidandrtfm 02.07.2012 21:50 # +2

        In a far far far far far far far far galaxy...
        Ответить

        • ava Lure Of Chaos 03.07.2012 09:05 # +5

          long().long().long().long().long().time( ).ago();
          Ответить

          • ava roman-kashitsyn 03.07.2012 10:44 # +1 

            I.like().fluent().api();
            Ответить

            • ava tir 03.07.2012 13:38 # +2

              а в groovy 2.0 можно написать
              I like fluent api
              Ответить

              • ava roman-kashitsyn 03.07.2012 15:01 # 0

                Попробовал запилить такое в Scala: она упорно отказывается кушать такие конструкции. Собственно, правильно делает: пойми тут, как это должно правильно интерпретироваться (пусть I - инстанс): 
                I.like().fluent().api()
I.like(fluent, api)
I.like(fluent).api()
                Ответить

                • ava tir 04.07.2012 07:21 # 0

                  груви скорее всего воспримет так:
                  I.like(fluent).getApi()
                  Ответить

                  • ava roman-kashitsyn 04.07.2012 10:40 # 0

                    Вот-вот: Scala тоже хочет fluent в качестве аргумента like и ругается на слишком большое количество аргументов.
                    Ответить

      • ava wvxvw 03.07.2012 22:27 # +3

        В Лиспе для таких вещей придуманы: caar, cadr, cddr, caadr, caddr, cdddr, cdadr, cdaar и т.д. В Й-квери можно было бы тоже что-нибудь такое добавить, например:
        $("element").pparent();
$("element").ppparent();
. . .
$("element").pppppppparent();

        Просто раньше моя идея про то, чтобы передавать параметр в parent(x) не прижилась :(
        Ответить

        • ava Lure Of Chaos 03.07.2012 22:42 # 0

          зато можно указать селектор парента
          Ответить

        • ava 3.14159265 04.07.2012 21:07 # 0

          И как это будет на уровне либы? Генерировать код для ста функций
          (вплоть до paaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa aaaaaaaaaarent) и проставлять их прототипы в цикле?
          > моя идея про то, чтобы передавать параметр в parent(x) не прижилась
          Ну она хоть звучит здраво и вполне вменяемо.
          Ответить

          • ava wvxvw 04.07.2012 21:56 # +1

            Ну почему, можно весь код завернуть в try-catch, если ловим ошибку о попытке доступа к неизвестному свойству, парсим сообщение об ошибке, и создаем свойство.

            Хотя, конечно, с идеей я сплоховал, копировать, так уж полностью:
            $("element").papaparent(); // эквивалентно parentNode().firstChild.parentNode().firstChild.parentNode();
$("element").ppparent(); // эквивалентно parentNode().parentNode().parentNode()
            :)
            Ответить

            • ava Fai 05.07.2012 06:34 # +2

              па-па-па па!
              Ответить

              • ava roman-kashitsyn 06.07.2012 12:46 # +3

                Нужно было ещё добавить методы
                $("element").mother().father().sister().brother();
// ppparnet is not readable enough!!!
$("element").grandgrandgrandmother()
                Ответить

                • ava roman-kashitsyn 06.07.2012 17:04 # 0

                  Кстати, python способен на такое 
                  http://ideone.com/vwUwo
                  а вот на JS, уверен, такое запилить не получится :)
                  Ответить

                  • ava wvxvw 06.07.2012 22:58 # 0

                    Только если свой интерпретатор запилить. Можно, как я уже говорил, ловить ошибку вызова несуществующей функции, но только внутри своей функции. С другой стороны - так многие и пишут, т.е. заворачивают весь код в одну функцию. Так что...
                    (function () {
    "use strict";
    try {
        var o = { toString : function () { return "[Object o]"; } };
        o.parent();
    } catch (e) {
        // called parent on [Object o]
        console.log("called " + e.arguments[0] + " on " + e.arguments[1]);
    }
})();
                    Ответить

                    • ava wvxvw 06.07.2012 23:18 # 0

                      Да, но это только в V8, в Симанки можно получить имя неправильной функции, а вот объект - не, только его "название". Но через eval() и до него добраться можно будет.
                      (function () {
    "use strict";
    var caller, method, o = { toString : function () { return "[Object o]"; } };
    try {
        o.parent();
    } catch (e) {
        // called parent on [Object o]
        caller = e.message.substring(0, e.message.indexOf("."));
        method = e.message.substring(caller.length + 1, e.message.indexOf(" "));
        console.log("called " + method + " on " + eval(caller));
    }
})();

                      Версия для Мозилы.
                      Ответить

                    • ava roman-kashitsyn 07.07.2012 01:02 # 0

                      Это совсем не то: клиент такого API должен сам заботиться о перехвате исключений и парсинге имён вызванных функций, что совершенно бесполезно. В python эта логика инкапсулирована в классе и прозрачна для пользователя API (пользователь не знает, реализован ли метод явно или генерируется налету, но в теории может узнать, если очень захочет).
                      Ответить

                      • ava wvxvw 07.07.2012 10:11 # 0 

                        /совсем не/не совсем/s
                        и все становится на свои места.
                        Ответить

                        • ava roman-kashitsyn 07.07.2012 11:43 # 0

                          Сфера - не совсем куб: есть гомеоморфизм, преобразующий одну поверхность в другую.
                          Сфера - совсем не "сфера с дыркой" - гомеоморфизма нет.
                          Ответить

                • ava Fai 06.07.2012 19:30 # +1

                  Загадка:
                  "Кем элементу приходится брат сестры отца его матери?"
                  Ответить

            • ava 3.14159265 05.07.2012 19:09 # 0

              >если ловим ошибку о попытке доступа к неизвестному свойству, парсим сообщение об ошибке, и создаем свойство
              адЪ. если бы такая фича появилась - я бы немедленно запостил на этот сайт.
              Ответить

              • ava wvxvw 05.07.2012 19:43 # 0

                По мотивам: 
                http://stackoverflow.com/questions/6240033/mutable-versions-of-cadr-caddr-etc
                Ответить

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