Стоить ли так делать? Или все таки нет?

Question

[adrenalinruslan]

Я сделал такой код, стоит ли его оставлять ? Я сделал этот код, чтобы было легче и можно было достать любую запись из любой таблицы Базы Данных. Или это быдло код ?

public $dbTables 	= [
		'settings' => [
			'id', 'rucaptcha',
		],
	];

/*
	**********************************************************************************************
	**********************************************************************************************
	*/

	public function searchTableToArray($table) 
	{
		if (array_key_exists($table, $this->dbTables)) 
		{
			return true;
		}
		return false;
	}

	public function searchColumnToArray($column, $table) 
	{
		if (in_array($column, $this->dbTables[$table])) 
		{
			return true;
		}
		return false;
	}

	/*
	**********************************************************************************************
	**********************************************************************************************
	*/

public function dbGetRecordToTableByColumn($table, $column, $value) 
	{
		if (!empty($table) && !empty($column) && !empty($value)) 
		{
			if ($this->searchTableToArray($table)) 
			{
				if ($this->searchColumnToArray($column, $table)) 
				{
					$result = $this->db->row("SELECT * FROM `{$table}` WHERE `{$column}` = :{$column} LIMIT 1", [ 
						$column => $value 
					]);
					if (!empty($result)) 
					{
						return $result[0];
					}
				}
			}
		}
		return [];
	}

	public function dbGetRecordsToTableByColumn($table, $column, $value) 
	{
		if (!empty($table) && !empty($column) && !empty($value)) 
		{
			if ($this->searchTableToArray($table)) 
			{
				if ($this->searchColumnToArray($column, $table)) 
				{
					return $this->db->row("SELECT * FROM `{$table}` WHERE `{$column}` = :{$column} ORDER BY `id` DESC", [ 
						$column => $value
					]);
				}
			}
		}
		return [];
	}

Comments

[Дархан]

sql injection?

[adrenalinruslan]

Дархан Камалиев, ну вроде не должно быть иньекции, я так понимаю PDO защищает, но не так хорошо уж

public function query($sql, $params = []) {
		$stmt = $this->db->prepare($sql);
		if (!empty($params)) {
			foreach ($params as $key => $val) {
				if (is_int($val)) {
					$type = PDO::PARAM_INT;
				} else {
					$type = PDO::PARAM_STR;
				}
				$stmt->bindValue(':'.$key, $val, $type);
			}
		}
		$stmt->execute();
		return $stmt;
	}

	public function row($sql, $params = []) {
		$result = $this->query($sql, $params);
		return $result->fetchAll(PDO::FETCH_ASSOC);
	}

[Алексей Уколов]

В чём конкретно ваш вопрос?

[adrenalinruslan]

Алексей Уколов, ну тип это быдло код? Все эти проверки на существование таблицы и столбцов лишнее ? Стоит ли это оставлять ?

Answer 1

[ThunderCat]

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

public function dbGetRecordsToTableByColumn($table, $column, $value) 
  {
    if (empty($table)  || empty($column) || empty($value)) return [];
    if (!$this->searchTableToArray($table)) return [];
    if (!$this->searchColumnToArray($column, $table)) return [];
    return $this->db->row("SELECT * FROM `{$table}` WHERE `{$column}` = :{$column} ORDER BY `id` DESC", [ 
       $column => $value
     ]);
  }

Comments

[adrenalinruslan]

Хорошо, я понял

[neol]

adrenalinruslan, Вопрос: если метод dbGetRecordsToTableByColumn вернул пустой массив, то:
A) нет подходящих строк
B) опечатка в названии таблицы/колонки
C) $value === NULL, 0 или ''
D) Для ответа на этот вопрос нужен дебагер, хрустальный шар, литр водки и ириска

[ThunderCat]

neol, тут уже пусть сам разбирается, может ему "не нужно чтобы стояло, а надо чтобы блестело" (с)

Answer 2

[morricone85]

не быдлокод, но использовать не стоит

"SELECT * FROM `{$table}` WHERE `{$column}` = :{$column} ORDER BY `id` DESC",

это слишком простые запросы, на практике они будут редко использоваться;
они, не гибкие: что делать, если нужно несколько таблиц соединить, группировка данных, сортировать по разным полям и т.д.

Answer 3

[T4rry]

А что вот эта часть делает WHERE `{$column}` = :{$column}

Comments

[ThunderCat]

тут как раз все ясно, в итоге получишь что-то типа where `data` = :data для препаред стэтментс.

[T4rry]

ThunderCat, а! Ну что-то это не тянет на "можно было достать любую запись из любой таблицы Базы Данных"

[ThunderCat]

T4rry, в самом базовом случае оно так и будет, естественно это не посторитель запросов, а тупо узкозаточенная функция выборки по 1 параметру.