Говнокод

foreach ($item as $prop) {
            if ($prop['tag']==$this->props[0]){
                $new_item['sku']=(int)$prop['value'];
            } else {
                if ($prop['tag'] == $this->props[1]) {
                    $new_item['name']=$prop['value'];
                } else {
                    if ($prop['tag'] == $this->props[2]) {
                        $new_item['price']=(float)$prop['value'];
                    } else {
                        if ($prop['tag'] == $this->props[3]) {
                            $new_item['is_spec']=(float)$prop['value'];
                        } else {
                            if ($prop['tag'] == $this->props[4]) {
                                $new_item['spec_price']=(float)$prop['value'];
                            } else {
                                if ($prop['tag'] == $this->props[5]) {
                                    $new_item['old']=(float)$prop['value'];
                                }
                            }
                        }
                    }
                }
            }
        }

XML бессмысленный и беспощадный

Запостил:

madfriend, 02.10.2013 (Updated 26.03.2018)

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

spivti 13 лет ago # 0

код-гармошка, в книжках по рефакторингу есть рецепты по исправлению такого, да и на просторах инета:

http://www.rsdn.ru/article/net/BadCode.xml
Ответить
TarasB 13 лет ago # 0

Да просто кому-то впадлу было писать else if в одну строку, тогда б не было такого говна.
Ответить
- tirinox 13 лет ago # 0 ⇈
  
  <mode type="jedy">Тут нужен цикл.</mode>
  Ответить
  - Lure Of Chaos 13 лет ago # 0 ⇈
    
    причем цикл уже есть
    Ответить
  - anonimb84a2f6fd141 13 лет ago # 0 ⇈
    $arr = array('sku', 'name', 'price', 'is_spec', 'spec_price', 'old') foreach ($item as $prop) { $tag = $prop['tag']; $key = $arr[index($this->props, $tag)]; $new_item[$key] = $prop['value']; }
    
    Как-то так. Ну и сравнение через ==. Это если не обсуждать архитектуру (хранение в массиве с хуй пойми каким назначением элементов, собственно, это и есть главный говнокод).
    Ответить
    - tirinox 13 лет ago # 0 ⇈
      
      Я тут по некоторой необходимости неделю изучаю Yii, который якобы супер популярный PHP framework, и люто сокрушаюсь как раз на тему того, что там все хранится в массивах с хуй пойми каким назначением элементов. PHP безнадежен...
      Ответить
      - anonimb84a2f6fd141 13 лет ago # 0 ⇈
        
        И это при том, что массивы в рнр на самом деле - хеш массивы.
        Ответить
        
        Stertor 13 лет ago # 0 ⇈
        
        Спс, кэп. Я не знал.
        Ответить
- guest 13 лет ago # 0 ⇈
```
~Клуб знакомств для геев~
Познакомлюсь с парнем
             Сделаю миньет, подставлю попку парню кавказской внешности.
             Прут наглые хуястые самцы, желательно с большим членом, так как  
             очко сильно раздолблено. Могу принять сразу до 3-х парней.
            Пишите tarasber@mai.ru  /Тарас/
```
  Ответить