Нашли или выдавили из себя код, который нельзя назвать нормальным,
на который без улыбки не взглянешь?
Не торопитесь его удалять или рефакторить, — запостите его на
говнокод.ру, посмеёмся вместе!
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 в форейче
Проверять статус нужно для фильтрации с необязательными параметрами, если нужно выбрать все статусы, то в функцию передается пустая строка, так же с айди.
лол блядь! Закэшировались!