Почему не работают исключения php?

Question

[lookingfor2]

в классе есть 4 метода
на проверку мима
на проверку размера
запись ошибок
загрузка файла на сервер

Я иду сверху вниз

Проверка мима и добавление исключений



вот допустимые типы файлов


вот результат

Comments

[Дмитрий]

Скрины маленькие, ни чего не видно, код выкладывайте в теги </> через кнопку в панеле

[Максим Федоров]

Функция checkMime() ожидает парметр, а вы не передаете туда параметр

Что и написано в результате и даже шторм подсветил

Error не отлавливаются через \Exception, они отлавливаются через более общий интерфейс \Throwable или именно явно через catch(\Error $e)

[lookingfor2]

Максим Федоров, я пытался передать туда параметр
пишет переменная не найдена

<?php
include 'components/fileComponent.php';

if (!empty($_FILES['Files'])) {
    $fileComponent = new fileComponent($_FILES['Files']); //Инициализируем класс
    try {
        $fileComponent->checkMime($mime);
        $fileComponent->startDownload();

    } catch (Exception $e) {
        $fileComponent->addError('Произошел RuntimeExeption, атеншон');
    } catch (Exception $e) {
        die($e->getMessage());
    }


    $errors = $fileComponent->isError();
    if ($errors) {
        foreach ($errors as $error) {
            echo "<div style='color: red'>Сообщение ошибки: {$error['msg']} [{$error['obj']}]</div>";
        }
    } else {
        echo 'Нет ошибок';
    }

}

<?php
include 'components/helpers.php';
require '../checlOnEmptyFiles.php';

class fileComponent
{
    public $files;
    public $dir;
    public $allowMime;
    public $normalSize;
    public $errors;
    public $helper;

    public function __construct($files, $abortDownload = false, $folder = 'files')
    {
        $this->files = $files;
        $this->dir = $_SERVER['DOCUMENT_ROOT'] . '/' . $folder . '/';

        $this->allowMime = [
            'application/vnd.ms-excel', 'image/jpeg', 'image/png'
        ];
        $this->normalSize = 12000;
        $this->helper = new Helpers();

    }



    public function checkMime($mime)
    {
        if (in_array($mime, $this->allowMime, false)) {
            throw new Exception("Тип данного файла не поддерживается:" . $mime);
        }

    }

    public function addError($msg, $obj = null)
    {
        $this->errors[] = [
            'msg' => $msg,
            'obj' => $obj
        ];
    }

    public function checkSize($size)
    {
        if (($this->files['size']) === ($this->normalSize)) {
            return true;
        }
        $this->addError('Размер данного файла большой', $this->helper->formatSize($size));
        return false;
    }

    public function isError()
    {
        if (!count($this->errors)) {
            return false;
        }

        return $this->errors;
    }


    public function startDownload()
    {
        foreach ($this->files['tmp_name'] as $key => $file) {

            if (!$this->checkMime($this->files['type'][$key])) {
                continue;
            }

            if (!$this->checkSize($this->files['size'][$key])) {
                continue;
            }

            move_uploaded_file($this->files['tmp_name'][$key], $this->dir . $this->files['name'][$key]);

        }
    }


}

[lookingfor2]

Дмитрий, спасибо за подсказку
вот код

<?php
include 'components/fileComponent.php';

if (!empty($_FILES['Files'])) {
    $fileComponent = new fileComponent($_FILES['Files']); //Инициализируем класс
    try {
        $fileComponent->checkMime($mime);
        $fileComponent->startDownload();

    } catch (Exception $e) {
        $fileComponent->addError('Произошел RuntimeExeption, атеншон');
    } catch (Exception $e) {
        die($e->getMessage());
    }


    $errors = $fileComponent->isError();
    if ($errors) {
        foreach ($errors as $error) {
            echo "<div style='color: red'>Сообщение ошибки: {$error['msg']} [{$error['obj']}]</div>";
        }
    } else {
        echo 'Нет ошибок';
    }

}

<?php
include 'components/helpers.php';
require '../checlOnEmptyFiles.php';

class fileComponent
{
    public $files;
    public $dir;
    public $allowMime;
    public $normalSize;
    public $errors;
    public $helper;

    public function __construct($files, $abortDownload = false, $folder = 'files')
    {
        $this->files = $files;
        $this->dir = $_SERVER['DOCUMENT_ROOT'] . '/' . $folder . '/';

        $this->allowMime = [
            'application/vnd.ms-excel', 'image/jpeg', 'image/png'
        ];
        $this->normalSize = 12000;
        $this->helper = new Helpers();

    }



    public function checkMime($mime)
    {
        if (in_array($mime, $this->allowMime, false)) {
            throw new Exception("Тип данного файла не поддерживается:" . $mime);
        }

    }

    public function addError($msg, $obj = null)
    {
        $this->errors[] = [
            'msg' => $msg,
            'obj' => $obj
        ];
    }

    public function checkSize($size)
    {
        if (($this->files['size']) === ($this->normalSize)) {
            return true;
        }
        $this->addError('Размер данного файла большой', $this->helper->formatSize($size));
        return false;
    }

    public function isError()
    {
        if (!count($this->errors)) {
            return false;
        }

        return $this->errors;
    }


    public function startDownload()
    {
        foreach ($this->files['tmp_name'] as $key => $file) {

            if (!$this->checkMime($this->files['type'][$key])) {
                continue;
            }

            if (!$this->checkSize($this->files['size'][$key])) {
                continue;
            }

            move_uploaded_file($this->files['tmp_name'][$key], $this->dir . $this->files['name'][$key]);

        }
    }


}

[Дмитрий]

lookingfor2,

public function checkMime($mime)
    {
        if (in_array($mime, $this->allowMime, false)) {
            throw new Exception("Тип данного файла не поддерживается:" . $mime);
        }

    }

Если расширение файла совпадает с текущим, то кидаем исключение
Наверное там надо !in_array()
Это как добавка к указанным ошибкам

[lookingfor2]

Дмитрий, да спасибо )

Answer 1

[coderisimo]

Выглядит странно ))
1) Конкретно в примере что на скринах - в метод не передается переменная, которая там нужна.
2) Если хотите перехватывать все исключения, используйте
catch(Throwable $e)

3) в комментариях вы привели код :

if (!empty($_FILES['Files'])) {
    $fileComponent = new fileComponent($_FILES['Files']); //Инициализируем класс
    try {
     $fileComponent->checkMime($mime); //$mime - атеншон!это откуда?незаметно подбросили полицаи?
..................

$mime ОТКУДА в нем ?

Answer 2

[xmoonlight]

......//Инициализируем класс
try {
......checkMime(!!!ТУТ!!!);