1

Не работает перебор значения в циклах for, foreach.

Конкретнее после удаления строки из таблицы, обновить столбец с id. Т.е. например мы имеем таблицу где в ней есть столбец id со значениями 1, 2, 3 ... послед удаления строки например 2 столбец с id принимает значения 1, 3.... Мне нужно осуществить обновление столбца id в соответствии количество строк, что бы стало id: 1, 2 ....

Логика такая удалить значение с бд и после обновить столбец с ID.

$delete = mysql_query("DELETE FROM test WHERE id='".$_GET['id']."'");
$id = mysql_query("SELECT id FROM test");
while($idCheck[] = mysql_fetch_assoc($id)) {
    #echo $idCheck["id"]."\n";
}

echo count($idCheck);
for($i = 0; $i < count($idCheck); $i++) {
    mysql_query("UPDATE test SET id=".$i);
}

получил массив со значениями в поле id он соответствует бд. Но в переборе в цикле значений в обновление бд вписывает общее значение строк после удаления . Т.е вместо 1, 2, 3 ... записывает 2, 2, 2...

9
  • забавно будет, если вам в качестве $_GET['id'] кто-то передаст 0' or id in (select id from test)# не правда ли?
    – teran
    27 окт 2019 в 17:31
  • забавно, вопрос не в этом 27 окт 2019 в 17:33
  • у вас в $rows просто число строк. То есть если у вас в таблице после удаление осталось 350 строк. вернется 350. И далее вы делаете 350 апдейтов где id= 350. вы ведь не извлекаете конкретные id из таблицы, а только используете их общее значение.
    – teran
    27 окт 2019 в 17:34
  • нет, после удаления будет 349 строк, и id будет 349 .. 351 27 окт 2019 в 17:39
  • исходную задачу объясните. вы апдейтите всю таблицу последним запросом
    – teran
    27 окт 2019 в 18:13

3 ответа 3

3

Другой вариант тоже не работает - тут стоит включить отображение ошибок на экран, потому что исходя из "конструкции"

foreach ($idCheck as $key => $value) {
    mysql_query("UPDATE test WHERE id='".$value[$key]."'");
}

вряд-ли существует индекс $value[$key] вообще

2
  • Существует, выдача общего числа строк. 27 окт 2019 в 20:39
  • @СтаниславФарфудинов не придумывайте, $value это число/
    – teran
    28 окт 2019 в 7:05
1
$delete = mysql_query("DELETE FROM test WHERE id=".$_GET['id']);
$id = mysql_query("SELECT id FROM test");
$rows = mysql_num_rows($id);
$fields = mysql_num_fields($id);
for($i = 0; $i < $rows; $i++) {
    mysql_query("UPDATE test WHERE id = ".$i);
}
7
  • Вы меняете в цикле переменную $i, а подставляете постоянную $rows. 27 окт 2019 в 17:49
  • не работает, посмотрите мою обновление в посте, как вы написали я уже делал 27 окт 2019 в 18:04
  • Мало исходных данных. Что вы хотите получить в итоге? 27 окт 2019 в 18:09
  • что делает ваш код? Обновиь значения столбца бд ID после удаления из бд строки 27 окт 2019 в 18:11
  • "UPDATE test WHERE id = ".$i эта строка ничего не делает 27 окт 2019 в 18:21
0

Если не брать во внимание в целом использование get-запроса для измеения данных, использование устаревшего расширения для работы с БД, отсутствие какой-либо проверки на валидность данных и защиты от инъекций, и самое главное непонятность, зачем это все надо, то задачу надо решать не через циклы на стороне пхп.

у вас есть цепочка идентификаторов 1 2 3 4 5. Вы удаляете 3, получаете 1 2 4 5. Далее вместо циклов по перебору нужно выполнить только один запрос на апдейт записей 4 и 5 уменьшив их на 1.

$id = $_GET['id'];
mysql_query("delete from test where id = $id");
mysql_query("update test set id = id - 1 where id > $id") ;

в своих попытках решения UPDATE test SET id=$i у вас нет условия where, так что вы данным значением обновляете всю таблицу целиком. вот и остаются значения только последней итерации цикла.

2
  • спасибо, работает. Оказывается все так просто. 29 окт 2019 в 4:15
  • @space_is_fun тогда ткните галку у вопроса
    – teran
    29 окт 2019 в 6:45

Ваш ответ

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

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