DoctrineのEntityのgetter、setterをやめてみたお話。

こんばんはポリドックです。
今日も元気にSymfonyにを書いています。

さて、今日はTwitterでこんなツッコミをされてしまったので、軽くまとめてみようと思います。

何がどう変わったのか詳しく。 https://t.co/q7vo2KOF9O
— Hidenori Goto (@hidenorigoto) November 18, 2016

そもそもなぜGetter/Setterをやめたかったのか？

「必要なくね？」って激しく思った。ただそれだけです。

privateでプロパティを定義してgetter/setterを用意するなんて、ほとんど意味がないことだと僕は思うんですが、違うんですかね？

1

$hoge->getUser()->getId();

ってのは

1

$hoge->user->id

ってなるのは書きやすさ全然違いますよね。
そんなSyntaxの話はどうでもいいとか、本質的じゃないって話はよくわかっています。
でも・・・ちょっとでも書く量が減ると嬉しいんですよね。

今現在、書くコード量が多い僕としては十分なメリットです。

Entityが本当に見やすくなります。
定義するメソッドが少なくなり、すっきりするんですよね。

getter/setterがなくなると、本当に必要なメソッドがパッとみでわかるようになります。（IDE使ってたらそんなにメリットでもない気がしますが・・・。）

こんな感じに、ManyToOneとかしていて、本当はEntity渡さなきゃいけないようなところでも、intとか平気で渡せちゃいますよね。

1

$hoge->fuga = 1; // 本来ならFugaオブジェクトをセットしなければいけないところ。

普通にsetterを定義していれば、タイプヒント使って防げますよね。

1
2
3
4
5
6
7


class Hoge {
    public function setFuga(Fuga $fuga) {
        $this->fuga = $fuga;
    }
}

$hoge->setFuga(1); // これだと、エラーになる。

PHPの型のゆるさが辛いところですね・・・。

こうやって書いてみると、getter/setterをやめたところで、本質的には何も変わらないような気もします。
でも、本質的にかわらないなら、よりシンプルになるのほうが良いのかなとも思います。

結論としては「どっちでもいい」と。
チームとかで話し合って好きな方を採用すればいいと思います。

「Getter/Setterをやめてたら心が豊かになりました。」とか調子にのったツイートしてごめんなさい。