Forum PHP.pl

Witam, chciałem zapytać czy jest jakieś polecenie/wywołanie do zamknięcia połączenia przy PDO i mysql ?

[PHP] pobierz, plaintext 
function dbInit(){
	$dbs=new PDO('mysql:host='.$DBVARS['hostname'].';port='.$DBVARS['port'].';dbname='.$DBVARS['db_name'],$DBVARS['username'],$DBVARS['password']);
	$dbs->query('SET NAMES utf8');
	$dbs->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
	$dbs->num_queries=0;
	$GLOBALS['db']=$dbs;
	return $dbs;
}
function dbQuery($query){
	$dbs=dbInit();
	$q=$dbs->query($query);
	$dbs->num_queries++;
	return $q;
}
function dbRows($query) {
	$q = dbQuery($query);
	//print_r($dbs->errorInfo());
	return $q->fetchAll(PDO::FETCH_ASSOC);
}
[PHP] pobierz, plaintext 

Znalazłem takie coś w skryptach/książkach, i bardzo ciekawie się to sprawdza tylko że niestety niewiem czemu generuje to 25tyś połączeń przy 20użytkownikach.. Czy ktoś jest mi wstanie pomóc/ przerobić to tak żeby nie było połączenia/inicjacji bazy przy każdym dbQuery() ? Ew czy to można jakoś zamknąć ?

http://www.php.net/manual/en/pdostatement.closecursor.php

tzn mam dołożyć $dbs->closeCursor(); ?

Radzę w ogóle zmienić podejście i ten antypattern zamienic na coś rozsądniejszego np:

[PHP] pobierz, plaintext 
<?php
class PdoFactory {
	private static $pdoInstances = array();
 
	private function __construct() {}
 
	/**
	 * @param string $key
	 * @throws PDOException
	 * @throws Exception
	 */
	private static function set($key = null) {
		try {
			//Tutaj wstaw fragment kodu ktory ustawi odpowiednie wartosci dla $dsn, $user, $password
 
			$pdo = new PDO($dsn, $user, $password);
			$pdo->setAttribute( PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION );
			$pdo->setAttribute( PDO::ATTR_DEFAULT_FETCH_MODE, PDO::FETCH_ASSOC);
			self::$pdoInstances[$key] = $pdo;
		} catch (PDOException $e) {
			echo 'Connection failed: '.$e->getMessage();
			throw $e;
		} catch (Exception $e) {
			echo $e->getMessage();
			throw $e;
		}
	}
 
	/**
	 * @param string $key
	 * @return PDO
	 */
	public static function get($key) {
		if (!isset(self::$pdoInstances[$key])) {
			self::set($key);
		}
		return self::$pdoInstances[$key];
	}
 
	public static function closeAll() {
		foreach ( self::$pdoInstances as $key => $pdo ) {
			self::$pdoInstances[$key] = null;
		}
	}
}
?>
[PHP] pobierz, plaintext 

Obiekt klasy PDO pobierasz przez PdoFactory::get('nazwaPolaczenia');
Jeśli korzystasz tylko z jednej bazy danych możesz zmodyfikować skrypt tak żeby nie trzeba było podawać w ogóle parametru do metody PdoFactory::get()
Przed zakończeniem skryptu lub w momencie kiedy jesteś pewien że nie będziesz już potrzebował komunikacji z bazą danych pamiętaj o wywołaniu PdoFactory::closeAll()

A mogłbyś jeszcze powiedzieć jak teraz przez to pobrać dane ?

Zamiast tworzyć jakieś bzdurne klasy korzystaj z PDO takim jakie jest. http://pl.wikibooks.org/wiki/PHP/Biblioteka_PDO

Trzymając się kodu z pierwszego posta i wykorztując klasę PdoFactory którą Ci podałem mogłoby wyglądać to np tak:

[PHP] pobierz, plaintext 
function dbQuery($query){
$dbs=PdoFactory::get('identyfikatorPołączenia');
//lub $dbs=PdoFactory::get(); - w zależności czy chcesz obsługiwać kilka połączeń czy tylko jedno
$q=$dbs->query($query);
$dbs->num_queries++;
return $q;
}
function dbRows($query) {
$q = dbQuery($query);
//print_r($dbs->errorInfo());
return $q->fetchAll();
}
[PHP] pobierz, plaintext 

