В чем ошибка в запросе к базе?

Question

[prostovlad]

Добрый день, есть запрос к базе, раньше работал, сейчас дает ошибку, в чем может быть проблема *
запрос

UPDATE `'.DB_PREFIX.'product` AS `p`
    LEFT JOIN (
        SELECT `t`.`sku`, MIN(`p`.`product_id`) AS `product_id`
            FROM (
                SELECT `sku`, MIN(`price`) AS `price`
                    FROM `'.DB_PREFIX.'product`
                    WHERE `quantity` != 0
                    GROUP BY `sku`
            ) AS `t`
            JOIN `'.DB_PREFIX.'product` AS `p`
                ON `p`.`sku` = `t`.`sku` AND `p`.`price` = `t`.`price`
            WHERE `p`.`quantity` != 0
            GROUP BY `t`.`sku`
    ) AS `i` ON `i`.`product_id` = `p`.`product_id`
    SET `p`.`status` = (`i`.`product_id` IS NOT NULL)

ошибка по этой строке
SET `p`.`status` = (`i`.`product_id` IS NOT NULL)
текст ошибки
Notice: Trying to get property of non-object in
смысл запроса, выборка нескольких товаров по одинаковому sku, поставить всем статус 0 ( отключено) и поставить статус одному товару 1 ( включено) у которого количество quantity больше 0 и самая минимальная цена.

Comments

[Василий Банников]

У тебя тут что-то странное в середине запроса:
'.DB_PREFIX.'

[prostovlad]

Василий Банников, это константа, префикс базы, подставляется, которая указывается в конфиге при установке,
ну если так принципиально, замените на `'ls_product`
но сути это не изменит

[Rsa97]

А причём тут MySQL, если ошибка из PHP?

[Akina]

LEFT JOIN (...) AS `t` можно смело заменить на INNER JOIN.

[prostovlad]

Akina, не помогло

[Akina]

prostovlad, вообще-то мой комментарий - не попытка решить проблему, а дополнительное замечание к кривому запросу.

А по проблеме я полностью согласен с предыдущим оратором. Показанное сообщение об ошибке никакого отношения к MySQL не имеет, это косяк на стороне PHP.

[Ищю в поисковиках]

Обычно, в ошибке пишется номер и строка, где это произошло.

Запрос я рекомендую упростить, тк джоин по вложенному селекту с джоином - это не совсем адекватный запрос.

[prostovlad]

Ищю в поисковиках, я же написал, ошибка вот по этой строке
SET `p`.`status` = (`i`.`product_id` IS NOT NULL)

Запрос я рекомендую упростить

и как его упростить ?

[Ищю в поисковиках]

prostovlad Ошибки PHP примерно выглядят так:

[22-May-2023 13:00:40 Europe/Moscow] PHP Parse error:  syntax error, unexpected end of file in /путь/до/скрипта/index.php on line 3

То, что вы скинули, это SQL запрос в базу данных.

> и как его упростить ?

Как минимум, убрать LEFT JOIN:

UPDATE `'.DB_PREFIX.'product` SET `p`.`status` = (`i`.`product_id` IS NOT NULL) WHERE {условие}

[Михаил Ливач]

покажите кусок кода выше - тот, который ведёт к

