Как быть с картинкой при записи в базу?

Question

[dev400]

В базу записываются строки, и путь к картинке.
скрипт загрузки картинки:

<?php
namespace Common;

class Upload {
    
    public function make($uploadDir, $file) {

        $pathToUpload = $uploadDir . basename($file['name']);

        if( move_uploaded_file($file['tmp_name'], $pathToUpload) ) {

            return $pathToUpload;

        }

        return false;
        
    }

}

Возвращает путь к файлу.

Кусок модели сохранения в базу:

$params = [
                "id" => $array['category_id'],
                "cat_name" => $array['category_name'],
                "cat_url" => $array['category_url'],
                "image_url" => $array['image_url'],
                "parent_cat" => $array['parent_cat'],
            ];

            $sql = "UPDATE {$this->table} SET `cat_name` = :cat_name, `cat_url` = :cat_url, `image_url` = :image_url, `parent` = :parent_cat WHERE `id` = :id";

Так вот, загрузили мы картинку, все хорошо. В следующий раз мы хотим поменять какую то строку, но картинку мы загружать не будем.
И соответственно получим ошибку, что image_url не передался. Как сделать так, чтобы image_url был не обязательным параметром при сохранении?

В голову приходит простыня:

public function updateCategory(Array $array) {

        if ( isset($array['image_url']) ) {

            $params = [
                "id" => $array['category_id'],
                "cat_name" => $array['category_name'],
                "cat_url" => $array['category_url'],
                "image_url" => $array['image_url'],
                "parent_cat" => $array['parent_cat'],
            ];

            $sql = "UPDATE {$this->table} SET `cat_name` = :cat_name, `cat_url` = :cat_url, `image_url` = :image_url, `parent` = :parent_cat WHERE `id` = :id";

        } else {

            $params = [
                "id" => $array['category_id'],
                "cat_name" => $array['category_name'],
                "cat_url" => $array['category_url'],
                "parent_cat" => $array['parent_cat'],
            ];

            $sql = "UPDATE {$this->table} SET `cat_name` = :cat_name, `cat_url` = :cat_url, `parent` = :parent_cat WHERE `id` = :id";

        }


        if( !parent::save( $params, $sql ) ) {

            throw new \Exception('Ошибка редактирования категории');

        }

    }

Но это же трэш. Помогите!

Answer 1

[Анатолий]

$params = [
                "id" => $array['category_id'],
                "cat_name" => $array['category_name'],
                "cat_url" => $array['category_url'],
                "image_url" => $array['image_url'],
                "parent_cat" => $array['parent_cat'],
            ];
if ($image) {
    $params["image_url"] = $image;
}

Comments

[dev400]

А с запросом как быть?

[Анатолий]

Андрей:
$sql = "UPDATE ...";
foreach ($params as $key => $param){
$sql .= " SET `{$key}` = :{$key} "
}

Answer 2

[Rsa97]

$params["image_url"] = (isset($array['image_url']) ? $array['image_url'] : NULL);

UPDATE ... `image_url` = IFNULL(:image_url, `image_url`) ...

Comments

[dev400]

да, sql у меня хромает. Есть куда развиваться. Спасибо!

Answer 3

[Максим Тимофеев]

if(isset($_FILE['name']))

Comments

[dev400]

так у меня по дефолту

if( !empty( $_FILES['cat_image']['name'] ) ) {

            $pathToUpload = $upload->make('path/to/upload', $_FILES['image']);

        }

[Максим Тимофеев]

Андрей: Ну так если нет empty $_FILES['cat_image']['name'] то и картинка не будет перезаписана, в чем проблема?

[dev400]

$upload это объект класса Common\Upload

[dev400]

Максим Тимофеев: в модели посмотрите апдейт

[Максим Тимофеев]

Андрей: я понимаю и что?

[dev400]

Максим Тимофеев: если нету $_FILES['cat_image']['name'] то ожидаемый `image_url` = :image_url даст ошибку

[Максим Тимофеев]

Максим Тимофеев: "image_url" => $array['image_url'], оттуда уберите и добавляйте если есть $_FILES['cat_image']['name']

[Максим Тимофеев]

$params = [
                "id" => $array['category_id'],
                "cat_name" => $array['category_name'],
                "cat_url" => $array['category_url'],
                "parent_cat" => $array['parent_cat'],
            ];
if( !empty( $_FILES['cat_image']['name'] ) ) {
                $params["image_url"] = $array['image_url'];
}
            $sql = "UPDATE {$this->table} SET `cat_name` = :cat_name, `cat_url` = :cat_url, `image_url` = :image_url, `parent` = :parent_cat WHERE `id` = :id";

[dev400]

Максим Тимофеев: тогда придется и из запроса тоже убрать это, тк prepared statements

[Максим Тимофеев]

Андрей: почему?

[dev400]

Максим Тимофеев: это же модель, причем тут $_FILES из контроллера. Это не ее дело что пришло и куда

[Максим Тимофеев]

Андрей: Ну так вынесете эту логику в контроллер.

[dev400]

Максим Тимофеев: ну в $params массив для подготовки к апдейту

[Максим Тимофеев]

Андрей: логика таже только if($_FILES) выносите в контроллер, а в моделе проверяете$array['image_url'] на null

[dev400]

Это даст ошибку

$params = [
                "id" => $array['category_id'],
                "cat_name" => $array['category_name'],
                "cat_url" => $array['category_url'],
                "parent_cat" => $array['parent_cat'],
            ];
if( !empty( $_FILES['cat_image']['name'] ) ) {
                $params["image_url"] = $array['image_url'];
}
            $sql = "UPDATE {$this->table} SET `cat_name` = :cat_name, `cat_url` = :cat_url, `image_url` = :image_url, `parent` = :parent_cat WHERE `id` = :id";

Если не передадим image_url. В запросе `image_url` = :image_url

[Максим Тимофеев]

Андрей: формируйте $sql перебирая $params через foreach

Answer 4

[Александр]

$params = [
    "id" => $array['category_id'],
    "cat_name" => $array['category_name'],
    "cat_url" => $array['category_url'],
    "image_url" => $array['image_url'],
    "parent_cat" => $array['parent_cat'],
];

$a = [];
foreach ($params as $k => $v) {
    $a[] = "`$k` = :$k";
}

if (count($a)) {
    $s = implode(", ", $a);
    $sql = "UPDATE {$this->table} SET $s WHERE `id` = :id";
}

И нет никаких костылей. Правда в этом случае стоит еще проверить, что каждая $k и правда есть в таблице.