Как можно сжать данный код, вызов функции?

@poss · 05.03.2016, 13:39. **Показов** 1270. **Ответов** 13

Студворк — интернет-сервис помощи студентам

PHP
1
2
3
4
5
6
7
 if ( isset($get['q']) && $get['q'] == 2) { 
                 $data["result"] = $this->functions->question2();
            } else if ( isset($get['q']) && $get['q'] == 3) { 
                 $data["result"] = $this->functions->question3();
            } else if ( isset($get['q']) && $get['q'] == 4) { 
                 $data["result"] = $this->functions->question4();
            }

Можно ли вызов функции записать так:

PHP
1
question$$get[;q'();

или так:

PHP
1
question.$get['q']();]

@Jewbacabra · 05.03.2016, 13:42

Сообщение от poss

Можно ли вызов функции записать так:

почему бы не попробовать выполнить данный код?

@WhiteMind · 05.03.2016, 13:42

PHP
1
2
3
4
$funTable = [2 => 'question2' , 3 => 'question3' , 4 => 'question4'];
if ( @($funName = $funTable[ $get['q'] ] ) ) {
    $data["result"] = $this->functions->{$funName}();
}

@Jewbacabra · 05.03.2016, 13:44

PHP
1
2
3
if (!empty($get['q'])) {
    $data['result'] = $this->functions->{"question$get[q]"}();
}

@WhiteMind · 05.03.2016, 13:46

PHP
1
2
3
if ( @( $i=$get['q'] ) && ($i >= 2 && $i<=4) ) {
    $data['result'] = $this->functions->{'question'.$i}();
}

@Gcom · 05.03.2016, 14:05

poss, Я бы сделал так:

PHP
1
2
3
if (isset($get['q'])) {
    $data["result"] = (2 <= $get['q'] && $get['q'] <= 4) ? $this->functions->{'question'.$get['q']}() : '' ;
};

@Para bellum · 05.03.2016, 14:37

Что-то я сомневаюсь в правильности подхода. Что вообще делают те методы и почему их именно так именуете?
В любом случае, такие проверки лучше не делать:

PHP
1
(2 <= $get['q'] && $get['q'] <= 4)

А что, если методов поприбавится?

PHP
1
2
3
4
5
6
7
8
9
10
if (!empty($get['q'])) {
    # Формируем имя метода
    $method = 'question' . $get['q'];
    
    # Если метод не существует
    if(!method_exists($this->functions, $method))
        throw new Exception("Метод {$method} не найден");
    
    $data['result'] = $this->functions->$method();
}

@Gcom · 05.03.2016, 15:02

Не по теме:

Сообщение от Para bellum

А что, если методов поприбавится?

Я всегда пытаюсь мыслить глобально, но нехватка знаний иногда мешает этому, а так отличное решение :good:

Добавлено через 5 минут
Para bellum, Но опять же, в нем не учитывается проверка:
$get['q'] == 2;
$get['q'] == 3;
$get['q'] == 4;
в частности может надо что то другое сделать когда $get['q'] выходит за рамки интервала, конечно это мало вероятно, но все же имеет быть место.

@Para bellum · 05.03.2016, 15:12

Сообщение от Gcom

Para bellum, Но опять же, в нем не учитывается проверка

Да, Вы правы. Только это

Сообщение от Gcom

в частности может надо что то другое сделать когда $get['q'] выходит за рамки интервала, конечно это мало вероятно, но все же имеет быть место.

действительно маловероятно, а вот если в объекте "$this->functions" содержатся ещё методы, к которым доступ через указание в "$get['q']" не предполагается — мой вариант действительно не подходит.
Поэтому я и спросил у автора темы:

Сообщение от Para bellum

Что вообще делают те методы и почему их именно так именуете?

Добавлено через 43 секунды
Чувствую, что не то что-то он творит.

@Gcom · 05.03.2016, 15:27

Сообщение от Para bellum

мой вариант действительно не подходит.

