Argomenti funzionali opzionali PHP con array. Come codice/modo migliore per codificare?

Question

Sono sicuro che c'è di meglio. Qualsiasi aiuto sarà molto apprezzato.

Voglio passare una matrice a una funzione php che contiene l'argomento e tutti gli argomenti sono facoltativi. Sto usando l'accenditore del codice e non sono affatto un esperto. Qui di seguito è quello che ho usato finora:

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

Si potrebbe fare così:

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

È sufficiente modificare l'array $ campi con la chiave e il suo valore di default.

Answer 2

Penso che l'esempio va bene. Non penso che ci sia molto più ottimizzazione da fare qui, a parte forse lasciare poche parentesi.

opzionalmente, è possibile controllare il funzionamento estratto in PHP

Answer 3

ne dite:

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

o in alternativa:

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

Con quello avvertendo che se si dispone di una chiave di serie di ' valori predefiniti 'in $defaults, sovrascriverà il globale $defaults.

Answer 4

L'utilizzo dell'array come argomento è una buona idea in questo caso. Tuttavia, è possibile semplificare un po 'il codice nella funzione utilizzando l'operatore ternario (http://dk.php.net/ternary):

$ title = isset ($ postDetailArray [' title '])? $ postDetailArray ['title']: 'Errore: Nessun titolo';

Si potrebbe semplificare ancora di più nel modo seguente:

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

Prova questa:

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

Si potrebbe fare tutti gli elementi dei parametri di matrice della funzione. Controlla se il primo è un array nella funzione e, in tal caso, estrai l'array.

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

     .... 
} 

Forse questo rende il codice un po 'più chiaro.

Answer 7

È possibile utilizzare un array di valori predefiniti e quindi unire l'array di argomenti con i valori predefiniti. I valori predefiniti verranno sovrascritti se compaiono nell'array degli argomenti. Un semplice esempio:

$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 
) 

*/ 

Nel tuo caso, che sarebbe:

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