1

让我先解释一下。我有一些全局数据库连接,并有一些简单的功能,它们使用每个连接并执行查询等。

因为我想不止一次地使用这些连接,并且为了节省我每次在每个函数中定义它们的时间,我已经将它们创建为文档顶部的全局变量。但是我只是想知道,而不是不得不写

global $mysql_db1, $mysql_db2, $mysql_db3, $mysql_db4, $mysql_db5;

有没有办法让我不必每次都复制和粘贴它?

我知道这很琐碎,但我只是想加快自己的发展,

4

4 回答 4

3

我会创建一个类 ConnectionManager 来存储这些连接并使用该类的实例来使用 db 的任何函数:)

如果您不想每次都输入参数,那么使用 Singleton 也是一个好主意。

于 2011-08-16T18:54:42.670 回答
1

最好的办法是以面向对象的方式声明它们,比如作为 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;
于 2011-08-16T18:55:16.777 回答
0

全局变量通常被认为是不好的做法。我不会对此高谈阔论,但请查看这篇文章: 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-16T19:01:35.683 回答
0

直接回答你的问题:不。

对您的问题的实际回答指出,全局变量并不是一个热门的想法。请注意,我认为那里的大多数论点都是无效的,并且围绕着其他人说“ewww,这太恶心了!” 或“你会释放魔鬼!”。

然而,他们都围绕着这个问题:如果你的程序的两个部分试图将同一个全局变量$foo用于不同的目的(即,他们不小心决定使用相同的名称),则不能保证会发生错误。这就是为什么建议使用不同的存储方式,比如一个类,因为两个类命名Foo肯定会报错。

在您的程序上下文中,可能会建议创建一个ConnectionManager单例,您可以在其中通过编写类似$conn1 = ConnectionManager::getConnection('conn1');.

于 2011-08-16T19:15:43.387 回答