Przypisanie obiektu w klasie Opcje

[rahul]

Witam. Otoz pisze sobie takie cos i wywala mi blad :/

[PHP] pobierz, plaintext 
 
$notification = $notification_M->getNotificationByTitle('unregistered');  // zwraca obiekt notifikacja
$users = $user_M->getUnregisteredNotified($notification);  // zwraca uzytwkonikow z bd + tworzy nowy obiekt notifikacja_wyslana ktora nadpisuje wczesniejsza. 
 
class UserManager { 
 
public function getUnregisteredNotified($notification) 
{ 
        $sql = mysql_query("SELECT users.id, notifications_sent.id as 'notification_sent_id' FROM users, notifications_sent
					WHERE users.status = 2
					AND users.id = notifications_sent.user_id
					AND notifications_sent.sent = 0
					AND users.time < DATE_SUB(NOW(), INTERVAL '264' DAY)
					ORDER BY users.time DESC
						   ");
		$this->handler($sql); 
		return $this->userContainer; 
}
 
private function handler($sql)
{
		while($row = mysql_fetch_assoc($sql))
		{
			$user = new User();
			$user->setId($row['id']);
			$this->userContainer[] = $user;
 
			if($row['notification_sent_id'])
			{
				notificationManager::setSentNotification($row['notification_sent_id']);
			}
	       }
}
 
}// koniec klasy userManager.
 
class notificationManager { 
 
 
	private $sentNotification;
 
        public function setSentNotification($param) 
	{
		$this->sentNotification = new SentNotification(); 
		$this->sentNotification->setSentId($param);
	}
 
	public function getSentNotification() 
	{
		return $this->sentNotification;
 
	}
} // koniec klasy 
[PHP] pobierz, plaintext 

Generalnie chodzi o to ze przypisuje do zmiennej private sentNotificaion klasy NotifiacionManager obiekt new SentNotification. I wszystko cacy, moge kozystac z metod tego obiektu dzieki tej referencji. Lecz jak sie odwoluje do tego obiektu spoza klasy to juz wtedy mi mowi ze jest gupi i ze " Call to a member function getSentId() on a non-object in "

[PHP] pobierz, plaintext 
$dupa = $notification_M->getSentNotification()->getSentId();
[PHP] pobierz, plaintext 

Czemu tak ? Co pominalem.

[nospor]

Nigdzie nie widzę, gdzie inicjalizujesz obiekt $notification_M ani nigdzie nie widzę byś na tym obiekcie wykonywał setSentNotification()
Nic więc dziwnego, że getSentNotification() zwraca ci null.

php nie jest głupi i jak ci mowi, że coś nie jest obiektem, znaczy że nie jest obiektem i lepiej szukaj błędu w swoim kodzie a nie zwalasz winę na php.

[rahul]

Nie widzisz bowiem nie umiescilem tego w opisie. Problem glownie polega na tym ze po przypisaniu obiektu wewnatrz klasy do zmiennej private nie moge sie do niego dostac. ale aby bylo bardziej klarownie to napisze ci wszystko:

[PHP] pobierz, plaintext 
$user_M= new UserManager($db); 
$notification_M = new notificationManager($db); 
 
$notification = $notification_M->getNotificationByTitle('unregistered'); 
$users = $user_M->getUnregisteredNotified($notification);
 
if($users) 
{	foreach($users as $user)
	{
		$notification_M->updateSentNotification($user, $notification);
		$dupa = $notification_M->getSentNotification()->getSentId();  // tu wywala blad 
                 $dupa = $notification_M->sendNotification; // Tak tez nie dziala jakby co ;) 
	}
}
 
class notificationManager { 
 
        private $db;
	public $sendNotification;   // TU PRZYPISUJE MOJ OBIEKT , spoza klasy nie dziala 
 
	public function __construct(Database $db)
        {
		$this->db = $db;
	}
 
	public function setSentNotification($param) 
	{
		$this->sendNotification= new SentNotification(); 
		$this->sendNotification->setSentId($param);
                print_r($this->sendNotification) // To zadziala pokaze mi wszystko co chce , ale juz spoza klasy niiii :( 
	}
 
