[Kohana] Nie widzi wartości zmiennej w metodach klasy Opcje

[Max Damage]

Cześć, mam dziwny problem, ale najpierw kod:

[PHP] pobierz, plaintext 
class Profil_Controller extends Index_Controller {
	protected $id;
	protected $profil;
	public function __construct(){
		parent::__construct();
		$this->profil=new Profil_Model;
		$this->template->menu=new View('profil_menu');
	}
	public function index($index){
		if(!$this->auth->logged_in() || !isset($index))
			url::redirect('index.php/index');
		$this->id=$index;	
		$this->template->menu->id=$index;
		$wynik=$this->profil->informacje($index);
		$this->template->content=new View('informacje');
		$this->template->content->informacje=$wynik;
		$this->template->render(TRUE);
	}
	public function informacje(){
		echo $this->id;
		$id = ($this->id == $this->session->get('id')) ? $this->session->get('id') : $this->id;
		$wynik=$this->profil->informacje($id);
		$this->template->content=new View('informacje');
		$this->template->content->informacje=$wynik;
		$this->template->render(TRUE);
	}
[PHP] pobierz, plaintext 

Przy wejściu do profilu użytkownika odpalam metodę index, i wszystko działa, gdy wybiorę w menu informacje, nie pojawiają się żadne informacje. Problemem jest to że nie wypisuje żadnej wartości $this->id.
W tym kontrolerze mam także inne metody: do zmiany hasła, swoich danych, uploadu avatara oraz ulubione. Nie jestem pewien czy dobrze to podzieliłem, czy nie porozbijać tego jakoś inaczej (dodatkowe kontrolery etc.)?

[skowron-line]

Odpalasz akcje informacje a wartość $id ustawiasz w ackoj index. Pomyśl chwile nad tym.

[Max Damage]

Hm, zawsze przy ładowaniu strony tworzony jest nowy obiekt klasy po czym wywoływana jest dla niego odpowiednia metoda z podanymi argumentami?
Jeśli tak, to czy mogę jakoś inaczej przechować taką zmienną w klasie? Pozmieniałem 3 linijki na takie:

[PHP] pobierz, plaintext 
protected static $id;
W index:
self::$id=$index;	
W informacje:
$idd =(self::$id == $this->session->get('id')) ? $this->session->get('id') : self::$id;
[PHP] pobierz, plaintext 

Ale nadal nie działa. Nie wiem jak to napisać.

[skowron-line]

@Max Damage nie mam bladego pojęcia o czym ty mówisz
nie możesz poprostu przekazać tego ID jako parametr methody informacje

[Max Damage]

No pewnie że mogę, równie dobrze mógłbym go przechowywać w sesji, ale nie chce Chcę to id przechowywać w klasie jako składową, na pewno jest na to jakiś sposób.

[skowron-line]

No pewnie że mogę, równie dobrze mógłbym go przechowywać w sesji, ale nie chce Chcę to id przechowywać w klasie jako składową, na pewno jest na to jakiś sposób.

A słyszałeś o zasadzie KISS

[PHP] pobierz, plaintext 
class A
{
	protected $id;
 
	public function index()
	{
		$this->id = 10;
	}
 
	public function informacje()
	{
		echo $this->id;
	}
}
[PHP] pobierz, plaintext 

i teraz odpal sobie to na 2 sposoby
1 tak jak ty robisz

[PHP] pobierz, plaintext 
$q = new A;
$q->informacje();
[PHP] pobierz, plaintext 

i 2 właściwy

[PHP] pobierz, plaintext 
$q = new A;
$q->index();
$q->informacje();
[PHP] pobierz, plaintext 

Edit:

[PHP] pobierz, plaintext 
self::$id=$index;
[PHP] pobierz, plaintext 

A tak też możesz sobie pobierać wartość pod warunkiem że ta zmienna ma zadeklarowaną wartość. podstawy OOP się kłaniają.

Ten post edytował skowron-line 7.11.2009, 17:40:30

[Max Damage]

O żadnej zasadzie kiss nie słyszałem, ale kod który podałeś jest równoważny mojemu z pierwszego postu. Jeśli chodzi o odpalenie tego to muszę napisać wtedy coś takiego: profil/index/3/informacje ,a ja właśnie chciałem uniknąć parametrów i mieć coś takiego: profil/informacje. A poza tym najpierw i tak odpalam index, więc zmienna powinna być przepisana do składowej, jak wywołuje potem informacje to powinienem ją odczytać. Ze static kombinowałem bo założyłem, że za każdym razem jest tworzony przez framework nowy obiekt klasy, ale w sumie też bez sensu bo przecież jest wielu użytkowników. Problem chyba chwilowo nie do przeskoczenia, więc jednak zrobię po prostu metodę informacje (oraz inne) z parametrem. Dzięki za chęć pomocy :)

[marcio]

Kod

A poza tym najpierw i tak odpalam index, więc zmienna powinna być przepisana do składowej, jak wywołuje potem informacje to powinienem ją odczytać

Problem w tym ze za kazdym refreshem strony tworzy sie nowy obiekt i jego pola tak jakby sie ustawialy na defaultowe wartosci.

[skowron-line]

O żadnej zasadzie kiss nie słyszałem,

http://pl.wikipedia.org/wiki/KISS_%28regu%C5%82a%29