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

    +158

    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
    <span class="n2"><?//$productAttr
            if ($cpage->title == 'Поршневой насос') {
			echo $cpage->title;
			}
			else {echo $nm." ".$cpage->title; }?></span>

if(isset($productA) && !empty($productA)){
				echo '<span class="tehno" style="font-size:12px; text-align: left; heifht:90px;">';
				foreach($productA as $k => $v){
					if($v['product_id']==$cpage->id){
						if(!empty($v['ProductAttrValue'])) {
						
							if ($v['AttrName'] == 'Вес') { $prep = 'кг'; }
							else if ($v['AttrName'] == 'Дальность подачи по горизонтали') { $v['AttrName'] = 'Дальность подачи'; $prep = 'м'; }
							else if ($v['AttrName'] == 'Производительность') { $prep = 'м3/ч'; }
							else if ($v['AttrName'] == 'Высота подачи') { $prep = ''; }
						
							echo $v['AttrName'].': '.$v['ProductAttrValue'].' '.$prep.'<br>';
						}
					}
				}	
				echo '</span>';
}

    И такого кода по всему проекту ооооооочень много =(

    Запостил: farit_slv, 26 Декабря 2014

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

    • ava farit_slv 26.12.2014 18:08 # 0

      И вот откуда берется переменная $nm

      $v=1;
      $nm="";
      if (isset($list[$_SERVER['REQUEST_URI']])) {
      $nm = $list[$_SERVER['REQUEST_URI']]." ";
      $v=2;
      }
      Ответить

    • ava bormand 26.12.2014 18:50 # 0

      А почему поршневой насос без $nm?
      Ответить

      • ava farit_slv 26.12.2014 18:52 # +1

        мне бы самому это знать))) Я сижу и только "радуюсь" от этого кода... Ребята не реально "круто знают" Yii framework
        Ответить

      • ava guest 26.12.2014 23:48 # 0

        поршневой насос без нанометров
        Ответить

    • ava guest 27.12.2014 20:33 # −1

      Типичное пыхоговно. Пыховцы иначе не умеют
      Ответить

      • ava inkanus-gray 27.12.2014 22:11 # 0

        Пыховцы ещё умеют так:
        <style type="text/css">
  .tehno {
    font-size:12px; 
    text-align: left; 
    height:90px;
  }
</style>
<?php
function filterParams($params) {
    return array_filter($params, function($value) {return $value !== 'Поршневой насос'; });
}

function combineTitle($params) {
    return implode(' ', filterParams($params));
}

function combineTwo($nm, $title) {
    return combineTitle(array($nm, $title));
}

function unit($attrname) {
    $table = array(
        'Вес' => 'кг',
        'Дальность подачи по горизонтали' => 'м',
        'Производительность' => 'м3/ч',
        'Высота подачи' => ''
    );
    if (array_key_exists($attrname, $table)) {
        return $table[$attrname];
    }
}

function combineAttrs($v) {
    if(!empty($v['ProductAttrValue'])) {
        return implode(' ', array($v['AttrName'].':', $v['ProductAttrValue'], unit($v['AttrName']));
    }
}

function product($v, $id) {
    if ($v['product_id'] === $id) {
         return combineAttrs($v).'<br>';
    }
}
?>
<span class="n2"><?= combineTwo($nm, $cpage->title) ?></span>
<?php if(isset($productA) && !empty($productA)){ ?>
    <span class="tehno">
        <?php foreach($productA as $k => $v){ ?>
            <?= product($v, $cpage->id)?>
	<?php }	 ?>
    </span>
<?php } ?>
        Ответить

        • ava bormand 27.12.2014 22:37 # +1

          А ведь в оригинале разобраться было проще...
          Ответить

          • ava inkanus-gray 27.12.2014 23:09 # 0

            А в enterprise edition завезём ООП, тогда будет ещё «понятнее»...
            Ответить

        • ava guest 28.12.2014 15:43 # 0

          Кто скажет, что пхп не может в шаблоны?
          Ответить

          • ava inkanus-gray 28.12.2014 15:45 # 0

            ПХП — лучший шаблонизатор! Переходите на ПХП!
            Ответить

            • ava bormand 28.12.2014 15:46 # 0

              "ПХП" — лучший шаблонизатор! Не то что эти ваши "c++".
              Ответить

              • ava inkanus-gray 28.12.2014 15:48 # 0

                Сравните, сколько времени нужно на изучение крестошаблонов, с тем, сколько времени уйдёт на изучение пыхошаблонов. Вывод: на ПХП изучение+разработка займут меньше времени, чем на C++.
                Ответить

                • ava heyzea1 29.12.2014 12:26 # 0

                  Вы говорите это так, как будто это что-то хорошее.
                  Ответить

      • ava guest 02.01.2015 16:34 # 0

        Еще и с шорт-тегами.
        Ответить

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