PHP opcjonalne argumenty funkcji z tablicą. Jak kodować/Lepszy sposób kodowania?

Question

Jestem pewien, że jest lepiej. Każda pomoc zostanie bardzo doceniona.

Chcę przekazać tablicę do funkcji php zawierającej argument, a wszystkie argumenty są opcjonalne. Używam inicjatora kodu i nie jestem ekspertem. Poniżej jest to, czego używam do tej pory:

function addLinkPost($postDetailArray) { 

    if (isset($postDetailArray['title'])) { 
     $title = $postDetailArray['title']; } 
    else { 
     $title = "Error: No Title"; 
    } 

    if (isset($postDetailArray['url'])) { 
     $url  = $postDetailArray['url']; 
    } else { 
     $url  = "no url"; 
    } 
    if (isset($postDetailArray['caption'])) { 
     $caption = $postDetailArray['caption']; 
    } else { 
     $caption = ""; 
    } 
    if (isset($postDetailArray['publish'])) { 
     $publish = $postDetailArray['publish']; 
    } else { 
     $publish = TRUE; 
    } 
    if (isset($postDetailArray['postdate'])) { 
     $postdate = $postDetailArray['postdate']; 
    } else { 
     $postdate = "NOW()"; 
    } 
    if (isset($postDetailArray['tagString'])) { 
     $tagString = $postDetailArray['tagString']; 
    } else { 
     $tagString = ""; 
    } 

Answer 1

Można to zrobić tak:

function addLinkPost(array $postDetailArray) 
{ 
    $fields = array(
     'key' => 'default value', 
     'title' => 'Error: No Title', 
    ); 

    foreach ($fields as $key => $default) { 
     $$key = isset($postDetailArray[$key]) ? $postDetailArray[$key] : $default; 
    } 
} 

prostu edytować tablicę $ pól z klucza i jego wartości domyślnej.

Answer 2

Myślę, że przykład jest w porządku. nie sądzę, że można tu zrobić o wiele więcej optymalizacji, chyba że pominiemy kilka nawiasów.

Opcjonalnie można sprawdzić działanie ekstraktu w PHP

Answer 3

Jak o:

function getdefault($value, $default = null) { 
    return isset($value) ? $value : $default; 
} 

function addLinkPost($postDetailArray) { 
    $title = getdefault($postDetailArray['title'], 'Error: No Title'); 
    $url = getdefault($postDetailArray['url'], 'no url'); 
    $caption = getdefault($postDetailArray['caption'], ''); 
    $publish = getdefault($postDetailArray['publish'], TRUE); 
    $postdate = getdefault($postDetailArray['postdate'], 'NOW()'); 
    $tagString = getdefault($postDetailArray['tagString'], ''); 
} 

lub alternatywnie:

$defaults = array(
    'title' => 'Error: No Title', 
    'url' => 'no url', 
    'caption' => '', 
    'publish' => TRUE, 
    'postdate' => 'NOW()', 
    'tagString' => '', 
); 

function addLinkPost($postDetailArray) { 
    global $defaults; 
    foreach ($defaults as $k => $v) { 
    $$k = isset($postDetailArray[$k]) ? $postDetailArray[$k] : $v; 
    } 
} 

z jednym ostrzeżeniem, że jeśli masz klucza tablicy " domyślne "w $defaults, nadpisze globalny $defaults.

Answer 4

Używanie tablicy jako argumentu jest dobrym pomysłem w tym przypadku. Można jednak nieco uprościć kod funkcji za pomocą operatora trójskładnikowego (http://dk.php.net/ternary):

$ title = isset ($ postDetailArray ['title'])? $ postDetailArray ['title']: 'Error: No Title';

Mogłeś uprościć jeszcze bardziej, wykonując następujące czynności:

function addLinkPost($data) { 

$arguments = array('title', 'url', 'caption', 'publish', 'postdate', 'tagString'); 

foreach ($arguments as $value) { 
    $$value = isset($data[$value]) ? $data[$value] : 'Error: No '.$value; 
} 

}

Answer 5

Spróbuj tego:

function addLinkPost($postDetailArray) { 

    foreach($array as $key=>$value){ 
    $$key = (isset($value) && !empty($value)) ? $value : ('no '.$key); 
    } 
    //all keys are available as variables 
    var_dump($url); var_dump($publish); //etc 
} 

Answer 6

Można zrobić wszystkie elementy parametrów wachlarz funkcji. Sprawdź, czy pierwsza jest tablicą w funkcji, a jeśli tak, wyodrębnij tablicę.

function addLinkPost($title = null, $url = null, $caption = null, $publish = null, $postDate = null, $tagString = null) 
{ 
     if(is_array($title)) { 
      extract($title); 
     } 

     .... 
} 

Może to sprawia, że ​​kod jest trochę bardziej przejrzysty.

Answer 7

Można użyć tablicy wartości domyślnych, a następnie scalić tablicę argumentów z wartościami domyślnymi. Wartości domyślne zostaną nadpisane, jeśli pojawią się w tablicy argumentów. Prosty przykład:

$defaults = array(
    'foo' => 'aaa', 
    'bar' => 'bbb', 
    'baz' => 'ccc', 
); 

$options = array(
    'foo' => 'ddd', 
); 


$merged = array_merge($defaults, $options); 

print_r($merged); 

/* 

Array 
(
    [foo] => ddd 
    [bar] => bbb 
    [baz] => ccc 
) 

*/ 

W twoim przypadku, to byłoby:

function addLinkPost($postDetailArray) { 
    static $defaults = array(
     'title'  => 'Error: No Title', 
     'url'  => 'no url', 
     'caption' => '', 
     'publish' => true, 
     'postdate' => 'NOW()', 
     'tagString' => '', 
    ); 

    $merged = array_merge($defaults, $postDetailArray); 

    $title  = $merged['title']; 
    $url  = $merged['url']; 
    $caption = $merged['caption']; 
    $publish = $merged['publish']; 
    $postdate = $merged['postdate']; 
    $tagString = $merged['$tagString']; 
}