1

Здравствуйте. Не могу понять, почему не работает такой цикл:

foreach($arr as $mail){
    $res=mail($mail,"Это может быть тема письма",$msg,$header);
    if($res) echo "OK"; else echo "Sorry..";
}

Выводит: OKSorry..Sorry..Sorry.. и т.д. В чем здесь может быть ошибка?

UPD:

Хедеры такие:

$header="Content-Type: text/html; charset=windows-1251\n";
$header.= 'From: vasya@pupkin.ru '. "\r\n"
Msg: '<h1>Hello!</h1>',
$arr: $arr=array([0]=>'mike@doe.com', [1]=>'mike@doe.com', [2]=>'mike@doe.com')
12
  • 1
    Что в массиве $arr? Расставьте фигурные скобки в условиях. У экстрасенсов сегодня выходной
    – iurii_n
    9 мар 2012 в 14:48
  • @Barton, в массиве $arr хранятся все E-mail-ы для рассылки. Впервые слышу, что фигурные скобки в условии обязательны )) Еще вопросы?
    – alex_90
    9 мар 2012 в 16:41
  • А разве я упомянул, что фигурные скобки обязательны? С ними код более удобочитаем.
    – iurii_n
    9 мар 2012 в 21:59
  • Какие хедеры отправляете? $msg?Все переменные в студию. Включая $arr чтоб мы могли у себя протестировать код с такими же параметрами
    – iurii_n
    9 мар 2012 в 22:01
  • >Выводит: OKSorry..Sorry..Sorry.. и т.д. В чем здесь может быть ошибка? Ошибка в $arr.
    – butteff
    14 мар 2012 в 13:56

3 ответа 3

2

Не вижу проблемы. Стало интересно, скопировал код, расставил скобки. запустил и вуаля - все работает. Попробовал на хостинге тоже без проблем. Вот код:

$arr=array('test1@mail.com','test2@mail.com');
$msg = '<h1>Hello!</h1>';
$header="Content-Type: text/html; charset=windows-1251\n";
$header.= 'From: vasya@pupkin.ru '. "\r\n";
foreach($arr as $mail){
    $res=mail($mail,"Это может быть тема письма",$msg,$header);
    if($res) {
        echo 'OK<br />';
    } else {
        echo 'Sorry..<br />';
    }
}
6
  • @Barton, в точности скопировал ваш код(поставил свои E-mail-ы) -- результат тот же. Отправляется только первое письмо...
    – alex_90
    15 мар 2012 в 13:09
  • Я тоже пробовал в денвере- все нормально уходит) Мне кажется, автор задал вопрос, потому что правильный код выполняется неправильно и это странно)
    – ReinRaus
    15 мар 2012 в 13:28
  • @alex_90, могу посоветовать попробовать на другой машине, а еще лучше на хостинге. По возможности протестируйте с реальными адресами.
    – iurii_n
    15 мар 2012 в 13:36
  • Ну на хостинге все нормально уходит. 30 писем в лет ушло.
    – Artem
    15 мар 2012 в 13:45
  • 4
    бесплатным хостингом не пользовался, но рискну предположить, что стоит ограничение на количество отправляемых писем в единицу времени.
    – ikoolik
    15 мар 2012 в 15:40
1

Написал письмо в службу тех поддержки.. Оказывается было ограничение на время между отправкой писем - 4 секунды.

@ikoolik, вы оказались правы))

-3

if($res) echo "OK"; else echo "Sorry..";

По моему ошибка в условии! Даже при твоем раскладе это должно было выглядеть так: if($res){ echo "ok"; }else{ echo "Sorry"; }

И где блин само условие то? Функция mail() должна вернуть true если все пучком, false если отправка не удастся Тоесть условие твоё должно выглядеть так

if($res == true){ echo "ok"; }else{ echo "Sorry"; }

3
  • 2
    @icedev if(true == true) return true замечательные условия у вас =) функция mail возвращает логическое значение, которое необходимо оператору if. Сравнение результата с true бессмысленно.
    – ikoolik
    15 мар 2012 в 4:06
  • Жесть! Ковычки нужны не всегда, смысл городить их куда только можно.
    – Artem
    15 мар 2012 в 6:12
  • 1
    Халк злой!!! Готов убить каждого кто напишет if($res == true)... Стереть с лица Земли этих быдло-кодеров... >__< 21 мар 2012 в 0:23

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими пользовательским соглашением, политикой конфиденциальности и политикой о куки

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.