让我从解释开始。我有几个全局数据库连接,并且有几个简单的函数来使用每个连接并执行查询等。
因为我想多次使用这些连接,并且为了省去每次在每个函数中定义它们的麻烦,我在文档顶部将它们创建为全局变量。然而,我只是想知道,与其写
global $mysql_db1, $mysql_db2, $mysql_db3, $mysql_db4, $mysql_db5;有没有什么方法可以做到这一点,而不需要我每次都复制和粘贴?
我知道这很微不足道,但我只是想加快我自己的发展,
发布于 2011-08-17 03:01:36
全局变量通常被认为是不好的做法。我不会对你说三道四,但可以看看这篇文章:http://blog.case.edu/gps10/2006/07/22/why_global_variables_in_php_is_bad_programming_practice
您可以使用$GLOBALS超级全局来访问全局作用域(docs)中定义的任何变量。因此,在您的示例代码中,简单地使用$GLOBALS['mysql_db1']等同于拥有global $mysql_db1;行,然后使用$mysql_db1。
我再怎么强调(不夸张)这是一个多么糟糕的计划都不为过。您可能在整个开发过程中都对它没有意见,但是可怜的Johnny Nextguy,如果您包含一个也使用全局变量的第三方脚本,那么愿代码之神拯救您……并且变量名之间存在冲突。现在你要为此付出代价了!
正如前面所建议的,最好将数据库功能封装在一个类中。如果您使用静态类,您仍然拥有全局变量的所有好处,而没有污染作用域或覆盖的危险。
下面是一个示例数据库类,以及用法:
// put this in a library file or some place that all scripts include
require_once('database_class.php');
$db = new db(array(
'host'=>'localhost'
'user'=>'db_user_name'
'password'=>'db_password',
'database_name'=>'my_database'
));
// now anywhere in code you want to use it
$array = db::getRows('SELECT id, name FROM users');
$field = db::getField('SELECT name FROM users WHERE id=10');
<?php
// database_class.php
class db {
static protected $resource_link = null;
function __construct($args=false) {
if ($args===false || !is_array($args))
return false;
if (
!isset($args['host']) ||
!isset($args['user']) ||
!isset($args['password']) ||
!isset($args['database_name'])
)
return critical_error('Missing database configuration data.');
self::$resource_link = @mysql_connect($args['host'],$args['user'],$args['password']);
if (!self::$resource_link)
return critical_error('Error connecting to database 2001. MySQL said:<br>'.mysql_error());
@mysql_select_db($args['database_name'], self::$resource_link);
return;
}
// return a single-dimmension array of fields as string
static public function getFields ($sql=false, $field=false) {
$res = null;
if ($sql!==false) {
$query_obj = mysql_query($sql, self::$resource_link);
if ($query_obj) {
$res = array();
while ($this_row = mysql_fetch_array($query_obj)) {
if ($field !== false && isset($this_row[$field]))
$res[] = $this_row[$field];
else
$res[] = $this_row[0];
}
} // end :: if query object is not null
} // end :: if $sql is not false
return $res;
}
// return a single-dimmension array of fields as string with keyfield as key
static public function getKeyFields ($sql=false, $key_field='id', $list_field='id') {
$res = null;
if ($sql!==false) {
$query_obj = mysql_query($sql, self::$resource_link);
if ($query_obj) {
while ($this_row = mysql_fetch_array($query_obj)) {
if (isset($this_row[$key_field]))
$res[$this_row[$key_field]] = $this_row[$list_field];
}
} // end :: if query object is not null
} // end :: if $sql is not false
return $res;
}
// return a single field as string from the first row of results
static public function getField ($sql=false) {
$res = null;
if ($sql!==false) {
$query_obj = mysql_query($sql, self::$resource_link);
if ($query_obj) {
$this_array = mysql_fetch_array($query_obj);
if (is_array($this_array))
return $this_array[0];
} // end :: if query object is not null
} // end :: if $sql is not false
return $res;
}
// return a single row as an array
static public function getRow ($sql=false) {
$res = null;
if ($sql!==false) {
$query_obj = mysql_query($sql, self::$resource_link);
if ($query_obj)
$res = mysql_fetch_assoc($query_obj);
} // end :: if $sql is not false
return $res;
}
// return an array of rows as arrays of strings
static public function getRows ($sql=false) {
$res = null;
if ($sql!==false) {
$query_obj = mysql_query($sql, self::$resource_link);
$res = array();
if ($query_obj) {
while ($this_row = mysql_fetch_assoc($query_obj)) {
$res[] = $this_row;
}
} // end :: if query object is not null
} // end :: if $sql is not false
return $res;
}
// return an array of rows as arrays of strings, using specified field as main array keys
static public function getKeyRows ($sql=false, $key_field='id', $include_key_in_results=true) {
$res = null;
if ($sql!==false) {
$query_obj = mysql_query($sql, self::$resource_link);
if ($query_obj) {
$res = array();
while ($this_row = mysql_fetch_assoc($query_obj)) {
if (isset($this_row[$key_field])) {
$res[$this_row[$key_field]] = $this_row;
if ($include_key_in_results == false)
unset($res[$this_row[$key_field]][$key_field]);
} // end :: if checking for key field in result array
} // end :: while looping query obj
} // end :: if query object is not null
} // end :: if $sql is not false
return $res;
}
// do an update query, return true if no error occurs
static public function update ($sql=false) {
$res = false;
if ($sql!==false) {
$query_obj = mysql_query($sql, self::$resource_link);
if ($query_obj && strlen(mysql_error()) < 1)
$res = true;
}
return $res;
}
// do an insert query, return the auto increment ID (if there is one)
static public function insert ($sql=false) {
$res = null;
if ($sql !== false) {
$query_obj = mysql_query($sql, self::$resource_link);
if ($query_obj)
$res = mysql_insert_id(self::$resource_link);
}
return $res;
}
}
?>发布于 2011-08-17 02:54:43
我将创建一个ConnectionManager类来存储这些连接,并将该类的实例用于任何使用db :)
如果您不想每次都放入参数,那么使用Singleton也是一个好主意。
发布于 2011-08-17 02:55:17
最好的办法是以面向对象的方式声明它们,比如说作为Singleton的属性,然后以这种方式访问它们。如果你坚持以一种会让后续开发人员感到悲伤的方式来做事情,你可以使用$GLOBALS数组。全局变量只是一个糟糕的想法,坦率地说,我希望这个关键字可以从语言中删除。
你可以使用的东西的一个非常简单的版本:
class ConnectionHolder
{
public $mysql_db1;
private static $inst;
public static &getInstance()
{
if( !self::$inst )
self::$inst = new ConnectionHolder();
return self::$inst;
}
private function __construct()
{
$this->mysql_db1 = // ... you may want another nameing convension.
// yada yada
}
}然后,在您的函数中:
$ch =& ConnectionHolder::getInstance()
$ch->mysql_db1;https://stackoverflow.com/questions/7083448
复制相似问题