Forum PHP.pl > Problemy z napisaniem klasy

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: Problemy z napisaniem klasy

Forum PHP.pl > Forum > Przedszkole

Gość_KRCreater

11.09.2005, 19:53:41

Wtiam!
Właśnie ucze się programowania obiektowego. Strukturalne mi się znudziło

, a po za tym jestem głudny wiedzy. Moje pytanie brzmi hehe dlaczego mi to niedziała

Tzn. zmienne z pliku config.php nie pojawiają się w klasie :/

klasa sql

[PHP] pobierz, plaintext 
<?php
include("config.php");
 
 
   class sql{
 
 
	 // DANE PODTRZEBNE DO POLACZENIA
	 var $sql_host;
	 var $sql_admin;
	 var $sql_pass;
	 var $sql_db;
 
 
 
	 // POLACZENIE Z BAZA DANYCH MYSQL
	 function sql(){
 
	  public $sql_host;
 
	  $sql = mysql_connect($sql_host,$sql_admin,$sql_pass) or die("BRAK POŁACZENIA Z BAZA DANYCH :(");
	  mysql_select_db($sql_db);
 
	  echo("polaczenie<br>");
	  echo($sql_host."<br>");
	  echo($sql_pass."<br>");
	  echo($sql_admin."<br>");
	  echo($sql_db."<br>");
 
	 }
 
   }
 
$connect = new sql;
?>
[PHP] pobierz, plaintext

plik config.php

[PHP] pobierz, plaintext 
<?php
$sql_host   = "localhost";
$sql_admin  = "krasnal";
$sql_pass   = "krasnal";
$sql_db   = "krasnal";
?>
[PHP] pobierz, plaintext

ps. Proszę na mnie nie krzyczeć

dr_bonzo

11.09.2005, 19:56:53

Bo sa poza scopem (zakresem widzialnosci zmiennych) klasy.
Przekaz je konstruktorowi, np

[PHP] pobierz, plaintext 
<?php
$connect = new sql( $user, $haslo, itd );
?>
[PHP] pobierz, plaintext

PS. Nie uzywaj 'global'

EDIT: usunalem wykrzykniki

Gość_KRCreater

11.09.2005, 20:03:45

Dzięki za szybką odpowiedź, ale ja albo jestem głupie (hehe to wyjście jest najprawdopodobniejsze :]) albo klasa nadal nie widzi zmiennych z pliku config.php wewnątrz siebie.

Zmieniłem:

[PHP] pobierz, plaintext 
<?php
 
$connect = new sql;
 
?>
[PHP] pobierz, plaintext

Na:

[PHP] pobierz, plaintext 
<?php
 
$connect = new sql($sql_host,$sql_admin,$sql_pass,$sql_db)
 
?>
[PHP] pobierz, plaintext

Gość_KRCreater

11.09.2005, 20:12:04

Kurczę no nie widzi tych zmiennych tylko global mi działa :/ A napisałeś, że mam globala nie używać (aha mam php 4).

[PHP] pobierz, plaintext 
<?php
include("config.php");
 
 
class sql{
 
 
	 // DANE PODTRZEBNE DO POLACZENIA
	 var $sql_host;
	 var $sql_admin;
	 var $sql_pass;
	 var $sql_db;
 
	 //POKAZ ZMIENNE Z PLIKU CONFIG
	 function pokaz_config(){
	 echo $this->sql_host."<BR>!";
	 echo $this->sql_admin."<BR>!";
	 echo $this->sql_pass."<BR>!";
	 echo $this->sql_db."<BR>!<BR><BR><BR><BR>";
 
	 }
 
 
 
	 // POLACZENIE Z BAZA DANYCH MYSQL
	 function sql(){
 
	// global $sql_host,$sql_admin,$sql_pass,$sql_db;
 
	  $sql = mysql_connect($sql_host,$sql_admin,$sql_pass) or die("BRAK POŁACZENIA Z BAZA DANYCH :(");
	  mysql_select_db($sql_db);
 
 
	  echo($sql_host."<br>");
	  echo($sql_pass."<br>");
	  echo($sql_admin."<br>");
	  echo($sql_db."<br>");
	  echo("polaczenie<br>");
 
	 }
 
}
 
