Как сделать один динамичный метод на php?

Question

[Dom Alfro]

Здравствуйте ! Хотел бы узнать как сделать такой метод который бы принимал имя таблици в переменную - что бы не делать постоянно каждый метод отдельно !
1 метод для выборки из таблици slider

public function getStuff() {
    	$result = $this->db->row('SELECT * FROM `slider`');
    	return $result;
    }

а вот второй

public function getMainDb() {
        $result = $this->db->row('SELECT * FROM `$tables`');
        return $result;
    }

из второго примера можно увидеть что я уже пытался писать переменную $tables а дальше не знаю как
Спасибо !

Comments

[Ninazu]

У методов есть параметры)))) Либо использовать __get для задания имени таблицы

[FanatPHP]

Два вопроса.
1. Какой смысл делать запрос вида SELECT * FROM `table`? Как часто требуется выбирать ВСЕ строки из таблицы?
2. Метод $this->db->row() возвращает одну строку или все?

[Dom Alfro]

FanatPHP, спасибо за участие ! видите ли у меня интернет магазин - и там у меня есть список лучших товаров - отдельный список магазинов и отдельный список лучших предложений - и если мне писать каждый метод с каждым именем таблици - мне сказали что я нарушаю правило" не повторйся"! и думал я как же мне быть что бы в переменную таблица записывалась динамично и что бы не было массу методов как то так.

[Dom Alfro]

Ninazu, и что же мне делать, их передавать ? return ? или как ?

[Dom Alfro]

FanatPHP, тут на запрос внимание не стоит обращать это утрированый запрос тут цель в том что бы передавать имя таблици динамично ! а как понять одну строку или не одну - return что то одно возвращает ( ну сорри что я такой нуб )

[FanatPHP]

обновил ответ

[Dom Alfro]

FanatPHP, а разве нужны подготовленные запросы при выводе из бд данных ? я видел что они делаются при добовление изменения удаления данных в бд ?

[FanatPHP]

Вячеслав Коростин, да разумеется.
Подготовленные запросы нужны для любых запросов.
Тут дело не в том, какой запрос, а в том, используются ли в нем переменные.
Если через переменную передаются данные, то их надо передавать через подстановки.
А если через переменну. передается имя колонки или таблицы, то так как у меня в ответе - через ручную фильтрацию

[Dom Alfro]

меня долго мучал этот вопрос ! Спасибо за ответы и за участие !!!

Answer 1

[FanatPHP]

Для вопроса из заголовка

abstract class BaseORM {
    protected $table;
    protected $db;
    public function __construct($db) {
        $this->db = $db;
    }
    public function getById($id) {
        $stmt = $this->db->prepare("SELECT * FROM `$this->table`" WHERE id=?);
        $stmt->execute([$id]);
        return $stmt->fetch();
    }
    public function getAll() {
        return $this->db->query("SELECT * FROM `$this->table`");
    }
}

class SliderDB extends BaseORM {
    protected $table = 'slider';
}

$slider = new SliderDB($db);
$allSliders = $slider->getAll();

Для вопроса из комментов

у меня интернет магазин - и там у меня есть список лучших товаров - отдельный список магазинов и отдельный список лучших предложений - и если мне писать каждый метод с каждым именем таблици - мне сказали что я нарушаю правило" не повторйся"! и думал я как же мне быть что бы в переменную таблица записывалась динамично и что бы не было массу методов как то так.

function getTableData($table) {
    $list = ['best_offers', 'shops', 'best_products'];
    if (!in_array($table, $list)) {
        throw new InvalidArgumentException('Table name not found');
    }
    return $this->db->row('SELECT * FROM `$table`');
}

$data = getTableData('shops');

Comments

[Newn]

можно не мучиться с подготовленными запросами, и использовать готовую библиотеку RedBean

[FanatPHP]

Newn, можно использовать готовую библиотеку, какую угодно только не это адово поделие из прошлого века.

Answer 2

[Александр]

Используйте магический метод __call() для написания таких методов

Answer 3

[Newn]

public function getMainDb($tables) {
        $result = $this->db->row('SELECT * FROM '.$tables);
        return $result;
    }

public function getMainDb($tables) {
        $result = $this->db->row("SELECT * FROM {$tables} ");
        return $result;
    }

public function getMainDb() {
        global $tables
        $result = $this->db->row("SELECT * FROM {$tables} ");
        return $result;
    }

Не?

Comments

[FanatPHP]

не

[Newn]

конечно, обижаешь

[FanatPHP]

Вопрос снимается. У тебя тут такой ад, что уже не до инъекций.

[Newn]

FanatPHP, В чем ад?)

[FanatPHP]

Например в использовании global

[Dom Alfro]

так я делал ... error undefined variable