Как исправить код php?

Question

[Марк Ешкилев]

Здравствуйте, помогите пожалуйста, есть такая функция

public function remove_icon($icon) {
		global $db;
		if(!isset($this->icon[$icon]))
			return;
		unset($this->icon[$icon]);
		$db->execute("UPDATE `players` SET `icon`='{$db->safe(json_encode($this->icon))}' WHERE `id`='{$db->safe($this->id)}'")or die($db->error());
	}

и есть вот такое использование этой функции
$pl->remove_icon((int) $_POST['icon']);

проблема такова, что при использовании данной функции, json который образует данная функция не совсем правильный для меня, к примеру возьмём json - [{"id":"5"},{"id":"10"}], если я захочу убрать из json - "id":"5", то на выходе получается, вот такая вот дичь, {"1":{"id":"10"}}, как можно сделать так чтобы json отображался после такой функции вот так - [{"id":"10"}]??? Помогите пожалуйста

Comments

[FanatPHP]

Я бы ответил, но у меня глаза кровоточат от этого говнокода, всех этих джейсонов в бд и "db-safe"

Ну и да, научиться работать с массмвами неплохо бы самостоятельно, а не бегать с каждым вопросом на хабр

[Марк Ешкилев]

FanatPHP, этот код не я писал.

Answer 1

[Rsa97]

array_values()

$t = ['value0', 'value1'];
unset($t[0]);
print json_encode($t); // {"1":"value1"}
$t = array_values($t);
print json_encode($t); // ["value1"]

Comments

[Марк Ешкилев]

и тут у меня проблема, я написал вот так

public function remove_icon($icon) {
		global $db;
		if(!isset($this->icon[$icon]))
			return;
		unset($this->icon[$icon]);
		array_values($this->icon[$icon]);
		$db->execute("UPDATE `players` SET `icon`='{$db->safe(json_encode($this->icon))}' WHERE `id`='{$db->safe($this->id)}'")or die($db->error());
	}

в итоге выдает ошибку

[15-Jun-2021 14:39:46 Europe/Moscow] PHP Warning:  array_values() expects parameter 1 to be array, null

[Rsa97]

Марк Ешкилев, Вы только что удалили $this->icon[$icon] через unset и тут же пытаетесь с ним работать.
$this->icon = array_values($this->icon);

[Марк Ешкилев]

Rsa97, а.. спасибо