$connect = new sql( $sql_host, $sql_admin, $sql_pass, $sql_db );
$connect->pokaz_config();
?>
[PHP] pobierz, plaintext

czachor

11.09.2005, 20:15:17

musisz gdzieś te zmienne odbierać

[PHP] pobierz, plaintext 
<?php
 
	 // POLACZENIE Z BAZA DANYCH MYSQL
	 function sql($sql_host, $sql_admin, $sql_pass, $sql_db){
 
?>
[PHP] pobierz, plaintext

KRC.Creater

11.09.2005, 20:44:42

A no tak

hehe
Wielkie dzięki za pomoc, napewno jeszcze się odezwe tutaj :]

Mam kolejny problem

Dopisałem troche linijek. Chciałem wyciągnąć za pomocą obiektu connect dane z tabeli jakiejść z mysql'a:

[PHP] pobierz, plaintext 
<?php
include("config.php");
 
 
class sql{
 
 
     // DANE PODTRZEBNE DO POLACZENIA
     var $sql_host;
     var $sql_admin;
     var $sql_pass;
     var $sql_db;
     var $n_tabeli;
     var $n_order;
     var $n_where_pole;
     var $n_where_zmienna;
 
 
     // POLACZENIE Z BAZA DANYCH MYSQL
     function sql($sql_host,$sql_admin,$sql_pass,$sql_db){
 
      $sql = mysql_connect($sql_host,$sql_admin,$sql_pass) or die("BRAK POŁACZENIA Z BAZA DANYCH :(");
      mysql_select_db($sql_db);
     }
 
     function tabela(){
 
 
 
       if($this->n_tabeli && $this->n_order && $this->n_where_pole && $this->n_where_zmienna){
        $w = mysql_query("SELECT * FROM '".$this->n_tabeli."' ORDER BY '".$this->n_order."' WHERE '".$this->n_where_pole."'='".$this->n_where_zmienna."' ");
 
         echo $this->n_tabeli."<BR>";
         echo $this->n_where_pole."<BR>";
         echo $this->n_where_zmienna."<BR>";
         echo $this->n_order."<BR>";
       } else {
        $w = mysql_query("SELECT * FROM '".$this->n_tabeli."' ORDER BY '".$this->n_order."'");
       }
 
        $t = mysql_fetch_array($w);
 
     }
 
}
 
$connect = new sql( $sql_host, $sql_admin, $sql_pass, $sql_db );
 
$connect->n_tabeli = 'news';
$connect->n_order = 'DESC';
$connect->n_where_pole= 'id';
$connect->n_where_zmienna= '3';
echo $connect->tabela();
 
?>
[PHP] pobierz, plaintext

Ale w 41 linijce wywala mi błąd

linijka 41

[PHP] pobierz, plaintext 
<?php
 
 $t = mysql_fetch_array($w);
 
?>
[PHP] pobierz, plaintext

czachor

11.09.2005, 21:05:41

Przjerzyj przykłady w manualu, pokaż, co wyświetla mysql_error()

Ive

11.09.2005, 21:25:51

ja tam sie na OOP wogole nie znam ale na moje oko to tez powinienes wstawic te zmienne jako argumenty do funkcji bo jesli ci nie wstawia danych do zapytania [a na pewno o to chodzi] to wywala ci blad mysql w momencie w ktorym sie do niego odwolujesz... walnij za mysql_query :

[PHP] pobierz, plaintext 
<?
mysql_query('zapytanie') or die(mysql_error());
?>
[PHP] pobierz, plaintext

zeby uniknac bledu ze dane nie przedostaja sie do funkcji musisz je znowu wlepic jako argumenty do funkcji:

[PHP] pobierz, plaintext 
<?
$connect->tabela('news', 'desc', 'id', '3');
?>
[PHP] pobierz, plaintext

KRC.Creater

11.09.2005, 22:56:33

Ok doszedłem do tego już

