Как правильно типизировать ответ в php?

Question

[Владимир Грабко]

Сейчас слепил вот такое гавнище

class ReturnApi {

    protected $code;
    protected $messages;
    protected $data;

    public function SetCode($code) {
        if (is_int($code)) {
            $this->code = $code;
        } else {
            throw new Exception("Не число");
        }
    }

    public function SetMessage($message) {
        if(is_string($message)){
        $this->messages = $message;
        }else{
            throw new Exception("Не строка");
        }
    }

    public function SetData($data) {
        if (is_array($data)) {
            $this->data = json_encode($data);
        } else {
            throw new Exception("Не масив");
        }
    }

    public function Ret() {
        return json_encode([
            "code" => $this->code,
            "message" => $this->messages,
            "data" => $this->data,
        ]);
    }
}

Как это делать правильно?

Comments

[Zhainar]

а какая задача стоит?

[Владимир Грабко]

Zhainar: у меня ответы api имеют 1 формат ответа.

[Zhainar]

Владимир Грабко: $this->data = json_encode($data); лишнее, $this->data = $data;

[Владимир Грабко]

Zhainar: по сути конструкция останется такой же уродливой?

[Владимир Грабко]

Zhainar: мля... как же меня бесит пхп.

[Zhainar]

Владимир Грабко: можно засунуть все это в один метод __construct

Answer 1

[Владимир Грабко]

На сколько я понял то конструкция останется такой же уродливой. Мля как же я не люблю этот пхп...

Answer 2

[Артем Сошников]

Можно заюзать JsonSerializable интерфейс.
Слепить абстрактный класс со стандартными значениями code, message data и т.п.
А еще лучше сделать интерфейс для ResponseData.

Answer 3

[Алексей Скобкин]

Посмотрите в сторону Symfony HTTPFoundation. А именно: Response и JsonResponse.

Answer 4

[novrm]

Сеттер - устанавливает переменную.
Геттер - проверяет тип переменно - вызывает исключение или возвращает переменную.
Кодировать/декодировать переменную нужно отдельным методом.

Например:

/**
     * Set tag array
     *
     * @param  array $tag
     * @return MessagesOptions
     */
    public function setTag($tag)
    {
        $this->tag = (array) $tag;
        return $this;
    }

    /**
     * Get tag array
     *
     * @return array
     */
    public function getTag()
    {
        if (!is_array($this->tag)) {
            throw new \InvalidArgumentException(sprintf('"%s" expects array.', __METHOD__));
        };
        return $this->tag;
    }

Comments

[Владимир Грабко]

facepalm. Вы вопрос читали?

[Владимир Грабко]

парень я сделал тоже самое! Я спросил как можно типизировать ответ. У меня апи должно отвечать по формату. Так для этого я попытался слепить подобие Golang структуры. А ты мне про сетеры чешешь

[novrm]

Вопрос: "Как это делать правильно?" Лепить гавнище или делать типизацию?

[Владимир Грабко]

novrm: what?

Answer 5

[shell_execute]

PHP7 strict mode
php.net/manual/ru/migration70.new-features.php

Answer 6

[shagguboy]

public function SetData(array $data)