[PHP]Błąd Call to a member function prepare() on a non-object, problem z kodem. Opcje

[casperii]

Panowie próbuje się zalogować poprzez ajax. Niestety otrzymuje bład w pliku class.php
Fatal error: Call to a member function prepare() on a non-object in line 13 czyli linia gdzie jest:

[PHP] pobierz, plaintext 
$sql = $this->db->prepare("SELECT * FROM tabelka WHERE user=:login") or die ($db->errorInfo());
[PHP] pobierz, plaintext 

class.php

[PHP] pobierz, plaintext 
class xx{
	private $db;
 
	function __construct($con){
		$this->pdo = $con;
 
	}
}
[PHP] pobierz, plaintext 

Może ktoś podpowiedzieć co robię źle?

Ten post edytował casperii 7.05.2016, 18:20:01

[rad11]

[PHP] pobierz, plaintext 
$this->pdo
[PHP] pobierz, plaintext 

a nie

[PHP] pobierz, plaintext 
$this->db
[PHP] pobierz, plaintext 

[casperii]

Dziękuje @rad11 .
Jeszcze mam pytanie odnośnie logowania, bo coś nie idzie.
Otóż loguje się poprzez Ajax i tu mam kilka pytań,
czy jeśli nie wpiszę loginu i hasła to błędy mam pluć w pliku ajaxowym coś na styl:

[PHP] pobierz, plaintext 
$value_login = trim($_POST['login']);
	$value_password = trim(md5($_POST['pass']));
 
	if(empty($value_login)){
		$message_error = 'Nie wpisano loginu.';
		echo $message_error;
	}
	elseif(empty($value_password)){
		$message_error = 'Nie wpisano hasła.';
		echo $message_error;
	}
[PHP] pobierz, plaintext 

czy da radę to zrobić w class.php ? Jeżeli tak to w jaki sposób i jak później przekazać komunikat do ajax:

class.php:

[PHP] pobierz, plaintext 
public function AjaxLogin($value_login, $value_password)
	{
		$sql = $this->db->prepare("SELECT * FROM `users` WHERE login=:user  AND pass=:password") or die ($db->errorInfo());
		$sql->bindValue(':user', $value_login, PDO::PARAM_STR);
		$sql->bindValue(':password', $value_password, PDO::PARAM_STR);
		$sql->execute(); 
                $row = $sql->fetch(PDO::FETCH_ASSOC);
		if($sql->rowCount()!=0){
			$test = 'supper';
		}
		return $sql;
	}
[PHP] pobierz, plaintext 

ajax.php:

