1. Список говнокодов пользователя Panda

    Всего: 3

  2. PHP / Говнокод #20367

    +3

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    public function favoriteCount($userId) {
		$query = $this->db->query("SELECT * FROM ".DB_PREFIX."favorite WHERE customer_id =".$this->db->escape($userId));
		$rows = $query->rows;
		return count($rows);
}

    Opencart.
    Потому что агрегатные функции для слабаков!

    Panda, 13 Июля 2016

    Комментарии (1)

  3. PHP / Говнокод #20366

    +2

    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
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    <table>       
        <?php 
        $showShowroomsIds =  array();
        foreach($available_sizes as $sizeName):
        $available  =  false;
        foreach($warehouses as $warehouse) :
            $uuid = $warehouse['uuid'];
            $name = $warehouse['name'];
            $quantity = $size_quantity[$uuid][trim($sizeName)];
            if($quantity >0) {
                $available  =  true;
            }
        endforeach;
        if(!$available) continue;
        ?>
        <?php
        foreach($warehouses as $warehouse) :
        $uuid = $warehouse['uuid'];
        //echo $uuid."<br>";
        
        if(isset($showrooms[$uuid])) {
            $showShowroomsIds[$uuid] = $uuid;
        }
        
        $name = $warehouse['name'];
        $quantity = $size_quantity[$uuid][trim($sizeName)];
        if($quantity) {
            $eshopAvailable = '';
            if($uuid == $eshopUuid && $quantity > 0) $eshopAvailable = 'eshop_available=\'y\'';
            $quantity  = ModelCommonWarehouse::getFormattedQuantity($quantity,$uuid);
            $qtA  =  "";
        } else {
            continue;
            $qtA  =  "not-available";
            $quantity  =  "нет в наличии";
        }
        ?>
        <tr <?php echo $eshopAvailable?> available='y'>
            <td class="td">
                <?php if(isset($showrooms[$uuid])):?><a href="#" class="showroom-info"  uuid="<?php echo $uuid;?>" icId="uuid" rel="nofollow"><?php endif;?>
                <?php echo $name?>
                <!--<?php echo $uuid;?>-->
                <?php if(isset($showrooms[$uuid])):?></a><?php endif;?>
            </td>
            <td class="qt <?php echo $qtA;?>"   ><?php echo $quantity;?></td>
        </tr>
        <?php endforeach;?>
        <tr>
            <td style="height: 10px;"></td>
        </tr>
        <?php endforeach;?>
    </table>

    Opencart. Прямо во вьюхе ровным слоем размазаны бизнес-логика и js.
    Что самое интересное, оборот магазина составляет 7 млн. руб (о__О)

    Panda, 13 Июля 2016

    Комментарии (5)

  4. PHP / Говнокод #20135

    +1

    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
    30. 30
    31. 31
    32. 32
    public function selectValueType($value)
    {
        $type = null;
        if (App::helperPhone()->parseMobile($value)) {
            $type = 1; //телефон    
        }
        if (App::helperEmail()->validate($value)) {
            $type = 2; //email
        }
        return $type;
    }



А потом  в десятке методов:

 public function getLogin($value)
    {
        $type = $this->selectValueType($value);
        switch ($type) {
            case 1:
                $type = 'phone';
                break;
            case 2:
                $type = 'email';
                break;
        }
        if ($type) {
            $value = $type == 'phone' ? App::helperPhone()->parseMobile($value) : $value;
           // ... прочий говнокод
        }
}

    Вот такие перлы достались в наследство. Приходится проводить полный рефакторинг :((

    Panda, 06 Июня 2016

    Комментарии (0)