Я обновлял с 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;
}
Когда форма отправлена, я получаю эту ошибку:
Неустранимая ошибка: имя функции должно быть строкой в /usr/local/www/panhistoria/MyBooks/email_alert.php в строке 241
Строка 241 является первой !preg_match
$local_array($i)
это массив, а не функция, поэтому к нему нужно обращаться с помощью []
, или же {}
,
Так что постарайтесь:
if (!preg_match("/[^A-Za-z'-]/",$local_array[$i])) {
Для получения дополнительной информации о доступе к массивам см .: http://php.net/manual/en/language.types.array.php.
Из руководства:
Как квадратные скобки, так и фигурные скобки могут использоваться взаимозаменяемо для доступа к элементам массива.
Также ваши более поздние регулярные выражения также отсутствуют разделители.
Например, ваш второй preg_match
должно быть:
preg_match("/^\[?[0-9\.]+\]?$/"
Если вам нужно использовать модификатор, он будет идти после второго /
, Если /
необходимо использовать в выражении, вы можете либо избежать его, либо изменить разделитель. Побег будет \/
, Как другой разделитель:
preg_match("~^\[?[0-9\.]+\]?$~"
Вы также должны работать над отступом каждого блока управления.
Других решений пока нет …