PHPでオブジェクト指向を実現したいです。助言をください。

Question

前提・実現したいこと

PHPにて、オブジェクト指向プログラミングを目指しています。
しかし、初めに、手続き型でプログラミングを学んだことから、なかなかオブジェクト指向での実装ができません。
そこで、次のプログラムを例にオブジェクト指向を実現するには、あるいは、みなさんだったらどう実装するかをご教授願えると助かります。

要求仕様

同じ長さの二つの文字列 A, Bが存在する。
A, Bを比較して、以下の条件で値を返す。
2つの文字列のスペルがまったく同じであれば、2を返す。
二つの文字列のスペルが1つだけ異なる場合、1を返す。
その他は、0を返す。

例)
A:"teratail"
B:"teratair"
この場合、1を返す。

該当のソースコード

<?php
    class Hoge{
        
        
        private $count = 0;
        private $output = 0;


        public function compare($a, $b){
            $this->count = 0;
            for($i = 0; $i < strlen($a); $i++) {
                if($this->isSame($a[$i], $b[$i]) == 1){
                    $this->count +=1;
                }
            }    
            if($this->count == 0){
                $this->output +=2;
            }
            if($this->count == 1 ){
                $this->output +=1;
            }

            return $this->output;
        }
        public function isSame($a, $b){
            if ($a !== $b) {
                return 1;
            }
            else{
                return 0;
            }
        }
        
    }
    $hoge = new Hoge;
    echo ($hoge->compare("hoge", "hogu"));　//1を返す
    
?>

補足情報

個人的に気にかかっている点として、カウンタを使用していることです。
カウンタを使うのは手続き型のイメージがあります。
もし、カウンタを用いることが望ましくない場合、用いない場合の実装などを教えていただければ幸いです。

参考情報

オブジェクト指向の参考記事
https://ameblo.jp/principia-ca/entry-11876117369.html

問題の元ネタ（paizaは提出コードを公開できないので、内容を改変しています。）
https://paiza.jp/student/challenges/116/page/problem

Comments

[m.ts10806]

オブジェクト指向については何か参考にした文献や記事などはありますか？

[plato]

主にこちらです。https://ameblo.jp/principia-ca/entry-11876117369.html

[m.ts10806]

正直この要件だけだとオブジェクト指向にしたとしてもほぼ恩恵を得られません。

[m.ts10806]

参考記事などは回答に必要な材料となるので質問本文に追記してください。

[plato]

失礼しました。追加しました

[退会済みユーザー]

なんでこういう、実務ではおおよそありえなそうな例題を出すかな。

[退会済みユーザー]

$this->outputを初期化する概念がないのがニントモカントモ

[m.ts10806]

オーソドックスな例で車とか「役割をしっかりわけられる」ようなものとか、掲示板とかDB絡めた方がしっかり使えそうですね。せめてプロパティ共有するとかくらいはしたいですが、この要件だとそもそもローカル変数のみで済むので・・・。

[退会済みユーザー]

paizaみれないけど、あそこの問題って実務ではおおよそ関係ない問題ばっかなのであんま、相手にしてない。

[退会済みユーザー]

オブジェクト指向といったら、キャットドア課題でしょ。

[退会済みユーザー]

2文字以上食い違っていたらどうしたらいいんだろう、teratailとraratailとか

[退会済みユーザー]

就活のチャレンジは公開されてない（会員登録必須）なので答えかねるほうにいきそうだな

[plato]

paiza見れないですよね。失礼いたしました...問題と内容を変えてはいるのですが、それでも不適切な場合は質問自体を消します。

[退会済みユーザー]

質問が不適切なわけじゃないけど、もっと現実味あるお題じゃないと座布団取りに行く食尻が湧かないっす。

[m.ts10806]

でしたら私の回答はベストアンサーとしては不適切なので、もう少し回答を待ってみては？

[退会済みユーザー]

だから、やっぱりキャットドア課題をやるのが一番なんですよ。

Answer 1

オブジェクト使うんなら、多分こんな方向性

