Tek tek dizi vs işlev çağrısında birden fazla bağımsız değişken

Bir parametreler kümesinde yer alan ve ardından bir SQL sorgusuna koşul olarak uygulanan bir işleve sahibim. Bununla birlikte, koşulları içeren tek bir argüman dizisini tercih ederken:

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

Meslektaşım açıkça tüm argümanları listelemeyi tercih etti:

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

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

Argümanı, argümanları açıkça listeleyerek, fonksiyonun davranışının daha belirgin hale gelmesiydi - gizemli argümanın ne olduğunu bulmak için koda atlamak zorunda kalmanın aksine $param.

Benim sorunum, 10+ gibi birçok argümanla uğraşırken bunun çok ayrıntılı olmasıydı. Tercih edilen bir uygulama var mı? Benim en kötü senaryom, şöyle bir şey görmek olacaktır:

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

IMHO meslektaşınız yukarıdaki örnek için doğrudur. Tercihiniz kısa olabilir, ancak aynı zamanda daha az okunabilir ve bu nedenle daha az bakım yapılabilir. Fonksiyonu neden ilk olarak yazarken neden rahatsız edici, 'fonksiyonunuz masaya ne getiriyor' sorusunu sorun - ne yaptığını ve nasıl yaptığını, sadece onu kullanmak için ne kadar ayrıntılı olduğunu anlamak zorundayım. Onun örneğiyle, bir PHP programcısı olmasam bile, işlev bildirimi içerisinde kendimi uygulamasıyla ilgilenmek zorunda kalmayacağım kadar ayrıntı görebiliyorum.

Argüman sayısı kadar, normalde bir kod kokusu olarak kabul edilir. Genelde işlev çok fazla şey yapmaya mı çalışıyor? Çok sayıda argümana gerçek bir ihtiyaç bulursanız, bir şekilde ilişkili olmaları ve bir veya birkaç yapı veya sınıfta bir araya gelmeleri muhtemeldir (belki bir adresdeki satırlar gibi ilgili öğelerin dizisi). Ancak, yapılandırılmamış bir diziyi iletmek, kod kokularını gidermek için hiçbir şey yapmaz.

Cevabım aşağı yukarı dil agnostik.

Argümanları karmaşık bir veri yapısında (tablo, kayıt, sözlük, nesne ...) gruplandırmanın tek amacı onları bir fonksiyona bir bütün olarak iletmekse, bundan kaçınmak daha iyidir. Bu işe yaramaz bir karmaşıklık katmanı ekler ve niyetinizi belirsizleştirir.

Eğer gruplanmış argümanlar kendi başlarına bir anlam ifade ediyorsa, o zaman bu karmaşıklık katmanı tüm tasarımın anlaşılmasına yardımcı olur: bunun yerine soyutlama katmanını adlandırın.

Bir düzine bağımsız argüman veya bir büyük dizi yerine, en iyi tasarımın her biriyle ilişkili verileri gruplayan iki veya üç argüman olduğunu görebilirsiniz.

Senin durumunda, meslektaşının yöntemini tercih ederim. Modeller yazıyorsan ve ben onların modellerini onların üzerinde geliştirmek için kullanıyordum. İş arkadaşınızın yönteminin imzasını görüyorum ve hemen kullanabiliyorum.

Bununla birlikte, searchQueryfonksiyonunuzdan hangi parametrelerin beklendiğini görmek için fonksiyonunuzun uygulanmasından geçmem gerekecekti .

Yaklaşımınızı yalnızca searchQuerytek bir tablo içinde arama yapmakla sınırlı kalması durumunda tercih ederim , bu nedenle katılım olmaz. Bu durumda fonksiyonum şöyle görünür:

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

Bu yüzden, dizi öğelerinin gerçekte, bu yönteme sahip olan sınıfın kodunuzda temsil ettiği belirli bir tablonun sütun adları olduğunu hemen biliyorum .

İkisini birden yap. array_mergeiş arkadaşınızın sevdiği gibi, parametreleri tercih ettiğiniz gibi parametreleri dengesiz tutmaktan koruyarak işlevin en üstünde açık bir listeye izin verir.

Ayrıca, @ chiborg'un önerisini soru yorumlarından kullanmanızı şiddetle tavsiye ediyorum - ne düşündüğünüzü çok daha açık bir şekilde.

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

Ayrıca, bir sorgu dizisine benzeyen bir dize iletebilir ve parse_str(PHP kullanıyormuş gibi göründüğünüzden, ancak diğer çözümler muhtemelen diğer dillerde de kullanılabilir) yöntemde bir dizide işlemek için kullanabilirsiniz:

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

ve buna benziyor

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

http_build_queryBir ilişkisel diziden bir dizgeye (tersini yapan parse_str) dönüştürmek için kullanabilirsiniz .