1. PHP / Говнокод #10409

    +64

    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
    /*
 *---------------------------Темка на одном seo форуме...
 *Проблемка вот в чём:
 *используется такая конструкция
*/

echo '<td class="play"><span class="doRadio" onclick="doRadio('.$row['alt_name'].', '.$row['name'].');">Play</span></td>';

/*
 *при правильном раскладе должно получиться так
 *<span class="doRadio" onclick="doRadio('kissfmua', 'KISS FM');">Play</span>
 *но теряются одиночные кавычки ('kissfmua', 'KISS FM') как я не менял выдает ошибку
 *подскажите!
*/

/*
 *---------------------------Ответ одного формучанина
echo '<td class="play"><span class="doRadio" onclick="doRadio(\''.$row['alt_name'].'\', \''.$row['name'].'\');">Play</span></td>';

    Я просто в ахуе...

    Запостил: kise97, 29 Мая 2012

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

    • ava Lure Of Chaos 30.05.2012 00:24 # +4

      уже и кавычки не осиливают? куда катится мир...
      Ответить

      • ava Vasiliy 30.05.2012 07:41 # +5

        Популярность пыха привела к тому, что он становится первым языком который пытаются освоить юные прогеры и как видно, Зря.
        Ответить

        • ava vistefan 30.05.2012 07:50 # +2

          Причём те, кто делает это неудачно, это мягко говоря не очень умные люди, и непонятно, как они вообще выживают, и почему именно на нашу святую область деятельности обрушивается ТАКОЕ их количество...
          Ответить

        • ava Lure Of Chaos 30.05.2012 08:42 # +1

          пожалуй, если так и дальше пойдет, рискуем увидеть "стать настоящимпрограммистом на php за 24 часа, не написав ни строчки кода, по методу Зелены Длявидувой: быстро, весело, прибыльно"
          Ответить

          • ava vistefan 30.05.2012 09:14 # +3

            Да, где в конце книги идёт более сложный материал, а именно - что такое компьютер.
            Ответить

    • ava 7ion 30.05.2012 15:39 # −2

      А чем неправилен ответ?
      Ответить

      • ava kise97 30.05.2012 16:57 # 0

        А зачем городить кучу слешей и т.п.?
        Ответить

        • ava 7ion 30.05.2012 17:04 # 0

          Кучу - это четыре?
          И т.п. - что?
          Как бы вывели вы?
          Через HEREDOC?
          Ответить

          • ava kise97 30.05.2012 19:57 # −5

            показать все, что скрытоecho "<td class='play'><span class='doRadio' onclick='doRadio(".$row['alt_name'].", ".$row['name'].");'>Play</span></td>";

            или, например

            <td class="play"><span class="doRadio" onclick="doRadio('<?=$row['alt_name']?>', '<?=$row['name']?>');">Play</span></td>
            Ответить

            • ava bormand 30.05.2012 20:04 # +1

              В вашем первом варианте не хватает кавычек вокруг аргументов doRadio.
              Ответить

              • ava kise97 30.05.2012 20:06 # −4

                ну Х с ними. главное, что смысл понятен.
                Ответить

                • ava bormand 30.05.2012 20:10 # +3

                  В варианте с <?= => да. А в том - вы тупо поменяли ' и " местами, и их точно также придется заэкранировать (да да, четырьмя слешами)
                  Ответить

            • ava volhv 30.05.2012 23:29 # 0

              Shorttags зло
              Ответить

              • ava Lure Of Chaos 30.05.2012 23:43 # 0

                как раз длинные - зло. и зло, что короткие можно через конфигу отключить.
                Ответить

                • ava roman-kashitsyn 30.05.2012 23:51 # +2 

                  <?xml version="1.0"?>
                  Ничего не напоминает?
                  Ответить

                  • ava Lure Of Chaos 31.05.2012 00:18 # 0 

                    <%
   String name = request.getParameter( "username" );
   session.setAttribute( "theName", name );
%>
<%= session.getAttribute( "theName" ) %>
                    в JSP теги злом никто не называет
                    Ответить

                    • ava roman-kashitsyn 31.05.2012 00:38 # 0

                      во-первых, их нельзя перепутать с инструкциями xml
                      во-вторых, xml-теги вроде как являются предпочтительной формой 
                      <c:set var="name" value="${param.username}"/>
                      Ответить

                • ava volhv 31.05.2012 00:53 # +1

                  а вот так 
                  <? MyScript detail="2"?>
                  каково?
                  причем, это вполне себе xml,
                  или так 
                  <?ScriptA emphasize="yes" ?>
                  Ответить

                • ava bormand 31.05.2012 05:49 # +4

                  Да, согласен, но надо было сделать их через % вместо ?.

                  P.S. Шаблон на PHP с длинными тегами смотрится как г-но.
                  Ответить

          • ava Vasiliy 31.05.2012 06:37 # −3

            echo "траляля тралаля '{array[$i]}' {array[$j]} траляля"; вот и читаемо и нетбианс нормально подсвечивает переменные, да и в блокноте видно. И городить кучу слешей не надо.
            Ответить

            • ava bormand 31.05.2012 07:38 # +3

              И чем это поможет в случае с тремя уровнями кавычек?
              Первый уровень - кавычки от эха.
              Второй уровень - кавычки от атрибута.
              Третий уровень - кавычки от параметра doRadio.

              Так?
              echo "<td class='play'><span class='doRadio' onclick='doRadio(\"{$row['alt_name']}\", \"{$row['name']}\");'>Play</span></td>";


              Предлагаю думать, и возможно даже писать пример, прежде чем писать, что "городить кучу слешей не надо".
              Ответить

    • ava oooZinka 31.05.2012 13:31 # +3

      Все было бы красиво, если не использовать такую связку php и javascript.
      var player_control = {};
player_control.play = document.getElementById('player_play');
player_control.play.onclick = function(){
	window.doRadio('<?=$row['alt_name'];?>','<?=$row['name'];?>');
	return false;
}

      Используем httprequest + json блеать.

      Если мы не знаем, что такое шаблонизатор:
      echo '<td class="play"><span id="player_play" class="doRadio">Play</span></td>';


      Если мы знаем:
      <td class="play"><span class="doRadio" onclick="doRadio('{alt_name}', '{name}');">Play</span></td>
      Ответить

      • ava roman-kashitsyn 31.05.2012 13:42 # +1

        Дело глаголишь, Зинка
        сам хотел об этом написать, но поленился
        Ответить

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