whats wrong in this code ? [i am learning]!

codefreek · 07-02-2008, 06:15 PM

PHP Code:


			
<?php


class db_connect {

private $dbn;
private $user;
private $pass;

public function specs($dbn, $user, $pass) {

$dbtestcon = mysql_connect('$dbn', '$user', '$pass');

if (!$dbtestcon) {
    die('Could not connect: ' . mysql_error());

echo 'Connected successfully';



     }



}

?>

Tanax · 07-02-2008, 06:31 PM

You can't use ' around the variables in the mysql_connection

codefreek · 07-02-2008, 07:11 PM

so how would i fix that instead ?

delayedinsanity · 07-02-2008, 07:17 PM

Just to expand on that, when you're passing arguments, you don't need any quotes at all around variables, ie

PHP Code:


			
mysql_connect($dbn, $user, $pass);

...and in the case of strings, variables won't be parsed inside of single quotes anyways, only inside of double quotes;

PHP Code:


			
$var = 'world!';

echo 'Hello, $var'; // will display Hello, $var
echo "Hello, $var"; // will display Hello, world!

-m

drewbee · 07-02-2008, 07:44 PM

to expand even further on this, your class variables are not being utilized.
These guys:

PHP Code:


			
private $dbn;
private $user;
private $pass;

The variables inside of the function ($dbn, $user, and $pass) are all local to the function, and will no longer be usable outside of the function. You need to assign them to use them outside of it.

Also, constructers are very useful as it will execute as soon as the object is initialized. I ahve the constructor called below to call the two functions. The constructor can be either called __constructor or the exact name of the class in this case: db_connect

PHP Code:


			
<?php
 
 
class db_connect 
{
 
    private $dbn;
    private $user;
    private $pass;
    
    public function db_connect()
    {
        $this->specs('localhost', 'username', 'password');
        $this->showConnectionDetails();
    }
 
    public function specs($dbn, $user, $pass) 
    {
        $this->dbn = $dbn;
        $this->user = $user;
        $this->pass = $pass;
 
        $dbtestcon = mysql_connect($dbn, $user, $pass);
        if (!$dbtestcon) 
        {
            die('Could not connect: ' . mysql_error());
        }
        echo 'Connected successfully';
    }
 
    // Now this function will work
    function showConnectionDetails()
    {
        echo 'We connected on ' . $this->dbn . ' with username: ' . $this->user . ' and password ' . $this->password;
    }
 
}
 
?>

PHP Code:


			
$database_connection = new db_connect();
// Will output:
// 'connected succesfully'
// We connected on 'localhost' with username: 'username' and password 'password'

codefreek · 07-02-2008, 09:35 PM

Thank you i get it now ;) just because it's private i need to keep it on that same page, so it can read the values i get it all..

TY!

codefreek · 07-02-2008, 09:53 PM

PHP Code:


			
<?php
error_reporting(E_ALL & ~E_NOTICE);
 
class db_connect 
{
 
    private $dbn;
    private $user;
    private $pass;
    private $db_selected;
    
    public function db_connect()
    {
        $this->specs('localhost', 'orb', '123123');
        $this->showConnectionDetails();
    }
 
    public function specs($dbn, $user, $pass, $db_selected) 
    {
    $this->db_selected = $db_selected;
        $this->dbn = $dbn;
        $this->user = $user;
        $this->pass = $pass;
 
        $dbtestcon = mysql_connect($dbn, $user, $pass);
        if (!$dbtestcon) 
        {
            die('Could not connect: ' . mysql_error());
        }
        echo 'Connected successfully';
    }
 
    // Now this function will work
    function showConnectionDetails()
    {
     
        $db_selected = mysql_select_db('zone', $dbtestcon);
    if (!$db_selected)
    die ('Can\'t use workspace : ' . mysql_error());

    }
 
}

?>

what am i doing wrong with this snippet of code??..

drewbee · 07-02-2008, 09:56 PM

Quote:

Thank you i get it now ;) just because it's private i need to keep it on that same page, so it can read the values i get it all..
TY!

Making it privage gives you the ability to access the variables inside of the class, IE $this->. However, you would not be able to get it outside of the class lets say after it has been initialized.

$this->dbn would return results inside of the class,

$database_connection = new db_connection();
echo $database_connection->dbn; Should throw some type of error.

drewbee · 07-02-2008, 10:04 PM

PHP Code:


			
// A few things... see comments in code
<?php
error_reporting(E_ALL & ~E_NOTICE);
 
class db_connect 
{
 
    private $dbn;
    private $user;
    private $pass;
    private $db_selected;
    