<?php
class ArrayOfString{
    public $str_org;
    public $str_arr;
    function __construct($str){
        $this->str_org = $str;
        $this->str_arr = str_split($str);
    }
    
}
$test_arr = ['hoge', 'hoga', 'hoeg', 'hgeo'];
$hoge = new ArrayOfString('hoge');
foreach($test_arr as $val){
    $str = new ArrayOfString($val);
    echo $hoge->str_org.'と'.$str->str_org.'の比較:';
    $count = count(array_diff_assoc($hoge->str_arr, $str->str_arr));
    echo $count>=2?0:!$count+1;
    echo PHP_EOL;
}

Answer 2

オブジェクト指向で捉えるべき箇所が見つかりませんでした＾＾；

<?php
class Hoge{
    public function compare($a, $b){
        $a_arr = str_split($a);
        $b_arr = str_split($b);
        $count = count(array_diff_assoc($a_arr, $b_arr));
        return $count>=2?0:!$count+1;
    }
}
$test_arr = ['hoge', 'hoga', 'hoeg'];
$hoge = new Hoge;
foreach($test_arr as $val){
    echo ($hoge->compare("hoge", $val)).PHP_EOL;        
}

オブジェクト指向で考えるなら、文字列 a,文字列 b をオブジェクト化してゴニョゴニョやるんじゃないかなぁ。。。

Comments

[退会済みユーザー]

全くもって、おっしゃる通りでございます。

Answer 3

そもそもPHPだけではオブジェクト指向分からないし難しいこと気にする前に
良いコード書けるようになるのが先。

試してないけど今回のならarray_diff_assoc()辺り使えば簡単に比較できるかも。

Comments

[退会済みユーザー]

何がそもそもなのか分からNice.

[退会済みユーザー]

サンプルコードを書いて検証したいところですね

Answer 4

質問に対する直接的な回答ではありません。あくまで参考記事紹介です。

記事としては下記とか、

• PHPオブジェクト指向入門（前半）
• PHPで学ぶオブジェクト指向

実践もあるなら下記とか

• PHPのオブジェクト指向入門

参考にされた方が良いかも。

Comments

[plato]

ご回答ありがとうございました。みなさんの意見を拝見するにオブジェクト指向を学習の目的にする場合、paizaなどではなく、教えていただいたサイトなどを参考にした方がよさそうですね。そちらで学習したいと思います。ありがとうございました。

Answer 5

2つの文字列のスペルがまったく同じであれば、2を返す。
二つの文字列のスペルが1つだけ異なる場合、1を返す。
その他は、0を返す。

3行それぞれを評価するのに、最も重い処理は2を返す条件じゃないかな、
次に重いのは1を返す条件で、
簡単に判別できて大多数の結果がそうなるであろう条件は0。
って考えると、
0を返す条件をもう少し深掘りする。

・互いの文字列長が異なる

って判定を最初に持ってくれば、さっさと0を返して終われる判定。

$this->outputをわざわざ使う理由がよくわからないけど、
メソッド内ローカル変数を使ってもいいんだよ。

一文字でも違っていれば1を返すのだから、
文字列のアタマから照合して

        public function compare($a, $b){
            if (strlen($a) !== strlen($b)) {
                return 0;
            }
            $count = 0;
            for($i = 0; $i < strlen($a); $i++) {
                if($this->isSame($a[$i], $b[$i]) == 1){
                    $count +=1;
                }
            }    
            if($this->count == 0){
                return 2;
            }
            else {
                return 1;
            }
        }

ってのはダメでしょうか。

Comments

[退会済みユーザー]

あのさぁ…

[退会済みユーザー]

アルゴリズム的にも気になったところがあったので、加筆してみました。

[m.ts10806]

m6uさん
おかしくないですよ。

$a = "abcdef";
echo $a[2];
//c

[退会済みユーザー]

あ、すみませんでした。substr()の例にちゃんと説明がありましたね。

[退会済みユーザー]

もう、その辺の緩さがPHPの最大の魅力なのに、それを否定されちゃうと…

[退会済みユーザー]

スコア２をもっていて　不一致文字があるごとにスコアー１で０が最低なら
ここまでシンプルにできるが（※　即興版なので マルチバイト文字は意識ｓていない ）

https://ideone.com/4pZotG

[退会済みユーザー]

5行程度でおわるのをどうしてクラスにして肥大化するんだろう

[退会済みユーザー]

asahina は寝てた方が良かったな。