33-Nagata / kadai

0 stars 0 forks source link

第12回レビュー #8

Open gs-teacher opened 9 years ago

gs-teacher commented 9 years ago

レビュー担当の増子です。

クラスを使ったり、データがある時ない時で分けているのは処理としてはバッチリです。 気になったところは、gnaviControllerをnewしているところで$_GETを渡しておりますが、これは使われておりますか?コンストラクタがないので使われていないような・・・ あと、クラス内の関数(メソッド)や変数にprotected修飾子が付いてますが、これは意味を理解して使っておりますでしょうか?protectedはクラス継承などを使用しない限りはprivateにしておいたほうがよいです。(クラス設計でそもそも継承される予定とかなら別ですが)

オブジェクト思考全部理解するのは難しいと思うので、ちょっとづつ理解を進めてみてください。

33-Nagata commented 9 years ago

159行目にコンストラクタを書いていて、175行目で使用している……と思います。 クラス継承の予定は特にありませんでしたが、継承した場合には使うかもしれないと思ったものにはprotectedをつけていました。 まだクラス自体とクラスから作られたインスタンスのprivateの動作の違いなど、いまいち感覚的に腑に落ちていない部分があるので少しずつ理解していきたいです。

gs-teacher commented 9 years ago

見落としてました! そして継承した場合使うかものところにprotectedなど意味を理解して使っているのであれば問題ないです!よく見るとpublicの部分もあったりしますね。使い分けできていない人が多かったので、、、失礼しました!