Помогите с ошибкой Undefined index

[maiki]

Всем привет!

опенкарт 2.1

Помогите пж исправить проблему. 

Ругается на этот код

 

public function getPosBarcode($product_id){
$sql = "SELECT barcode FROM `" . DB_PREFIX . "pos_barcode` WHERE product_id = ".$product_id;
$query = $this->db->query($sql)->row;

------- return $query['barcode']; ----- ругается на нее
}

 

Undefined index: barcode in httpdocs/admin/model/pos/products.php on line 89
 

Змінено 12 серпня 2019 користувачем maiki

[legioner26]

Значит пуста

[legioner26]

Надо смотреть получает значение или нет в запросе

[maiki]

2 минуты назад, legioner26 сказал:

Надо смотреть получает значение или нет в запросе

Это где смотреть ?

[legioner26]

public function getPosBarcode($product_id){
$sql = "SELECT barcode FROM `" . DB_PREFIX . "pos_barcode` WHERE product_id = ".$product_id;
$query = $this->db->query($sql)->row;

echo '<pre>'; print_r($query); die('<pre>');
// return $query['barcode']; 
}

 

[thentru]

значит для этого товара нет результата выборки. Добавить проверку
return isset($query['barcode']) ? $query['barcode'] : false;

[legioner26]

Только что, thentru сказал:

значит для этого товара нет результата выборки. Добавить проверку
return isset($query['barcode']) ? $query['barcode'] : false;

ну это уже потом после проверки результата))))

[legioner26]

Надо всегда проверять метод когда пишешь его.

[maiki]

10 минут назад, thentru сказал:

значит для этого товара нет результата выборки. Добавить проверку
return isset($query['barcode']) ? $query['barcode'] : false;

Гений!

Я как раз empty читал. 

Ваш код сработал ! Громадное Вам спасибо ! 

22 минуты назад, legioner26 сказал:

Надо смотреть получает значение или нет в запросе

Спасибо за помощь!

[maiki]

5 минут назад, legioner26 сказал:

ну это уже потом после проверки результата))))

public function getPosBarcode($product_id){
$sql = "SELECT barcode FROM `" . DB_PREFIX . "pos_barcode` WHERE product_id = ".$product_id;
$query = $this->db->query($sql)->row;

return isset($query['barcode']) ? $query['barcode'] : false;
}

 

Так правильно ?

[legioner26]

Только что, maiki сказал:

Гений!

Я как раз empty читал. 

Ваш код сработал ! Громадное Вам спасибо ! 

Спасибо за помощь!

empty это проверка на пустое значение, а isset это проверка на существование

[legioner26]

Только что, maiki сказал:

public function getPosBarcode($product_id){
$sql = "SELECT barcode FROM `" . DB_PREFIX . "pos_barcode` WHERE product_id = ".$product_id;
$query = $this->db->query($sql)->row;

return isset($query['barcode']) ? $query['barcode'] : false;
}

 

Так правильно ?

Я не про это ))). Я клоню к тому что проверять результат через print_r(); для теста метода.

[thentru]

8 минут назад, legioner26 сказал:

ну это уже потом после проверки результата))))

а что там проверять? ошибки, что такой колонки в таблице не существует - нет
значит для этого товара записи просто нет

[legioner26]

Только что, thentru сказал:

а что там проверять? ошибки, что такой колонки в таблице не существует - нет
значит для этого товара записи просто нет

Да я вкурсе. Я так понимаю человек написал метод. Но обычно после правок какого либо метода, надо проверить его работоспособность. 

[legioner26]

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

[maiki]

3 минуты назад, thentru сказал:

а что там проверять? ошибки, что такой колонки в таблице не существует - нет
значит для этого товара записи просто нет

 

 

Там да генерация идет не автоматом а в ручную.

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

Ваш код отлично сработал, 

[maiki]

2 минуты назад, legioner26 сказал:

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

Спасибо!  Учту обязательно!