Nieprawidłowy argument dla mysql_query Opcje

[Wojteq]

Witam!
Jeśli wywołam poniższy kod z wnętrza klasy, dostanę następujący komunikat:

supplied argument is not a valid MySQL result resource

Jeśli natomiast wykonam kod poza klasą (nieumieszczony w funcji), wszystko będzie działało poprawnie. W czym leży problem?

[PHP] pobierz, plaintext 
<?php
	protected function mysql($query)
	{
		//$query = 'SELECT `subtitle` FROM `table` WHERE `id`=1';
		$sql_query = mysql_query('SELECT `subtitle` FROM `table` WHERE `id`=1');
		return $sql_array = mysql_fetch_array($sql_query, MYSQL_BOTH);
	}
?>
[PHP] pobierz, plaintext 

[hwao]

nie wiem w ktorej lini problem, ale jezeli w tej z mysql_query" title="Zobacz w manualu php" target="_manual() to dodaj res. od bazy danych.

[M4chu]

Może dodaj obsługę błędów i zobacz co wyświetliło. No i:

[PHP] pobierz, plaintext 
<?php
return $sql_array = mysql_fetch_array($sql_query, MYSQL_BOTH);
?>
[PHP] pobierz, plaintext 

To '$sql_array = ' jest przecież niepotrzebne.

----
Gorąco polecam zmianę nazw zmiennych i metody, na bardziej intuicyjne - metoda powinna, moim zdaniem, nazwać się query, a zmienna $sql_query, raczej $result (patrz: komunikat - MySQL result resource) Ale to takie głupie gadanie, zrobisz jak chcesz, jednak moim zdaniem warto się nad tym zastanowić i następnym razem przyłożyć się do tego dla własnego i naszego dobra
----
TomASS: $var = false; też zwroci true? ;>

[TomASS]

M4chu ma rację Sprawdź też, czy ta metoda nie zwraca Ci zawsze True? (bo operator przypisania zawsze będzie dawał true)


--------edit--------------
Masz rację, mój bład. Jak dasz:

[PHP] pobierz, plaintext 
<?php
 
return $zmienna=12;
//zwroci wartosc 12
 
?>
[PHP] pobierz, plaintext 

Wydało mi się to dziwne, z tego względu, że w logice operator przypisanaia zawsze będzie dawał wynik true, widocznie tak w php nie jest. Przepraszam bardzo za wprowadzenie w bład.

Ten post edytował TomASS 16.09.2005, 14:44:48

[dr_bonzo]

(bo operator przypisania zawsze będzie dawał true)

Nie prawda, zwroci $sql_array.

Linia z bledem to ta z "return $sql_array = mysql_fetch_array($sql_query, MYSQL_BOTH);", php podaje, ze podano nieprawidlowy "result resource" -- wynik zapytania, czyli blad jest w mysql_query() -- pewnie, jak mowil hwao, nieokresliles polaczenia z baza, ani sie nie polaczyles z nia (nie widac tego w kodzie).
Rada: nie uzywaj domyslnego polaczenia z baza, tylko ZAWSZE przekazuj resource polaczenia do kazdej funkcji tego wymagajacej.

[Wojteq]

Z tymi nazwami zmiennych i funcji to rzeczywiście macie racje. Wybrałem trochę nieintuicyjne nazwy. Do bazy łączę się przez ten kod:

[PHP] pobierz, plaintext 
<?php
	function __autoload()
   {
	  include('myslq_connect.php');	
   }
?>
[PHP] pobierz, plaintext 

Czy to może być problemem?
Jeśli tak, to czym to zastąpić oraz czy

[PHP] pobierz, plaintext 
<?php
return mysql_fetch_array($query, MYSQL_BOTH);
?>
[PHP] pobierz, plaintext 

będzie działało bez żadnego problemu?

[M4chu]

Z tymi nazwami zmiennych i funcji to rzeczywiście macie racje. Wybrałem trochę nieintuicyjne nazwy. Do bazy łączę się przez ten kod:

[PHP] pobierz, plaintext 
<?php
	function __autoload()
   {
	  include('myslq_connect.php');	
   }
?>
[PHP] pobierz, plaintext 

Czy to może być problemem?
Jeśli tak, to czym to zastąpić oraz czy

[PHP] pobierz, plaintext 
<?php
return mysql_fetch_array($query, MYSQL_BOTH);
?>
[PHP] pobierz, plaintext 

