PHP. Несколько вариантов в switch case. Какие есть решения?

Question

[Дмитрий Коршунов]

Пример стандартного кода:

$format = 'png';

SWITCH ($format):
   case 'txt': // условие 1
      break;
   case 'jpg': case 'png': case 'gif':  // условие 2
      break;
   default: // условие по умолчанию
      break;
ENDSWITCH;

Как вы избавляетесь от такой записи: case 'jpg': case 'png': case 'gif':
Раздражает отсутствие вариантов записи case 'jpg', 'png', 'gif': или case ['jpg', 'png', 'gif']:

Comments

[FanatPHP]

Что-то бессмысленные вопросы пошли косяком.

Answer 1

[Сергей Соколов]

switch ... case только так записывается.
Можно сделать что-то типа этого:

$format = 'png';

$ext = array(
    'text' => array(
        'txt' => TRUE,
    ),
    'pics' => array(
        'jpg' => TRUE,
        'png' => TRUE,
        'gif' => TRUE,
    ),
    'music' => array(
        'mp3' => TRUE,
        'aiff' => TRUE,
        'wav' => TRUE,
    ),
);

if( isset( $ext['text'][$format])) // сделать что-то для текста
else if( isset( $ext['pics'][$format])) // что-то для картинок
else if( isset( $ext['music'][$format])) // что-то для музыки
else // что-то по умолчанию
;

К тому же, перформанс у if...else, говорят, лучше, чем у switch.

Comments

[Дмитрий Коршунов]

Решение нашёл по указанной ссылке:

$format = 'png';

SWITCH ($format):
   case 'txt': // условие 1
      break;
   case 'jpg' || 'png' || 'gif':  // условие 2
      break;
   default: // условие по умолчанию
      break;
ENDSWITCH;

Вот:
case 'jpg' || 'png' || 'gif':
---
Спасибо :)

[Сергей Соколов]

@dkorshunov Нет, это не решение, это ошибка! 'azaza' || 'ololo' равен true, и этот вариант будет выбираться всегда, кроме пустой строки. Там же в комментариях предупреждают об этом.

[Дмитрий Коршунов]

@sergiks
Извиняюсь, что вчера не написал... понял , что это ошибка после запуска сайта))) отказало всё, что можно :)

[Сергей Соколов]

@dkorshunov отредактируйте свой вопрос, пожалуйста, чтобы там не светился больше этот ошибочный код, а то будут ещё жертвы поспешной невнимательности : )

[Дмитрий Коршунов]

@sergiks Готово :)

[Сергей Соколов]

@dkorshunov Спасибо! : )

Answer 2

[GHua]

abstract class AbsctactFileProcessor
{
    abstract public function supports($extension);
    abstract public function process($file);
}

class ProcessingProvider
{
    
    private $processors;

    public function __construct()
    {
        $this->processors = array();
    }

    public function addProcessor(AbsctactFileProcessor $processor)
    {
        $this->processors[] = $processor;

        return $this;
    }

    public function process($file)
    {
        $file = new \SplFileInfo($file);
        $extension = $file->getExtension();
        foreach ($this->processors as $processor) {
            if ($processor->supports($extension)) {

                return $processor->process($file);
            }
        }

        throw new \LogicException();
    }

}

class SimpleTextProcessor extends AbsctactFileProcessor
{

    public function process($file)
    {
        // return ...
    }

    public function supports($extension)
    {
        return in_array($extension, array('txt'));
    }

}

class ImageProcessor extends AbsctactFileProcessor
{

    public function process($file)
    {
        // return ...
    }

    public function supports($extension)
    {
        return in_array($extension, array('png', 'gif', 'bmp', 'jpg'));
    }
}

$provider = new ProcessingProvider();
$provider->addProcessor(new \SimpleTextProcessor());
$provider->addProcessor(new \ImageProcessor());

$provider->process('mybestfile.exe');

Comments

[Юрий Чернышев]

не Вы автор библиотеки для сложения чисел на php? :) https://github.com/Herzult/SimplePHPEasyPlus

[Сергей Протько]

шутка не шутка, но это по провославному. Этот метод очень легко читаем и расширяем.

Answer 3

[Назар Мокринский]

Никак, switch имеет именно такой формат записи

Comments

[Дмитрий Коршунов]

Жаль :(
Надоела эта громоздкая запись...

[Сергей Протько]

Вообще для данной задачи свитчи не нужны.

[Дмитрий Коршунов]

@Fesor
Это просто как вариант, пример то есть...

Answer 4

[nikel303]

Как вариант:

switch(true) {

   case ($format == 'text'):
      break;

   case (in_array($format, array('jpg', 'png', 'gif'))):
      break;

   default:
      break;
}

Но, этот проще воспринимается:

case 'text':
   break;

case 'jpg':
case 'png':
case 'gif':
   break;

default:
   break;

Comments

[Павел]

case 'jpg':
case 'png':
case 'gif':

вопрос - часом последующий

case 'png':

не выполняется как вложенное условие

case 'jpg':

и т.д.?

Answer 5

[Sild]

if-ом...

Comments

[Дмитрий Коршунов]

Это понятно, интересует switch