Email Validation Функция задирая ошибку

Question

я модернизировал с PHP 4 на PHP 5.7 и у меня есть функция, я работал на:

function is_valid_email($email) { 
    // First, we check that there's one @ symbol, and that the lengths are right 
if (!filter_var($email, FILTER_VALIDATE_EMAIL)) { 
    // Email invalid because wrong number of characters in one section, or wrong number of @ symbols. 
return false; 
} 
// Split it into sections to make life easier 
$email_array = explode("@", $email); 
$local_array = explode(".", $email_array[0]); 
for ($i = 0; $i < sizeof($local_array); $i++) { 
    if (!preg_match("/[^A-Za-z'-]/",$local_array($i))) { 
    return false; 
    } 
} 
if (!preg_match("^\[?[0-9\.]+\]?$",'/' . $email_array[1])) { // Check if domain is IP. If not, it should be valid domain name 
    $domain_array = explode(".", $email_array[1]); 
    if (sizeof($domain_array) < 2) { 
    return false; // Not enough parts to domain 
    } 
    for ($i = 0; $i < sizeof($domain_array); $i++) { 
    if (!preg_match("^(([A-Za-z0-9][A-Za-z0-9-]{0,61}[A-Za-z0-9])|([A-Za- z0-9]+))$", '/' . $domain_array[$i])) { 
    return false; 
    } 
    } 
} 
return true; 
} 

Когда форма отправлена, я получаю эту ошибку:

Fatal error: Function name must be a string in /usr/local/www/panhistoria/MyBooks/email_alert.php on line 241

линия 241 является первым !preg_match

Answer 1

$local_array($i) является массивом, а не функция, поэтому она должна решаться с [] или {}.

Так попробуйте:

if (!preg_match("/[^A-Za-z'-]/",$local_array[$i])) { 

Для получения дополнительной информации о доступе к Массивы см: http://php.net/manual/en/language.types.array.php.

Из инструкции:

Both square brackets and curly braces can be used interchangeably for accessing array elements.

Также ваши поздние regexs также отсутствуют delimiters.

Например ваш второй preg_match должно быть:

preg_match("/^\[?[0-9\.]+\]?$/" 

Если вам нужно использовать модификатор он будет идти после второго /. Если в выражении необходимо использовать /, вы можете либо сбежать от него, либо изменить разделитель. Побег будет \/. В качестве другого разделителя:

preg_match("~^\[?[0-9\.]+\]?$~" 

Вы также должны работать с отступом каждого блока управления.