Где костыль ?

[zomo]

Доброго времени суток, вопрос  как лучше поступить... Есть массив items которые относятся к определенному товару... Когда мне нужно получить одно значение из выборки я делаю так

				foreach ($this->model_catalog_product->getProductItems($this->request->post['product_id']) as $item) {
					if ($item['item_id'] == $this->request->post['item_id']) {

			            $json = array(
						    'item_id'            => $item['item_id'],
				            'item_name'          => $item['name'],
				            'price'                => $this->currency->format($item['price'], $this->session->data['currency']),
				            'href'                 => $this->url->link('product/item',  '&item_id=' . $item['item_id']),							
		   	            );
				    }
				}

Вот начал сомневаться не костыль ли это? Или лучше все же написать отдельную модель с указанием двух параметров ( product_id и item_id ) для вывода конкретного item...

[Otvet]

во первых getProductItems в переменную $items

во вторых $this->request->post['product_id'] привести к int

в третьих если $items['product_id'] существует то уже делать что нужно

 

а теперь всё что выше вычеркиваем (кроме 2го) и делаем нормально создав метод get_item_by_product_Id

[AlexDW]

1 час назад, zomo сказал:

лучше все же написать отдельную модель

 

[zomo]

19 часов назад, AlexDW сказал:

 

 

Хорошо тогда другой вопрос, мне нужно получить items которые относятся к product, у меня два варианта

1. Писать свою функцию в модели PRODUCT и проходится по таблице item, product и item_to_product ;

2.Писать функцию в модели ITEMS и использовать $this->getItems($result['product_id']) для вывода

Как правильнее?

Грубо говоря своя функция в модели которая все собирает воедино, либо же использовать стандартные методы $this->getProduct и в контроллере форичами соединять все...

[Yesvik]

Однако ты мастер излагать мысли...