load config.php with in a class

Question

I want to load a configuration file in a class. Here is the content of config.php

<?php
$__error_reporting_level=1;
$server='localhost';
$user='root';
$password='';
$dbase='eauction';
?>

Content of sql.php

<?php
include ('config.php');
error_reporting($__error_reporting_level);

class sql
{
 function connect()
 {
  $connections = mysql_connect($server, $user,$password) or die ('Unabale to connect to the database');
  mysql_select_db($dbase) or die ('Unable to select database!');
  return;
 }

 function login($email, $pwd)
 {
  $this->connect();
  $result = $this->qry("SELECT uid,nameF FROM user WHERE email='".$email."' AND password='".$pwd."'");
  $row=mysql_fetch_array($result);
  if (mysql_num_rows($result)>0)
   return array($row[0],$row[1]);
  else
   return array(0,0);
 }
}
?>

I execute the code using

include ('core/sql.php');
$obj = new sql;
$data=$obj->login($email,$pwd);
print_r($data);

I get this error

Unable to select database!

Ignore mysql injection issue, I just need to execute the code perfectly

You can't use $server,$user etc in class functions directly . — Duke
– Duke, Commented Feb 28, 2012 at 3:51
why are you all still creating the "database classes" instead of using PDO or MySQLi ( if you live in mysql-only realm ) ? — tereško
– tereško, Commented Feb 29, 2012 at 2:17

Mark Meyer · Accepted Answer · 2016-04-11 02:38:48Z

Why not use a .ini file?

config.ini

server=test
user=root
pass=pass
dbname=mydb

and with in your class have something like

class A {

    public $config;

    public function __construct() {
       $this->config = parse_ini_file('config.ini', true);
    }

    public function sql() {
        $connections = mysql_connect($this->config['server'], $this->config['user'],$this->config['password']) or die ('Unabale to connect to the database');
  mysql_select_db($this->config['dbase']) or die ('Unable to select database!');
  return;
    }
}

just another way of doing it, make sure your database is correctly named too.

Additionally, if you want to use your current config.php then you will need to include in the method you are using the variables in. It cannot be used from outside that scope.

function connect()
 {
    include ('config.php');
  $connections = mysql_connect($server, $user,$password) or die ('Unabale to connect to the database');
  mysql_select_db($dbase) or die ('Unable to select database!');
  return;
 }

Michael Berkowski · Accepted Answer · 2012-02-28 04:07:01Z

Read up on variable scope in the PHP manual.

You've included the file before the class declaration, at global scope, not accessible to the class method scope. If you want to use those variables inside class methods, you'll need to either access them globally via $GLOBALS[] or the global keyword, or better yet, pass them into the function that uses them.

include ('config.php');
// Now all your variables are defined at global scope!!!
error_reporting($__error_reporting_level);

class sql
{
 // Pass as params to the function
 function connect($server, $user, $password, $dbase)
 {
  $connections = mysql_connect($server, $user,$password) or die ('Unabale to connect to the database');
  mysql_select_db($dbase) or die ('Unable to select database!');
  return;
 }
 // etc...
 // etc...
}

You might also consider setting them up as class properties, and passing them in the constructor:

class sql
{
  public $server;
  public $user;
  public $password;
  public $dbase;

  public function __construct($server, $user, $password, $dbase) {
    $this->server = $server;
    $this->user = $user;
    // etc...
    $connections = mysql_connect($this->server, $this->user, $this->password);
  }
}

Jay D. · Accepted Answer · 2012-02-28 03:52:28Z

1

use defined variables in your config.

http://php.net/manual/en/function.define.php

answered Feb 28, 2012 at 3:52

Jay D.

6825 silver badges12 bronze badges

Comments

Eric Aya · Accepted Answer · 2018-02-13 09:22:34Z

1

try this

config.ini
[database]
username = root
password = 1
dbname = users
dsn = localhost
......
class Database{
public $koneksi;
  function __construct(){
    try{
        $config = parse_ini_file('config.ini'); 
        $this->koneksi = new PDO("mysql:host=" . $config['dsn'] . ";dbname=" . $config['dbname'], $config['username'],$config['password'],array(PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION, PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES utf8"));
        $this->koneksi->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
            }catch(PDOException $e){
        $e->getMessage();
    }
  }
}

edited Feb 13, 2018 at 9:22

Eric Aya

70.2k36 gold badges190 silver badges266 bronze badges

answered Feb 13, 2018 at 7:48

Rahmad Saleh

513 bronze badges

Comments

Bert · Accepted Answer · 2012-02-28 03:57:35Z

can do it this way.. the issue on your code must be that the variable you declare in config.php is not accessible. you can make it global or try this code of mine.

<?php
class ConnectionSettings {
    private $hostname = 'localhost';
    private $username = 'root';
    private $password = '';
    private $db = 'cordiac_db';
    protected $connLink;

    // protected 'connect()' method
    protected function connect(){

        // establish connection
        if(!$this->connLink = mysql_connect($this->hostname, $this->username, $this->password)) {
            throw new Exception('Error connecting to MySQL: '.mysql_error());
        }

        // select database
        if(!mysql_select_db($this->db, $this->connLink)) { 
            throw new Exception('Error selecting database: '.mysql_error());
        }
    }
}

Duke · Accepted Answer · 2012-02-28 04:14:16Z

0

Try this

config.php
define('SERVER',$server);
define('USER',$user);
define('PASSWORD',$password);
define('DBASE',$dbase); 



----------class----------     
<?php
include ('config.php');
error_reporting($__error_reporting_level);

class sql
{
 protected  $server=SERVER;
 protected $user=USER;
 protected $password=PASSWORD;
 protected $dbase=DBASE;
 private function connect()
 {
  $connections = mysql_connect($this->server, $this->user,$this->password) or die ('Unabale to connect to the database');
  mysql_select_db($this->dbase) or die ('Unable to select database!');
  return;
 }
........
........

edited Feb 28, 2012 at 4:14

answered Feb 28, 2012 at 3:53

Duke

37.3k13 gold badges57 silver badges74 bronze badges

1 Comment

Sourav Over a year ago

I want to save and load mysql_connect parameter from the config.php so I dont need to update this !

cweiske · Accepted Answer · 2020-03-11 20:40:35Z

0

You need to load the config file in the same scope that you want to use the variables:

class sql
{
    function connect()
    {
        include 'config.php';
        $connections = mysql_connect($server, $user,$password) or die ('Unabale to connect to the database');
        mysql_select_db($dbase) or die ('Unable to select database!');
    }
}

answered Mar 11, 2020 at 20:40

cweiske

31.4k15 gold badges150 silver badges206 bronze badges

Collectives™ on Stack Overflow

load config.php with in a class

7 Answers 7

Comments

Comments

Comments

Comments

Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

7 Answers 7

Comments

Comments

Comments

Comments

Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related