咨询：下述CodeIgniter模型方法是否属于可复用方法？

嘿，咱们来好好唠唠你写的这个模型方法的可复用性问题——目前它还不算一个高可复用的方法，原因主要有这几点：

• 职责不单一，违反单一职责原则：一个方法同时承担了「获取产品ID数组」和「统计产品数量」两个完全独立的逻辑。这种“一锅炖”的写法会让后续维护、扩展变得很麻烦，比如以后要加个获取产品名称列表的功能，难道还要往switch里塞新case？久而久之这个方法会变得臃肿不堪。
• 存在潜在bug：你在ids分支里直接用了data['ids'][$counter]，但data变量根本没声明（应该加$data = [];），而且counter变量也没有自增的逻辑，最后所有数组元素都会被最后一个product_id覆盖，这会导致返回的ID数组完全不符合预期。
• 灵活性极差：查询条件只固定了user_id，如果后续需要按产品状态、分类或者其他条件来获取ID或统计数量，这个方法完全派不上用场，只能重写新的方法，这完全违背了复用的初衷。

优化建议：拆分为单一职责的方法 #

要提升复用性，最直接的方式就是把这两个逻辑拆成独立的方法，每个方法只做一件事，比如：

// 获取指定用户的产品ID数组
public function getProductIdsByUserId($user_id) {
    $query = $this->db->select('product_id')
                      ->from('products')
                      ->where('user_id', $user_id)
                      ->get();
    
    // 用array_column直接提取数组中的product_id字段，简洁高效
    return array_column($query->result_array(), 'product_id');
}

// 统计指定用户的产品数量
public function countProductsByUserId($user_id) {
    // CodeIgniter提供了count_all_results方法，无需手动查询再统计行数
    return $this->db->where('user_id', $user_id)
                    ->count_all_results('products');
}

这样拆分后，每个方法的职责清晰，命名一目了然，其他开发者（或者未来的你）一看就知道怎么用。如果后续需要扩展，比如支持按产品状态筛选，只需要给方法加可选参数即可：

public function getProductIdsByUserId($user_id, $status = null) {
    $this->db->select('product_id')
             ->from('products')
             ->where('user_id', $user_id);
    
    // 可选参数，灵活扩展查询条件
    if ($status !== null) {
        $this->db->where('status', $status);
    }
    
    $query = $this->db->get();
    return array_column($query->result_array(), 'product_id');
}

总结 #

原来的方法因为把多个逻辑耦合在一起，还存在bug，复用性很低；拆分成单一职责的方法后，不仅代码更健壮，还能灵活扩展，复用性也大大提升啦！

内容的提问来源于stack exchange，提问作者Dween