[PHP] pobierz, plaintext 
if($_POST['action'] == 'user_login'){
 
	if($cos->AjaxLogin($value_login, $value_password)){
	var_dump($sql);
	}
[PHP] pobierz, plaintext 

Gdzie i jak powinno wyglądać czy user znajduje się w bazie? w strukturalnym to bym przeleciał pętlą while i sprawdził czy jest większy od 0 i stosowny komunikat.
Tylko jak to teraz połączyć tą klase z wstawieniem tego do Ajax. Czy mógłbyś pomóc?

Ten post edytował casperii 7.05.2016, 19:42:21

[CuteOne]

Hmm, a nie lepiej rzucać json'a jako odpowiedź serwera? np

[PHP] pobierz, plaintext 
class MySpecialException extends Exception {}
 
///...
 
try {
    if(empty($value_login)){
         throw new \MySpecialException('Nie wpisano loginu.');
    }
 
    echo json_encode([
         'status' => 1,
         'content' => 'bla bla bla'
    ]);
} catch (\MySpecialException $e) {
    $this->logger->add($e); //logowanie błedów
    echo json_encode([
         'status' => 0,
         'error' => $e->getMessage()
    ]);
} catch (\Exception $e) {
    $this->logger->add($e); //logowanie błedów
    echo json_encode([
         'status' => 0,
         'error' => 'Nie udało się wykonać'
    ]);
}
exit;
[PHP] pobierz, plaintext 

Ten post edytował CuteOne 7.05.2016, 21:45:33

[com]

rad11 co za bzdury wypisujesz, właśnie że db tylko w konstruktorze wypadało zmienić z $this->pdo na db, bo tak zrobił zmienna publiczną tam.

[casperii]

@com czy mógłbyś mi wytłumaczyć jak powinna wyglądać komunikacja pomiędzy dwoma plikami? :

[PHP] pobierz, plaintext 
<?php
class cos{
	private $db;
 
	function __construct($con){
		$this->db = $con;
 
	}
	public function AjaxLogin($value_login, $value_password) //
	{
		$sql = $this->db->prepare("SELECT * FROM `users` WHERE login=:user OR email=:mail AND pass=:password") or die ($db->errorInfo());
		$sql->bindValue(':user', $value_login, PDO::PARAM_STR);
		$sql->bindValue(':mail', $value_login, PDO::PARAM_STR);
		$sql->bindValue(':password', $value_password, PDO::PARAM_STR);
		$sql->execute();
		return $sql->fetch(PDO::FETCH_ASSOC);
	}
}
?>
[PHP] pobierz, plaintext 

oraz plik ajaxowy.php do którego odwołuje się w pliku index.php. Wiem, że najłatwiej było by po prostu w w pliku ajaxowym bez kombinowania zrobić zapytanie i w zależności czy user istnieje czy nie lub plucie komunikatem jeśli czegoś user nie wpisał i jsonem lub echem wysłać do indexa stosowny komunikat.
Ale ja chce to zrobić na classach które są wczytywane do pliku ajaxowego z pliku class.php , czyli musi być komunikacja pomiędzy dwoma plikami ajaxowy.php , class.php a stosowny komunikat wyświetlić w index.php poprzez json lub echo.

ajaxowy.php

[PHP] pobierz, plaintext 
if($_POST['action'] == 'user_login'){
//tu trzeba pobrać klase AjaxLogin ze stosownym komunikatem.
}
[PHP] pobierz, plaintext 

Rozumiem, że jeśli user nie wpisze loginu lub hasła to mogę pluć komunikatem bezpośrednio z pliku ajaxowy.php np.

[PHP] pobierz, plaintext 
if(empty($value_login)){
//tu json lub echo
}
elseif(empty($value_password)){
//tu json lub echo
}
[PHP] pobierz, plaintext 

ale jak zrobić tą komunikację pomiędzy ajaxowy.php a class. że ajaxowy pobierze wartości przekaże je do class.php i w zależności jeśli user istnieje zwróci zalogowany==true i wtedy ajaxowy tworzyłby sesje? a jeśli user nie istnieje klasa AjaxLogin zwróci zalogowany==false do pliku ajaxowy.php i tu sobie wyświetle odpowiedni komunikat.

Myślę, że dobrze kombinuje ?

[com]

ale w czym jest problem bo nie bardzo rozumiem? przecież możesz w jeden metodzie wywołać klasę z innego pliku? tylko najpierw musisz ja wczytać, jak używasz composera to nie musisz się tym martwić, jak nie to ją importujesz, w jakimś głównym pliku a potem przekazujesz obiekt do metody i wykonujesz operacje jaka potrzebujesz

[casperii]

@com gdybym nie miał problemu to bym nie pytał się na forum.

w ajaxowym mam tak:

[PHP] pobierz, plaintext 
$value_login = trim($_POST['login']);
$value_password = trim(md5($_POST['pass']));
 
 
	if($cos->AjaxLogin($value_login, $value_password)){
	// tu chcę odebrać return $msg_err oraz $msg_ok
	}
[PHP] pobierz, plaintext 

w powyższym pliku chciałbym odebrać informacje z pliku class.php

class.php:

[PHP] pobierz, plaintext 
	public function AjaxLogin($value_login, $value_password)
	{
		$sql = $this->db->prepare("SELECT * FROM `users` WHERE (login=:user OR email=:mail) AND pass=:password") or die ($db->errorInfo());
		$sql->bindValue(':user', $value_login, PDO::PARAM_STR);
		$sql->bindValue(':mail', $value_login, PDO::PARAM_STR);
		$sql->bindValue(':password', $value_password, PDO::PARAM_STR);
		$sql->execute();
		$row = $sql->fetch(PDO::FETCH_ASSOC);
 
		if(empty($value_login)){
			//echo 'Proszę podać login.'; //jeśli robię echem to w indexie się pokazuje 
			$msg_err = 'Proszę podać login';
		}
		elseif(empty($value_password) || $value_password != $row['pass']){
			$msg_err = 'Proszę podać prawidłowe hasło.';
		}
 
		if(empty($msg_err)){
		echo 'super';
		$msg_ok = 'ok';
		}
 
return $msg_err;
return $msg_ok; 
} 
[PHP] pobierz, plaintext 

Chodzi mi teraz jak przenieść info z 1 pliku do drugiego. AjaxLogin($msg_err); nie działa.

[CuteOne]

[PHP] pobierz, plaintext 
class A {
    public function make($dane) {
        echo 'a' . $dane;
    }
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
include 'A.php';
 
$dane = 'b';
$a = new A();
$a->make($dane);
[PHP] pobierz, plaintext 

[com]

po pierwsze to nie return, return bo drugi nigdy nie zadziała.

no to

[PHP] pobierz, plaintext 
class Foo 
{
 public function bar($bar)
 {
    if($bar === 'bar') {
      return true;
    }
    return false;
}
 
class Bar 
{
 private $foo;
 public function __construct(Foo $foo)
 {
    $this->foo = $foo;
 }
 public function foo($bar)
 {
   $result = $this->foo->bar($bar);
   // no i tu cokolwiek z tym robisz
 }
 
[PHP] pobierz, plaintext 

czyli AjaxLogin($value_login, $value_password) powinno zwracać true || false, a komunikaty o błędach przechowujesz w property i w razie potrzeby zwracasz geterem.


Ten post edytował com 8.05.2016, 15:16:32