    public function db_connect()
    {
        // you added a 4th parameter to specs. you would need to pass it here
        // public function specs($dbn, $user, $pass, $db_selected)
        $this->specs('localhost', 'orb', '123123');
        $this->showConnectionDetails();
    }
 
    public function specs($dbn, $user, $pass, $db_selected) 
    {
        $this->db_selected = $db_selected;
        $this->dbn = $dbn;
        $this->user = $user;
        $this->pass = $pass;
 
        $dbtestcon = mysql_connect($dbn, $user, $pass);
        if (!$dbtestcon) 
        {
            die('Could not connect: ' . mysql_error());
        }
        echo 'Connected successfully';
    }
 
    // Now this function will work
    function showConnectionDetails()
    {
         // We have no idea what the variable 'dbtestcon' is. Remember scope. 
        //  $dbtestcon scope is limited to the function spec();
        // You can handle this two ways: 1) ommit the identifier (php is smarty enough to know which
        // connection you are referring to so you could use:
        // $db_selected = mysql_select_db('zone');
        // Option two is to create a class variable for $dbtestcon
        // 1) create variable:  private $dbtestcon;
        // 2) in function specs() when doing the initial mysql_connect(), assign it to the new var:
        //    $this->dbtestcon = mysql_connect($dbn, $user, $pass);
        // 3) Reference it below
        //    $db_selected = mysql_select_db('zone', $this->dbtestcon);
        $db_selected = mysql_select_db('zone', $dbtestcon);
        
        // Also, one other note. Since you added the variable 'db_selected', you should use that as well IE
        // $db_selected = mysql_select_db($this->db_selected, $this->dbtestcon);
            if (!$db_selected)
        {
            die ('Can\'t use workspace : ' . mysql_error());
        }

    }
 
}

?>

codefreek · 07-03-2008, 06:28 AM

whats wrong now :S?

Quote:

//i know i am outputting the value the wrong way i think..

PHP Code:


			
<?php
error_reporting(E_ALL & ~E_NOTICE);

class db_connect 
{
 
    private $dbn;
    private $user;
    private $pass;
    private $db_selected;
    private $dbtestcon;

    
    public function db_connect()
    {

        
        $this->specs('localhost', 'orb', '123123');
        $this->showConnectionDetails('zone');
    
    
    }
 
    public function specs ($dbn, $user, $pass) 
    {
        
        $this->dbn = $dbn;
        $this->user = $user;
        $this->pass = $pass;

        
        $this->dbtestcon = mysql_connect($dbn, $user, $pass);

        if ( ! $this->$dbtestcon) 
        {
            die('Could not connect: ' . mysql_error());
        }

        echo 'Connected successfully';
    } 


    // Now this function will work
    function showConnectionDetails($db_selected)
    {
    $this->db_selected = $db_selected;
    
    $db_selected = mysql_select_db($this->db_selected, $this->dbtestcon);
            if (!$db_selected)
        {
            die ('Can\'t use workspace : ' . mysql_error());
        }
        echo 'db_selected';
    }
 
}

?>

delayedinsanity · 07-03-2008, 07:12 AM

This part here:

PHP Code:


			
    public function specs($dbn, $user, $pass) 
    {
        
        $this->dbn = $dbn;
        $this->user = $user;
        $this->pass = $pass;
     $this->dbtestcon = $dbtestcon;
     
    
        $dbtestcon = mysql_connect($dbn, $user, $pass);
    
        if (!$dbtestcon) 
        {
            die('Could not connect: ' . mysql_error());
        }
        echo 'Connected successfully';
    }

Should be written:

PHP Code:


			
    public function specs ($dbn, $user, $pass) 
    {
        
        $this->dbn = $dbn;
        $this->user = $user;
        $this->pass = $pass;

        // You were originally trying to assign an empty variable, $dbtestcon to $this->dbtestcon
        $this->dbtestcon = mysql_connect($dbn, $user, $pass);

        // You need to use $this to assign properties to a global reference inside your class
        // Any variables that are called within a function directly (such as $dbtestcon as
        // opposed to $this->dbtestcon) are local to that function and do not exist in the global
        // scope of a class.
        if ( ! $this->$dbtestcon) 
        {
            die('Could not connect: ' . mysql_error());
        }

        echo 'Connected successfully';
    }

-m

codefreek · 07-03-2008, 07:22 AM

it is still not working...
.."No database selected"..

Tanax · 07-03-2008, 07:25 AM

Meaning: You didn't select a database.. ?

? So select a database

codefreek · 07-03-2008, 08:09 AM

so what you call this then ?
$this->showConnectionDetails('zone');
Read the code ;)