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

    0

    1. 1
    2. 2
    3. 3
    4. 4
    if($category_id!='')
            $this->db->where('group_category =', $category_id);
            if($group_by == TRUE)
            $this->db->group_by('blog_cat_name');

    У меня создалось впечатление, что здесь специально замаскировали все ифы. Чтобы читающий думал, что это линейный код. На скобках - экономят, на пробелах - экономят.

    Запостил: gorsash, 17 Августа 2017

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

    • У первой строчки такой-же отступ, как и у остальных. Неправильно запостилось
      Ответить
    • Для маскировки if-ов лучше ruby ничего не придумали.
      Ответить
    • Смотрите, как надо:
      ($category_id == '') || intval($this->db->where('group_category =', $category_id));
      ($group_by == TRUE) && intval($this->db->group_by('blog_cat_name'));
      Ответить
    • > У меня создалось впечатление, что здесь специально замаскировали все ифы.

      на одном проекте индуса видел который точно так же делал. объяснение: его лиды запугали - и он боялся чужой код трогать. строчку вставить - ок, чужой код менять - нет, нет, нет. там примеры были еще и похлеще. индус был глупый, а лиды мудаки. я тех лидов сам стороной потом обходил, потому что хуже того индуса были.
      Ответить
      • Я тоже в чужой говнокод стараюсь минимально залезать, когда возникает такая необходимость. И сам проблюешься, и на ревью потом гемор.
        Ответить

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