[php][mysql] Odwołanie do składowej rzekomo nieistniejącego obiektu, Fatal error: Call to a member function query() on a non-object in... Opcje

[Geston]

Witam, z tego co widziałem/czytałem, problem jest dość popularny. Mimo tego przeglądnięcie dziesiątek tematów nie pomogło mi go rozwiązać :/

Zacznę może od kodu i jego opisu.

W napisanej przeze mnie klasie User pod lupę trafiają dwie jej funkcje, publiczna "set" i prywatna "is_exists", reszta nie ma związku z problemem:

[PHP] pobierz, plaintext 
<?php #class_User.php
 
class User {
 
	public $error;
 
	private $login;
	private $email;
	private $password;
	private $firstname;
	private $lastname;
	private $street;
	private $city;
	private $postal;
 
	public function set($name, $var) {
		// funkcja ustawiająca wartości zmiennych
 
		User::validate($name, $var);
 
		// dla loginu i maila
		if ($name == 'login' || $name == 'email') {
			User::is_exists($name, $var);
		}
 
		if ($this->error == '') {
			$this->$name = $var;
		}
	}
 
	private function is_exists($name, $var) {
		// funkcja sprawdzająca dostępność loginu/maila
 
		// utworzenie połączenia z bazą danych
		require_once('../db_connect.php');
 
		$query = sprintf("SELECT %s FROM adresses WHERE %s = '%s'", $name, $name, $var);
 
		echo $query;
 
		$result = $dbc->query($query);
 
		// zakończenie połączenia z bazą
		$dbc->close();
 
		if ($result->num_rows) {
			$result->close();
			$this->error .= "Podany $name już istnieje!\n";
 
			return true;
		}
	}
}
[PHP] pobierz, plaintext 

Funkcja set wywoływana jest w pliku który korzysta z utworzonego obiektu klasy User i przekazuje do obiektu za pomocą funkcji set wartości pobrane z formularza. Kod poniżej:

[PHP] pobierz, plaintext 
if (isset($_POST['submitted'])) {
	// obsługa formularza
 
	// dołączenie pliku z definicją klasy user
	require_once('../classes/class_User.php');
 
	// usunięcie zbędnych znaków z danych wejściowych
	$trimmed = array_map('trim', $_POST);
 
	// tworzenie nowego obiektu klasy user
	$user = new User();
 
	// ustawianie wartości zmiennych klasy user
	$user->set(login, $trimmed['login']);
	$user->set(firstname, $trimmed['firstname']);
	$user->set(lastname, $trimmed['lastname']);
	$user->set(email, $trimmed['email']);
	($trimmed['password1'] == $trimmed['password2']) ? $user->set(password, $trimmed['password1']) : $user->error .= "Podane hasła różnią się!\n";
	$user->set(street, $trimmed['street']);
	$user->set(city, $trimmed['city']);
	$user->set(postal, $trimmed['postal']);
	$user->set(birthday, $trimmed['birthday']);
 
	// wyświetlenie błędu w przypadku nieudanej próby ustawienia zmiennej klasy user,
		// brak błędu = wywołanie funkcji register klasy user dodającej użytkownika do bazy i wysyłającej email aktywacyjny
	if (!$user->error == '') echo "\n" . '<p class="error">' . "\n" . str_replace("\n", "<br />\n", $user->error) . "</p>\n";
	else $user->register();
}
[PHP] pobierz, plaintext 

Jak widać do kontroli błędów podczas walidacji danych służy mi publiczna zmienna error, dzięki niej wiem że nie mam błędu na etapie walidacji który występuje przed felernym sprawdzeniem czy w bazie danych istnieje już podany wcześniej login lub email. Również jak widać wyświetlam sobie utworzone zapytanie przed wykonaniem go, jest ono w 100% poprawne.

W funkcji "is_exists" poprzez dołączenie pliku odpowiedzialnego za utworzenie obiektu przez który łączę się z bazą tworzę właśnie taki obiekt. Błąd połączenia nie jest zwracany a więc teoretycznie też powinienem mieć pewność że obiekt ten zostaje utworzony. Również jak widać obiekt jest tworzony wewnątrz funkcji - a więc lokalnie, dzięki czemu nie muszę się do niego odwoływać globalnie itd, a bezpośrednio za pomocą zmiennej $dbc.

Poniżej kod odpowiedzialny za połączenie z bazą danych:

[PHP] pobierz, plaintext 
<?php #db_connect.php
 
// połączenie z bazą danych
 
// stałe dostępu do bazy
DEFINE ('DB_USER', 'root');
DEFINE ('DB_PASSWORD', '');
DEFINE ('DB_HOST', 'localhost');
DEFINE ('DB_NAME', 'library');
 
// nawiązanie połączenia
$dbc = new mysqli(DB_HOST, DB_USER, DB_PASSWORD, DB_NAME);
 
if (mysqli_connect_errno()) {
    echo "Connection failed: " . mysqli_connect_error() . "\n";
    exit();
}
 
?>
[PHP] pobierz, plaintext 

Proszę o pomoc, nie pierwszy raz działam w podobny sposób i zawsze radziłem sobie z tym problemem, teraz jednak nie mam zielonego pojęcia skąd on się bierze (IMG:style_emoticons/default/sad.gif)

Fatal error: Call to a member function query() on a non-object in...

Rozumiem że wywołując składową klasy mysqli - query() - wywala mi błąd bo obiekt nie istnieje...

Przepraszam za zamieszanie, wątek do zamknięcia. Problem najczęściej leży nie tam gdzie go szukamy (IMG:style_emoticons/default/smile.gif) Może podając na czym polegał przyczynię się do pomocy innym którzy kiedyś trafią na ten temat.

Otóż funkcja is_exists zostaje wywołana w jednym skrypcie dwukrotnie, co za tym idzie require_once() nie dołącza drugi raz pliku odpowiedzialnego za połączenie z bazą. Zamiana na require() rozwiązuje problem (IMG:style_emoticons/default/smile.gif) w takim wypadku dobrym zwyczajem jest każdorazowa likwidacja utworzonego obiektu połączenia z bazą danych zaraz po jego zakończeniu (IMG:style_emoticons/default/smile.gif)

Ten post edytował Geston 29.11.2010, 15:43:44