php mysql : how to avoid multiple connections to database in a Class?

Question

Here is my index.php

<?
require("Annonce.php");
$annonce = new Annonce();
$annonce->id=1;
$annonce->delete();

?>

My Annonce.php is!

<?php 

require("DB.php");
class Annonce extends DB {
}    
 ?>

and finally DB is:

<?php 
$db = new PDO('mysql:host=localhost;dbname=annonces;charset=utf8', 'root', '');
$db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
$db->setAttribute(PDO::ATTR_EMULATE_PREPARES, false);


class DB {
    public static $debug = true;    
    function __construct() {
        # code...
    }
    function delete() {
        $id = $this->id;
        $table = get_class($this);
        $sql="delete from $table where id=$id";     
        if ($this::$debug) echo $sql;

    }

    function get() {
        $sql="select *...";
    }

}
 ?>

I don't know what is the best strategy to define the $db connection ?

If I define it inside DB class, it will make a connection any time an "Annonce" object is created. Maybe using $db as GLOBAL (which I think is not clean)

Any suggestion for that ?

Maybe you will be interested by the Singleton pattern.

Debflav
– Debflav

2014-03-14 10:03:26 +00:00
Commented Mar 14, 2014 at 10:03 — Debflav
– Debflav, Commented Mar 14, 2014 at 10:03
Or the Dependency Injection Container Pattern

Andresch Serj
– Andresch Serj

2014-03-14 10:05:09 +00:00
Commented Mar 14, 2014 at 10:05 — Andresch Serj
– Andresch Serj, Commented Mar 14, 2014 at 10:05

Raymond Cheng · Accepted Answer · 2014-03-14 10:07:58Z

1

Create a Singleton ,some thing like this:

class DB
{
   private $_db;
   private $_instance;
   private function __construct() 
   {

    $this->_db = $db = new PDO('mysql:host=localhost;dbname=annonces;charset=utf8', 'root', '');

}

public static function getInstance()
{
    if(!(self::$_instance instanceof DataBase)){
      $c = __CLASS__;
      self::$_instance = new $c;
    }
    return self::$_instance;
}
//.....other code

}

use it by :

$db = DB::getInstance();

answered Mar 14, 2014 at 10:07

Raymond Cheng

2,51523 silver badges35 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Community · Accepted Answer · 2017-05-23 12:29:38Z

0

You could define your connection as a service and use a Dependency Injuection Container (DIC) to provide the connection for any other feature/module/class needing it by providing the Dependency Injection Container. Pimple is a simple DIC Solution by "the people who brought us Symfony".

Having a DIC is a good idea anyway.

THis might seem "over the top" for your problem (since a Singleton would definetly solve your problem) but considering a DIC will improve your work a lot more. And you asked for a clean Solution. The Singleton Design Pattern is only clean if you do not use Unit Tests since Singletons will get you into trouble with mocking.

Collectives™ on Stack Overflow

php mysql : how to avoid multiple connections to database in a Class?

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related