このPHPコードを簡略化しますか？

Question

アイデア？とにかく二ケース$ vは、私がIS_ARRAY（$ V）を行うことができ、配列 ですが、私は場合/他を繰り返さなければならないと思いで

foreach(array('city', 'location') as $f) { 
     $ors[$f] = array(); 

     if (!isset($_POST[$f])) continue; 

     $v = $_POST[$f]; 
     if (isset($df[$f][$v])) { 
       array_push($ors[$f], $df[$f][$v]); 
     } 
     else { 
       security_error(); 
     } 
} 

foreach(array('age', 'sex') as $f) { 
     $ors[$f] = array(); 

     if (!isset($_POST[$f])) continue; 

     foreach($_POST[$f] as $v) { 
       if (isset($df[$f][$v])) { 
         array_push($ors[$f], $df[$f][$v]); 
       } 
       else { 
         security_error(); 
       } 
     } 
} 

Answer 1

function pushIfSetOrSecurityError($source, $target, $key) { 
     if (isset($source[$key])) 
       array_push($target, $source[$key]); 
     else 
       security_error(); 
} 

foreach(array('city', 'location') as $f) { 
     $ors[$f] = array(); 
     if (isset($_POST[$f])) 
       pushIfSetOrSecurityError($df[$f], $ors[$f], $_POST[$f]); 
} 

foreach(array('age', 'sex') as $f) { 
     $ors[$f] = array(); 
     if (isset($_POST[$f])) 
       foreach($_POST[$f] as $v) 
         pushIfSetOrSecurityError($df[$f], $ors[$f], $v); 
} 

Answer 2

使用両方のループのための三項条件演算子：

$result = isset($df[$f][$v]) ? array_push($ors[$f], $df[$f][$v]) : security_error(); 

および/または機能のために：

function pushIfSetOrSecurityError($source, $target, $key) { 
    $result = isset($source[$key]) ? array_push($target, $source[$key]) : security_error(); 
}