Как вынести подключение к БД?

Question

[Twelfth Doctor]

Здравствуйте! Такой вопрос: у меня есть класс, и много методов, в каждом из которых я подключаюсь к базе данных. Для сокращения количества кода хотелось бы вынести подключение к БД в конструктор класса, а из методов его убрать. Это возможно?

Answer 1

[Руслан Полин]

Можно, только в конструктор лучше передавать подключение к бд, а не выносить туда его.

public function __construct($pdo) {
		$this->pdo = $pdo;
}

Comments

[Twelfth Doctor]

Не совсем понял, о чём Вы. Т.е. где-то устанавливаем подключение к БД, а затем уже созданное подключение передавать в конструктор класса?

[Руслан Полин]

verdex: именно или как выше посоветовали использовать Singleton

[Falseclock]

Руслан: лучше будет, если сам класс в конструкторе будет делать коннект и возвращать свой инстанс, а методы там будут делать что-то с базой. Можно вообще сделать свой Singleton, который extends PDO

[Twelfth Doctor]

Falseclock: То есть получается вот так:
Конструктор класса:

//constructor
    public function __construct($login, $id = 0, $mainname = 0, $charset = 0, $email = 0, $index = 0)
    {
        $this->id = $id;
        $this->userid = $login;
        $this->mainname = $mainname;
        $this->charset = $charset;
        $this->email = $email;
        $this->index = $index;
        //connect to database
        $dbhost = "127.0.0.1";
        $dbuser = "***";
        $dbpass = "****";
        $dbname = "****";
        $connect = new PDO("mysql:host=$dbhost;dbname=$dbname", $dbuser, $dbpass , [PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION]);
    }

Метод класса:

public function getsiteinfo($login, $id)
    {
      
        $getquery = $connect->prepare("SELECT * FROM `***` WHERE `userid` = :login AND `unid` = :unid");
        $getquery->execute(array(":login" => $login,  ":unid" => $id));
        $info = $getquery->fetch_assoc();
        return $info;
    }

[Falseclock]

verdex: в правильном направлении мыслите. НО!!! Большое НО! Ваш основной класс должен быть отделен от логики. Как мухи от котлет. В принципе то что вы делаете - нормально. Рано или поздно вы придете к абстрактным классам и интерфейсам.

Тем не менее. Конструктор должен что-то возвращать, чтобы вы могли позже обращаться к публичным методам класса.

Публичные методы как правило возвращают самих себя. То есть ваш метод getsiteinfo по идее должен записать результат в переменную класса, а уже другими методами вы обрабатываете сам результат запроса.

Но. еще раз скажу. Вы в правильном направлении. Это не говорит, что вы делаете что-то не так. Все верно. Если я вам сейчас скажу как принято - у вас будет каша в голове. А пока делайте так как делаете. Все идет своим чередом.

[Twelfth Doctor]

Единственное, что меня напрягает - PHPStorm подчёркивает переменную $connect и говорит, что она не определена.

[Twelfth Doctor]

Проблему уже решил в другом вопросе.

Answer 2

[Никита]

Да, можно в синглон закинуть подключение к бд

Comments

[Twelfth Doctor]

что такое синглон?

[Falseclock]

verdex: Singleton правильней

[Никита]

verdex: в гугле набери синглтон по примере подключения к бд - первая ссылка полный пример

[Falseclock]

verdex:

<?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 */}
}

[Никита]

Falseclock: сорян, опечатался

[trevoga_su]

Falseclock: этот код -- говнокод
а если нужно к 2 базам подключение сделать?

[Falseclock]

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;
	}
}

Answer 3

[trevoga_su]

на те мою оберточку www.database.phpinfo.su