Помогите с ошибкой 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]

  В 12.08.2019 в 08:39, legioner26 сказав:

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

Expand  

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

[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]

  В 12.08.2019 в 08:44, thentru сказав:

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

Expand  

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

[legioner26]

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

[maiki]

  В 12.08.2019 в 08:44, thentru сказав:

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

Expand  

Гений!

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

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

  В 12.08.2019 в 08:39, legioner26 сказав:

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

Expand  

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

[maiki]

  В 12.08.2019 в 08:56, legioner26 сказав:

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

Expand  

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]

  В 12.08.2019 в 09:01, maiki сказав:

Гений!

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

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

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

Expand  

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

[legioner26]

  В 12.08.2019 в 09:02, 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;
}

 

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

Expand  

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

[thentru]

  В 12.08.2019 в 08:56, legioner26 сказав:

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

Expand  

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

[legioner26]

  В 12.08.2019 в 09:06, thentru сказав:

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

Expand  

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

[legioner26]

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

[maiki]

  В 12.08.2019 в 09:06, thentru сказав:

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

Expand  

 

 

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

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

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

[maiki]

  В 12.08.2019 в 09:09, legioner26 сказав:

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

Expand  

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