"PHP" / Говнокод #8029 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  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 amazondev, (Updated )

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

    • ava nethak # Ссылка на оригинал 0 ⇈

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

  • ava nethak # Ссылка на оригинал 0

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

  • ava autor_mudak # Ссылка на оригинал 0

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

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

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

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

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

    • ava Nemoden # Ссылка на оригинал 0 ⇈ 

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

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

    • ava p5ych0 # Ссылка на оригинал 0 ⇈

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

  • ava guest # Ссылка на оригинал 0

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

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

Семь раз отмерь — один отрежь, guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