Trying to get property of non-object [problem] Opcje

[thoran]

Wrzucilem skrypty na serwer z obsługa php 5 i wywala mi takiego notice "Trying to get property of non-object in". Co to oznacza i jak się tego pozbyć?

Ten post edytował thoran 9.11.2005, 17:31:38

[dr_bonzo]

1. Zadzwon do wrozki/jasnowidza -- ona ci pomoze
2. Jesli cie nie stac na nia: to podaj nazwe skryptu, fragment kodu, w ktorym blad wystepuje + wskaz, ktora to linia.

----edit----
Hehe, chyba się spieszyłeś z pisaniem tego posta.
Weś popraw go na polski.
~mike_mech

ED: pisalem jednym palcem w niewygodnej pozycji (ale bez ogonkow -- plisss:))

Ten post edytował dr_bonzo 8.11.2005, 21:12:57

[bela]

Próbujesz wywołać metodę na zmiennej, która nie jest obiektem.
Zrób print_r i zobaczysz czym jest ta zmienna.

[dr_bonzo]

Gwoli scislosci: nie metode ale wlasciwosc/pole/property (i ofkorz pobrac nie wywolac).

[thoran]

[PHP] pobierz, plaintext 
<?php
		 
if ($player -> race == 'Tauren')
 
		  {
 
			$ap = ($ap + 6);
 
		}
 
			  else
 
			{
 
			$ap = ($ap + 5);
 
		}
 
?>
[PHP] pobierz, plaintext 

bład pojawia się w 3 linijce, kod ten znajduje się w funkcji której poczatek wygląda tak

[PHP] pobierz, plaintext 
<?php
 
function checkexp ($exp,$expgain,$level,$rasa,$user,$eid,$enemyid,$enemyuser,$player,$skill,$amount)
 
