Не работает update запрос после mysqli_fetch_assoc?

Question

[dudedude]

структура таблицы

в функции update запрос не работает после mysqli_fetch_assoc, до mysqli_fetch_assoc( ) работает, точнее, update запрос возвращает истину, но никаких изменений в таблице не происходит. 7.4 php, database MariaDB 10.3, server last openserver. функция должна просто менять state, текущему на 0, следующему на 1

функция

// $link = mysqli_connect($host, $user, $pass, $db_name);
function setNewColor($link) {
	$query = "SELECT * FROM colors";
	$result = mysqli_query($link, $query);
	for ($data = []; $row = mysqli_fetch_assoc($result); $data[] = $row);
	mysqli_free_result($result);
		for ($i=0;$i<count($data);$i++) {
			if ($data[$i]['state'] == 1) {
				//print_r($data[$i]);
				$id = $data[$i]['id'];

				$query = "UPDATE colors SET state = 0 WHERE id=$id";
				$result = mysqli_query($link, $query);
				//Если state = 1 у последнего цвета, меняем state у первого, иначе меняем state у i++ (следующего).
				if ($i == count($data)-1) {

					$id = $data[0]['id'];
					$query = "UPDATE colors SET state = 1 WHERE id = $id";
					$result = mysqli_query($link, $query);
				} else {

					$id = $data[$i++]['id'];
					$query = "UPDATE colors SET state = 1 WHERE id = $id";
					$result = mysqli_query($link, $query);
				}
				break;
			}
		}

}

такой вариант тоже не работает

function setNewColor($link) {
	$query = "SELECT * FROM colors";
	$result = mysqli_query($link, $query);
	for ($data = []; $row = mysqli_fetch_assoc($result); $data[] = $row);
	mysqli_free_result($result);
		$i;
		$id;
		for ($i=0;$i<count($data);$i++) {
			if ($data[$i]['state'] == 1) {
				//print_r($data[$i]);
				$id = $data[$i]['id'];
				break;
			}
		}
	$query = "UPDATE colors SET state = 0 WHERE id=$id";
				$result = mysqli_query($link, $query);
				//Если state = 1 у последнего цвета, меняем state у первого, иначе меняем state у i++ (следующего).
				if ($i == count($data)-1) {

					$id = $data[0]['id'];
					$query = "UPDATE colors SET state = 1 WHERE id = $id";
					$result = mysqli_query($link, $query);
				} else {

					$id = $data[$i++]['id'];
					$query = "UPDATE colors SET state = 1 WHERE id = $id";
					$result = mysqli_query($link, $query);
				}
}

Comments

[Tendor]

$id = $data[$i++]['id']; а что если ++$i ?)))

[alexalexes]

В этом месте var_dump что выводит?

var_dump($data);
$query = "UPDATE colors SET state = 0 WHERE id=$id";

[dudedude]

alexalexes, array(10) { [0]=> array(3) { ["id"]=> string(1) "1" ["color"]=> string(7) "#FFD3D3" ["state"]=> string(1) "1" } [1]=> array(3) { ["id"]=> string(1) "2" ["color"]=> string(7) "#FFE9D3" ["state"]=> string(1) "0" } [2]=> array(3) { ["id"]=> string(1) "3" ["color"]=> string(7) "#FFFFD3" ["state"]=> string(1) "0" } [3]=> array(3) { ["id"]=> string(1) "4" ["color"]=> string(7) "#DAFFD3" ["state"]=> string(1) "0" } [4]=> array(3) { ["id"]=> string(1) "5" ["color"]=> string(7) "#D3FFE2" ["state"]=> string(1) "0" } [5]=> array(3) { ["id"]=> string(1) "6" ["color"]=> string(7) "#D3FFF8" ["state"]=> string(1) "0" } [6]=> array(3) { ["id"]=> string(1) "7" ["color"]=> string(7) "#D3F0FF" ["state"]=> string(1) "0" } [7]=> array(3) { ["id"]=> string(1) "8" ["color"]=> string(7) "#D3DAFF" ["state"]=> string(1) "0" } [8]=> array(3) { ["id"]=> string(1) "9" ["color"]=> string(7) "#E2D3FF" ["state"]=> string(1) "0" } [9]=> array(3) { ["id"]=> string(2) "10" ["color"]=> string(7) "#FFD3DC" ["state"]=> string(1) "0" } } 1

не знаю зачем это тебе, выводит массив

[Spartak (Web-StyleStudio)]

