Кто может допилить код PHP

[xatchikzzz]

<?php
#########################################
##
#########################################
set_time_limit(0);
$handle = fopen("emails.txt", "r");
$pattern = "#\.com$|\.info$|\.net$|\.org$|\.us$|\.de$|\.tv$|\ .ws$|\.tr$|\.gov$|\.biz$|\.uk$|\.name$|\.mil$|\.ed u$#"; #маски для домена \.{{EXT}}$, разделитель "|"
$wantEmails = 'good_emails.txt';
$otherEmails = 'other_emails.txt';
while (!feof($handle)){
$email = trim(fgets($handle, 1024));
if (preg_match($pattern, $email, $matches)>0){
file_put_contents($wantEmails, $email .PHP_EOL,FILE_APPEND);
}else file_put_contents($otherEmails, $email.PHP_EOL,FILE_APPEND);


}

fclose($handle);
?>

вот код ... он фильтрует email адреса которые оканчиваются на .com$|\.info$|\.net$|\.org$|\.us$|\.de$|\.tv$|\.ws $|\.tr$|\.gov$|\.biz$|\.uk$|\.name$|\.mil$|\.edu

надо дописать что бы была проверка если ли в строке знак @ , что был конкретный адрес почты ..

кто может допилить немного???

[Painkiller]

Сообщение от DimaX

PHP код:

<?php

$pattern = "#\.com$|\.info$|\.net$|\.org$|\.us$|\.de$|\.tv$|\.ws$|\.tr$|\.gov$|\.biz$|\.uk$|\.name$|\.mil$|\.edu$#";

        if (strpos($email, '@') > 0 && preg_match($pattern, $email))

?>

есть же стандартная функция

PHP код:

filter_var($email, FILTER_VALIDATE_EMAIL) 


и никакой ебли

[Алёша]

\.com$|\.info$|\.net$|\.org$|\.us$|\.de$|\.tv$|\.w s$|\.tr$|\.gov$|\.biz$|\.uk$|\.name$|\.mil$|\.edu$ "

\.(com|info|net|org|us|de|tv|ws|tr|gov|biz|uk|name |mil|edu)$
но ты походу не в курсе, что у каждой страны есть свой домен и можно слегка подзаебаться, отлавливая их всех, поэтому делается проще:
\.[a-zA-Z]{2,6}$
(да, есть домены на 6 букв: .museum)

[DimaX]

Сообщение от Painkiller

есть же стандартная функция

PHP код:

filter_var($email, FILTER_VALIDATE_EMAIL) 


и никакой ебли

filter_var(PHP 5 >= 5.2.0)
Не везде будет работать.

Сообщение от Алёша

\.(com|info|net|org|us|de|tv|ws|tr|gov|biz|uk|name |mil|edu)$
но ты походу не в курсе, что у каждой страны есть свой домен и можно слегка подзаебаться, отлавливая их всех, поэтому делается проще:
\.[a-zA-Z]{2,6}$
(да, есть домены на 6 букв: .museum)

ТС как написал, какие зоны ему нужны, я так и сделал. В голову не приходило, что с других зон мыла ему не нужны? И да, про .музеи и прочие .коопы в курсе, ага.

[Алёша]

Сообщение от DimaX

ТС как написал, какие зоны ему нужны, я так и сделал. В голову не приходило, что с других зон мыла ему не нужны? И да, про .музеи и прочие .коопы в курсе, ага.

я ТСу и писал, это же его строка то изначально