อาร์กิวเมนต์หลายตัวในการเรียกใช้ฟังก์ชันเทียบกับอาร์เรย์เดียว

ฉันมีฟังก์ชั่นที่ใช้ในชุดของพารามิเตอร์จากนั้นนำไปใช้กับพวกเขาเป็นเงื่อนไขในแบบสอบถาม SQL อย่างไรก็ตามในขณะที่ฉันชอบอาร์เรย์อาร์กิวเมนต์เดียวที่มีเงื่อนไขตัวเอง:

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

เพื่อนร่วมงานของฉันต้องการแสดงรายการอาร์กิวเมนต์ทั้งหมดแทนอย่างชัดเจน:

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

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

การโต้เถียงของเขาคือการระบุข้อโต้แย้งอย่างชัดเจนพฤติกรรมของฟังก์ชั่นจะชัดเจนมากขึ้นเมื่อเทียบกับการต้องเจาะลึกรหัสเพื่อค้นหาว่าข้อโต้แย้งลึกลับ$paramคืออะไร

ปัญหาของฉันคือการได้รับ verbose มากเมื่อจัดการกับข้อโต้แย้งมากมายเช่น 10+ มีวิธีปฏิบัติที่ต้องการหรือไม่? สถานการณ์กรณีที่เลวร้ายที่สุดของฉันจะเห็นบางสิ่งดังต่อไปนี้:

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

IMHO เพื่อนร่วมงานของคุณถูกต้องสำหรับตัวอย่างข้างต้น การตั้งค่าของคุณอาจสั้น แต่ก็ยังอ่านง่ายและบำรุงรักษาน้อย ถามคำถามว่าทำไมการเขียนฟังก์ชั่นในตอนแรกฟังก์ชั่นของคุณ 'มาที่โต๊ะ' - ฉันต้องเข้าใจสิ่งที่มันทำและวิธีการมันในรายละเอียดที่ดีเพียงใช้ ด้วยตัวอย่างของเขาแม้ว่าฉันไม่ใช่โปรแกรมเมอร์ PHP แต่ฉันสามารถเห็นรายละเอียดเพียงพอในการประกาศฟังก์ชั่นที่ฉันไม่ต้องกังวลกับการใช้งาน

เท่าที่มีการโต้แย้งจำนวนมากซึ่งโดยปกติถือว่าเป็นกลิ่นรหัส โดยทั่วไปแล้วฟังก์ชั่นพยายามทำอะไรมากเกินไป? หากคุณพบว่ามีความต้องการที่แท้จริงสำหรับอาร์กิวเมนต์จำนวนมากอาจเป็นไปได้ว่าพวกเขามีความสัมพันธ์ในทางใดทางหนึ่งและอยู่ร่วมกันในโครงสร้างหรือคลาสหนึ่งหรือสองสามอัน อย่างไรก็ตามการผ่านอาร์เรย์ที่ไม่มีโครงสร้างนั้นไม่ได้ช่วยแก้ไขปัญหาเรื่องโค้ด

คำตอบของฉันคือไม่เชื่อเรื่องภาษามากหรือน้อย

หากจุดประสงค์เพียงอย่างเดียวของการจัดกลุ่มข้อโต้แย้งในโครงสร้างข้อมูลที่ซับซ้อน (ตาราง, บันทึก, พจนานุกรม, วัตถุ ... ) คือการส่งพวกเขาโดยรวมไปยังฟังก์ชั่นให้หลีกเลี่ยงมันได้ดีกว่า สิ่งนี้จะเพิ่มความซับซ้อนที่ไร้ประโยชน์และทำให้ความตั้งใจของคุณชัดเจน

หากอาร์กิวเมนต์ที่จัดกลุ่มมีความหมายด้วยตัวเองเลเยอร์ของความซับซ้อนนั้นจะช่วยให้เข้าใจการออกแบบทั้งหมด: ตั้งชื่อเลเยอร์ของสิ่งที่เป็นนามธรรมแทน

คุณอาจพบว่าแทนที่จะเป็นอาร์กิวเมนต์แต่ละตัวเป็นโหลหรือเป็นอาเรย์ขนาดใหญ่การออกแบบที่ดีที่สุดคือการมีสองหรือสามข้อโต้แย้งในการจัดกลุ่มข้อมูลที่มีความสัมพันธ์กัน

ในกรณีของคุณฉันต้องการวิธีการของเพื่อนร่วมงานของคุณ หากคุณกำลังเขียนโมเดลและฉันใช้โมเดลของคุณเพื่อพัฒนาโมเดลเหล่านั้น ฉันเห็นลายเซ็นต์ของวิธีการของเพื่อนร่วมงานของคุณและสามารถใช้งานได้ทันที

ในขณะที่ฉันจะต้องผ่านการใช้งานsearchQueryฟังก์ชั่นของคุณเพื่อดูพารามิเตอร์ที่ฟังก์ชั่นของคุณคาดหวัง

ฉันต้องการแนวทางของคุณเฉพาะในกรณีที่searchQueryถูก จำกัด ให้ค้นหาเฉพาะในตารางเดียวดังนั้นจะไม่มีการเข้าร่วม ในกรณีนี้ฟังก์ชั่นของฉันจะเป็นดังนี้:

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

ดังนั้นฉันรู้ทันทีว่าองค์ประกอบของอาร์เรย์เป็นชื่อคอลัมน์ของตารางเฉพาะซึ่งคลาสที่มีวิธีนี้แทนรหัสของคุณ

ทำทั้งสองอย่าง array_mergeอนุญาตให้มีรายการที่ชัดเจนที่ด้านบนของฟังก์ชั่นตามที่เพื่อนร่วมงานของคุณชอบในขณะที่รักษาพารามิเตอร์จากการเทอะทะตามที่คุณต้องการ

ฉันขอแนะนำอย่างยิ่งให้ใช้คำแนะนำของ @ chiborg จากความคิดเห็นของคำถาม - ชัดเจนกว่าที่คุณต้องการ

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

นอกจากนี้คุณสามารถส่งผ่านสตริงที่คล้ายกับสตริงการสืบค้นและใช้parse_str(เพราะดูเหมือนว่าคุณกำลังใช้ PHP แต่อาจมีโซลูชันอื่นในภาษาอื่น) เพื่อประมวลผลเป็นอาเรย์ภายในเมธอด:

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

และเรียกว่าชอบ

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

คุณสามารถใช้http_build_queryในการแปลงจากอาเรย์แบบสัมพันธ์เป็นสตริง (กลับด้านparse_str)