UPDATE `'.DB_PREFIX.'

. Ошибка там, скорее всего.

Answer 1

[ThunderCat]

Notice: Trying to get property of non-object in

Чтение ошибки внимательно может дать подсказку, особенно если понять что за проблема описана в ошибке.

я же написал, ошибка вот по этой строке
SET `p`.`status` = (`i`.`product_id` IS NOT NULL)

Очевидно это не так, читайте всю ошибку внимательно, в том числе смотрите на файл в котором она возникает, так же не лишним будет использовать нормальные IDE, с правильным отображением нумерации строк и подсветкой ошибок.

Comments

[prostovlad]

вы считаете что я не вижу в какой строке ошибка ? хм, может и так, спорить не буду, буду рад если вы укажите что я не прав и ошибка не в этой строке.
https://skr.sh/sJxENoqfgIM?a
https://skr.sh/sJxB9jyw26D?a

[Сергей delphinpro]

prostovlad, строку вы видите, но к вопросу приложили совершенно не тот код.
Как уже заметили, это php-ошибка

$this->db->query()
   ->rows; // <-- И она здесь

Метод query вернул совсем не объект, вот вам и ошибка.

[prostovlad]

Сергей delphinpro, я конечно понимаю, что я дурак, но тогда как вы обьясните такой момент, в базе около 15000 товаров, ошибка вываливается где то на 7000, то есть 7000 товаров обрабатываются нормально, а потом вдруг перестает нравится код ?

Метод query вернул совсем не объект, вот вам и ошибка.

это я и так понимаю )
вопрос почему возвращается не то что нужно ?
проблема в коде, в рнр или базе ?
что конкретно не нравится ?

[Сергей delphinpro]

prostovlad, надо было сразу добавить эту информацию в вопрос. Тогда бы вас не тыкали в последствия проблемы, а сосредоточились на поиске ее корня.
Для начала было бы неплохо выяснить в чем проблема с базой. Почему query() не возвращает объект с результатом, как должен. Почему он при ошибке не выбрасывает исключение с понятным описанием произошедшего.

[prostovlad]

Сергей delphinpro,

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

так в том и проблема, если бы возвращало понятное описание проблемы то и проблем бы не было (
базу проверил, везде во всех полях все прописано. дело в том что я не могу конкретно отловить на каком товаре она вываливается. если бы точно знать sku или id товара/. тогда можно было бы конкретно смотреть записи в базе по этому товару..

[Сергей delphinpro]

prostovlad, ну а как работа с базой ведется? Если это допустим модуль mysqli, то следует включить режим исключений при ошибках.

[Дмитрий]

prostovlad, а где у вас в коде какое то указание на то какой товар обрабатывается? у вас вроде как выполняется для всех записей таблицы удовлетворяющих условию. Не видно что у вас там в методе update происходит, но может как раз в один прекрасный момент у вас апдейт не находит записей для обновления и у вас вылетает эта ошибка.

[prostovlad]

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

[Дмитрий]

prostovlad, вы код метода query приведите. и не скриншотом. те кто приводят код скриншотом - будут прокляты и всю жизнь все задания тоже скриншотами получают. ну и правилами это запрещено

[prostovlad]

Дмитрий,

<?
ini_set('display_errors', 1);
ini_set('display_startup_errors', 1);
ini_set('error_reporting', E_ALL);
class ControllerExtensionModuleDubli extends Controller {
    public function index() {
    $this->db->query('UPDATE `'.DB_PREFIX.'product` AS `p`
    INNER JOIN (
        SELECT `t`.`sku`, MIN(`p`.`product_id`) AS `product_id`
            FROM (
                SELECT `sku`, MIN(`price`) AS `price`
                    FROM `'.DB_PREFIX.'product`
                    WHERE `quantity` != 0
                    GROUP BY `sku`
            ) AS `t`
            JOIN `'.DB_PREFIX.'product` AS `p`
                ON `p`.`sku` = `t`.`sku` AND `p`.`price` = `t`.`price`
            WHERE `p`.`quantity` != 0
            GROUP BY `t`.`sku`
    ) AS `i` ON `i`.`product_id` = `p`.`product_id`
    SET `p`.`status` = (`i`.`product_id` IS NOT NULL)')->rows;
    echo "ok";
 }
}

[Дмитрий]

prostovlad, вы привели код метода index. Я просил привести код метода db->query - что бы увидеть почему он возвращает null.

[prostovlad]

Дмитрий, если вы считаете что эт поможет, то нет проблем

namespace DB;
final class MySQLi {
	private $connection;

	public function __construct($hostname, $username, $password, $database, $port = '3306') {
		$this->connection = new \mysqli($hostname, $username, $password, $database, $port);

		if ($this->connection->connect_error) {
			throw new \Exception('Error: ' . $this->connection->error . '<br />Error No: ' . $this->connection->errno);
		}

		$this->connection->set_charset("utf8");
		$this->connection->query("SET SQL_MODE = ''");
	}

	public function query($sql) {
		$query = $this->connection->query($sql);

		if (!$this->connection->errno) {
			if ($query instanceof \mysqli_result) {
				$data = array();

				while ($row = $query->fetch_assoc()) {
					$data[] = $row;
				}

				$result = new \stdClass();
				$result->num_rows = $query->num_rows;
				$result->row = isset($data[0]) ? $data[0] : array();
				$result->rows = $data;

				$query->close();

				return $result;
			} else {
				return true;
			}
		} else {
			throw new \Exception('Error: ' . $this->connection->error  . '<br />Error No: ' . $this->connection->errno . '<br />' . $sql);
		}
	}

[Дмитрий]

prostovlad, помогло.

у вас срабатывает вот эта часть

} else {
        return true;
}

если к типу boolean обратиться как к обьекту - то есть то что делаете вы true->rows - то вам php выдаст ровно ту ошибку что вы видите.

[prostovlad]

Дмитрий, хорошо, но это не обьясняет почему возникает эта ошибка ) ведь она возникает не сразу, а примерно на половине обработанной базы. почему половину обрабатывает нормально, а потом вдруг ей перестает нравится ?

[Дмитрий]

prostovlad, ну возьмите и проверьте.

public function index() {
$query = 'UPDATE `'.DB_PREFIX.'product` AS `p`
    INNER JOIN (
        SELECT `t`.`sku`, MIN(`p`.`product_id`) AS `product_id`
            FROM (
                SELECT `sku`, MIN(`price`) AS `price`
                    FROM `'.DB_PREFIX.'product`
                    WHERE `quantity` != 0
                    GROUP BY `sku`
            ) AS `t`
            JOIN `'.DB_PREFIX.'product` AS `p`
                ON `p`.`sku` = `t`.`sku` AND `p`.`price` = `t`.`price`
            WHERE `p`.`quantity` != 0
            GROUP BY `t`.`sku`
    ) AS `i` ON `i`.`product_id` = `p`.`product_id`
    SET `p`.`status` = (`i`.`product_id` IS NOT NULL)';
    $response = $this->db->query($query)->rows;
    if(is_object($response)){
          echo 'Все ок '.$query.' вернул обьект '.PHP_EOL;
    }else{
         echo 'Все пропало! '.$query.' не вернул обьект! '.PHP_EOL;
    }
    echo "ok";
 }

[prostovlad]

попробовал вставить этот запрос через phpmyadmin просто как sql запрос, все отработало без ошибок (((
честно говоря не понимаю почему не хочет работать как рнр запрос, пробовал менять версии рнр, от 5.6 до 7.4 разницы нет.

[Дмитрий]

prostovlad, потому что сначала надо разобраться - потом действовать. Попытки решать методом научного тыка - меняя версии, чего там еще - редко дают результат, а если и дают то временный. Вариант как пройти дальше в отладке - я вам предложил.
Метод connection->query я не вижу, но сильное подозрение что там стоит какой нибудь mysql_query - а он при успешном update обьект не вернет - только bool

[ThunderCat]

хорошо, но это не обьясняет почему возникает эта ошибка ) ведь она возникает не сразу, а примерно на половине обработанной базы. почему половину обрабатывает нормально, а потом вдруг ей перестает нравится ?

Абсолютно бесполезно гадать почему, надо взять и посмотреть. Для начала добавить в ваш замечательный класс MySQLi команду для драйвера, позволяющую ему отдавать ошибки запросов:

mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);

После этого уже можно что-то смотреть, а не тыкать пальцем в небо...

[Дмитрий]

ThunderCat, да там нет ошибки в бд. "Notice: Trying to get property of non-object in" - это PHP ругается

[ThunderCat]

Дмитрий,

да там нет ошибки в бд.

Сильно сомневаюсь. То что его нет в выводе не значит что его нет вообще. 99% ошибка в данных и драйвер тупо молчит, так как ему не давали слова.

[Дмитрий]

ThunderCat, может быть.

[prostovlad]

всем спасибо, вопрос закрыт, ->rows не нужен.
что то ступил )))

Answer 2

[Константин Цветков]

SET `p`.`status` = NOT ISNULL(`i`.`product_id`)

Comments

[prostovlad]

насколько я знаю, такого NOT ISNULL вообще то не, но попробовал на всякий случай, не работает.