这个PHP代码可以改进吗？

Question

有没有更好的方法来做这个简单的任务如下？像数组或甚至另一种方法？

<?PHP 
// current way 
if ($city != NULL) { 
    $city = FilterALLHTML($city); 
} 
if ($state != NULL) { 
    $state = FilterALLHTML($state); 
} 
if ($title != NULL) { 
    $title = FilterALLHTML($title); 
} 
if ($division != NULL) { 
    $division = FilterALLHTML($division); 
} 
?> 

这里是我当前的功能

function FilterALLHTML($document) { 
    //old array line //"'<[\/\!]*?[^<>]*//?//>'si",// strip html 
    $text = strip_tags($document); 
    $search = array ("/f.?u.?c.?k/i", 
       "/(s|$).?h.?i.?t/i", 
       '/(potspace|mycrib|palbolt)/i'); 
    $text = preg_replace ($search, '', $text); 
    return $text; 
} 

---

更新 - 好我的新功能后，从这个职位的建议，谢谢你们

function FilterALLHTML($var) { 
    //old array line //"'<[\/\!]*?[^<>]*//?//>'si",// strip html 
    if ($var != null){ 
     $text = strip_tags($var); 
     $search = array ("/f.?u.?c.?k/i", 
        "/(s|$).?h.?i.?t/i", 
        '/(potspace|mycrib|palbolt|pot space)/i'); 
     $text = preg_replace ($search, '', $text); 
     return $text; 
    } 
    return null; 
} 

Answer 1

更改FilterALLHTML功能做null检查并返回null？ 然后你可以扔掉所有的if s。

例子：

function FilterALLHTML($input) 
{ 
    if ($input === null) 
     return null; 

    // Original code, I'll just use strip_tags() for a functional example 
    return strip_tags($input); 
} 

编辑：

我觉得像共享一个替代变量的变量，因为我真的不喜欢使用字符串常量，而不是变量名的想法。参考一路:)

function FilterALLHTML(&$text) 
{ 
    if ($text !== null) 
    { 
     // Omitted regex bit for simplicity 
     $text = strip_tags($text); 
    } 
} 

$city = "<b>New York</b>"; 
$state = null; 
$title = "<i>Mr.</i>"; 

$fields = array(&$city, &$state, &$title); 
foreach ($fields as &$var) 
    FilterALLHTML($var); 

（注：FilterALLHTML实现从第一个例子不同）

Answer 2

我不认为你可以提高性能，但可以缩短语法，但它会最终被同样的解释

<?PHP 
    $city = ($city == NULL) ? "default value" : FilterALLHTML($city); 
    $state = ($state == NULL) ? "default value" : FilterALLHTML($state); 
    $title = ($title == NULL) ? "default value" : FilterALLHTML($title); 
    $division = ($division == NULL) ? "default value" : FilterALLHTML($division); 
?> 

“默认值”应与你想什么值更换了，如果变量为空

Answer 3

那么，你可能已经考虑编写一个函数，因为你做了四次完全相同的事情。

假设FilterALLHTML不是自定义函数。

function Filter($var) 
{ 
    if ($var != null) 
    { 
     return FilterALLHTML($var); 
    } 
    return null; 
} 

或者只是在FilterALLHTML函数中包含null检查，并且如果需要的话从那里返回null。

所以，如果你可以改变FilterALLHTML，那么你会做这样的：

function FilterALLHTML($var) 
{ 
    if ($var == null) 
    { 
     return null; 
    } 
    else 
    { 
     //do your filtering 
     return $filteredVar; 
    } 
} 

Answer 4

是，使用PHP的variable variables。

$vars = array('city','state','title','division'); 
foreach($vars as $v) { 
    if ($$v != null) $$v = FilterAllHTML($$v); 
} 

如果您知道所有变量先前已定义的事实，那么您不需要空检查。否则，空检查将防止触发E_NOTICE错误。

Answer 5

添加到Thorarin的答案，你可以改变你的filterall函数，以接受一个数组作为输入，并通过引用传递它将修改数组的内容。

$tofilter = array($city,$state,$division,$title); 


filterall($tofilter); 

Answer 6

foreach (array('city', 'state', 'title', 'division') as $var) { 
    if ($$var != null) { 
     $$var = FilterALLHTML($$var); 
    } 
} 

像Thorarin我建议具有空的FilterALLHTML功能检查，而不是虽然。

Answer 7

zombat的答案是最好的，但我想补充一点，你不应该真的在检查null。如果由于某种原因，FilterAllHTML与null值有问题，但不应该将null的检查置于FilterAllHTML函数定义中。

$vars = array('city', 'state', 'title', 'division'); 
foreach($vars as $var) { 
    $$var = FilterAllHTML($$var); 
} 

Answer 8

我没有看到它提到，你总是可以通过引用传递的参数跳过重复分配：

function FilterALLHTML(&$var) 
{ 
    if ($var == null) 
    { 
     $var = null; 
    } 
    else 
    { 
     $var = strip_tags($var); 
    } 
} 

我相信你也可以存储在阵列中的引用，但我的天堂” t试过了。

foreach (array(&$city, &$state, &$title, &$division) as $var) 
{ 
    FilterALLHTML($var); 
}