[php] Konstruktor inicjujący połączenie z bazą Opcje

[pjamorski]

Witam,

Próbuje pisać swoją klasę na której chcę popracować z bazą danych. Oto ona:

[PHP] pobierz, plaintext 
class MySQL_class
{
        // fields
        var $server;
	var $username;
	var $password;  
 
        /* Database fields */
        var $db_name;
        var $db_connection;
        var $db;
        var $row;
 
        /* Table user fields */
        var $nick;
        var $passwd;
 
	// constructor
	function _construct()
	{
	    $this->server="localhost";
	    $this->username="root";
	    $this->password="krasnal";
	    $this->db_connection = mysql_connect($this->server,$this->username,$this->password) or die("Connection failed");
	    $this->db = mysql_select_db("baza") or die("Database selection failed");
        }        
	// properties SET / GET
	function Set_nick($nickname)
	{
	    $this->nick = $nickname;	
	} 
	function Set_password($password)
	{
	    $this->passwd = $password;
	}
	function Get_nick()
	{
	    return $this->nick;	
	} 
	function Get_password()
	{
	    return $this->passwd;
	}
 
	// functions
	function Login()
	{
 
 
	    $nick = $this->nick;
	    $haslo = $this->passwd;
	    $tmp1;
	    $tmp2;
	    $querry = "SELECT *FROM users WHERE nick='$nick' AND passwd='$haslo'";
	    $equal = mysql_query($querry) or die("Error DB querry");
	    while($this->row = mysql_fetch_array($equal))
	    {
	        $tmp1[] = $this->row['nick'];
	        $tmp2[] = $this->row['passwd'];
	        $licznik++;
	    }
	    if($nick && $haslo)
	    {
	        for($i=0;$i<$licznik;$i++)
	        {
	 	    echo '<br>'.$tmp1[$i].' '.$tmp2[$i];
		    echo '<br> JESTES ZALOGOWANY';	
		    echo '<a href="table.html"> start </a>';
		}
		echo '<br><b>Wszystkie operacje identyfikacji zostały zakończone: </b>'.$licznik;
	    }
	    else
	    {
		echo 'NIE MOZESZ SIE ZALOGOWAC';
	    }
	}
}
[PHP] pobierz, plaintext 

Funkcję Loguj() i jakieś watości, obiekt uruchamiam tak, w osobnym pliku "loguj.php":

[PHP] pobierz, plaintext 
<?php
	require_once('sql_class.php');
	$object = new MySQL_class();
	$object->Set_nick("user");
	$object->Set_password('dddd');
	$object->Login();
?>
[PHP] pobierz, plaintext 

Zadada wydaje mi się jest ok, ale konstruktor nie inicjuje mi tych rzęczy które ma inicjować, przez to mam problem z pracą na bazie, wiadomo jak nie ma z nią połączenia to nic się nie zrobi. Jak zrobię kopiuj w klej zawartość konstruktora do funcji Loguj() to działa normalnie. Jak w PHP powinno w moim przypadku wyglądać ciało konstruktora?

dzięki z góry

Ten post edytował pjamorski 15.04.2010, 19:43:19

[mortus]

Nie _construct, ale __construct (dwa znaki podkreślenia). Wydaje mi się, że do logowania powinna służyć jednak inna klasa, której możemy przekazać instancję obiektu obsługującego bazę danych.

Poza tym takie dane jak host, nazwa użytkownika, hasło i może jeszcze nazwę bazy danych powinniśmy raczej przekazać jako parametry w konstruktorze. Dlaczego? Ano dlatego, że każdy może mieć inny host, inną nazwę użytkownika i inne hasło, a dane może przechowywać w innej tabeli.

Ten post edytował mortus 15.04.2010, 19:45:46

[pjamorski]

Też nie wykonuje swojej funkcji właściwie. W dalszym ciągu ten sam problem. Wywala mi komunikat

[PHP] pobierz, plaintext 
$equal = mysql_query($querry) or die("Error DB querry");
[PHP] pobierz, plaintext 

