Почему не записываются данные в БД?

Question

[thekot24]

Короче, ребят. Есть код:

for($i = 0; $i < count($groups); $i++){
  /*  $find = mysql_query("SELECT * FROM groups_data WHERE group_id=".$groups[$i]);
    $find = mysql_fetch_assoc($find);
    if(empty($find['group_id'])){
         mysql_query("INSERT INTO groups_data (group_id,post_id) VALUES (".$groups[$i].",0)") or die(mysql_error()); //добавляем наши группы в базу, стави post_id по умолчанию 0
    } else {*/
        $wallGet = api("https://api.vk.com/method/wall.get?owner_id=-".$groups[$i]."&count=1");
        $wallGet = json_decode($wallGet);
        $id =  $wallGet->response[1]->id.' ';
        $group = $groups[$i];
   

    $res =   mysql_query("UPDATE groups_data SET `post_id`='".$id[$i]."' WHERE `group_id`='".$groups[$i]."';");
       echo $res.'<br>';
 #   }
 
    
}

Он почему-то записывает данные только первого group_id, а дальше - нет, что делать?
в $groups это:

$groups = array('122860616','122860645','122860660');

Comments

[thekot24]

Neoline: неет, ВК возвращает все данные, проверял. только в БД ничего не записывается

[OVK2015]

Это очень интересная конструкция
$find = mysql_fetch_assoc($find);
php уже на 2-ой итерации ругнуться должен

[thekot24]

OVK2015: ошибок вообще нет.

Answer 1

[Ruslan]

Я один вижу что запрос со вставкой закоментирован? Выполняется только апдейт.

Comments

[thekot24]

Код до "else" не столь важен, в данной ситуации вставил его сюда только для полноты картины.

[Ruslan]

thekot24: Сделайте так
$res_sql = "UPDATE groups_data SET `post_id`='".$id[$i]."' WHERE `group_id`='".$groups[$i]."';";
$res = mysql_query($res_sql);
echo $res.'
';

[thekot24]

Ruslan: сработало, спасибо.

[Ruslan]

Neoline: там в выводе был не sql скрипт, а результат работы mysql_query, который вероятнее всего был с ошибкой. Автор вывел на экран скрипт и увидел ошибку.

[thekot24]

Neoline: не знаю, но это сработало:)

Answer 2

[ThunderCat]

Оох, такой каша вариться, моя не смотреть! (с) Jar Jar Binks.
1) Каша внатуре, ну да ладно, это пол беды.
2) Используйте prepared statements и PDO.
3) Формируйте запросы в переменные, только потом передавайте запрос на исполнение, так хоть можно дебаггить что там в запрос попадает.
4) Если нифига не ясно(как тут) - через строчку ставим print_r(), и так для всего что шевелится.
5) Profit

Comments

[thekot24]

Для чего мне PDO? Скрипт используется узким кругом лиц. (1-2 человека)

[ThunderCat]

thekot24: разве это критерий делать/не делать? Во первых - это современно, во вторых mysql функции депрекатед, в третьих - вы же просили совета у людей - вам советуют, там и отладка нормальная, и завтра работать не перестанет. Да и кроме того - все остальное сделайте, никто вам ничего не навязывает же. Скорее всего как все выполните найдете свой баг за пару минут.

[thekot24]

ThunderCat: сделаем когда-нибудь. :)