Как обратиться к функции через $this в цикле?

Question

[Li]

Имеется класс в котором находится функция вывода всех товаров. У каждого товара в базе имеется своя стоимость но функция get_price() выполняет различные манипуляции с ценой

class Product {
# получаем товары
public function get_all() {

    $all_products = SELECT * FROM ...

    foreach($all_products as $products) {
        $this->comission = 5;
        $this->base_price = 500;
    }

    return $this;
}
}
# формируем цену
public function get_price() {
    $price = (( $this->base_price ) + ( $this->comission / 100 ));

    return $price;
}
}

в итоге синтаксис должен получиться такой

$product = new Product;

foreach($product ->get_all() as $item) {
echo $item->get_price(); // тут надо применить функцию get_price() к каждой иттерации
}

Но конструкция не верна. get_price берет значение только первого ключа массива и применяет это значение ко всему. Каким образом можно применить функцию в цикле? Спасибо за помощь

Answer 1

[Александр Аксентьев]

Чтобы это работало надо при вытаскивании из базы как-нибудь превратить массив данных в объект Product.
И собирать массив объектов.

А у вас пока какая-то каша малаша получается.

Вот это бессмысленно и беспощадно

$all_products = SELECT * FROM ...

    foreach($all_products as $products) {
        $this->comission = 5;
        $this->base_price = 500;
    }

    return $this;

Comments

[Li]

Спасибо. Можете выложить примерный алгоритм?

[Александр Аксентьев]

Alex,
этим всем ORM занимается, велосипеды строить нет смысла.

Но допустим так

$all_products = SELECT * FROM ...
$models = [];
foreach($all_products as $products) {
    $model = new Product();
    $model->comission = 5;
    $model->base_price = 500;
    $models[] = $model;
}

return $models;

[Li]

Александр Аксентьев, благодарен! У меня смежный вопрос, учитывая ваш опыт подаскажите пожалуйста, каким образом лучше обратиться к функции get_price();

В методе класса: $model->price = $this->get_price();
Или в массиве при выводе: $product->get_price();

Спасибо

[Александр Аксентьев]

Alex, тут уж кому как больше нравится.
Если цена рассчитывается по каким-то критериям/параметрам, то логично через функцию её получать, чтобы было понятно что там не просто столбец в базе.
Если она в базе лежит уже обсчитанная, то конечно выдавать сразу через свойство "сырые данные".

Answer 2

[ThunderCat]

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