Nhiều đối số trong hàm gọi so với mảng đơn

Tôi có một hàm lấy một tập các tham số, sau đó áp dụng cho chúng làm điều kiện cho truy vấn SQL. Tuy nhiên, trong khi tôi ủng hộ một mảng đối số duy nhất chứa các điều kiện:

function searchQuery($params = array()) {
    foreach($params as $param => $value) {
        switch ($param) {
            case 'name':
                $query->where('name', $value);
                break;
            case 'phone':
                $query->join('phone');
                $query->where('phone', $value);
                break;
        }
    }
}

Thay vào đó, đồng nghiệp của tôi thích liệt kê tất cả các đối số một cách rõ ràng:

function searchQuery($name = '', $phone = '') {
    if ($name) {
        $query->where('name', $value);
    }

    if ($phone) {
        $query->join('phone');
        $query->where('phone', $value);
    }
}

Lập luận của ông là bằng cách liệt kê các đối số một cách rõ ràng, hành vi của hàm trở nên rõ ràng hơn - trái ngược với việc phải đi sâu vào mã để tìm hiểu xem đối số bí ẩn $paramlà gì.

Vấn đề của tôi là điều này trở nên rất dài dòng khi xử lý nhiều tranh luận, như 10+. Có thực hành ưa thích? Trường hợp xấu nhất của tôi sẽ là một cái gì đó như sau:

searchQuery('', '', '', '', '', '', '', '', '', '', '', '', 'search_query')

IMHO đồng nghiệp của bạn là chính xác cho ví dụ trên. Sở thích của bạn có thể là ngắn gọn, nhưng nó cũng ít đọc hơn và do đó ít bảo trì hơn. Đặt câu hỏi tại sao phải viết chức năng ở vị trí đầu tiên, chức năng của bạn 'mang đến cái gì' - Tôi phải hiểu nó làm gì và làm thế nào, rất chi tiết, chỉ để sử dụng nó. Với ví dụ của anh ấy, mặc dù tôi không phải là lập trình viên PHP, tôi có thể thấy đủ chi tiết trong khai báo hàm mà tôi không phải lo lắng về việc triển khai nó.

Theo như một số lượng lớn hơn các đối số, thường được coi là một mùi mã. Điển hình là chức năng đang cố gắng làm quá nhiều? Nếu bạn tìm thấy một nhu cầu thực sự cho một số lượng lớn các đối số, thì có khả năng chúng có liên quan theo một cách nào đó và thuộc về nhau trong một hoặc một vài cấu trúc hoặc lớp (thậm chí có thể là mảng các mục liên quan như đường trong một địa chỉ). Tuy nhiên, việc truyền một mảng không có cấu trúc sẽ không có gì để giải quyết mã có mùi.

Câu trả lời của tôi là ít nhiều ngôn ngữ bất khả tri.

Nếu mục đích duy nhất của việc nhóm các đối số trong một cấu trúc dữ liệu phức tạp (bảng, bản ghi, từ điển, đối tượng ...) là để chuyển toàn bộ chúng cho một hàm, tốt hơn là tránh nó. Điều này thêm một lớp phức tạp vô dụng và làm cho ý định của bạn bị che khuất.

Nếu các đối số được nhóm có một ý nghĩa riêng, thì lớp phức tạp đó sẽ giúp hiểu toàn bộ thiết kế: thay vào đó là lớp trừu tượng.

Bạn có thể thấy rằng thay vì một tá các đối số riêng lẻ hoặc một mảng lớn, thiết kế tốt nhất là với hai hoặc ba đối số mỗi nhóm dữ liệu tương quan.

Trong trường hợp của bạn, tôi thích phương pháp của đồng nghiệp của bạn. Nếu bạn đang viết mô hình và tôi đang sử dụng mô hình của bạn để phát triển chúng. Tôi thấy chữ ký của phương pháp của đồng nghiệp của bạn và có thể sử dụng nó ngay lập tức.

Trong khi, tôi sẽ phải thực hiện searchQuerychức năng của bạn để xem những tham số nào được chức năng của bạn mong đợi.

Tôi chỉ thích cách tiếp cận của bạn trong trường hợp khi searchQuerygiới hạn chỉ tìm kiếm trong một bảng duy nhất, vì vậy sẽ không có tham gia. Trong trường hợp đó, chức năng của tôi sẽ trông như thế này:

function searchQuery($params = array()) {
    foreach($params as $param => $value) {
        $query->where($param, $value);
    }
} 

Vì vậy, tôi biết ngay rằng các phần tử của mảng thực sự là tên cột của một bảng cụ thể mà lớp có phương thức này đại diện trong mã của bạn.

Làm cả hai, sắp xếp. array_mergecho phép một danh sách rõ ràng ở đầu chức năng, như đồng nghiệp của bạn thích, trong khi giữ cho các tham số không bị khó sử dụng, như bạn muốn.

Tôi cũng đề nghị sử dụng đề xuất của @ chiborg từ các nhận xét câu hỏi - rõ ràng hơn nhiều những gì bạn dự định.

function searchQuery($params = array()) {
    $defaults = array(
        'name' => '',
        'phone' => '',
        ....
    );
    $params = array_merge($defaults, $params);

    if(!empty($params['name'])) {
        $query->where('name', $params['name']);
    }
    if (!empty($params['phone'])) {
        $query->join('phone');
        $query->where('phone', $params['phone']);
    }
    ....
}

Ngoài ra, bạn có thể chuyển một chuỗi giống như chuỗi truy vấn và sử dụng parse_str(vì có vẻ như bạn đang sử dụng PHP, nhưng các giải pháp khác có thể có sẵn bằng các ngôn ngữ khác) để xử lý chuỗi đó thành một mảng bên trong phương thức:

/**
 * Executes a search in the DB with the constraints specified in the $queryString
 * @var $queryString string The search parameters in a query string format (ie
 *      "foo=abc&bar=hello"
 * @return ResultSet the result set of performing the query
 */
function searchQuery($queryString) {
  $params = parse_str($queryString);
  if (isset($params['name'])) {
    $query->where('name', $params['name']);
  }
  if (isset($params['phone'])) {
    $query->join('phone');
    $query->where('phone', $params['phone']);
  }
  ...

  return ...;
}

và gọi nó như

$result = searchQuery('name=foo&phone=555-123-456');

Bạn có thể sử dụng http_build_queryđể chuyển đổi từ một mảng kết hợp thành một chuỗi (ngược lại parse_str).