記述方法について

Question

phpでdb処理を書こうとしています。以下はその一部です。
いちいちprepare〜bind〜executeで記述してそのたびにifで判定して$this->errorにセットしてreturnで帰るやり方をしていますが
正直、もっと別のやり方はないものだろうかとも思います。
同様の処理を短く、かつ可読性の高い処理方法はないものでしょうか？

     private function searchAndCreateTable($tableName){
                require(dirname(__FILE__)."/db_config.php");

                //テーブルそのものの確認
                $string="show tables like :tableName";
                $stmt=$this->PDO->prepare($string);
                if(     $stmt===false   ){
                        $this->errorMessage="seaechAndCreateTable showTables prepare error";
                        $this->$tableName=$string;
                        $this->PDO=null;
                        return false;
                }
                $stmt->bindvalue(":tableName",$tableName);
                $result=$stmt->execute();
                if(     $result===false ){
                        $this->errorMessage="seaechAndCreateTable showTables execute error";
                        $this->$tableName=$string;
                        $this->PDO=null;
                        return false;
                }else if(       $result===0     ){
                        //結果が0ならテーブル作成
                        $string="create table :tableName ()"


                }

                //指定カラムの確認
                $string="show columns from ".$tableName." where field in (".
                        implode(',',array_fill(0,count($$tableName),'?')).
                        ")";
                $stmt=$this->PDO->prepare($string);
                if(     $stmt===false   ){
                        $this->errorMessage="searchAndCreateTable search colum prepare error";
                        $this->$tableName=$string;
                        $this->PDO=null;
                        return false;
                }
                $result=$stmt->execute( array_keys($$tableName) );
                if(     $result===false ){
                        $this->errorMessage="searchAndCreateTable search colum execute error";
                        $this->PDO=null;
                        $this->$tableName=$string;
                        return false;
                }
                return $result;
        }

Answer 1

進み方としては間違っていないと思います。

多分、あなたが今考えているのは似たようなコードを何回も書いていることでしょう。

そうしたら、Exception と set_error_handler()について学びましょう。

Comments

[gyojin]

ありがとうございます。set_error_handler()はかなり使えそうな感じがします。
>>似たようなコード
大した処理をするものでもないのに延々とコードが伸びてしまいます。
可読性はこれでもまぁ、いいのかなとは個人的に思いますがいいものを教えてくださいました。
基本独学なので覚えることの要点を抑えるということがやりづらく、大変ありがたいです
重ねてお礼申し上げます。