. Jak zawartośc konstruktora jest w funkcji dziaął poprawnie.

[mortus]

To zastąp tę linię

[PHP] pobierz, plaintext 
$equal = mysql_query($querry) or die("Error DB querry");
[PHP] pobierz, plaintext 

w funkcji Login() na:

[PHP] pobierz, plaintext 
$equal = mysql_query($querry) or die(mysql_error());
[PHP] pobierz, plaintext 

bo wygląda na to, że zapytanie się nie wykonuje, a tak dowiemy się, jaka jest przyczyna.

Ten post edytował mortus 15.04.2010, 20:05:02

[pjamorski]

Nie ma żadnego error'a. Właśnie nie wiem co jest grane.

[luck]

[SQL] pobierz, plaintext 
SELECT * [brak spacji] FROM
[SQL] pobierz, plaintext 

[pjamorski]

Tutaj mam dobrze.

Konstruktor z parametrami też zrobiłem:
/

[PHP] pobierz, plaintext 
/ constructor
	function __construct($server, $username, $password)
	{
	    $server="localhost";
	    $username="root";
	    $password="krasnal";
	    $this->db_connection = mysql_connect($server,$username,$password) or die("Connection failed") or die(mysql_error());
	    $this->db = mysql_select_db("baza") or die("Database selection failed") or die(mysql_error());   
        }  
[PHP] pobierz, plaintext 

Tak go uruchamiam i tą funkcję i w dalszym ciągu to samo:

[PHP] pobierz, plaintext 
    require_once('sql_class.php');
	$object = new MySQL_class("localhost","root","krasnal");
	$object->Login();
[PHP] pobierz, plaintext 

Jak zrobię tak to jest dobrze:

[PHP] pobierz, plaintext 
function Login()
	{
                $server="localhost";
	    $username="root";
	    $password="krasnal";
	    $this->db_connection = mysql_connect($server,$username,$password) or die("Connection failed") or die(mysql_error());
	    $this->db = mysql_select_db("baza") or die("Database selection failed") or die(mysql_error());  
	    $nick = $this->nick;
	    $haslo = $this->passwd;
	    $tmp1;
	    $tmp2;
	    $querry = "SELECT * FROM users WHERE nick='$nick' AND passwd='$haslo'";
	    $equal = mysql_query($querry) or die("Error DB querry") or die(mysql_error());
	    while($this->row = mysql_fetch_array($equal))
	    {
	        $tmp1[] = $this->row['nick'];
	        $tmp2[] = $this->row['passwd'];
		$licznik++;
	    }
	    if($nick && $haslo)
	    {
	        for($i=0;$i<$licznik;$i++)
	        {
	 	    echo '<br>'.$tmp1[$i].' '.$tmp2[$i];
		    echo '<br> JESTES ZALOGOWANY';	
		    echo '<a href="table.html"> start </a>';
		}
		echo '<br><b>Wszystkie operacje identyfikacji zostały zakończone: </b>'.$licznik;
	    }
	    else
	    {
		echo 'NIE MOZESZ SIE ZALOGOWAC';
	    }
	}
[PHP] pobierz, plaintext 

Normalnie zgupłem

[MateuszS]

Ale zobacz, po co przekazujesz parametry konstruktorowi skoro potem w konstruktorze na nowo przypisujesz dane do bazy? Poza tym jeszcze

[PHP] pobierz, plaintext 
  $this->db = mysql_select_db("baza", $this->db_connection) or die("Database selection failed") or die(mysql_error());  
[PHP] pobierz, plaintext 

może się przydać

No i przed wywołaniem metody logującej, ustaw parametry.

Ten post edytował MateuszScirka 15.04.2010, 20:20:31

[pjamorski]

[PHP] pobierz, plaintext 
// constructor
	function __construct($server, $username, $password)
	{
	    $this->db_connection = mysql_connect($server,$username,$password) or die("Connection failed") or die(mysql_error());
	    $this->db = mysql_select_db("itworld_cba_pl",$this->db_connection) or die("Database selection failed") or die(mysql_error());   
        }        
	//
