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

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

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

PHP

 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

$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

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

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

case()

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

PHP

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

@Para bellum 5753 / 4133 / 1507 Регистрация: 06.01.2011 Сообщений: 11,276
	05.03.2016, 15:36	11
	Сообщение от Gcom Вроде и правильно и проверки есть Тогда там моя проверка вообще не нужна. В итоге приходим назад, к Вашему варианту. Допустим, есть метод "questionFoo". Мою проверку $get['q'] равное "Foo" пройдёт. А вот Вашу — нет, ибо в диапазон не вписывается. В таком случае не получится перехватить ошибку. Эти всякие "throw" и нужны, чтобы держать под контролем приложение. Чтобы можно было потом в отчёт записать, что метод неверный вызван и т.д. и т.п. 0

@Jewbacabra 4925 / 3920 / 1620 Регистрация: 24.04.2014 Сообщений: 11,441
	05.03.2016, 13:42	2
	Сообщение от poss Можно ли вызов функции записать так: почему бы не попробовать выполнить данный код? 0

@Gcom 82 / 82 / 18 Регистрация: 03.02.2016 Сообщений: 564 Записей в блоге: 1
	05.03.2016, 15:02	8
	Не по теме: Сообщение от Para bellum А что, если методов поприбавится? Я всегда пытаюсь мыслить глобально, но нехватка знаний иногда мешает этому, а так отличное решение :good: Добавлено через 5 минут Para bellum, Но опять же, в нем не учитывается проверка: $get['q'] == 2; $get['q'] == 3; $get['q'] == 4; в частности может надо что то другое сделать когда $get['q'] выходит за рамки интервала, конечно это мало вероятно, но все же имеет быть место. 0

@Para bellum 5753 / 4133 / 1507 Регистрация: 06.01.2011 Сообщений: 11,276
	05.03.2016, 15:12	9
	Сообщение от Gcom Para bellum, Но опять же, в нем не учитывается проверка Да, Вы правы. Только это Сообщение от Gcom в частности может надо что то другое сделать когда $get['q'] выходит за рамки интервала, конечно это мало вероятно, но все же имеет быть место. действительно маловероятно, а вот если в объекте "$this->functions" содержатся ещё методы, к которым доступ через указание в "$get['q']" не предполагается — мой вариант действительно не подходит. Поэтому я и спросил у автора темы: Сообщение от Para bellum Что вообще делают те методы и почему их именно так именуете? Добавлено через 43 секунды Чувствую, что не то что-то он творит. 0

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

Решение