- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
- 16
- 17
- 18
- 19
- 20
- 21
- 22
- 23
- 24
- 25
- 26
- 27
- 28
- 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>
Эта хуита выводит цену. И так все шаблоны. Разработчик - мудак.
Teddy_Brown 29.09.2011 11:55 # −2
nethak 29.09.2011 11:57 # +1
Ну а школьнику желаю удачи, пусть учится, все мы когда писали не лучше, а когда и хуже.
DeusModus 29.09.2011 11:59 # 0
nethak 29.09.2011 12:05 # −2
amazondev 29.09.2011 12:09 # 0
Teddy_Brown 29.09.2011 12:30 # +1
Teddy_Brown 29.09.2011 12:10 # −1
Teddy_Brown 29.09.2011 12:01 # 0
nethak 29.09.2011 11:56 # −1
Welcome to Hell.
И пожелать Вам удачи, в подтирании «...» за чужим «разработчиком»
autor_mudak 29.09.2011 12:23 # +2
Помимо сроков, задача менялась и видоизменялась параллельно с разработкой. Что-то дорабатывалась уже после задачи готового проекта, и красоту наводить там не надо было, а надо было чтоб работало.
Само собой рефакторинг некому не нужен был. К тому же поддерживаю проект как видите - не я. Да и бюджет был, мягко скажем, не щедрый.
Ну а бедному автору могу помочь и переписать чутка, а то вон как разозлился:
roman-kashitsyn 29.09.2011 12:26 # +5
autor_mudak 29.09.2011 12:28 # +7
DeusModus 29.09.2011 12:26 # +1
Мир тебе и не пиши больше так, а то придет Тедди Браун :P
Teddy_Brown 29.09.2011 12:30 # −4
Nemoden 29.09.2011 14:08 # +1
в противном случае, если у массива $uno_el индекс 'link' не определен, вывалится E_NOTICE. То же касается и <?php if ($uno_el['pr'.$i]):?>
И все-равно корявая вьюха получается - можно было бы данные подготовить и прогнать foreach'ем просто, а тут for и проверки еще внутри. А еще нихера не понятно зачем $flag_k, если он всегда false. Ну, а вообще, это очень мелкие придирки уже, конечно.
autor_mudak 29.09.2011 14:29 # 0
i-- 29.09.2011 16:08 # 0
p5ych0 30.09.2011 15:32 # 0
один раз открыл, один закрыл, эхом вывел все что надо. даже читать удобнее было бы
roman-kashitsyn 30.09.2011 15:34 # 0
i-- 30.09.2011 17:53 # 0
Teddy_Brown 29.09.2011 12:26 # −1
guest 30.09.2011 12:34 # −2
Да, реализация немножко хромает, но блять, она работает!