[PHP] pobierz, plaintext 

Niestety dalej to samo, czyli nic.
Jak powinien wyglądać konstruktor inicjujący połączenie z bazą?

[luck]

A tak:

[PHP] pobierz, plaintext 
mysql_query($querry, $this->db_connection)
[PHP] pobierz, plaintext 

albo i

[PHP] pobierz, plaintext 
mysql_query($querry, $this->db)
[PHP] pobierz, plaintext 

Choć to drugie to tak na słowo honoru - nie pamiętam czy działało

Ten post edytował luck 15.04.2010, 20:28:51

[pjamorski]

Warning: mysql_query(): supplied argument is not a valid MySQL-Link resource in c:\usr\krasnal\www\games\sql_class.php on line 52
Error DB querry

[MateuszS]

Najlepiej napisz do obsługi db osobną klasę lub skorzystaj z Mysqli (gotowa klasa). Ewentualnie, masz tu taki prosty przykład

[PHP] pobierz, plaintext 
<?
 
class mysql
{
	protected $dbHost;
	protected $dbUsername;
	protected $dbPassword;
	protected $dbBasename;
 
	public function __construct($dbHost, $dbUsername, $dbPassword, $dbBasename)
	{
		$connect = mysql_connect($dbHost, $dbUsername, $dbPassword) or die(mysql_error());
		$db = mysql_select_db($dbBasename, $connect) or die(mysql_error());
	}
}
 
$obMysql = new mysql("localhost", "root", "pass", "baza");
 
?>
 
 
[PHP] pobierz, plaintext 

[mortus]

Spróbuj jeszcze tak:

[PHP] pobierz, plaintext 
class MySQL_class {
    // fields
    var $server;
    var $username;
    var $password;
 
    /* Database fields */
    var $db_name;
    var $db_connection;
    var $db;
    var $row;
 
    /* Table user fields */
    var $nick;
    var $passwd;
 
    // constructor
    function __construct($host, $username, $password) {
        $this->server = $host;
        $this->username = $username;
        $this->password = $password;
        $this->db_connection = mysql_connect($this->server, $this->username, $this->password) or die("Connection failed. Error: " . mysql_error());
        $this->db = mysql_select_db("baza") or die("Database selection failed. Error " . mysql_error());
    }
    // properties SET / GET
    function Set_nick($nickname) {
        $this->nick = $nickname;
    }
    function Set_password($password) {
        $this->passwd = $password;
    }
    function Get_nick() {
        return $this->nick;
    }
    function Get_password() {
        return $this->passwd;
    }
 
    // functions
    function Login() {
 
 
        $nick = $this->nick;
        $haslo = $this->passwd;
        $tmp1;
        $tmp2;
        $querry = "SELECT * FROM users WHERE nick='$nick' AND passwd='$haslo'";
        $equal = mysql_query($querry) or die("Error DB querry. Error: " . mysql_error());
        while ($this->row = mysql_fetch_array($equal)) {
            $tmp1[] = $this->row['nick'];
            $tmp2[] = $this->row['passwd'];
            $licznik++;
        }
        if ($nick && $haslo) {
            for ($i = 0; $i < $licznik; $i++) {
                echo '<br>'.$tmp1[$i].' '.$tmp2[$i];
                echo '<br> JESTES ZALOGOWANY';
                echo '<a href="table.html"> start </a>';
            }
            echo '<br><b>Wszystkie operacje identyfikacji zosta�y zako�czone: </b>'.$licznik;
        } else {
            echo 'NIE MOZESZ SIE ZALOGOWAC';
        }
    }
}
[PHP] pobierz, plaintext 

[luck]

Ja bym do klasy Mateusza dołożył jeszcze coś takiego i powinno zagadać:

[PHP] pobierz, plaintext 
class Login {
  private $db;
 