Funkcja dbInit() nie jest Ci już potrzebna. Obiekty klasy PDO pobierasz teraz używając klasy PdoFactory. Zaletą tego jest to że PdoFactory::get() nie będzie łączyć się za każdym razem z bazą danych. Zrobi to raz a potem będzie już zwracane wcześniej utworzony obiekt klasy PDO.

Widziałem tą strone, tylko że $stmt -> closeCursor(); które tutaj jest podane powoduje u mnie błąd Call to undefined method PDO::closeCursor() '


@Methestel
To zwykłe pdo nie potrafi zrobić jednego połączenia ?

Znalazłem coś takiego:
$dbs->db = null;

Czy jak wstawie takie coś do dbquery i dbrows na koniec to połączenie zostanie zamknięte po każdym wczytaniu/zmianie ?

Ten post edytował Mayka 10.12.2013, 19:32:03

Po co Ci to manualne zamykanie połączenia?

Jak już to $dbs = null;

Ten post edytował Damonsson 10.12.2013, 20:10:15

Zwykłe PDO jak najbardziej potrafi zrobić jedno połączenie. Tylko chodzi o to żeby nie robił tych połączeń przy każdym zapytaniu tylko maksymalnie raz podczas jednego requesta. Nie wiem jak to prościej wyjaśnić ... może wyobraź sobie że masz do wykonania 100 zapytań do bazy danych.
Masz do wyboru:
- przed każdym zapytaniem tworzysz nowe połączenie z bazą danych i wykonujesz zapytanie (łącznie wykonujesz 100 połączeń z bazą danych i 100 zapytań)
- tworzysz jedno połączenie z bazą danych i używając tylko tego jednego połączenia wykonujesz kolejno 100 zapytań (łącznie 1 połączenie z bazą danych i 100 zapytań)

Efekt końcowy będzie taki sam. Jak myślisz, które rozwiązanie jest lepsze?


Tak, jak wstawisz $dbs = null na koniec dbquery to połączenia będą zamykane - zresztą masz to jak wół napisane w dokumentacji http://www.php.net/manual/en/pdo.connections.php
Jeśli nie przeszkadza Ci że przy każdym zapytaniu będziesz od nowa łączył się z bazą danych to nie ma problemu.

Ten post edytował Methestel 10.12.2013, 20:26:32

tak jak napisał @Methestel problemem w Twoim skrypcie nie jest zamykanie połączeń ale ich nadmierne "otwieranie" - a podejrzewam, że Twoim jest to

Jeśli przy każdym zapytaniu nawiązujesz połączenie z bazą przez dbInit() to jest bezsensu. Jak piszesz w OOP to w konstruktorze nawiązujesz połączenie z bazą i później tylko się odwołujesz przez np. $this->db, albo tworzysz instancję new PDO i działasz na tym w całym dokumencie.

A jak nie pisze w oop ?
To jak to zrobić żeby połączenie było raz wykonywane a nie przy każdym requescie ?



A czemu ma mi to przeszkadzać bo nie rozumiem ?

przecież @Methestel Ci napisał jak tu



może twój serwer będzie miał coś przeciwko ;-) ? pewnie masz apache , to zrób sobie test wydajności twojego rozwiązania i tego z jednym inicjowanym połączeniem (np. ab -n 1000 -c 10 )

Sama się pytasz w pierwszym poście jak zrobić "żeby nie było połączenia/inicjacji bazy przy każdym dbQuery()" a teraz pytasz dlaczego łączenie się z bazą przy każdym zapytaniu ma Ci przeszkadzć :/

Sorry ale ja się poddaje.

Hahaha, chodziło mi bardziej o to, że teraz wiem jak je rozłączyć wiec moge przy każdym pobraniu rozłączyć z bazą i wtedy nie bedzie przekroczenia ilości jednoczesnych połączeń.


Nie zrobie bo niemam dostępu do tego serwera -zwykły hosting w nazwie.

Czyli dobrze rozumiem że trzeba zrobić to w oop bo inaczej bedzie za każdym dbquery łączył z baza ?

Aha i PdoFactory::closeAll() na samym końcu strony tak ?

Ten post edytował Mayka 11.12.2013, 18:06:21

Żadne OOP.

[PHP] pobierz, plaintext 
$db = new PDO('mysql:host=;dbname=', '', '', array(PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES 'utf8'")) or die();
[PHP] pobierz, plaintext 

Dalej już wszystkimi zapytaniami operujesz na $db, np. $db->query(...)

powiedz szczerze, nie masz pojęcia co ta klasa PdoFactory robi ?

a z ciekawości po co robisz to Twoje dbquery wstaw więcej kodu jak możesz