	public function getSentNotification() 
	{
		return $this->sendNotification;
	}
 
 
        public function getNotificationByTitle($title) // FETCHES ALL INFORMATION (ID, SUBJECT, CONTENT , INTERVAL ) OF NOTIFICATION
	{
		$sql = mysql_query("SELECT * FROM notifications WHERE title = '$title' "); 
		$notification = $this->handler($sql);
		return $notification;
	}
 
        private function handler($sql)
	{
		$row = mysql_fetch_assoc($sql); 
 
			$notification = new Notification(); 
			$notification->setId($row['id']); 
			$notification->setSubject($row['subject']); 
			$notification->setBody($row['body']); 
			$notification->setInterval($row['interval']); 
 
		return $notification;
	}
 
}// koniec notificationManagera
 
 
class UserManager { 
 
        public function getUnregisteredNotified($notification) // zwraca tez note_id
	{ 
		$sql = mysql_query("SELECT users.id, notifications_sent.id as 'notification_sent_id' FROM users, notifications_sent
							WHERE users.status = 2
							AND users.id = notifications_sent.user_id
							AND notifications_sent.sent = 0
							AND users.time < DATE_SUB(NOW(), INTERVAL '264' DAY)
							ORDER BY users.time DESC
						   ");
		$this->handler($sql); 
		return $this->userContainer; 
	}
 
       private function handler($sql)
	{
		while($row = mysql_fetch_assoc($sql))
		{
			$user = new User();
			$user->setId($row['id']);
			$this->userContainer[] = $user;
 
			if($row['notification_sent_id'])
			{
				notificationManager::setSentNotification($row['notification_sent_id']);  // TUTAJ wywoluje metode do stworzenia nowej Notificaji. 
			}
		}
	}
 
 
 
}// koniec klasy 
 
[PHP] pobierz, plaintext 

Kurcze, to chyba wszystko , chyba ze mam jeszcze umiescic opisy kontenerow ..

Ten post edytował rahul 6.09.2011, 14:29:05

[nospor]

Powtarzam:
Nigdzie nie wywołujesz setSentNotification() na obiekcie $notification_M

Jeśli myślisz, że to:
notificationManager::setSentNotification($row['notification_sent_id']);
załatwia sprawę to się mylisz. To coś odpala metodę statyczną klasy, która działa na $this. Jest to błędne bo mieszasz w tej chwili metody statyczne klasy z odwołaniem do obiektu klasy - to są dwie różne rzeczy.

[rahul]

ok, dzieki.
Widzisz , ja myslalem ze ta statyczna metoda wszystko zalatwi. Czy zatem musze przekazac obiekt notificationManager do klasu UserManager i wtedy odpalic funkcje ?

[nospor]

I tak i nie
TAK: składniowo wszystko będzie ok
NIE: logicznie będzie bez sensu. Przecież do setSentNotification przekazujesz ID a to chyba z każdym userem jest inne i po każdym wywołaniu będzie się nadpisywać.

[rahul]

bedzie sie nadpisywac to prawda, tylko jest on w petelce wiec bedzie dobrze przypisywal swoje parametry do uzytkownika w tym akurat przypadku. Ewentualnie moge go wrzucic do jakiegos kontenera w sensie array'a. tylko wtedy tez bede po nim musial przejsc jakas petla.

[PHP] pobierz, plaintext 
 
$notification = $notification_M->getNotificationByTitle('unregistered'); 
$users = $user_M->getUnregisteredNotified($notification); 
 
if($users) 
{	foreach($users as $user)
	{
		$notification_M->updateSentNotification($user, $notification);
		$notification_M->InsertTimesWhenNotified($notification_M->getSentNotification()->getId);	
	}
}
 
 
 
[PHP] pobierz, plaintext 

A moje pytanie jeszcze takie dodatkowe - czy jakims prostym sposobem moge bez problemu odziedziczyc wszystkie wartosci poprzedniej notyfikacji do tej nowej czy musze je od nowa przypisywac :/
Chce tylko dodac jeden parametr a reszte zostawic.