Методы выборки из БД?

Question

[Andrey]

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

Вот сам фрагмент:

function GetHashtagInfo($id = null, $tag = null, $type = null)
  {
    if(!is_null($id) && is_numeric($id) && !is_null($tag) && !is_numeric($tag) && !is_null($type) && !is_numeric($type))
      $sql = "SELECT * FROM `hashtags` WHERE `id` = '$id' AND `tag` = '$tag' AND `type` = '$type'"; //1, 1, 1
    else if(!is_null($id) && is_numeric($id) && !is_null($tag) && !is_numeric($tag) && is_nullt($type))
      $sql = "SELECT * FROM `hashtags` WHERE `id` = '$id' AND `tag` = '$tag'"; //1, 1, 0
    else if(!is_null($id) && is_numeric($id) && is_null($tag) && is_null($type))
      $sql = "SELECT * FROM `hashtags` WHERE `id` = '$id'"; //1, 0, 0
    else if(!is_null($id) && is_numeric($id) && is_null($tag) && !is_null($type) && !is_numeric($type))
      $sql = "SELECT * FROM `hashtags` WHERE `id` = '$id' AND `type` = '$type'"; //1, 0, 1
    else if(is_null($id) && !is_null($tag) && !is_numeric($tag) && !is_null($type))
      $sql = "SELECT * FROM `hashtags` WHERE `tag` = '$tag' AND `type` = '$type'"; //0, 1, 1

    $sql_result = $this->mysql->Query($sql);

    if(($sql_row = mysqli_fetch_assoc($sql_result)) != null)
      return $sql_row;
    else
      return $sql_row = null;
  }

Входные переменные: $id = Идентификатор тэга из бд, $tag = Системный тэг для фильтрации, $type = маркер для назначения тэга к примеру стоит articles

Как вы видите очень много ветвления, что приводит к путанице, можно ли иначе реализовать данный подход?
В голове крутится switch, но он ведь тоже из той оперы.

Comments

[FeST1VaL]

А есть какое то свойство которое всегда есть?
т.е Я бы просто через if добавлял в переменную что то типо такого:

$where = "";
if($id) $where .= " AND `id` = '{$id}'";
if($tag) $where .= " AND `tag` = '{$tag}'";
...
...
$sql = "SELECT * FROM `hashtags` WHERE `show` = 1{$where}";
...

[FanatPHP]

Про SQL инъекции, как я понимаю, современные писатели блогов не слышали.

[Andrey]

FanatPHP, Слышали, что вы этим хотите сказать?

[Andrey]

Артем Будин, Думаю можно сказать, что идентификатор является обязательным свойством.

Answer 1

[profesor08]

Вообще-то достаточно знать несколько базовых функций для работы с массивами и уметь работать с данными. Либо освоить какой-то sql query builder

function BuildGetHashtagInfoQuery($id = null, $tag = null, $type = null) {
    $sql = "SELECT * FROM `hashtags`";
    $where = [];
    
    if (is_numeric($id)) {
        $where["id"] = $id;
    }
    
    if (is_string($tag)) {
        $where["tag"] = $tag;
    }
    
    if (is_string($type)) {
        $where["type"] = $type;
    }
    
    if (count($where) > 0) {
        $sql .= " WHERE ";
        
        $sql .= implode(" AND ", array_map(function($key) use($where) {
            return "`{$key}` = '{$where[$key]}'";
        }, array_keys($where)));
    }
    
    return $sql;
}

echo BuildGetHashtagInfoQuery(123, "cat", "animal");

Comments

[FanatPHP]

Вообще-то желательно иметь хотя бы теоретическое представление о такой базовой уязвимости, как SQL инъекция.

[profesor08]

FanatPHP, вообще-то это не моя забота.

[FanatPHP]

конечно, твоя забота - показать нубам какой ты крутой и как лихо управляешься с базовыми функциями. До остальных-то не дорос

[Andrey]

FanatPHP, Покажите тогда вы, как стоит поступить?

[profesor08]

FanatPHP, в чем проблема вообще? Не нравится - пиши свой вариант, с полной санитарной обработкой и полноценным билдером.

Answer 2

[Aricus]

$sql = "SELECT * FROM `hashtags` WHERE ";
if (...) {
  $sql .= "`id` = '$id' AND ";
}
if (...) {
  $sql .= "`tag` = '$tag' AND ";
}
if (...) {
  $sql .= "`type` = '$type' AND ";
}
$sql = substr($sql, 0, -5);

Comments

[profesor08]

И получаешь ошибку SQL запроса

[Aricus]

Ну так выведи $sql, посмотри, что там не так. Может, у тебя все три if оказались ложными. Я сами условия не закладывал, просто показал суть. Надеюсь эту суть ты понял? Если нет, почитай про конкатенацию строк и substr.

[profesor08]

Aricus, мне это объяснять не надо. Так как лучше представить аргументы как ассоциативный массив и красиво обработать, чем говнокодить на substr и считать пробелы.

[Aricus]

profesor08, Интересно, как ты собираешься красиво обрабатывать ассоциативный массив в рамках данной задачи? Можно сделать обычный массив и соединить через implode, а вот на ассоциативный моей фантазии не хватает.

[profesor08]

Aricus, пример выше

[Aricus]

profesor08, Выше - это в твоём примере, что ли? Так у тебя во время формирования $sql никаких ассоциативных массивов не применяется.

[profesor08]

Aricus, $where

[Aricus]

Извини, думал, пишет автор вопроса. Ну можно и так. Хотя городить такую конструкцию внутри implode ради экономии нескольких символов на условие как-то странно. И ещё более странно делать функцию для частного случая, формирующего конкретный запрос.

Answer 3

[Boris Köln]

Используйте любой "query builder".
Например, https://laravel.com/docs/master/queries или https://www.yiiframework.com/doc/guide/2.0/en/db-q...