1. Куча / Говнокод #2460

    +129.4

    1. 1
    2. 2
    3. 3
    <a class="btn" href="#" onclick="{if $client->getId()}xmlrpc_choose_playlist(this,{$mp3->getId()},1,'top_month_mp3');return false;{else}xmlrpc_load_popup('login', null);{/if}">
<img src="views/default/images/add.gif" width="8" height="8" alt="Добавить в плейлист" title="Добавить в плейлист" />
</a>

    Это кусок смартявского шаблона
    Типа песенку в плейлист добавляем


    а иногда таких песен на странице ни одна сотня )))

    Запостил: paranoid, 20 Января 2010

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

    • ava guest 21.01.2010 07:34 # 0

      С нормальным мозгом и jQuery это сокращается раза в 3-4.
      Ответить

    • ava striker 21.01.2010 08:27 # 0

      в чем говнокод? в том, что есть инлайн обработчик onclick?
      а как иначе? с учетом того, что у каждой функции свой аргумент в виде {$mp3->getId()}
      Ответить

      • ava cheef 21.01.2010 11:57 # 0

        а низя сразу все данные кинуть в json, а потом с jquery повесить событие на onclick?
        Ответить

        • ava striker 21.01.2010 12:38 # 0

          а как определить к какому элементу вешать тот или иной элемент json? по порядку?
          Ответить

          • ava guest 21.01.2010 12:57 # 0

            Можно по порядку, можно по id, можно вообще эти ссылки при выполнении создавать.
            Ответить

            • ava striker 21.01.2010 13:17 # 0

              а если жс отключен?
              Ответить

              • ava Pencioner 21.01.2010 13:23 # 0

                если отключен, то и инлайн обработчик onclick сцуко не выполнится ;)
                Ответить

                • ava striker 21.01.2010 13:24 # 0

                  я про третий вариант, когда ссылки создаются при загрузке страницы
                  Ответить

              • ava cheef 21.01.2010 17:46 # 0

                js - стандарт де факто, а если пользователь слепой? 0-о
                Ответить

                • ava striker 22.01.2010 01:51 # 0

                  В идеале сайт должен работать и без жс, а уже всякие вкусности добавляются через него.
                  Попробуйте, скажем, запустить Говнокод.ру без жс. Всё будет работать, хоть и не так красиво
                  Ответить

                  • ava cheef 22.01.2010 12:25 # 0

                    это не всегда реально сделать, если речь идёт, например, о нагруженном яваскриптами сайтом с медиа-контентом, хоть и признак хорошего тона..
                    Ответить

                    • ava paranoid 22.01.2010 15:25 # 0

                      но в приведённом мной примере это можно было элементарно сделать
                      Ответить

          • ava cheef 21.01.2010 17:45 # −5

            показать все, что скрытоа не надо ничего определять, весь контент должен быть привязан к js
            Ответить

    • ava noisiaman 23.01.2010 23:07 # 0

      Все нормально, шеф.
      Ответить

    • ava wvxvw 29.01.2010 22:32 # 0

      Плюсанул и вот почему:
      в xmlrpc_choose_playlist достаточно просто передать this а не плодить повторяющиеся переменные по шалблону. Это тот же "китайский" подход, только немного замаскированый.
      Ответить

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