Question

我想就设计理念寻求建议。我创建了自己的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方法需要首先处理变量。

Answer 1

您提到的Sql类对我来说就像是一个Repository模式。确实具有维护优势，但也有助于单元测试，更好地分离内容和更清晰的代码。

Answer 2

这是我在使用数据库时如何设计自己的框架类的个人帐户。

通常，在设计MVC框架时，我列出了两个用于开发类的核心概念：

管理各种方面的单个数据库类，例如查询构建，行计数返回，插入ID等。
一个接口数据类，用于确保每个类保持一致（例如，每个类必须具有所有四个CRUD操作）

数据库类

我的数据库类通常类似于以下内容。你会注意到底部有你指定的mutator函数。这些仅专门用于设置和获取类本身的数据。虽然我听说这种代码应该在Model级别，但是在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();

我正在我的模型类中正确执行SQL语句吗？

2 个答案: