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

    −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
    public function getRegProductDetails($product_id = '', $status = 'yes'){

            $this->db->select('*');

            if ($product_id != '') {

                $this->db->where('product_id', $product_id);

            }

            if ($status != '') {

                $this->db->where('active', $status);

            }

            $query = $this->db->get('registration_package');

            foreach ($query->result_array() as $row) {

                $product_details[] = $row;

            }

            return $product_details;

    Работаю с коммерческой системой, писали Индусы. У меня 100 Мб анекдотов в исходном коде. В БД есть столбец статус, они туда пишут статус yes OR no. И проверка везде тоже идет через строку. а выборку делают вот такой функцией. Зачем тут проверять статус на пустоту? В других подобных функциях встречаю RETURN в форейче

    Запостил: traney, 16 Июня 2016

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

    • ava Vince 16.06.2016 12:46 # 0

      >>$this->db->select('*');

      лол блядь! Закэшировались!
      Ответить

    • ava guesto 16.06.2016 14:34 # −1

      PHPшник не может писать нормальный код, увы
      Ответить

    • ava andrew_blablabla 19.07.2016 08:39 # 0

      Проверять статус нужно для фильтрации с необязательными параметрами, если нужно выбрать все статусы, то в функцию передается пустая строка, так же с айди.
      Ответить

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