Помогите составить скрипт PHP/SQL

[Andkorol]

Я что то не понял, а зачем вообще нужно разбираться с форматом датой и временем, сравнений ведь уже никаких нет.

- разбираться с этим нужно потому,что вот это

SELECT * FROM `anketa` WHERE `date` = date(NOW())

работает исключительно потому,что разработчиками учтены такие вот "форс-мажоры" в коде...
НО - как я уже говорил выше - учитесь писать простой,правильный и понятный код - в будущем это сэкономит вам(и не только) кучу времени и нервов.

Что касается этого:

PHP код:

$current_time = time("H:i"); 


- я по-прежнему советую никому этого не показывать...почитайте описание функции time()

Вообще - скачайте себе толковый мануал по PHP со справочником функций - полезнейшая вещь,всегда можно посмотреть,что нужно той или иной функции для корректной работы.

Ещё совет - примените-таки дополнительное поле `sent`, как я описывал выше - cron cron-ом - но всякое с ним бывает,а отметки в Базе всегда дадут вам понять,какие анкеты вы отправили,а какие - ещё нет.

Кстати - mail() также требует 3 параметра,а не 2 как у вас:

bool mail ( string $to , string $subject , string $message [, string $additional_headers [, string $additional_parameters ]] )

[erwerwe]

можно узнать этот код правильный?:

PHP код:

 <?php 
 include "config.php"; 
 $SelectQuery="SELECT * FROM  `anketa` WHERE `sent`=0 `date` = date(NOW(),'%m-%d-%Y') AND DATE_FORMAT(time,'%H%i') = DATE_FORMAT(NOW(),'%H%i')" 
  $SelectReslut = mysql_query($SelectQuery); 
  //сколько записей получили
  $Count = mysql_num_rows($SelectReslut); 
  for ($i=1; $i<=$Count; $i++) { 
    //получаем очередную строку
    $Fetch = mysql_fetch_array($SelectReslut); 
    //выводим результат
    echo $Fetch['text']." : ".$Fetch['email']." : ".$Fetch['time']." : ".$Fetch['date']; 
  // $date - заведомо известная переменная
  // $time - заведомо известная переменная
//(взято из базы в формате ГГГГ-ММ-ДД)
//(взято из базы в формате ч-м)

$to = $Fetch['email']; 
 $message .=" Текст письма : ".$text; 

 if(mail($to, $subject,$message)){ 
    echo $query = "update `anketa` set `sent`=1 where `id`='.$id";
}  
else{  
    echo "есть некоторые ошибки, чтобы отправить, проверьте параметры сервера"; 
} 
?>

[Andkorol]

Первый показатель правильности кода - его работоспособность.
1.

`date` = date(NOW(),'%m-%d-%Y')

PHP код:

`date` = DATE_FORMAT(NOW(),'%m-%d-%Y') 


2. Собственно нет проверки на то,есть ли результат у запроса.

3. $subject - ???

4. Это что такое???:

PHP код:

echo $query = "update `anketa` set `sent`=1 where `id`='.$id"; 


неверный синтаксис,должно быть примерно так:

PHP код:

$query = "update `anketa` set `sent`= 1 where `id` = $id ";
mysql_query($query); 


- кстати что такое $id ???

Совет: привыкайте правильно форматировать SQL-запросы,примерно так:

PHP код:

$query = "UPDATE `anketa` SET `sent` = 1 WHERE `id` = $id " 


- т.е. старайтесь SQL-синтаксис писать большими буквами - это позволит легко разобраться в коде и визуально выделяет текст SQL-запроса.

Ещё такой момент - раз уж вы планируете использовать cron для запуска этого файла 1 раз в минуту - ЗАЧЕМ вам выводы (echo) в этом файле?? Какой в них смысл?
Правильнее будет реализовать запись сообщения об ошибке отправки анкет(если таковые будут) в какой-нить специально созданный "файл ошибок отправки анкет" - это я образно...
Ведь если файл вызывает не браузер, а cron - соответственно никто ничего не увидит... никогда ....

[erwerwe]

Ведь если файл вызывает не браузер, а cron - соответственно никто ничего не увидит... никогда ....

Это как раз для проверки работоспособности, временный параметр-потом удалю

[erwerwe]

Parse error: syntax error, unexpected T_VARIABLE in - путь до скрипта script.php on line 4

Вот походу 4-ая строчка:

PHP код:

$SelectReslut = mysql_query($SelectQuery); 


В чём может быть ошибка?

PHP код:

<?php 
 include "config.php"; 
 $SelectQuery="SELECT * FROM  `anketa` WHERE `sent`=0 `date` = DATE_FORMAT(NOW(),'%m-%d-%Y') AND DATE_FORMAT(time,'%H%i') = DATE_FORMAT(NOW(),'%H%i')" 
  $SelectReslut = mysql_query($SelectQuery); 
  //сколько записей получили
  $Count = mysql_num_rows($SelectReslut); 
  for ($i=1; $i<=$Count; $i++) { 
    //получаем очередную строку
    $Fetch = mysql_fetch_array($SelectReslut); 
    //выводим результат
    echo $Fetch['text']." : ".$Fetch['email']." : ".$Fetch['time']." : ".$Fetch['date']; 
  // $date - заведомо известная переменная
  // $time - заведомо известная переменная
//(взято из базы в формате ГГГГ-ММ-ДД)
//(взято из базы в формате ч-м)

$to = $Fetch['email']; 
$message .=" Текст письма : ".$text; 
$subject = "Заголовок письма";
 if(mail($to, $subject, $message)){ 
    echo $query = "UPDATE `anketa` SET `sent`= 1 WHERE `id` = 0 "; 
mysql_query($query);
}  
else{  
    echo "Есть некоторые ошибки, чтобы отправить, проверьте параметры сервера"; 
} 
?>

[Johnatan]

Parse error: syntax error, unexpected T_VARIABLE in - путь до скрипта script.php on line 4

перевод

ОШИБКА! Вы забыли поставить ; в "путь до скрипта script.php" линия 3

[Andkorol]

Вот вы убрали $id :

where `id` = $id

сделали так :

WHERE `id` = 0

- к чему..???

Я готов допустить,что у вас в базе есть анкета с `id` = 0 - но дальше-то что..???
Вы при каждой удавшейся отправке письма собираетесь устанавливать `sent` = 1 исключительно для этой записи?
Для тех что вы отправили - этого делать не нужно?

Вводите в таблицу поле `id` INT(11) Auto_increment, ставьте его Primary Key (т.к. я допускаю что в вашей теперешней таблице индексов и ключей нет совсем) - и возможно будет вам радость....

[erwerwe]

Johnatan спасибо исправил

Я готов допустить,что у вас в базе есть анкета с `id` = 0 - но дальше-то что..???
Вы при каждой удавшейся отправке письма собираетесь устанавливать `sent` = 1 исключительно для этой записи?
Для тех что вы отправили - этого делать не нужно?

Насколько я понял лучше удалить `

PHP код:

id` = 0 


так как этого поля (id) у меня нет.

Думаю оставить такую запись:

PHP код:

echo $query = "UPDATE `anketa` SET `sent`= 1"; 


PHP код:

Вводите в таблицу поле `id` INT(11) Auto_increment, ставьте его Primary Key (т.к. я допускаю что в вашей теперешней таблице индексов и ключей нет совсем) - и возможно будет вам радость.... 


А зачем они нужны (индексов и ключей)? Имеется ввиду вместо поля sent? Но sent вроде работает-когда данные попадают в таблицу sent=0, а когда отправляются-я пока не могу проверить.

Таблица имеет вид:

[Andkorol]

А зачем они нужны (индексов и ключей)?

В первом разделе данного форума книг полно - почитайте (просмотрите) хоть одну.Вам без этого - НИКАК.

Имеется ввиду вместо поля sent?

С чего вы взяли что вместо..???
Поле `id` будет уникальным для каждой анкеты - оно позволит вам чётко указать,для каких именно анкет нужно проставить `sent` = 1 в текущей операции.Тоесть вам нужно будет собрать id всех только что отправленных анкет - и именно для этих записей проставить в БД `sent` = 1.

PHP код:

echo $query = "UPDATE `anketa` SET `sent`= 1"; 


- это ваше echo - глупость несусветная,уж простите.... - Я вам уже о нём говорил ранее.

Ваша таблица должна иметь вид(примерно):
id
text
email
date
time
sent

- хотя я бы не разделял date и time - а сделал бы одно поле (`added`,например, тип поля - DATETIME)

[Johnatan]

erwerwe
Крайне настойчиво рекомендую вам почитать современную литературу по PHP/MySQL и логике программирования. Вы сейчас пытаетесь сделать автомобиль из "лего", и чтобы он сам ездил.