dudedude,
А затем, что теперь по логике работы вашей функции, вы сначала в первой найденной строке у которой state == 1
Сначала вы обновляете на state равное 0, а потом же в этой функции обновляете эту же строку на 1

[dudedude]

Spartak (Web-StyleStudio),
даже если закомментировать этот код update не происходит.

if ($i == count($data)-1) {
	$id = $data[0]['id'];
					$query = "UPDATE colors SET state = 1 WHERE id = $id";
					$result = mysqli_query($link, $query);
				} else {

					$id = $data[$i++]['id'];
					echo $i;
					$query = "UPDATE colors SET state = 1 WHERE id = $id";
					$result = mysqli_query($link, $query);
				}

[Spartak (Web-StyleStudio)]

Вы же говорите что этот код выполняется

echo $i;

Значит и это условие выполняется и вы обновляете state на 1, которое вы перед этим обновили на 0

[Spartak (Web-StyleStudio)]

И ещё включите отображение ошибок

error_reporting(E_ALL);

// или то же, что и error_reporting(E_ALL);
ini_set('error_reporting', E_ALL);

[dudedude]

Spartak (Web-StyleStudio), да оказалась проблема именно в этом, пойду прочитаю о $I++, ++$I. Чувак куда тебе ещё можно лайк влепить? Огромное спасибо

[Spartak (Web-StyleStudio)]

dudedude,
Ну не ужели допер, что я хотел изначально донести))

Answer 1

[Spartak (Web-StyleStudio)]

$id = $data[$i++]['id'];

Это будет не у следующего, а через один => 0, 2, 4, 6 итд...

Если
$id = $data[++$i]['id'];
Будет тоже через один, но вот так уже => 1, 3, 5, 7 итд..

Если вам надо следующий то тогда так:
$id = $data[$i+1]['id'];

Comments

[Spartak (Web-StyleStudio)]

//Если state = 1 у последнего цвета, меняем state у первого

И не факт что это у последнего!

По логике работы цикла, это первая найденная строка у которой state равен 1, если это так и это не последняя строка то это условие:

if ($i == count($data)-1)

уже не выполнится никогда, так как работа цикла прекращается (break)

[dudedude]

добавив echo $i после этой строчки $id = $data[$i++]['id'];, я убедился что всё работает как надо и без $i+1

[Spartak (Web-StyleStudio)]

dudedude,

$data[$i++]
$data[++$i]
$data[$i+1]

Три варианта, во всех трех вариантах будет разный индекс массива
В первом варианте индексы 0, 2, 4, 6...
Во втором 1, 3, 5, 7...
В третьем 0, 1, 2, 3, 4 итд...

[FanatPHP]

ладно здесь ты верно заметил, привильно циферки посчитал
вот на такого уровня вопросы пиши ответы, разрешаю

[Spartak (Web-StyleStudio)]

dudedude,

добавив echo $i после этой строчки

Выводить надо

/*не echo $i, а*/
var_dump($data)

И увидите что индексы у дата будут не те которые вы ожидаете

иначе меняем state у i++ (следующего).

У вас это не следующего, а через один

Answer 2

[FanatPHP]

Где отладка?
Написал кучу бессмысленных предложений и не написал главного.
Вопросы вида "у миня ниработаит" надо задавать вот в таком формате

Вот результат селекта:
код с селектом строки и выводом результата
Вот результат апдейта этой же самой строки:
код с апдейтом и выводом результата

И сразу, ещё на этапе написания вопроса, все станет ясно, где в твоем коде косяк

Comments

[dudedude]

да, у меня не робит, никаких ошибок нет, всё что мог я написал, все функции возвращают true (1)

[FanatPHP]

Ещё раз:
Пиоешь свой вопрос в таком виде:
Вот результат селекта:
код с селектом строки и выводом результата
Вот результат апдейта этой же самой строки:
код с апдейтом и выводом результата

что здесь неясно?

[dudedude]

FanatPHP, Как выводить die(mysqli_error($link)?

[FanatPHP]

никак, это бессмысленная строчка
Пишешь свой вопрос в таком виде:
Вот результат селекта:
код с селектом строки и выводом результата
Вот результат апдейта этой же самой строки:
код с апдейтом этой же строки и выводом результата
что здесь неясно?

[dudedude]

FanatPHP, Как выводить?

[FanatPHP]

через echo
но если совсем не понимаешь что делают функции, которые ты пишешь в своем коде, то var_dump()
она сама за тебя посмотрит, что лежит в переменной, полученной из базы

[dudedude]

FanatPHP, все функции возвращают true (1)

[FanatPHP]

что, mysqli_fetch_array() тоже позвращает true?