让我从一个小小的介绍开始。我正在学习PHP中的OOP。我也研究过设计模式,但还没有完全掌握不同类型的概念。我正处于这样一个阶段,每隔几个月我就会意识到我做事的方式不对,必须改变我的风格。这太令人沮丧了。因此,我想一劳永逸地找出正确的做事方式。我已尝试全面阅读有关以下主题的 Stackoverflow:
ORM
数据映射器
Singleton
Globals 是邪恶的
一切相关
但是,我仍然不清楚一些事情。我以清晰简洁的方式在这里发布我的代码,希望人们能够指出好的做法和坏的做法。我会在最后列出我所有的问题。
请不要重复关闭,我已经诚实地搜索了几乎所有关于该主题的问题,但我仍然想知道一些我无法澄清的事情。对不起,它太长了,但我已经尝试组织它,所以它应该读得很好!
我将首先发布我的数据库类的要点。
数据库.php
<?php
class DatabaseMySQL{
private static $dbh;
public function __construct(){
$this->open_connection();
}
public function open_connection(){
if(!self::$dbh){
return (self::$dbh = new PDO(DB_TYPE.':host='.DB_HOST.';dbname='.DB_NAME, DB_USER,DB_PASSWORD)) ? true : false;
}
return true;
}
public function query($sql, $params=array()){
$this->last_query = $sql;
$stmt = self::$dbh->prepare($sql);
$result = $stmt->execute($params);
return $result ? $stmt : $stmt->errorInfo();
}
public function fetch_all($results, $class_name=''){
return $results->fetchAll(PDO::FETCH_CLASS, $class_name);
}
}
?>
这是我的数据库类文件。此类允许我创建任意数量的此类实例,并且它将重用已实例化的 PDO 对象,该对象存储为该类的静态属性。它还使用 PDO 从结果集中获取数据,以将数据作为指定类的对象获取。
我拥有的下一个文件是我的所有其他类都继承自的类。我称它为 MainModel。我不知道这是否符合惯例。
主模型.php
<?php
abstract class MainModel{
protected static $table;
public function __construct($array=array()){
$this->assign_known_properties($array);
}
public function assign_known_properties($array){
foreach($array as $key=>$value){
$this->$key = $value;
}
}
public static function find_by_id($id){
$db = new DatabaseMySQL();
self::intialise_table_name();
$id = (int) $id;
$sql = "SELECT * FROM ".static::$table." ";
$sql .= "WHERE id = {$id} ";
$result = self::find_by_sql($sql);
return array_shift($result);
}
public static function find_all(){
$db = new DatabaseMySQL();
self::intialise_table_name();
$sql = "SELECT * FROM ".self::$table." ";
return self::find_by_sql($sql);
}
public static function fetch_as_objects($results){
$db = new DatabaseMySQL();
$called_class = get_called_class();
$results = $db->fetch_all($results, $called_class);
return $results;
}
public static function find_by_sql($sql){
$db = new DatabaseMySQL();
$results = $db->query($sql);
return $results ? self::fetch_as_objects($results) : false;
}
public static function intialise_table_name(){
$called_class = get_called_class();
static::$table = strtolower($called_class).'s';
}
public function get_table_fields(){
self::intialise_table_name();
$sql = "SHOW FIELDS FROM ".static::$table." ";
return self::find_by_sql($sql);
}
public function set_table_details(){
$fields = $this->get_table_fields();
$total = count($fields);
$array = array();
foreach($fields as $object){
$array [] = $object->Field;
}
$this->table_details = array('objects'=>$fields,'array'=>$array,'total'=>$total);
$this->set_placeholders_for_new_record();
$this->set_properties_as_array();
$this->set_properties_as_array(true);
}
public function set_properties_as_array($assoc=false){
$array = array();
if (!$assoc){
foreach($this->table_details['array'] as $field){
if(isset($this->$field)){
$array [] = $this->$field;
}else{
$array [] = NULL;
}
}
$this->table_details['values'] = $array;
}else{
foreach($this->table_details['array'] as $field){
if(isset($this->$field)){
$array[$field] = $this->$field;
}else{
$array [$field] = NULL;
}
}
$this->table_details['assoc_values'] = $array;
}
}
public function set_placeholders_for_new_record(){
$string = '';
for($i=0; $i<$this->table_details['total']; $i++){
$string .= '? ';
if(($i+1) != $this->table_details['total'] ){
$string .= ", ";
}
}
$this->table_details['placeholders'] = $string;
}
public function create(){
$db = new DatabaseMySQL();
$this->set_table_details();
$sql = "INSERT INTO ".static::$table." ";
$sql .= " VALUES({$this->table_details['placeholders']}) ";
$result = $db->query($sql, $this->table_details['values']);
// If array is returned then there was an error.
return is_array($result) ? $result : $db->insert_id();
}
public function update(){
$db = new DatabaseMySQL();
$this->set_table_details();
$sql = "UPDATE ".static::$table." ";
$sql .= " SET ";
$count = 1;
foreach($this->table_details['array'] as $field){
$sql .= "{$field} = :{$field} ";
if($count < $this->table_details['total']){
$sql .= ", ";
}
$count++;
}
$sql .= " WHERE id = {$this->id} ";
$sql .= " LIMIT 1 ";
$result = $db->query($sql, $this->table_details['assoc_values']);
return $result;
}
public function save(){
return isset($this->id) ? $this->update() : $this->create();
}
}
?>
总结这个文件。我使用静态方法find_by_id($int)
来动态生成被调用类的对象。我正在使用后期静态绑定来访问被调用类的名称,并使用$stmt->fetchAll(PDO::FETCH_CLASS, $class_name)
数据库中的数据自动转换为对象来实例化这些对象。
在每个静态方法中,我都会实例化 DatabaseMySQL 类的一个实例。在每个静态方法中,我$table
通过获取类的名称并将其附加s
到它来动态设置要在 SQL 查询中使用的正确静态名称。因此,如果我的班级是User
,那将使表名users
。
在我的构造函数中,我放置了一个可选数组,可用于在创建对象时插入一些变量作为对象的属性。这样,一切都是动态完成的,这让我进入了项目的最后阶段。我的继承类。
用户.php
class User extends MainModel{
}
问题.php
class Question extends MainModel{
}
我现在做的很简单。我可以说:
$user = new User(array('username'=>'John Doe'));
echo $user->username; // prints *John Doe*
$user->save(); // saves (or updates) the user into the database.
我可以通过对用户的静态调用来检索用户。
$user = User::find_by_id(1);
echo $user->username; // prints users name
所以现在我的问题:
1) 你会给这个设计模式起什么名字(如果它甚至是一个)。数据映射器?依赖注入?领域模型(无论是什么)?数据访问层?
2) 就目前而言,这种实现是否被认为结构良好?
3)如果它很好,我应该注意我的代码中缺少的命名约定吗?
4)如果它被认为是好的,你介意指出你特别喜欢什么,这样我就知道你一定要保留哪个部分?
5)如果您认为这不好,您是否愿意详细解释为什么会这样?
6) 我的对象的所有方法 my , 是否应该与 my 属于同一类,
create
它们仅update
被静态调用并实际返回对象。如果他们应该在两个不同的班级,我该怎么做?delete
find_by_id
find_all
7)为什么其他人都在使用
$load->('UserClass')
像mappers这样的函数和花哨的词,而我一次都不需要它们?