Разделение кода многофайлового загрузчика CodeIgniter DropZone на MVC?

Ваша функция createThumbnail — хороший пример того, что должно быть либо в модели, либо в библиотеке, либо в помощнике. Помощник просто содержит функции, тогда как библиотека или модель основаны на классах и, следовательно, могут обращаться к свойствам класса и иметь конструкторы. Я бы склонялся к тому, чтобы сделать его либо библиотекой, либо помощником, поскольку он не связан (таким образом, повторно используемым) с чем-либо конкретным или другими функциями.

Как правило, вы хотите возвращать или генерировать исключения только в любой функции, которая не находится в контроллере.

1. Это позволяет вам установить выходные данные в один файл, а не искать в ваших моделях, библиотеках и т. д. если вы хотите изменить свой вывод в будущем (например, если вы планируете использовать AJAX).
2. Также позволяет использовать кодовые базы, которые не являются тесно связанными и могут быть легко перенесены на другие установки, которые могут отличаться.
3. Хорошо для удобочитаемости.

Модели

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

Контроллеры

Ваш контроллер используется только для загрузки моделей и любых переменных, которые могут потребоваться для просмотра. Его также следует использовать для установки ответа, будь то флэш-сообщение или закодированный json. Во многих примерах он также используется для form_validation или проверки данных перед выполнением операций CRUD, которые существуют в связанной модели.

tl;dr все, что связано с базой данных, передается в модель, все, что можно повторно использовать и не тесно связано, отправляется в библиотеку, или, если это не имеет конкретного отношения к oop, передается в помощник (например, алгоритм сортировки массива), все, что угодно, из формы/ввода /view/response related поступает в контроллер.

Примерно вы можете сделать что-то вроде этого (код может не работать из коробки; я использую свою собственную реализацию, которая отличается от поведения dz по умолчанию, поскольку я также использую jquerys sortable... Я также не уверен в обработке ошибок, как я что с использованием другой техники):

// byte_helper.php

/**
 * Gets max upload size from post_max_size and upload_max_filesize
 * Returns whichever is smaller
 *
 * @return int
 */
function file_upload_max_size() {
    $max_size = convert_to_bytes(ini_get('post_max_size'));
    $upload_max = convert_to_bytes(ini_get('upload_max_filesize'));
    if ($upload_max > 0 && $upload_max < $max_size) {
        $max_size = $upload_max;
    }
    return $max_size;
}

/**
 * Converts KB (K) through GB (G) to bytes
 * 
 * @param string $from
 * @return int bytes
 */
function convert_to_bytes($from) {
    $number = filter_var($from, FILTER_SANITIZE_NUMBER_INT);
    $type = strtoupper(str_replace($number, '', $from));
    switch ($type) {
        case "KB":
        case "K":
            $number = $number * 1024;
            break;
        case "MB":
        case "M":
            $number = $number * pow(1024, 2);
            break;
        case "GB":
        case "G":
            $number = $number * pow(1024, 3);
            break;
    }
    return fix_integer_overflow($number);
}

/**
 * Fixes integer overflow
 * 
 * @param int $size
 * @return int
 */
function fix_integer_overflow($size) {
    if ($size < 0) {
        $size += 2.0 * (PHP_INT_MAX + 1);
    }
    return $size;
}

class Dropzone_controller extends CI_Controller {

    public function upload() {
        if (!$this->input->is_ajax_request()) {
            exit('No direct script access allowed');
        }
        $this->load->helper('byte');
        $config['upload_path'] = ''; // upload directory realpath
        $config['file_name'] = md5(time()); // I like to set my own filename
        $config['overwrite'] = true;
        $config['allowed_types'] = 'jpg|jpeg|png|gif';
        $config['max_size'] = file_upload_max_size();
        $config['file_ext_tolower'] = true;
        $this->load->library('upload', $config);
        if (!$this->upload->do_upload('file')) {
            $this->dz_error($this->upload->display_errors('', '<br>'));
        }
        $data = $this->upload->data();
        $full_path = realpath($data['full_path']); // CI uses weird slashes
        // here you can use imagelib to resize if you want, I personally
        // like to use verot's class upload for that
        $this->load->model('dropzone_model');
        if (!$this->dropzone_model->add_image($full_path)) {
            @unlink($full_path); // remove orphaned image
            $this->dz_error('Failed to add image to the database!');
        }
        $this->dz_success();
    }

    // https://stackoverflow.com/questions/27030652/how-to-display-error-message-of-jquery-dropzone
    private function dz_error($error) {
        $this->output
                ->set_header("HTTP/1.0 400 Bad Request")
                ->set_output($error)
                ->_display();
        exit;
    }

    private function dz_success() {
        // don't really need to do anything here?
    }

}

class Dropzone_model extends CI_Model {

    public function add_image($full_path) {
        return $this->db->insert('sometable', array('image' => $full_path));
    }

}