będzie działało bez żadnego problemu?

Prawde mowiac nawet nie wiedzialem, ze mozna tak z __autoload kozystac Ja wole tak jak w manualu jest:

[PHP] pobierz, plaintext 
<?php
function __autoload($class) {
	require_once 'classes/' . $class . '.class.php';
}
 
?>
[PHP] pobierz, plaintext 

To zresztą nie ma znaczenia, jakbyś uważnie przeczytał posty przedmówców, to byś wiedział, że nie w tym rzecz . Musisz do funckji mysql_query(manual!) przekazywać identyfikator połączenia zwrócony przez mysql_connect(). Czyli np (to samo w manualu znajdziesz ):

[PHP] pobierz, plaintext 
<?php
 
$connection = mysql_connect($host, $user, $pass) or die(mysql_error());
mysql_select_db($db) or die(mysql_error($connection);
if(!$result = mysql_query($query, $connection)) {
	die(mysql_error($connection));
}
 
?>
[PHP] pobierz, plaintext 

ehh rozpisalem sie
pozdro

[Wojteq]

OK. Dzięki za pomoc.

Mam jednak jeszcze jedno pytanie: podobno można stworzyć nowy obiekt łączący się z bazą za pomoćą

[PHP] pobierz, plaintext 
<?php
$dbc = mysqli_connect($host, $user, $pass);
?>
[PHP] pobierz, plaintext 

I z tym związane jest moje pytanie: jak wewnątrz klasy utworzyć nowy obiekt? Przez konstruktor czy przez funkcję, która będzie iniciowana przez konstruktor i czy istnieje jakiś sposób, aby nie pisać za każdym razem takich samych konstruktorów w podklasie, które znajdują się w nadklasie?

[M4chu]

I z tym związane jest moje pytanie: jak wewnątrz klasy utworzyć nowy obiekt? Przez konstruktor czy przez funkcję, która będzie iniciowana przez konstruktor i czy istnieje jakiś sposób, aby nie pisać za każdym razem takich samych konstruktorów w podklasie, które znajdują się w nadklasie?

Zależy co rozumiesz pod pojęciem "utworzyć nowy obiekt wewnątrz klasy"? Co do takich samych konstruktorów w podklasie jak w nadklasie, to też nie rozumiem... mówisz o pod i nad klasach, a nie słyszałeś o dziedziczeniu?
Konkrety!

[Wojteq]

Chodziło mi właśnie o dziedziczenie.

Czyli czy istnieje coś takiego jak 'dzedziczenie konstruktorów'. bo nie chcę w każdej klasie pisać identycznych konstruktorów, króre będą wykonywały te same operacje. Bo jeśli utowrzę klasę B, która będzie dziedziczyła od klasy A wszystkie zmiene i funkcje, a zestępowała jedną funkcję (np. display()). I po co w takiej sytuacji pisać nowy konstruktor?

A co do mysqli_connect() to chodzi mi o utworzenie obiektu który będzie się łączył (już wewnątrz klasy) i bazą danych.

[mike]

W klasie potomnej można co najwyżej wywołać konstruktor klasy bazowej:

[PHP] pobierz, plaintext 
<?php
 
class A extends B
{
	public function __construct()
	{
		parent::__construct();
	}
 
	// ...
}
?>
[PHP] pobierz, plaintext 

Konstruktory nie są dziedziczone.

[splatch]

Zatem co wyświetli ten przykład?

[PHP] pobierz, plaintext 
<?php
class B {
	function __construct() {
		echo "B";
	}
}
 
class A extends B { }
$x = new A;
?>
[PHP] pobierz, plaintext 

Ten post edytował splatch 22.09.2005, 08:17:11

[dr_bonzo]

Takie cos nie przejdzie:

[PHP] pobierz, plaintext 
<?php
function __autoload()
   {
	  include('myslq_connect.php');	
   }
?>
[PHP] pobierz, plaintext 

__autoload sluzy zupelnie do czego innego -> manual.
Dla tego nigdy ci nie uruchamialo tej funkcji.

A co do mysqli_connect() to chodzi mi o utworzenie obiektu który będzie się łączył (już wewnątrz klasy) i bazą danych.

[PHP] pobierz, plaintext 
<?php
 
class Klasa
{
	private $db;
 
	public function __construct()
	{
		$this->db = new mysqli( parametry polaczenia);
		// lub
		$this->db = mysqli_connect( ... );
	}
}
 
?>
[PHP] pobierz, plaintext 

W razie niepowodzenia zwracaja FALSE, ja uzywam mysqli_connect gdyz konstruktor nie powinien zwracac FALSE. Dalej korzystam z otrzymanego obiektu.

[mike]

Zatem co wyświetli ten przykład?

[PHP] pobierz, plaintext 
<?php
class B {
	function __construct() {
		echo "B";
	}
}
 
class A extends B { }
$x = new A;
?>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
 
class B {
	function __construct() {
		echo "B";
	}
}
 
class A extends B
{
	public function __construct()
	{
		parent::__construct();
	}
}
 
$a = new A();
 
?>
[PHP] pobierz, plaintext 

Wynik:

B

[splatch]

@mike_mech po co pisać taki kod, skoro działa krótszy:

[PHP] pobierz, plaintext 
<?php
class B {
	function __construct() {
		echo "B";
	}
}
 
class A extends B { }
$x = new A;
?>
[PHP] pobierz, plaintext 

To wyświetli samo B.
No i dodatkowo, gdyby konstruktory nie były dziedziczone jaki by był sens tworzyć konstruktory finalne?

[PHP] pobierz, plaintext 
<?php
class B {
	final function __construct() {
		echo "B";
	}
}
 
class A extends B { }
$x = new A;
?>
[PHP] pobierz, plaintext 

[Wojteq]

Dzięki za pomoc i wyjaśnienie. Właśnie o to mi chodziło. Mam jednak jeszcze jedno pytanie dotyczące wykonywania zapytań do bazy. Jakie parametry mam podać pod czas wykonywania zapytania przez mysqli_connect? Wystarczy sam string, czy muszę wcześniej podać w parametrach utworzony obiekt klasy mysqli?

[mike]

Jakie parametry mam podać pod czas wykonywania zapytania przez mysqli_connect? Wystarczy sam string, czy muszę wcześniej podać w parametrach utworzony obiekt klasy mysqli?

No bez przesaday. My też się cieszymy że Ci pomogliśmy ale to już sam możesz sprawdzić: mysqli_connect(). Przkłady są bardzo przejrzyste.

[Wojteq]

Chodziło mi o funkcję mysqli_query a nie connect
Bo tam przykłady nie są (przynajmniej dla mnie) zbyt oczywiste...

Chodzi mi o to, czy mogę użyć takiego zapisu wewnątrz klasy:

[PHP] pobierz, plaintext 
<?php
class Page
{
   protected $db;
   function __construct()
   {
	   $this->db = new mysqli('localhost', 'username', 'password', 'database');
	   $this->db->query('zapytanie'); // chodzi mi o to
	}
}
?>
[PHP] pobierz, plaintext 

Ten post edytował Wojteq 30.09.2005, 17:31:07

[mike]

No nie przesadzaj. Jak byk stoi w manualu:

Styl zorientowany obiektowo:

[PHP] pobierz, plaintext 
<?php
 
$mysqli = new mysqli( 'localhost', 'user', 'pass', 'db' );
$result = $mysqli->query( 'SELECT * FROM table' );
// ...
 
?>
[PHP] pobierz, plaintext 

Styl proceduralny:

[PHP] pobierz, plaintext 
<?php
 
$link = mysqli_connect( 'localhost', 'user', 'pass', 'db' );
$result = mysqli_query( $link, 'SELECT * FROM table' );
// ...
 
?>
[PHP] pobierz, plaintext 

Czego tu można nie zrozumieć :?:

A poza tym napisałeś sobie kod i co? Nawet nie sprawdziłeś czy zadziała.
Zawsze najlepiej empirycznie sprawdzić.

[Wojteq]

To co było w manualu raczej zrozumiałem zrozumiałem, ale co zrobić jeśli ten kod znajduje się wewnątrz klasy?

[PHP] pobierz, plaintext 
<?php
 
class Page
{
protected $mysqli;
 
function __construct()
{
$this->mysqli = new mysqli( 'localhost', 'user', 'pass', 'db' );
//$result = $mysqli->query( 'SELECT * FROM table' );
// jak zrobić zapytanie w tym przypadku, bo należy użyć $this->mysqli 
}
}
?>
[PHP] pobierz, plaintext 

Ten post edytował Wojteq 30.09.2005, 18:47:26