我写的MySQL在PHP数据库交互层。 但我认为这是一个普遍的OOP问题（见最后一行）。
我有一个基本的dbTables类。 而且它有一个
public static function getBy($method='name', $value) {
// Gets flat table array of db Tables matching by $method == $value
// later could implement some lookup tables.
$allowed = array('name');
$query_format = SHOW TABLES LIKE '%s'";
if(in_array($method,$allowed)) {
dbConnection::connect(MAIN_DB); // makes db connection
$safe_value = mysql_real_escape_string($value);
// MAY want to change this query to a SCHEMA query in CHILD classes
$sql = sprintf($query_format,$safe_value);
// e.g. $sql = "SHOW TABLES LIKE '$safe_value'";
$result = mysql_query($sql);
if($result === false) {
debug::add('errors', __FILE__, __LINE__, __METHOD__,"Query Error for query '$sql'. MySQL said: " . mysql_error());
}
while($row = mysql_fetch_row($result)) {
$db_table = new static($row[0]); // creates instance of $this class
$object_array[] = $db_table; // add to $object_array for return value
}
} else {
debug::add('errors',__FILE__, __LINE__, __METHOD__, ' - Wrong method: ' . $method . '. Currently allowed: ' . print_r($allowed,true));
return false;
}
return $object_array;
// END public static function getBy($method='name', $value)
}
但是，子类将不同的查询来获取信息。 他们将有搜索其它允许$方法。
这里是我的解决方案，但我不知道这是否是很好的做法，如果稍后将带来更多的痛苦。 而不是覆盖每个子类中这个功能，我可以创建一组专用静态属性，将作为改性剂的功能。
像这样：
protected static $get_by_methods = array('name'); // array('name','id','frontend_name'…) in CHILDREN
protected static $get_by_query_format = "SHOW TABLES LIKE '%s'"; // for sprintf. Changes in children
protected static $get_by_handles_arrays = false; // true in CHILDREN
protected static $get_by_query_format_array = " SELECT * FROM %s` WHERE `$method` IN ($safe_values)"; // used in CHILDREN ONLY
public static function getBy($method, $value) {
$allowed = self::$get_by_methods;
$query_format = self::$get_by_query_format;
$handle_arrays = self::$get_by_handles_arrays; // false here,,, true in children
$query_format_array = self::$get_by_query_format_array; // used in children
if(is_array($value) && $handle_arrays === true) {
return false; // in child class, $handle_arrays can be set to true outside of function
// without rewriting function. just change the static property
}
if(in_array($method,$allowed)) {
dbConnection::connect(MAIN_DB);
if(!is_array($value)) { // handle string values
$safe_value = mysql_real_escape_string($value);
$sql = sprintf($query_format,$safe_value);
} else {
// arrays used only in children
e.g.
$safe_values = mysql_real_escape_string(implode(',',$value)); // convert to string
$sql = sprintf($query_format_array,$safe_values); // used in children
}
$result = mysql_query($sql);
if($result === false) {
debug::add('errors', __FILE__, __LINE__, __METHOD__,"MySQL Error num " . mysql_errno() . " for query [$sql] - MySQL said: " . mysql_error());
}
while($row = mysql_fetch_row($result)) {
$db_table = new dbTables($row['name']);
$object_array[] = $db_table;
}
} else { // if bad method chosen above
debug::add('errors',__FILE__, __LINE__, __METHOD__, ' Wrong method: ' . $method . '. Must use one of these: ' . print_r($allowed,true));
return false;
}
return $object_array;
// END public static function getBy($method='name', $value)
}
综上所述，这样会让我从来没有覆盖getBy（）方法。 我将只需要重写它去保护的静态属性。 对于干（不要重复自己），这似乎不错。 我将只需要一遍又一遍写4行代码，而不是20+。 但是我在这个新的，不知道这可能是某些其他原因一个可怕的错误。
它是安全和良好做法采取继承覆盖了方法并付诸帮手属性？
