Здравствуйте! Такой вопрос: у меня есть класс, и много методов, в каждом из которых я подключаюсь к базе данных. Для сокращения количества кода хотелось бы вынести подключение к БД в конструктор класса, а из методов его убрать. Это возможно?
Руслан: лучше будет, если сам класс в конструкторе будет делать коннект и возвращать свой инстанс, а методы там будут делать что-то с базой. Можно вообще сделать свой Singleton, который extends PDO
verdex: в правильном направлении мыслите. НО!!! Большое НО! Ваш основной класс должен быть отделен от логики. Как мухи от котлет. В принципе то что вы делаете - нормально. Рано или поздно вы придете к абстрактным классам и интерфейсам.
Тем не менее. Конструктор должен что-то возвращать, чтобы вы могли позже обращаться к публичным методам класса.
Публичные методы как правило возвращают самих себя. То есть ваш метод getsiteinfo по идее должен записать результат в переменную класса, а уже другими методами вы обрабатываете сам результат запроса.
Но. еще раз скажу. Вы в правильном направлении. Это не говорит, что вы делаете что-то не так. Все верно. Если я вам сейчас скажу как принято - у вас будет каша в голове. А пока делайте так как делаете. Все идет своим чередом.
<?php
/***************************************************************************
* Copyright (C) 2004-2007 by Sveta A. Smirnova *
* *
* This program is free software; you can redistribute it and/or modify *
* it under the terms of the GNU Lesser General Public License as *
* published by the Free Software Foundation; either version 3 of the *
* License, or (at your option) any later version. *
* *
**************************************************************************/
abstract class Singleton
{
private static $instances = array();
protected function __construct() {/* you can't create me */}
/// @example singleton.php
final public static function getInstance(
$class, $args = null /* , ... */
)
{
// for Singleton::getInstance('class_name', $arg1, ...) calling
if (2 < func_num_args())
{
$args = func_get_args();
array_shift($args);
}
if (!isset(self::$instances[$class]))
{
$object =
$args
? new $class($args)
: new $class();
if (!($object instanceof Singleton))
{
throw new Exception(
"Class '{$class}' is something not a Singleton's child"
);
}
return self::$instances[$class] = $object;
}
else
{
return self::$instances[$class];
}
}
final public static function getAllInstances()
{
return self::$instances;
}
final private function __clone() {/* do not clone me */}
}
trevoga_su: редко видел проекты с двумя подключениями к базе... тем не менее у меня 3 коннекда к базе и к ODATA ресурсу. Убрал синглтон и формализую подключения через DBPool
<?php
/***************************************************************************
* Copyright (C) 2006-2007 by Konstantin V. Arkhipov *
* *
* This program is free software; you can redistribute it and/or modify *
* it under the terms of the GNU Lesser General Public License as *
* published by the Free Software Foundation; either version 3 of the *
* License, or (at your option) any later version. *
* *
**************************************************************************/
final class DBPool extends Singleton implements Instantiatable
{
private $default = null;
private $pool = array();
/**
* @return DBPool
**/
public static function me()
{
return Singleton::getInstance(__CLASS__);
}
/**
* @param DBD\DBD) $db
* @return DBPool
*/
public function setDefault(DBD\DBD $db)
{
$this->default = $db;
return $this;
}
/**
* @return DBPool
**/
public function dropDefault()
{
$this->default = null;
return $this;
}
/**
* @param $name
* @param DBD\DBD $db
* @return DBPool
* @throws Exception
*/
public function addLink($name, DBD\DBD $db)
{
if (isset($this->pool[$name]))
throw new Exception(
"DBPool already have '{$name}' link"
);
$this->pool[$name] = $db;
return $this;
}
/**
* @param $name
* @return DBPool
* @throws Exception
*/
public function dropLink($name)
{
if (!isset($this->pool[$name]))
throw new Exception(
"DB link '{$name}' not found"
);
unset($this->pool[$name]);
return $this;
}
/**
* @param null $name
* @return DBD\DBD
* @throws Exception
*/
public function getLink($name = null)
{
$link = null;
// single-DB project
if (!$name) {
if (!$this->default)
throw new Exception(
'DBPool:: i have no default link'
.' and requested link name is null'
);
$link = $this->default;
} elseif (isset($this->pool[$name]))
$link = $this->pool[$name];
if ($link) {
# if (!$link->isConnected())
# $link->connect();
return $link;
}
throw new Exception(
"DBPool: can't find link with '{$name}' name"
);
}
/**
* @return DBPool
**/
public function shutdown()
{
$this->disconnect();
$this->default = null;
$this->pool = array();
return $this;
}
/**
* @return DBPool
**/
public function disconnect()
{
if ($this->default !== null) {
$this->default->disconnect();
}
foreach ($this->pool as $db)
$db->disconnect();
return $this;
}
}