{
 
	global $db, $newdate;
 
 
 
	$poziom = 0;
 
	$ap = 0;
 
	$pz = 0;
 
	$energia = 0;
 
	$texp = ($exp + $expgain);
 
	if ($level < 100)
 
	{
 
		$expn = ($level * $level) * 100;
 
	}
 
	if ($level > 99 && $level < 200)
 
	{
 
		$expn = ($level * $level) * 250;
 
	}
 
	if ($level > 199 && $level < 300)
 
	{
 
		$expn = ($level * $level) * 500;
 
	}
 
	if ($level > 299 && $level < 400)
 
	{
 
		$expn = (pow($level,2) * 1000);
 
	}
 
	if ($level > 399 && $level < 500)
 
	{
 
		$expn = (pow($level,2) * 2500);
 
	}
 
	if ($level > 499 && $level < 600)
 
	{
 
		$expn = (pow($level,2) * 5000);
 
	}
 
	if ($level > 599 && $level < 700)
 
	{
 
		$expn = (pow($level,2) * 10000);
 
	}
 
	if ($level > 699 && $level < 800)
 
	{
 
		$expn = (pow($level,2) * 25000);
 
	}
 
	if ($level > 799 && $level < 900)
 
	{
 
		$expn = (pow($level,2) * 50000);
 
	}
 
	if ($level > 899 && $level < 1000)
 
	{
 
		$expn = (pow($level,2) * 100000);
 
	}
 
	$energy = $db -> Execute("select max_energy from players where id=".$eid);
 
	while ($texp >= $expn)
 
	{
 
		  $poziom = ($poziom + 1);
 
		  if ($player -> race == 'Tauren')
 
		  {
 
			$ap = ($ap + 6);
 
		}
 
			  else
 
			{
 
			$ap = ($ap + 5);
 
		}
 
		$texp = ($texp - $expn);
 
		$level = ($level + 1);
 
		if ($level < 100)
 
			{
 
				$expn = (pow($level,2) * 100);
 
			}
 
?>
[PHP] pobierz, plaintext 

Na php 5 jeszcze nie mialem okazji pracowac dlatego nie wiem co z tym zrobić.



Wlasnie zauwazylem ze wyskakuje to w linijkach w których uzywam isset lub empty

[PHP] pobierz, plaintext 
<?php
			if (isset($arrows -> fields['id'])) //tutaj
			{
				$test = $db -> Execute("SELECT id FROM equipment WHERE name='".$arrows -> fields['name']."' AND status='U' AND owner=".$player -> id);
			}
			if (empty ($test -> fields['id']))  //tutaj
			{
				$db -> Execute("UPDATE equipment SET status='U' WHERE type='R' AND owner=".$player -> id." AND status='E'");
			} 
				else 
			{
				$db -> Execute("UPDATE equipment SET wt=wt+".$arrows -> fields['wt']." WHERE id=".$test -> fields['id']);
				$db -> Execute("DELETE FROM equipment WHERE id=".$arrows -> fields['id']);
			}
			if (isset($test -> fields['id']))  //tutaj
			{
				$test -> Close();
			}
 
?>
[PHP] pobierz, plaintext 

Ten post edytował thoran 8.11.2005, 22:31:20

[dr_bonzo]

A czym jest $player (jest przekazywany jako argument funkcji), pokaz gdzie wywolujesz ta funkcje, i pokaz kod klasy, ktorej instancja jest $player.

[thoran]

jako $player przekazywany jest id gracza

przykład

[PHP] pobierz, plaintext 
<?php
 
		checkexp($attacker['exp'],$expgain,$attacker['level'],$attacker['race'],$attacker['user'],$attacker['id'],$defender['id'],$defender['user'],$attacker['id'],'',0);
 
?>
[PHP] pobierz, plaintext 

klasa PLayer po krótce wygląda tak

[PHP] pobierz, plaintext 
<?php
 
class Player
{
	var $user;
	var $id;
	var $level;
	var $exp;
	var $hp;
	var $max_hp;
	var $mana;
	var $energy;
	var $max_energy;
	var $credits;
	var $bank;
	var $platinum;
	var $tribe;
	var $rank;
	var $podrank;
	var $location;
	var $ap;
	var $race;
	var $mask_race;
	var $clas;
	var $agility;
	var $strength;
	var $inteli;
	var $pw;
	var $wins;
	var $losses;
	var $lastkilled;
	var $lastkilledby;
	var $age;
	var $logins;
	var $smith;
	var $attack;
	var $miss;
	var $magic;
	var $ip;
	var $browser;
	var $speed;
	var $cond;
	var $alchemy;
	var $gg;
	var $avatar;
	var $wisdom;
	var $shoot;
	var $tribe_rank;
	var $fletcher;
	var $immunited;
	var $corepass;
	var $trains;
	var $fight;
	var $deity;
	var $maps;
	var $rest;
	var $page;
	var $profile;
	var $crime;
	var $gender;
	var $style;
	var $leadership;
	var $klan;
	var $burdel;
	var $nastroj;
	var $stan;
	var $koncesja;
	var $zbrodnia;
	var $gryz;
	var $changetype;
	var $pokoj;
	var $uzdrawianie;
	var $prawo;
	var $charyzma;
	var $breeding;
	var $battlelog;
	var $resth;
	var $antidote;
	var $poll;
	var $inscription;
	var $bonus;
	var $premium;
// konstruktor klasy - pobranie danych z bazy danych oraz przypisanie ich do odpowiednich zmiennych
	function Player($pid)
	{
	global $db;
		$stats = $db -> Execute("SELECT * FROM players WHERE id=".$pid." LIMIT 1");
		$this -> user = $stats -> fields['user'];
		$this -> id = $stats -> fields['id'];
		$this -> level = $stats -> fields['level'];
		$this -> exp = $stats -> fields['exp'];
		$this -> hp = $stats -> fields['hp'];
		$this -> max_hp = $stats -> fields['max_hp'];
		$this -> mana = $stats -> fields['pm'];
		$this -> energy = $stats -> fields['energy'];
		$this -> max_energy = $stats -> fields['max_energy'];
		$this -> credits = $stats -> fields['credits'];
		$this -> bank = $stats -> fields['bank'];
		$this -> platinum = $stats -> fields['platinum'];
		$this -> tribe = $stats -> fields['tribe'];
		$this -> rank = $stats -> fields['rank'];
		$this -> podrank = $stats -> fields['podrank'];
		$this -> location = $stats -> fields['miejsce'];
		$this -> ap = $stats -> fields['ap'];
		$this -> race = $stats -> fields['rasa'];
		$this -> mask_race = $stats -> fields['mask_rasa'];
		$this -> clas = $stats -> fields['klasa'];
		$this -> agility = $stats -> fields['agility'];
		$this -> strength = $stats -> fields['strength'];
		$this -> inteli = $stats -> fields['inteli'];
		$this -> pw = $stats -> fields['pw'];
		$this -> wins = $stats -> fields['wins'];
		$this -> losses = $stats -> fields['losses'];
		$this -> lastkilled = $stats -> fields['lastkilled'];
		$this -> lastkilledby = $stats -> fields['lastkilledby'];
		$this -> age = $stats -> fields['age'];
		$this -> logins = $stats -> fields['logins'];
		$this -> smith = $stats -> fields['ability'];
		$this -> attack = $stats -> fields['atak'];
		$this -> miss = $stats -> fields['unik'];
		$this -> magic = $stats -> fields['magia'];
		$this -> ip = $stats -> fields['ip'];
		$this -> browser = $stats -> fields['browser'];
		$this -> speed = $stats -> fields['szyb'];
		$this -> cond = $stats -> fields['wytrz'];
		$this -> alchemy = $stats -> fields['alchemia'];
		$this -> gg = $stats -> fields['gg'];
		$this -> avatar = $stats -> fields['avatar'];
		$this -> wisdom = $stats -> fields['wisdom'];
		$this -> shoot = $stats -> fields['shoot'];
		$this -> tribe_rank = $stats -> fields['tribe_rank'];
		$this -> fletcher = $stats -> fields['fletcher'];
		$this -> immunited = $stats -> fields['immu'];
		$this -> corepass = $stats -> fields['corepass'];
		$this -> trains = $stats -> fields['trains'];
		$this -> fight = $stats -> fields['fight'];
		$this -> deity = $stats -> fields['deity'];
		$this -> maps = $stats -> fields['maps'];
		$this -> rest = $stats -> fields['rest'];
		$this -> page = $stats -> fields['page'];
		$this -> profile = $stats -> fields['profile'];
		$this -> crime = $stats -> fields['crime'];
		$this -> gender = $stats -> fields['gender'];
	$this -> style = $stats -> fields['style'];
		$this -> leadership = $stats -> fields['leadership'];
		$this -> klan = $stats -> fields['klan'];
		$this -> burdel = $stats -> fields['burdel'];
		$this -> nastroj = $stats -> fields['nastroj'];
		$this -> stan = $stats -> fields['stan'];
		$this -> koncesja = $stats -> fields['koncesja'];
		$this -> zbrodnia = $stats -> fields['zbrodnia'];
		$this -> gryz = $stats -> fields['gryz'];
		$this -> changetype = $stats -> fields['changetype'];
		$this -> pokoj = $stats -> fields['pokoj'];
		$this -> uzdrawianie = $stats -> fields['uzdrawianie'];
		$this -> prawo = $stats -> fields['prawo'];
		$this -> charyzma = $stats -> fields['charyzma'];
		$this -> breeding = $stats -> fields['breeding'];
		$this -> battlelog = $stats -> fields['battlelog'];
		$this -> resth = $stats -> fields['resth'];
		$this -> antidote = $stats -> fields['antidote'];
		$this -> poll = $stats -> fields['poll'];
		$this -> inscription = $stats -> fields['inscription'];
		$this -> bonus = $stats -> fields['bonus'];
		$this -> premium = $stats -> fields['premium'];
		$stats -> Close();
	}
// funkcja zwracajaca wartości z charakterystyk w postaci tabeli
	function stats($stats)
	{
		$arrstats = array();
		foreach ($stats as $value)
		{
			$arrstats[$value] = $this -> $value;
		}
		return $arrstats;
	}
}
 
 
?>
[PHP] pobierz, plaintext 

hm chyba po woli dochodze o co w tym chodzi

A i mam jeszcze pytanie czego mozna uzyć zamist globali w funkji, bo wszedzie słysze "nieuzywaj globali" ale co dac w zamian?

[dr_bonzo]

1. global
W parametrze konstruktora przekazujesz $db a potem je zapisujesz w polu klasy

[PHP] pobierz, plaintext 
<?php
class Player
{
 //...
var $db;
 
function Player( ...., $db_ )
{
	$this->db = $db_;
}
}
?>
[PHP] pobierz, plaintext 

2.
$player zostaje zadeklarowany tu:

[PHP] pobierz, plaintext 
<?php
function checkexp ($exp,$expgain,$level,$rasa,$user,$eid,$enemyid,$enemyuser,$player,$skill,$amount)
?>
[PHP] pobierz, plaintext 

jako parametr $player podajesz $attacker['id'], a czym jest ta wartosc? Obiektem klasy Player?
Bo potem sie odwolujesz
$player->race, i tu jest blad.

3.

[PHP] pobierz, plaintext 
<?php
function stats($stats)
	{
		$arrstats = array();
		foreach ($stats as $value)
		{
			$arrstats[$value] = $this -> $value;
		}
		return $arrstats;
	}
?>
[PHP] pobierz, plaintext 

Nie lepiej po prostu zapisac wyniku z query

[PHP] pobierz, plaintext 
<?php
function Player($pid)
	{
		$this->stats = $db -> Execute("SELECT * FROM players WHERE id=".$pid." LIMIT 1");
?>
[PHP] pobierz, plaintext 

i potem to zwracac? No chyba ze jakos wykorzystujesz te dziesiatki pol klasy Player.

[thoran]

Dzięki za pomoc poradzilem sobie

PLayer ma takie same pola jak tabela w bazie, sa tam zapisane wszelkie dane dotyczące graczy, potem jednorazowo pobierane, w sumie uzywam tylko tej jednej klasy, jesli wiesz jak to inaczej rozwiązac to powiedz

Dzięki i pozdrawiam




Moze mi ktos wyjasnić pewną rzecz

Uzywam adodb i mam cos takiego

[PHP] pobierz, plaintext 
<?php
 
$cos = $db -> execute("Select id from cos where id=1");
if (isset($cos -> fields['id']))
{
   instrukcje
}
$cos -> Close();
 
?>
[PHP] pobierz, plaintext 

Wywala mi notice ze w isset chce uzyć non object dzieje się tak zapewne gdy zapytanie zwróci 0 ale przeciez to jest jakas wartosc dzieje sie tak samo przy uzyciu !empty(). Dlaczego tak sie dzieje skoro skrypt powinien się wykonać tylko w momencie pobrania czegos z bazy.

Ten post edytował thoran 11.11.2005, 14:16:28

[marcman]

Wywala się, bo '0' to nie jest obiekt, którego pola można sobie odczytywać.

Jeśli zwraca 0, to sprawdzaj:

[PHP] pobierz, plaintext 
<?php
 
if (!empty($cos))
{
  //robie cos
}
 
?>
[PHP] pobierz, plaintext 

Albo poczytaj sobie, co dokładnie zwraca AdoDB po wykonaniu Execute() i będziesz wiedział, jak na to zareagować.

Poza tym, na pewno jest w AdoDB funkcja zwracająca liczbę odczytanych z bazy wierszy ('0' - nie ma gracza o id=1, '1' - jest taki gość, '>1' - masz skopaną bazę, bo jest kilku graczy o tym samym id).

[SongoQ]

@thoran Radze popracowac nad sama budowa klasy bo trudno polapac sie co jest co, tyle tych pol, do metody przekazujesz tyle parametrow ze nie wiadomo co jest do czego. Klasa ma odzwierciedlac pewna rzeczywistosc a nie zawierac wszystko w sobie.

[thoran]

W sumie to nie moja klasa, ja osobiscie o oop jeszcze wiem niewiele poza samymi podstawami, dopiero ucze sie uzywać go odpowiednio Po za tym ta klasa jest bardzo uzyteczna, ze strony zaś logicznej moze się wydawac nieefektywna.