Скорость записи в бд?

Question

[emerysh]

Получаю json - декодирую - через foreach обновляю в бд - около 2-3к записей - выполняется 3-5 мин. В чем проблема?
Можно ли ускорить до 1 мин?
Нормальная ли это скорость?

Сам код

$url = $site;
	$file = file_get_contents($url);
	$jsondate = json_decode($file, true);


	foreach ($jsondate as $k => $value) {
		$keys = array_values($value);
		foreach ($keys as $bo) {
			$name= $bo['name'];
			$price = $bo['price'];
			///////////////////////
			$name= mysqli_real_escape_string($link, $name);
			///////////////////////
			$sql = mysqli_query($link,"UPDATE 	`price` SET `price`='$price' WHERE `name`='$name'");
		}
	}

Comments

[jasonOk]

Если колонка name уникальна, то нужно обязательно добавить limit 1

Answer 1

[Boris Köln]

Ненормально.
1. Обновлять надо по первичному ключу (ID), а не имени
2. Выполнять надо не 3-4К запросов, а 4 шт. Как сформировать 1 запрос UPDATE с тысячей значений?

Comments

[Boris Köln]

нужно сделать отдельный скрипт который будет по имени искать id?

Нужно ЭТОТ скрипт заменить на нормальный.

обновлять этот id?

Обновит не "id", а "по id"!

[emerysh]

Boris Korobkov: я все равно не понимаю, откуда я могу знать id товара, когда я парсером достаю названия из других сайтов?

Answer 2

[Arris]

1. READ https://dev.mysql.com/doc/refman/5.7/en/create-ind...
2. USE https://dev.mysql.com/doc/refman/5.7/en/insert-on-...

Answer 3

[FanatPHP]

Как всегда, милилон ответов, все не в кассу.

Правильный ответ - завернуть запросы в транзакцию.
Все остальные правки в коде не имеют отношения к принципиальному ускорению, и сделаны только затем, чтобы код выглядел не так позорно.

mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
$link->begin_transaction();
$stmt = $link->prepare("UPDATE `price` SET `price`=? WHERE `name`=?");
$stmt->bind_param("ss", $price, $name);
foreach ($jsondate as $k => $value) {
    foreach ($value as $bo) {
        $name = $bo['name'];
        $price = $bo['price'];
        $stmt->execute();
    }
}
$link->commit();

Comments

[emerysh]

Скрипт выполнялся за 229.98372888565 сек
что несильно изменило положение дел..

[emerysh]

в то время как временная таблица создается за 4-6 сек, но как обновить основную таблицу используя временную?
к сожалению, внешние ключи не работают..

[FanatPHP]

emerysh: полный размер таблицы какой? если больше 2-3 тысяч, то нужен индекс на name.