Не запутает ли такой полиморфизм разработчика?

Question

[myspace]

класс request, method() без аргументов возвращает имя request_method`а, а при передаче строкой нужного метода, возвращает массив с данными(или false)

class Request {

    public function method($method = null) {
        switch ($method) {
            case null:
                return $_SERVER['REQUEST_METHOD'];
                break;
            case 'post':
                return $this->post();
                break;
            case 'get':
                return $this->get();
                break;
            default:
                return false;
        }
    }

    private function post() {
       return $this->method() === 'post' ? $_POST : false;
    }

    private function get() {
        return $this->method() === 'get' ? $_GET : false;
    }
///

Стоит ли использовать такие способы, или сделать классически?

Comments

[Евгений]

адепты ооп обкончаются

[myspace]

Евгений: ))

Answer 1

[rPman]

Я не буду говорить про ООП, читаемость кода или его модифицируемость в будущем.

Меня просто корежит использование строк для передачи управляющих команд в методы... тем более там где это совершенно не нужно. Вдумайтесь, исключительно для вашего синтаксического сахара, в базовые методы коммуникации вы вставляете сравнение строк! Ну что мешает завести численный дефайн или воспользоваться делегатами (передать сам метод в качестве параметра)?

К сожалению это за последние десять лет стало нормой на столько что можно увидеть абсолютно везде... а потом удивляются, а почему все работает так медленно, и памяти кушает много.

p.s. по теме, если параметр, который подставляют в вызов method() будет браться из-вне, например значение одного из параметров в GET, то я бы порекомендовал пропустить эти параметры через специальный метод-фильтр, конвертирующий строки во внутреннее представление (числовые дефайны или ссылки на методы), он же проверяет их валидность... вот это удобство для программиста, когда выделяется отдельная задача в отдельный метод/класс... ООП тут совершенно не требуется, так как нечего наследовать, хотя может быть использовано просто как синтаксический способ объединить код и данные в одном месте.

Comments

[myspace]

да, можно расширить как нибудь так

///
    const REQUEST_METHOD_POST = 1;
    const REQUEST_METHOD_GET = 2;

    public function method($method = null) {

        switch ($method) {
            case null:
                return function() {
                    if($_SERVER['REQUEST_METHOD'] === 'post') {
                        return self::REQUEST_METHOD_POST;
                    } elseif($_SERVER['REQUEST_METHOD'] === 'get') {
                        return self::REQUEST_METHOD_GET;
                    } else {
                        return false;
                    }
                };
///

[rPman]

я не понимаю где будет вызываться метод method ти как будет вызываться делегейт, но чую вы что то не понимаете или путаете.

1. метод в котором строки подменяются на константы должен быть вынесен отдельно (у вас в последнем примере это анонимный метод), в идеале он же должен проверять все параметры и их валидность, организационно это правильнее размазывания по коду этого функционала.

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

я не понимаю, почему вы городите какую-то фигню с методами get и post?

обычно делают описанное мной с поддействиями (например в методе заведен параметр action а значение определяет действие, типа add,move,payment,...)

[Евгений]

myspace: у вас внутри класса используется суперглобальный массив $_SERVER, это уже странно, его лучше передавать параметром или в конструктор.

Answer 2

[Евгений]

Вы понимаете что строчка ниже делает тоже самое?

var_dump( $_SERVER['REQUEST_METHOD'] == 'post' ? $_POST : ($_SERVER['REQUEST_METHOD'] == 'get' ? $_GET : false) );

Comments

[myspace]

да, но стоить добавить еще методов, или переопределить существующие и получится больше кода. Плюс когда разбито по частям это удобнее.

[myspace]

oxidmod: это уже работающий ужасно написанный самописный проект, миграция на фреймворк пока не рассматривается