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

---

UPDATE 내 현재 기능입니다 - 좋아 내 새로운 기능이 게시물의 제안들 덕분에

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을 버릴 수 있습니다.

예 :

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

난 당신이 성능을 향상시킬 수 있다고 생각하지 않습니다,하지만 당신은 구문을 단축 할 수 있지만 것 변수가 null

,691 경우로

<?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

글쎄, 똑같은 일을 네 번하기 때문에 이미 함수 작성을 고려해 볼 수 있습니다.

가정 FilterLLHTML은 사용자 지정 함수가 아닙니다.

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

또는 FilterALLHTML 함수에 null 체크를 포함하고 필요한 경우 null을 반환하십시오. 예를

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

Answer 4

을 PHP의 variable variables를 사용하면 FilterALLHTML를 변경할 수있는 경우

그래서, 당신은 이런 식으로 그것을 할 것입니다.

$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처럼 내가 대신하지만 널 (null)에 대한 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); 
}