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

    • ava Teddy_Brown 29.09.2011 11:55 # −2

      разработчик - школьник.
      Ответить

      • ava nethak 29.09.2011 11:57 # +1

        Когда у тебя максимально сжатые сроки, такое из под палки не только школьник напишет ;) Хотя тут пожалуй и перебор...
        Ну а школьнику желаю удачи, пусть учится, все мы когда писали не лучше, а когда и хуже.
        Ответить

        • ava DeusModus 29.09.2011 11:59 # 0

          Там просто цикл. Даже если БД через жопу, даже если руки растут из ада, можно же использовать for вместо бесконечных if else.
          Ответить

          • ava nethak 29.09.2011 12:05 # −2

            Ну здесь не спорю, ну всё ведь ещё и зависит от уровня «автора», он низок, смерись =), будь добрее и к тебе люди потянутся)
            Ответить

        • ava Teddy_Brown 29.09.2011 12:01 # 0

          Я такое напишу только если " из под палки ". И то наврядли.
          Ответить

    • ava nethak 29.09.2011 11:56 # −1

      Не новшество среди «гавнокода», не удивило. Остаётся добавить только:
      Welcome to Hell.
      И пожелать Вам удачи, в подтирании «...» за чужим «разработчиком»
      Ответить

    • ava autor_mudak 29.09.2011 12:23 # +2

      Разработчик - я. Полтора года назад.

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

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

      Ну а бедному автору могу помочь и переписать чутка, а то вон как разозлился:
      <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>
      Ответить

      • ava roman-kashitsyn 29.09.2011 12:26 # +5

        "Найди меня", govnokod edition
        Ответить

      • ava DeusModus 29.09.2011 12:26 # +1

        :D Все собрались. Никто не разозлился, так подохуели чуток. В WebAsyst Shop-Script есть пожоще места.
        Мир тебе и не пиши больше так, а то придет Тедди Браун :P
        Ответить

      • ava Nemoden 29.09.2011 14:08 # +1 

        if (!empty($uno_el['link']))

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

        • ava autor_mudak 29.09.2011 14:29 # 0

          Да в цикле несомненно флаг должен быть true.
          Ответить

          • ava i-- 29.09.2011 16:08 # 0

            И несомненно надо закрыть не условие, а цикл.
            Ответить

      • ava p5ych0 30.09.2011 15:32 # 0

        все никак понять не могу. а нафига каждый раз <?php и ?> ???!!!
        один раз открыл, один закрыл, эхом вывел все что надо. даже читать удобнее было бы
        Ответить

    • ava Teddy_Brown 29.09.2011 12:26 # −1

      прям передача "Жди меня" какая-то!
      Ответить

    • ava guest 30.09.2011 12:34 # −2

      а чо вы здесь срете кирпичами? php - это в первую очередь шаблонизатор.
      Да, реализация немножко хромает, но блять, она работает!
      Ответить

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