オブジェクト指向の考え方について

Question

オブジェクト指向の考え方がいまいちわからないので質問させて頂きます。

あくまで例えばなのですが、犬の行動処理を作る場合この設計は正しくないのでしょうか？
犬が「歩く」→「吠える」→「眠る」という処理を書く場合

class Dog{
　public function action(){
　　walk();
　　bark();
　　sleep();
　}

　private function walk(){
　　// 歩く
　}

　private  function bark(){
　　// 吠える
　}

　private function sleep(){
　　// 眠る
　}
}

$dog = new Dog();
$dog->action();

始めはこのように処理を書いていただのですが、次第にこれはおかしいのではないかと思い始め下記の書き方に変更しました。

class Dog{
　public function walk(){
　　// 歩く
　}

　public function bark(){
　　// 吠える
　}

　public function sleep(){
　　// 眠る
　}
}

$dog = new Dog();
$dog->walk();
$dog->bark();
$dog->sleep();

この場合どちらの書き方が正しいのでしょうか？
それともどちらも正しくないのでしょうか？
個人的には下記の書き方は呼び出し側で大量の関数呼び出しを行う可能性も発生するので、
そこは書き方としてきれいなのかな？という疑問も感じています。

Comments

[m.ts10806]

タグは言語限定せずに「オブジェクト指向」だけで良いのでは？

Answer 1

walk bark sleep を別個に呼び出すことがある場合、分けてください。

上のZuishinさんと同意見です。

ケースバイケースです。
クラス内でしか使わないものはprivate、外から呼び出したいならpublicというだけです。

難しく考えすぎなくて良いです。

https://nowokay.hatenablog.com/entry/20140718/1405691217

また、もしオブジェクト指向設計を突き詰めるとしてもDogという実務で出てこない例で考えても意味がないです。

Answer 2

walk bark sleep を別個に呼び出すことがある場合、分けてください。必ずひとまとまりで呼び出す場合はまとめてください。

Answer 3

あなたがこの世界の中で

• 犬はactionする存在である
• 犬がactionする時は必ずwalk, bark, sleepの順にそれぞれ1回ずつおこなう存在である
• 犬はactionする時以外ではwalk, bark, sleepをすることはない

と認識しているなら、質問の前者のコードは正しいです。そうでないなら正しくありません。

あるいはもっと狭い世界、例えばあなたが作っているゲームやシミュレーションプログラムの中などで、犬とは上記のようなものであるとあなたが定義できるなら、質問の前者のコードは正しいです。そうでないなら正しくありません。

---

正しいか正しくないかという質問をされたらこういう回答になります。

実のところは、正しいか正しくないかなどという考えをする必要はありません、正しいか正しくないかという尺度が間違っています、というのが回答になるのですけれども。

Comments

[Hina_0917]

ベストアンサー遅くなり申し訳ありません。
回答いただきありがとうございました！

Answer 4

あながち、書き方は間違っていません。
実際に、デザインパターンのTemplateMethodパターンとかありますし。

ただし、質問者さんが「Dogはactionというメソッドを持つのがふさわしい」と考えるなら。です。
( つまり、quiquiさんに同意 )