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

    +160

    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
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    <div class="pmi_cont pr" style="width:60px;float: left; min-height: 20px;z-index: 5;"><?php $k = 0;?>
                <?php if ($uno_el['pr1']): ?>
                    <div class="price_1"
                         style="height:25px;"><?php echo $uno_el['pr1']; $k = 1; echo is_numeric($uno_el['pr1']) ? ' руб.'
                            : ''; ?></div><?php endif;?>
                <?php if ($uno_el['pr2']): ?>
                    <div class="price_2"
                         style="height:25px;"><?php echo $uno_el['pr2']; $k = 1; echo is_numeric($uno_el['pr2']) ? ' руб.'
                            : ''; ?></div><?php endif;?>
                <?php if ($uno_el['pr3']): ?>
                    <div class="price_3"
                         style="height:25px;"><?php echo $uno_el['pr3']; $k = 1; echo is_numeric($uno_el['pr3']) ? ' руб.'
                            : ''; ?></div><?php endif;?>
                <?php if ($uno_el['pr4']): ?>
                    <div class="price_4"
                         style="height:25px;"><?php echo $uno_el['pr4']; $k = 1; echo is_numeric($uno_el['pr4']) ? ' руб.'
                            : ''; ?></div><?php endif;?>
                <?php if ($uno_el['pr5']): ?>
                    <div class="price_5"
                         style="height:25px;"><?php echo $uno_el['pr5']; $k = 1; echo is_numeric($uno_el['pr5']) ? ' руб.'
                            : ''; ?></div><?php endif;?>
                <?php if ($k): ?>
                    <?php if ($uno_el['link']): ?>
                        <div style="margin-left: -60px; width: 200px;"><?php echo $uno_el['link'];?></div><?php endif; ?>
                    <?php else: ?>
                    <?php if ($uno_el['link']): ?>
                        <div style="position:relative;left: -80px; width: 140px;"><?php echo $uno_el['link'];?></div><?php endif; ?>
                    <?php endif;?>
            </div>

    Эта хуита выводит цену. И так все шаблоны. Разработчик - мудак.

    Запостил: amazondev, 29 Сентября 2011

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

    • разработчик - школьник.
      Ответить
      • Когда у тебя максимально сжатые сроки, такое из под палки не только школьник напишет ;) Хотя тут пожалуй и перебор...
        Ну а школьнику желаю удачи, пусть учится, все мы когда писали не лучше, а когда и хуже.
        Ответить
        • Там просто цикл. Даже если БД через жопу, даже если руки растут из ада, можно же использовать for вместо бесконечных if else.
          Ответить
          • Ну здесь не спорю, ну всё ведь ещё и зависит от уровня «автора», он низок, смерись =), будь добрее и к тебе люди потянутся)
            Ответить
        • Я такое напишу только если " из под палки ". И то наврядли.
          Ответить
    • Не новшество среди «гавнокода», не удивило. Остаётся добавить только:
      Welcome to Hell.
      И пожелать Вам удачи, в подтирании «...» за чужим «разработчиком»
      Ответить
    • Разработчик - я. Полтора года назад.

      Помимо сроков, задача менялась и видоизменялась параллельно с разработкой. Что-то дорабатывалась уже после задачи готового проекта, и красоту наводить там не надо было, а надо было чтоб работало.

      Само собой рефакторинг некому не нужен был. К тому же поддерживаю проект как видите - не я. Да и бюджет был, мягко скажем, не щедрый.

      Ну а бедному автору могу помочь и переписать чутка, а то вон как разозлился:
      <div class="price_list">
      	<?php $flag_k = false;?>
      	<?php for($i=0; $i<=5; $i++):?>
      		<?php if ($uno_el['pr'.$i]):?>
      			<div class="price price_<?php echo $i; ?>">
      			 <?php
      				$flag_k = false;			
      				echo $uno_el['pr'.$i] . is_numeric($uno_el['pr'.$i]) ? ' руб.' : ''; 
      			?>
      			</div>
      		<?php endif;?>
      	<?php endif; ?>
      	<?php if ($uno_el['link']): ?>
      		<div class="link link_type_<?php echo ($falg_k)?'1':'2';?>"><?php echo $uno_el['link'];?></div>
      	<?php endif; ?>
       </div>
      Ответить
      • "Найди меня", govnokod edition
        Ответить
      • :D Все собрались. Никто не разозлился, так подохуели чуток. В WebAsyst Shop-Script есть пожоще места.
        Мир тебе и не пиши больше так, а то придет Тедди Браун :P
        Ответить
      • if (!empty($uno_el['link']))

        в противном случае, если у массива $uno_el индекс 'link' не определен, вывалится E_NOTICE. То же касается и <?php if ($uno_el['pr'.$i]):?>
        И все-равно корявая вьюха получается - можно было бы данные подготовить и прогнать foreach'ем просто, а тут for и проверки еще внутри. А еще нихера не понятно зачем $flag_k, если он всегда false. Ну, а вообще, это очень мелкие придирки уже, конечно.
        Ответить
        • Да в цикле несомненно флаг должен быть true.
          Ответить
          • И несомненно надо закрыть не условие, а цикл.
            Ответить
      • все никак понять не могу. а нафига каждый раз <?php и ?> ???!!!
        один раз открыл, один закрыл, эхом вывел все что надо. даже читать удобнее было бы
        Ответить
    • прям передача "Жди меня" какая-то!
      Ответить
    • а чо вы здесь срете кирпичами? php - это в первую очередь шаблонизатор.
      Да, реализация немножко хромает, но блять, она работает!
      Ответить

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