0

デザインコンセプトについてアドバイスをお願いします。独自のMVCフレームワークを作成しました。

モデルクラスの場合、これらはデータベースからデータをメインに取得するクラスです。

たとえば（User）のクラスでは、次のメソッドがあります

  • get_user。
  • get_recent_activity。

基本的に、たとえばget_recent_activityメソッドでは、データを取得するために実行を行うsqlステートメントがあります。

私は別の設計を考えています。それは、各クラス（たとえばUser）をクラスで拡張することですSql。このクラスには、SQLに必要なすべてのクエリが含まれます。

たとえば、sqlクラスを使用した挿入クエリは次のようになります。

$sqlqueries->select("table","user=john&email=john@hotmail.com");

これはより良いデザインになるでしょうか？メンテナンスかそこら？メソッドからデータを取得するために、パフォーマンスの問題が心配です。まず、SQLクラスからのselectメソッドが最初に変数を処理する必要があります。

4

2 に答える 2

0

あなたが言及したSQLクラスは、私にはリポジトリパターンのように見えます。確かにメンテナンスの利点がありますが、単体テスト、懸念事項のより良い分離、よりクリーンなコードにも役立ちます。

于 2012-06-20T12:40:21.190 に答える
-1

これは、データベースを操作するときに独自のフレームワーククラスを設計する方法についての個人的な説明です。

通常、MVCフレームワークを設計するときは、クラスを開発するための2つのコアコンセプトをレイアウトします。

  • クエリの作成、行数の戻り、IDの挿入などのさまざまな側面を管理する単一のデータベースクラス。
  • 各クラスの一貫性を維持するためのインターフェイスデータクラス（たとえば、各クラスには4つのCRUD操作すべてが必要です）

データベースクラス

私のデータベースクラスは通常、次のようになります。下部に向かって、指定したようなミューテーター関数があることに気付くでしょう。これらは、クラス自体からデータを設定および取得するためだけに特別に設計されています。（get_connectionに）マイナーな検証がありますがif(is_resource(...))、この種のコードはモデルレベルにあるべきだと聞いています。 

    public $error;

    private $location = '';
    private $username = '';
    private $password = '';
    private $database = '';

    public function __construct()
    {
        $connection = mysql_connect($this->location,$this->username,$this->password);

        $this->set_connection($connection);

        $db_select = mysql_select_db($this->database,$this->get_connection());
    }

    public function query()
    {
        $query = mysql_query($this->query,$this->connection);

        $this->set_insert_id(mysql_insert_id($this->connection));

        switch(substr(strtolower($this->query),0,6))
        {
            case 'select':
                $this->set_row_count(mysql_num_rows($query));

                $row_count = $this->get_row_count();

                if($row_count == 0)
                {
                    $this->set_results(0);
                }
                else
                {
                    while($result = mysql_fetch_assoc($query))
                    {
                        $results = $this->get_results();
                        $results[] = $result;
                        $this->set_results($results);
                    }
                }
            break;
            case 'insert':
            case 'update':
            case 'delete':
            case 'replace':
            default:
                $this->set_results(($query === true));
            break;
        }
    }

    public function get_query()
    {
        return $this->query;
    }

    public function set_query($query)
    {
        $this->query = $query;
    }

    public function get_insert_id()
    {
        return $this->insert_id;
    }

    public function set_insert_id($insert_id)
    {
        if(is_numeric($insert_id))
        {
            $this->insert_id = $insert_id;
        }
    }

    public function get_results()
    {
        return $this->results;
    }

    public function set_results($results)
    {
        $this->results = $results;
    }

    public function get_row_count()
    {
        return $this->row_count;
    }

    public function set_row_count($row_count)
    {
        if(is_numeric($row_count))
        {
            $this->row_count = $row_count;
        }
    }

    public function get_connection()
    {
        return $this->connection;
    }

    public function set_connection($connection)
    {
        if(is_resource($connection))
        {
            $this->connection = $connection;
        }
        else
        {
            $this->connection = false;
        }
    }
}
?>

データクラス

インターフェイスデータクラスは他のクラスの単なるガイドラインであるため、非常に簡単です。このアプローチを行う場合、データを操作するのは簡単です。

<?php
interface Data {
    public function create();
    public function read();
    public function update();
    public function delete();
}
?>

サンプルクラス

これは、DatabaseクラスとDataクラスを操作する前述のクラスで通常使用するクラスの例です。典型的なCRUD操作を使用する場合、Databaseクラスを呼び出し、クエリを設定し、データベースにクエリを実行して、結果を返します。これは可能な限り単純であり、その他の要件を各関数に追加できます。

<?php
require_once('classes/class.Data.php');
require_once('classes/class.Database.php');
class Settings implements Data {
    private $profile_id;
    private $settings = array();

    public function create()
    {
        $settings_instance = new Database;
        $sql = 'INSERT INTO settings (' . implode(',',array_keys($this->settings)) . ') VALUES ("' . implode('","',array_values($this->settings)) . '");';
        $settings_instance->set_query($sql);
        $settings_instance->query();
        return $settings_instance->get_results();
    }

    public function read()
    {
        $settings_instance = new Database;
        $sql = 'SELECT * FROM settings WHERE profile_id = "' . $this->profile_id . '"';
        $settings_instance->set_query($sql);
        $settings_instance->query();

        $settings_results = $settings_instance->get_results();

        if($settings_results)
        {
            $this->set_settings($settings_results[0]);
        }
    }

    public function update()
    {
        $keys = array_keys($this->settings);
        $values = array_values($this->settings);
        $settings_instance = new Database;
        $sql = 'UPDATE settings SET ';
        foreach($keys as $index => $key)
        {
            if($keys[$index] == "profile_id")
                continue;
            $sql .= $keys[$index] . ' = "' . $values[$index] . '", ';
        }
        $sql = rtrim($sql,', ');
        $sql .= ' WHERE profile_id = "' . $this->profile_id . '"';
        $settings_instance->set_query($sql);
        $settings_instance->query();
        return $settings_instance->get_results();
    }

    public function delete()
    {
        $settings_instance = new Database;
        $sql = 'DELETE FROM settings WHERE profile_id = "' . $this->profile_id . '"';
        $settings_instance->set_query($sql);
        $settings_instance->query();
        return $settings_instance->get_results();
    }

    public function get_profile_id()
    {
        return $this->profile_id;
    }

    public function set_profile_id($profile_id)
    {
        if(is_numeric($profile_id))
        {
            $this->profile_id = $profile_id;
        }
    }

    public function get_setting($index)
    {
        if(isset($this->settings[$index]))
        {
            return $this->settings[$index];
        }
    }

    public function set_setting($index,$new_value)
    {
        if(isset($this->settings[$index]))
        {
            $this->settings[$index] = $new_value;
        }
    }

    public function get_settings()
    {
        return $this->settings;
    }

    public function set_settings($settings)
    {
        if(is_array($settings))
        {
            $this->settings = $settings;
        }
    }

    public function get_setting_types()
    {
        return $this->setting_types;
    }

    public function set_setting_types($setting_types)
    {
        if(is_array($setting_types))
        {
            $this->setting_types = $setting_types;
        }
    }
}
?>

いくつかのサンプルコード

これは、このアプローチに沿ったモデルコードの例です。これはかなり読みやすいと思います。これにより、profile_id=1でデータベースに新しい設定行が作成されます。

require_once('class.Settings.php');
$Settings = new Settings;
$Settings->set_profile_id(1);
$Settings->create();
于 2012-06-20T12:49:54.890 に答える