Как я могу упростить этот избыточный код? - PullRequest
Новая
       35

Как я могу упростить этот избыточный код?

4 голосов
/ 21 мая 2010

Может кто-нибудь помочь мне упростить этот избыточный кусок кода? 

if (isset($to) === true)
{
    if (is_string($to) === true)
    {
        $to = explode(',', $to);
    }

    $to = array_filter(filter_var_array(preg_replace('~[<>]|%0[ab]|[[:cntrl:]]~i', '', $to), FILTER_VALIDATE_EMAIL));
}

if (isset($cc) === true)
{
    if (is_string($cc) === true)
    {
        $cc = explode(',', $cc);
    }

    $cc = array_filter(filter_var_array(preg_replace('~[<>]|%0[ab]|[[:cntrl:]]~i', '', $cc), FILTER_VALIDATE_EMAIL));
}

if (isset($bcc) === true)
{
    if (is_string($bcc) === true)
    {
        $bcc = explode(',', $bcc);
    }

    $bcc = array_filter(filter_var_array(preg_replace('~[<>]|%0[ab]|[[:cntrl:]]~i', '', $bcc), FILTER_VALIDATE_EMAIL));
}

if (isset($from) === true)
{
    if (is_string($from) === true)
    {
        $from = explode(',', $from);
    }

    $from = array_filter(filter_var_array(preg_replace('~[<>]|%0[ab]|[[:cntrl:]]~i', '', $from), FILTER_VALIDATE_EMAIL));
}

Я пытался использовать переменные переменные, но безуспешно (с тех пор как я их использовал) уже давно.

Ответы [ 7 ]

4 голосов
/ 21 мая 2010

Переменные: 

$vars = array('to', 'cc', 'bcc', 'from');
foreach ($vars as $varname) {
    if (isset($$varname)) {
        if (is_string($$varname)) {
            $$varname = explode(',', $$varname);
        }
        $$varname = array_filter(filter_var_array(preg_replace('~[<>]|%0[ab]|[[:cntrl:]]~i', '', $$varname), FILTER_VALIDATE_EMAIL));
    }
}

Обычный (без использования переменных переменных): 

$vars = compact('to', 'cc', 'bcc', 'from'); 
foreach ($vars as $name => &$var) {
    if (is_string($var)) {
        $var = explode(',', $var);
    }
    $var = array_filter(filter_var_array(preg_replace('~[<>]|%0[ab]|[[:cntrl:]]~i', '', $var), FILTER_VALIDATE_EMAIL));
}
extract ($vars);

Обратите внимание, вам не нужен isset, потому что compact будет импортировать только установленные переменные Все остальные игнорируются ...

Кстати: вам не нужно === правда. isset () или is_string () всегда возвращают логическое значение. Так что === истина избыточна ...

2 голосов
/ 21 мая 2010

Поместить это в функцию? 

function validate($str) {
    if (isset($str) === true)
    {
        if (is_string($str) === true)
        {
            $str = explode(',', $str);
        }

        $str = array_filter(filter_var_array(preg_replace('~[<>]|%0[ab]|[[:cntrl:]]~i', '', $str), FILTER_VALIDATE_EMAIL));
    }
    return $str;
}

$to = validate($to);
$cc = validate($cc);
$bcc = validate($bcc);
$from = validate($from);
2 голосов
/ 21 мая 2010

Вы можете сделать (не проверено) 

$vars = array($from, $to, $cc, $bcc);

foreach ($vars as $var)
        {
        $var = explode(',', $var);
        ....
        ...
        }

$from = $vars[0];
$to = $vars[1];
$cc = $vars[2];
$bcc = $vars[3];
1 голос
/ 21 мая 2010

Для начала можно избавиться от isset() === true; isset() возвращает либо true, либо false.

И, конечно, поместите это в функцию, поскольку все ваши if утверждения, кажется, делают то же самое, но это уже упоминалось ранее ...

Кстати, ваша array_filter линия работает, если вход не является array()?

Если нет, вам нужно включить это утверждение в оператор if (is_string()).

1 голос
/ 21 мая 2010

Даже не пройдя полный путь переменных переменных, вы могли бы упростить это, просто поместив проверки в общую функцию, а затем выполнив: 

$to = cleanup_email_addrs($to);
$cc = cleanup_email_addrs($cc);
$bcc = cleanup_email_addrs($bcc);
$from = cleanup_email_addrs($from);
1 голос
/ 21 мая 2010

Просто вставьте значения в массив и переберите его. 

function cleanEmails($value) {
    if (is_string($value)) {
        $value = explode(',', $value);
    }
    return array_filter(filter_var_array(preg_replace('~[<>]|%0[ab]|[[:cntrl:]]~i', '', $value), FILTER_VALIDATE_EMAIL));
}

$fields = array();
if (isset($to)) {
    $fields['to'] = $to;
}
if (isset($from)) {
    $fields['from'] = $from;
}
if (isset($cc)) {
    $fields['cc'] = $cc;
}
if (isset($bcc)) {
    $fields['bcc'] = $bcc;
}
$result = array_map('cleanEmails', $fields);

Конечным результатом будет двумерный массив, первый индекс будет полями, которые были установлены, второй индекс будет соответствующими адресами электронной почты ...

1 голос
/ 21 мая 2010

Я мог бы сделать это:

Возможно, вы можете создать для этого функцию: 

function checkIt($var)
{
    if (isset($var) === true)
    {
        if (is_string($var) === true)
        {
          $var = explode(',', $var);
        }

        $to = explode(',', $var);
        $to = array_filter(filter_var_array(preg_replace('~[<>]|%0[ab]|[[:cntrl:]]~i', '', $to), FILTER_VALIDATE_EMAIL));
    }

    return $to;
}

И теперь вы можете передавать свои переменные этой функции.

...