Berbagai argumen dalam panggilan fungsi vs array tunggal

Saya memiliki fungsi yang mengambil set parameter, kemudian berlaku untuk mereka sebagai syarat untuk query SQL. Namun, sementara saya memilih array argumen tunggal yang berisi kondisi itu sendiri:

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;
        }
    }
}

Rekan saya lebih suka mendaftarkan semua argumen secara eksplisit sebagai gantinya:

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

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

Argumennya adalah bahwa dengan mendaftar argumen secara eksplisit, perilaku fungsi menjadi lebih jelas - sebagai lawan harus mempelajari kode untuk mencari tahu apa argumen misterius $paramitu.

Masalah saya adalah ini menjadi sangat bertele-tele ketika berhadapan dengan banyak argumen, seperti 10+. Apakah ada praktik yang disukai? Skenario terburuk saya akan melihat sesuatu seperti berikut:

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

IMHO kolega Anda sudah benar untuk contoh di atas. Preferensi Anda mungkin singkat, tetapi juga kurang dapat dibaca dan karenanya kurang dapat dipertahankan. Ajukan pertanyaan mengapa repot-repot menulis fungsi di tempat pertama, apa fungsi Anda 'bawa ke meja'- Saya harus memahami apa fungsinya dan bagaimana melakukannya, secara sangat rinci, hanya untuk menggunakannya. Dengan contohnya, meskipun saya bukan seorang programmer PHP, saya dapat melihat cukup detail dalam deklarasi fungsi sehingga saya tidak perlu khawatir dengan implementasinya.

Sejauh sejumlah besar argumen, itu biasanya dianggap sebagai bau kode. Biasanya fungsi ini mencoba melakukan terlalu banyak? Jika Anda benar-benar menemukan kebutuhan untuk sejumlah besar argumen, kemungkinan mereka terkait dalam beberapa cara dan termasuk bersama dalam satu atau beberapa struktur atau kelas (bahkan mungkin berbagai item terkait seperti baris dalam alamat). Namun, melewatkan array yang tidak terstruktur tidak menyebabkan bau kode.

Jawaban saya kurang lebih agnostik bahasa.

Jika satu-satunya tujuan pengelompokan argumen dalam struktur data yang kompleks (tabel, catatan, kamus, objek ...) adalah meneruskannya secara keseluruhan ke suatu fungsi, lebih baik menghindarinya. Ini menambah lapisan kompleksitas yang tidak berguna dan membuat niat Anda menjadi tidak jelas.

Jika argumen yang dikelompokkan memiliki makna sendiri, maka lapisan kompleksitas membantu memahami keseluruhan desain: sebagai gantinya beri nama layer abstraksi.

Anda mungkin menemukan bahwa alih-alih selusin argumen individual atau satu array besar, desain terbaik adalah dengan dua atau tiga argumen yang masing-masing mengelompokkan data yang berkorelasi.

Dalam kasus Anda, saya lebih suka metode kolega Anda. Jika Anda menulis model dan saya menggunakan model Anda untuk mengembangkannya. Saya melihat tanda tangan metode kolega Anda dan dapat segera menggunakannya.

Sementara, saya harus melalui implementasi searchQueryfungsi Anda untuk melihat parameter apa yang diharapkan oleh fungsi Anda.

Saya lebih suka pendekatan Anda hanya dalam kasus ketika searchQueryterbatas hanya mencari dalam satu tabel, sehingga tidak akan ada bergabung. Dalam hal ini fungsi saya akan terlihat seperti ini:

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

Jadi, saya segera tahu bahwa elemen-elemen array sebenarnya adalah nama-nama kolom dari tabel tertentu yang diwakili oleh kelas yang memiliki metode ini dalam kode Anda.

Lakukan keduanya, semacam. array_mergememungkinkan daftar eksplisit di bagian atas fungsi, seperti yang disukai rekan kerja Anda, sambil menjaga agar parameter tidak menjadi berat, seperti yang Anda inginkan.

Saya juga sangat menyarankan menggunakan saran @ chiborg dari komentar pertanyaan - itu jauh lebih jelas apa yang Anda inginkan.

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']);
    }
    ....
}

Anda juga bisa melewatkan string yang menyerupai string kueri, dan menggunakan parse_str(karena sepertinya Anda menggunakan PHP, tetapi solusi lain mungkin tersedia dalam bahasa lain) untuk memprosesnya menjadi array di dalam metode:

/**
 * 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 ...;
}

dan menyebutnya seperti

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

Anda dapat menggunakan http_build_queryuntuk mengonversi dari array asosiatif ke string (kebalikan yang parse_strterjadi).