Klasa mi działa, ale czy wszystko robie poprawnie ?

[PHP] pobierz, plaintext 
<?php
include("config.php");
 
 
class sql{
 
 
	 // DANE PODTRZEBNE DO POLACZENIA
	 var $sql_host;
	 var $sql_admin;
	 var $sql_pass;
	 var $sql_db;
	 var $n_tabeli;
	 var $n_order_by;
	 var $n_order_jak;
	 var $n_where_pole;
	 var $n_where_zmienna;
 
 
	 // POLACZENIE Z BAZA DANYCH MYSQL
	 function sql($sql_host,$sql_admin,$sql_pass,$sql_db){
 
	  $sql = mysql_connect($sql_host,$sql_admin,$sql_pass) or die("BRAK POŁACZENIA Z BAZA DANYCH :(");
	  mysql_select_db($sql_db) or die("BRAK POŁACZENIA Z BAZA DANYCH :(");
	 }
 
	 function tabela(){
 
 
 
	   if($this->n_tabeli && $this->n_order && $this->n_where_pole && $this->n_where_zmienna){
		$w = mysql_query("SELECT * FROM ".$this->n_tabeli." WHERE ".$this->n_where_pole."='".$this->n_where_zmienna."' ORDER BY ".$this->n_order_by." ".$this->n_order_jak) or die(mysql_error());
 
		 echo $this->n_tabeli."<BR>";
		 echo $this->n_where_pole."<BR>";
		 echo $this->n_where_zmienna."<BR>";
		 echo $this->n_order."<BR>";
	   } else {
		$w = mysql_query("SELECT * FROM ".$this->n_tabeli." ORDER BY ".$this->n_order_by." ".$this->n_order_jak) or die(mysql_error());
		echo("TO!");
	   }
 
		$t = mysql_fetch_array($w) or die(mysql_error());
		return $t;
 
	 }
 
}
 
$connect = new sql( $sql_host, $sql_admin, $sql_pass, $sql_db );
 
$connect->n_tabeli = "news";
$connect->n_order_by = "id";
$connect->n_order_jak = "DESC";
$connect->n_where_pole= "id";
$connect->n_where_zmienna= "3";
 
$t = $connect->tabela();
echo $t['temat'];
 
?>
[PHP] pobierz, plaintext

eS...

12.09.2005, 11:42:24

pewnie i dobrze jest, pokaże ci inne rozwiązanie, gdzie troch eprzejzysciej jest

[PHP] pobierz, plaintext 
<?
class sql {
  var $dbhost = 'localhost';
  var $dbuser = 'root';
  var $dbpass = '';
  var $dbname = '';
 
  var $result;
  var $row;
 
  function sql_connect()
  {
     mysql_connect($this -> dbhost, $this -> dbuser, $this -> dbpass,$this -> dbname) or die('Nie moge nawiazac polaczenia z baza! Blad: '.mysql_error());
     mysql_select_db($this -> dbname) or die('Nie znalazlem bazy danych! Blad: '.mysql_error());
  }
 
  function sql_query($query)
  {
     if($this -> result = mysql_query($query)){
        return $this -> result;
     }
      if(mysql_errno()){
         die('Błąd MySQLa: '.mysql_error());
    }
 
  }
 
    function sql_fetch_array(){
        if($this->row = mysql_fetch_array($this->result)){
          return $this->row;
        }
    }
}
$sql = new sql;
$sql -> sql_connect();
 
//pytanie do bazy
$a=$sql->sql_query('zapytanie');
//wyniki
$sql->sql_fetch_array($a);
 
?>  
[PHP] pobierz, plaintext

pozdrawiam

KRC.Creater

12.09.2005, 12:14:52

dzięki za tą class'e eS... rzeczywiście jest lepiej napisana, prościej :]
Postaram się ją rozwinąć w tym kierunku, pozdrawiam.
ps. widze, że też się uczysz OOP

eS...

12.09.2005, 14:20:25

ucze sie, raczej mam przerwe teraz

ale powoli bieram sie na napisanie jakiejs nowej kasy
jak by co JID : eska@chrome.pl

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.