0

У меня есть sql запрос,но он не обрабатывается, посмотрите пожалуйста и подскажите в чём ошибка

  $currentTime = (int)date('H');
  $currentdate = date("Y-m-d");
  $visible = 1;

for($i = 1; $i < 15; $i++){

  $time_set1 = find_all_time_f($i);
  $arrayOfFotball1 = mysqli_fetch_assoc($time_set1);
  $time_count1 = mysqli_num_rows($time_set1);

  $time_f = $arrayOfFotball1['time'];
  $rdate_f = $arrayOfFotball1['rdate'];



  if(strtotime($currentdate) > $rdate_f)  //HERE we should check Date if date < then  change visibility
  {
  //2.Perfor database querys
    //exit("Error");
    $query  = "UPDATE sport_football SET ";
    $query .= "visible = {$visible} "; 
    $query .= "WHERE time = {$currentdate} ";
    $result = mysqli_query($connection, $query);
  }else if($currentTime > $time_f){
    $query  = "UPDATE sport_football SET ";
    $query .= "visible = {$visible} "; 
    $query .= "WHERE time = {$currentdate} ";
    $result = mysqli_query($connection, $query);

  }//if statement

}//for loop

//check function for football
1
  • А у вас в базе точно в поле time лежит дата без времени. потому как иначе по равенству не найдет. И что то не вижу, чем у вас отличаются два update. может их надо в один if положить, дублировать код незачем. Ну и про дату вам ответили. Хотя его еще можно в кавычки заключить просто ...
    – Mike
    29 янв 2016 в 21:06

1 ответ 1

1

проблема тут

$query .= "WHERE time = {$currentdate} ";

В этот момент $currentdate содержит дату в формате 'XXXX-XX-XX', а это "текст"

надо вот так

$query .= "WHERE time = ".strtotime($currentdate);
4
  • интресено... а за что минус? что не так?
    – cyadvert
    29 янв 2016 в 18:10
  • mysql в принципе поймет дату строкой, если она будет в кавычках. так что можно и time='{$currentdate}'. У нас тут минусатор на sql похоже завелся, за последние несколько дней у меня несколько минусов на нормальных ответах по теме ...
    – Mike
    29 янв 2016 в 21:04
  • @Mike молодые люди, вы себе невероятно льстите, считая свои попытки что-то сочинить на ходу "нормальными ответами". 30 янв 2016 в 5:53
  • 1
    @Ипатьев Тогда пишите в комментариях к ответу, в чем именно мы не правы, давая тот или иной ответ. SO призван обучать. А если ученику ставят двойку и пр этом не объясняют причин, позволяя дальше заблуждаться - это не обучение. И еще странно выглядит заминусованный единственный, рабочий, ответ на вопрос в отсутствии каких либо других ответов.
    – Mike
    30 янв 2016 в 7:25

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

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