Все подходит, что мешает сделать это:

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
if (isset($get['q'])) {
    if (!empty($get['q'])) {
        # Формируем имя метода
        $method = 'question' . $get['q'];
    
        # Если метод не существует
        if(!method_exists($this->functions, $method))
            throw new Exception("Метод {$method} не найден");
    
    $data['result'] = (2 <= $get['q'] && $get['q'] <= 4) ? $this->functions->$method() : '' ;
    
    };
};

Вроде и правильно и проверки есть

@Para bellum · 05.03.2016, 15:36

Сообщение от Gcom

Вроде и правильно и проверки есть

Тогда там моя проверка вообще не нужна. В итоге приходим назад, к Вашему варианту.
Допустим, есть метод "questionFoo". Мою проверку $get['q'] равное "Foo" пройдёт. А вот Вашу — нет, ибо в диапазон не вписывается.
В таком случае не получится перехватить ошибку. Эти всякие "throw" и нужны, чтобы держать под контролем приложение. Чтобы можно было потом в отчёт записать, что метод неверный вызван и т.д. и т.п.

@Gcom · 05.03.2016, 15:53

Сообщение от Para bellum

Допустим, есть метод "questionFoo". Мою проверку $get['q'] равное "Foo" пройдёт.

Дак у нас проверка идет на 2,3,4 если нужно Foo, то надо добавить и эту проверку, и тогда будет все гуд: как пример:

PHP
1
$data['result'] = (2 <= $get['q'] && $get['q'] <= 4 && $get['q'] == 'Foo') ? $this->functions->$method() : '' ;

но опять же, если число проверок увеличиться, то с этим:

PHP
1
(2 <= $get['q'] && $get['q'] <= 4 && $get['q'] == 'Foo')

явно что-то надо будет делать...

Добавлено через 1 минуту
даже скорее всего так:

PHP
1
((2 <= $get['q'] && $get['q'] <= 4) || $get['q'] == 'Foo')

Добавлено через 1 минуту
или отдельной строкой:

PHP
1
2
$data['result'] = (2 <= $get['q'] && $get['q'] <= 4) ? $this->functions->$method() : '' ;
$data['result'] = ($get['q'] == 'Foo') ? $this->functions->$method() : '' ;

@Para bellum · 05.03.2016, 16:03

Сообщение от Gcom

Дак у нас проверка идет на 2,3,4 если нужно Foo, то надо добавить и эту проверку, и тогда будет все гуд

Да нет, я не о том. Я про это:

Сообщение от Para bellum

а вот если в объекте "$this->functions" содержатся ещё методы, к которым доступ через указание в "$get['q']" не предполагается — мой вариант действительно не подходит

Доступ к Foo не нужен. Например, создатель кода не хочет, чтобы был доступ к Foo, но Ваш вариант ничего не выдаст пользователю (или составителю отчётов) — он просто оборвёт работу и всё.

А обрывать так не стоит. Лучше бросать исключения, как я показал. Типа такого:

PHP
1
2
if(!(2 <= $get['q'] && $get['q'] <= 4))
    throw new Exception('сообщение');

Добавлено через 2 минуты
P.S. Но мне подобная проверка не нравится.

@Gcom · 05.03.2016, 19:04

Сообщение от Para bellum

P.S. Но мне подобная проверка не нравится.

Я подумал и увидел, что тут тоже есть "грешки", например что будет если $get['q'] = 3.5 понятное дело, что произойдет вызов метода question3.5(), что не есть гуд, это так то тоже желательно учесть, думаю это дело делать так придется:

PHP
1
($get['q'] == 2 || $get['q'] == 3 || $get['q'] == 4)

ну или через

PHP
1
case()

если параметров больше будет, при необходимости запихать в функцию, в которой как вариант сравнивать $get['q'] с массивом значений

PHP
1
array (2, 3, 4, ...)

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Программа принимает математическое выражение в виде строки и выдаёт его производную в виде строки и вычисляет значение производной при заданном х Логарифм записывается как: (x-2)log(x^2+2) -. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .

Как можно сжать данный код, вызов функции?

Решение