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

    +149

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    <?php
while($menu = mysql_fetch_assoc($q))
{
	?>
	<option value="<?=$menu['id']?>"<?=($id === $menu['id']) ? ' selected="selected"' : ''?>><?=$menu['title']?></option>
	<?php
}
?>

    Адовейший пиздец наговнокодил сегодня. Ну не люблю я HTML-теги в PHP писать, о шаблонизаторах никогда не слышал

    Запостил: 3Jlou_OguHo4ka, 30 Июня 2013

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

    • ava bormand 30.06.2013 11:08 # +2

      Ну так PHP сам по себе вполне пригоден в качестве шаблонизатора. Правда есть и недостаток - нет автоматической экранировки подставляемых значений, и х.з. как ее прикрутить. Да и <?php длинновато, короткие теги не поощряются, а ASP теги, емнип, обычно отключены.
      Ответить

      • ava 3Jlou_OguHo4ka 30.06.2013 11:19 # 0

        ASP-теги у меня включены, но мне кажется юзать их !== Ъ
        Ответить

      • ava inkanus-gray 30.06.2013 12:02 # +1

        Я подумал о том, что всякий программизм типа ифов лучше вынести в хелперы. Т. е. написать так: 
        function helper($id, $menu)
  {
    return ($id === $menu['id']) ? ' selected="selected"' : '';
  }
        А в шаблоне так:
        <option value="<?=$menu['id']?>"<?=helper($id, $menu)?>><?=$menu['title']?></option>
        Или вообще все атрибуты собирать в хелпере. Так и проблему экранировки решить можно.
        Ответить

        • ava bormand 30.06.2013 13:22 # +2

          Ну не все и не всегда можно вынести в хелперы. Тот же цикл по коллекции или проверку коллекции на пустоту и вывод ее заменителя, имхо, в хелпер выносить не стоит.
          Ответить

    • ava roman-kashitsyn 30.06.2013 11:22 # +6

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

      • ava 3Jlou_OguHo4ka 30.06.2013 11:33 # 0

        Ваш комментарий настолько толстый, что не поместился на моем 28-дюймовом дисплее, а жир потек прямо из корпуса.
        Ответить

      • ava eth0 30.06.2013 14:49 # +7

        Типичный второго рода. Типичный первого рода выводил бы всё через echo.
        Ответить

    • ava inkanus-gray 30.06.2013 11:43 # 0 

      <?php
while($menu = mysql_fetch_assoc($q))
{
  echo '<option value="'.$menu['id'].'"'.(($id === $menu['id']) ? ' selected="selected"' : '').'>'.$menu['title'].'</option>';
}
?>
      Так не лучше? Или это выглядит слишком по-программерски?
      Ответить

      • ava 3Jlou_OguHo4ka 30.06.2013 11:49 # 0

        Это все выглядит замечательно, честное слово. Раньше так и писал. Потом упоролся по выносу HTML-кода за пределы PHP.
        Ответить

        • ava inkanus-gray 30.06.2013 11:56 # +6

          Можно и не выносить. Можно вообще не писать HTML: 
          $tugoment = new DOMDocument();
while($menu = mysql_fetch_assoc($q))
  {
    $element = $tugoment->createElement('option', $menu['title']);
    $element->setAttribute('value', $menu['id']);
    if ($id === $menu['id']) $element->setAttribute('selected', 'selected');
    $tugoment->appendChild($element);
  }
echo $tugoment->saveXML();
          Ответить

    • ava nonamez 01.07.2013 09:13 # 0

      а не проще ли всё это сначало в массив, а затем одной строчкой через implode?
      Ответить

    • ava adkazakov 01.07.2013 21:49 # +1

      А выше наверно еще и SQL код, с неэкранированной вставкой переменных, и комментариями на русском языке. Все по Попову (сами думайте какому).
      Ответить

      • ava 3Jlou_OguHo4ka 01.07.2013 22:06 # 0

        Нет, слава аллаху, до видеоуроков я не докатился, а в плане фильтрации - в этом плане я параноик, чекаю $_SERVER['REMOTE_ADDR'] два раза - через explode(".") и регуляркой
        Ответить

        • ava Stallman 01.07.2013 22:14 # +2

          Лолшто? Код в студию.
          Ответить

        • ava bormand 01.07.2013 22:20 # +1

          > через explode(".") и регуляркой
          ipv6 на вас нет.
          Ответить

          • ava 3Jlou_OguHo4ka 01.07.2013 22:21 # 0

            Использую register_globals on
            Ответить

            • ava bormand 01.07.2013 22:24 # +1

              Сорри, я пост поправил, потер свой первый вопрос ;)

              >> А $_GET[...] и $_POST[...] тоже чекаете?
              > Использую register_globals on
              А чем оно помогает в проверке данных, пришедших от юзера?
              Ответить

            • ava Lure Of Chaos 01.07.2013 23:31 # +3

              > Использую register_globals on

              (далее следует непереводимая игра слов с использованием местных идиоматических выражений)


              однажды меня попросили перенести пхпшный сайт с одного хостинга на другой. скопировал файлы и БД, один к одному, проверил - не работает.
              как вы наверное догадались, дело было как раз в глобалсах, но пришлось потратить время на выяснение этого факта. обматерив разработчика, сделал на скорую руку софверную эмуляцию глобалсов (с помощью extract).
              так я обрел сакральное знание из мира пхп, что php.ini может сломать сайт.
              а также открыл для себя, что тот сайт - чуть менее, чем полностью состоял из отборнейшего говна с мегаswitchем и инъекциями - кстати, наиболее смешные куски попали сюда.
              Ответить

              • ava bormand 02.07.2013 05:37 # 0

                А еще php.ini может лишить говносайт призрачной безопасности, если выключить maagic_quotes...
                Ответить

                • ava Vasiliy 02.07.2013 12:07 # 0

                  Есть еще более тонкий вброс включить magic_quotes_runtime.
                  Ответить

    • ava guest 02.07.2013 00:02 # −14

      показать все, что скрыто
      * g o a t s e x * g o a t s e x * g o a t s e x *
g                                               g  
o /     \             \            /    \       o
a|       |             \          |      |      a
t|       `.             |         |       :     t
s`        |             |        \|       |     s
e \       | /       /  \\\   --__ \\       :    e
x  \      \/   _--~~          ~--__| \     |    x  
*   \      \_-~                    ~-_\    |    *
g    \_     \        _.--------.______\|   |    g
o      \     \______// _ ___ _ (_(__>  \   |    o
a       \   .  C ___)  ______ (_(____>  |  /    a
t       /\ |   C ____)/      \ (_____>  |_/     t
s      / /\|   C_____)       |  (___>   /  \    s
e     |   (   _C_____)\______/  // _/ /     \   e
x     |    \  |__   \\_________// (__/       |  x
*    | \    \____)   `----   --'             |  *
g    |  \_          ___\       /_          _/ | g
o   |              /    |     |  \            | o
a   |             |    /       \  \           | a
t   |          / /    |         |  \           |t
s   |         / /      \__/\___/    |          |s
e  |           /        |    |       |         |e
x  |          |         |    |       |         |x
* g o a t s e x * g o a t s e x * g o a t s e x *
      Ответить

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