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

    • ava gorsash 17.08.2017 14:46 # 0

      У первой строчки такой-же отступ, как и у остальных. Неправильно запостилось
      Ответить

    • ava Stallman 17.08.2017 14:51 # 0

      Для маскировки if-ов лучше ruby ничего не придумали.
      Ответить

    • ava inkanus-gray 17.08.2017 15:26 # 0

      Смотрите, как надо:
      ($category_id == '') || intval($this->db->where('group_category =', $category_id));
($group_by == TRUE) && intval($this->db->group_by('blog_cat_name'));
      Ответить

    • ava Dummy00001 17.08.2017 22:19 # +2

      > У меня создалось впечатление, что здесь специально замаскировали все ифы.

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

      • ava sos 17.08.2017 23:23 # +1

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

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