  public function __construct($objMysql) {
    $this->db = $objMysql;
  }
 
  /* reszta metod logowania */
}
[PHP] pobierz, plaintext 

[pjamorski]

Teraz wywala

[PHP] pobierz, plaintext 
Error DB querry. Error: Nie wybrano żadnej bazy danych
[PHP] pobierz, plaintext 

Funcja . mysql_error()); przynajmniej zadziałała. Nazwa bazy jest poprawna. W dalszym ciągu nie działa.

[luck]

Wiesz, może wklej jak to teraz u Ciebie wygląda, bo się powoli pogubimy...

[pjamorski]

Klasa:

[PHP] pobierz, plaintext 
<?php
class MySQL_class {
 
    // fields
    var $server;
    var $username;
    var $password;
 
    /* Database fields */
    var $db_name;
    var $db_connection;
    var $db;
    var $row;
 
    /* Table user fields */
    var $nick;
    var $passwd;
 
    // constructor
    function __construct($host, $username, $password) {
        $this->server = $host;
        $this->username = $username;
        $this->password = $password;
        $this->db_connection = mysql_connect($this->server, $this->username, $this->password) or die("Connection failed. Error: " . mysql_error());
        $this->db = mysql_select_db("baza") or die("Database selection failed. Error " . mysql_error());
    }
    // properties SET / GET
    function Set_nick($nickname) {
        $this->nick = $nickname;
    }
    function Set_password($password) {
        $this->passwd = $password;
    }
    function Get_nick() {
        return $this->nick;
    }
    function Get_password() {
        return $this->passwd;
    }
 
    // functions
    function Login() {
        $nick = $this->nick;
        $haslo = $this->passwd;
        $tmp1;
        $tmp2;
        $querry = "SELECT * FROM users WHERE nick='$nick' AND passwd='$haslo'";
        $equal = mysql_query($querry) or die("Error DB querry. Error: " . mysql_error());
        while ($this->row = mysql_fetch_array($equal)) {
            $tmp1[] = $this->row['nick'];
            $tmp2[] = $this->row['passwd'];
            $licznik++;
        }
        if ($nick && $haslo) {
            for ($i = 0; $i < $licznik; $i++) {
                echo '<br>'.$tmp1[$i].' '.$tmp2[$i];
                echo '<br> JESTES ZALOGOWANY';
                echo '<a href="table.html"> start </a>';
            }
            echo '<br><b>Wszystkie operacje identyfikacji zosta?y zako?czone: </b>'.$licznik;
        } else {
            echo 'NIE MOZESZ SIE ZALOGOWAC';
        }
    }
}
?>
[PHP] pobierz, plaintext 

plik loguj.php

[PHP] pobierz, plaintext 
<?php
	require_once('sql_class.php');
	$object = new MySQL_class("localhost","root","krasnal");
	$object->Login();
?>
[PHP] pobierz, plaintext 

Ten post edytował pjamorski 15.04.2010, 20:50:45

[mortus]

Proponuje w funkcji login na samym początku dopisać:

[PHP] pobierz, plaintext 
echo '<pre>'; print_r($this); echo '</pre>';
[PHP] pobierz, plaintext 

[pjamorski]

To dostalem w odpowiedzi:

[PHP] pobierz, plaintext 
mysql_class Object
(
    [server] => 
    [username] => 
    [password] => 
    [db_name] => 
    [db_connection] => 
    [db] => 
    [row] => 
    [nick] => 
    [passwd] => 
)
Error DB querry. Error: Nie wybrano żadnej bazy danych
[PHP] pobierz, plaintext 

[mortus]

A testujesz to na hostingu, czy na localhoscie? Jeśli na localhoście, to czy na pewno na PHP 5, bo Krasnal Serv może się przełączać pomiędzy PHP4... i PHP5.0.2.

Z wydruku z powyższego postu wnioskuję, że obiekt nie jest w ogóle tworzony, po prostu go nie ma. Dlatego stawiam na to, że masz włączone PHP4.3.9.