Forum PHP.pl > zmiana pól obiektu

Pomoc - Szukaj - Użytkownicy - Kalendarz

angerthor

18.02.2009, 16:41:37

Witam,

Oto mój problem : mam klasę strona.inc

[PHP] pobierz, plaintext 
<?php
class Strona
 {
protected $tytul
 
 
function __construct ($tytul ="moja www")
{
$this -> tytul = $tytul
}
 
function wyswietl ()
{
 
echo '<html><head><title>'.$this -> tytul.'</head><body>';
wyswietl_body();
echo '</body></html>';
}
 
function wyswietl_body()
{
// tutaj jakis skrypt ktory includuje odpowiednie strony
}
 
 
}
?>
[PHP] pobierz, plaintext

pozniej plik index.php

[PHP] pobierz, plaintext 
<?php
require(strona.inc);
$strona = new Strona();
$strona ->wyswietl();
?>
[PHP] pobierz, plaintext

funkcja wyswietl body na podstawie urla includuje opowiedni plik, co onzacza, ze zawsze operujemy na pliku index.php

i teraz moj problem, mamy np. podstrona.php, ktora jest includowana do czesci body i chcemy w niej zmienic pole tytul

[PHP] pobierz, plaintext 
<?php
$this - > tytul = "nowy tytul";
?>
[PHP] pobierz, plaintext

rzeczywiscie po wpisaniu echo $this ->tytul, tytuł rzeczywiście jest zmieniony, ale w gornym pasku przegladarki pozostaje poprzedni.

Czy mozna cos na to poradzic z poziomu strony "podstrona.php", czy jedyne wyjscie to odpowiedni kod w pliku index.php, ktory bedzie dawal konstruktorowi opcjonalny parametr tytul ?

Mam nadzieje, ze to w miare jasno wytlumaczylem

Z gory dzieki

bim2

18.02.2009, 18:45:27

protected $tytul
zamień na
public $tytul

Foxx

18.02.2009, 18:54:24

bim2 - nie w tym rzecz. Przecież ta klasa może korzystać z własnego pola protected.
Analizowałem Twój kod, angerthor i dochodzę do wniosku, że z ogólną koncepcją jest
coś nie tak: wyświetlasz stronę nadając jej tytuł w konstruktorze, potem wyświetlasz title za
pomocą metody wyswietl() a potem tresc w body. Ale jeżeli w tej tresci body zmienisz title to
żeby go wyświetlić w pasku przeglądarki, musiałbyś ponownie wyświetlić fragment kodu z <title>.
Ale jeżeli przeładujesz stronę - cały proces powtórzy się od nowa i znowu będziesz miał domyślnie
przydzielony title w konstruktorze + zmienną ustawioną na nowy tytuł ale niewyświetloną.

angerthor

18.02.2009, 20:41:26

No właśnie tak samo myślę. Zastanawiałem się tylko czy to można jakoś obejść

, ale jak się nie da to trudno.

nexik

18.02.2009, 21:29:46

renderowanie kodu powinno byc na koncu - robienie takiego spaghetti jest zlym pomyslem, oczywiscie jak sie upierasz przy tym rozwiazaniu mozna

napisac funkcje wyswietlbody ktora podmienia title i zwraca string (kod html) przypisac go do zmiennej $body
a potem dopiero wyswietlic wszystko w templatce

angerthor

20.02.2009, 21:12:18

oky, to proszę napiszcie, czy moja koncepcja jest optymalna:

Opierajac sie na formie strony z pierwszego postu doszedlem do wniosku, ze aby zmieniac temat zamieszcze odpowiedni kod w pliku index.php, tam gdzie jest wywolywany konstruktor.

Mój problem: mam stronę z różnymi podstronami, m.in. z artykułami. Na wszystkich podstronach chce mieć title="moja www", na stronie z artykułami chcę mieć title="tytuł" pobrany z bazy danych. Rozbijając urla mogę się dowiedzieć, czy ten url prowadzi do artykułu, czy do jakiejś innej podstrony, i modyfikując plik index.php mam :

[PHP] pobierz, plaintext 
<?php
require(strona.inc);
 
$adres = robij_url(); // rozbija url, zwracajac tablice
 
$pozycja = "";
if ($adres[2] == "artykul")
{
        // tutaj lacze sie z baza Mysql i pobieram z niej tytuł odpowiedniego artu
 
    $tytul = $art['tytul'];
 
    $strona = new Strona($tytul);
 
}
else {
$strona = new Strona();
}
 
$strona ->wyswietl();
?>
[PHP] pobierz, plaintext

czy taka koncepcja jest w miarę wydajna ? Pytam bo dopiero nie dawno przestawiłem się ze strukuralnego php, na obiektowy i do konca nie jestem pewien czy moj kod jest wydajny ;P

Crozin

20.02.2009, 21:47:33

[PHP] pobierz, plaintext 
<?
 
$strona = new Strona();
 
if(/* jest artykulem */){
  $strona->setTitle('Tytul');
}
 
$strona->display();
?>
[PHP] pobierz, plaintext

angerthor

22.02.2009, 17:47:33

fakt, wielki dzięki,

a jeszcze takie pytanie ogólne: czy lepiej jest tworzyć wszystkie settery (np. setTitle, setDesciption, setSth itd.), czy stworzyc jedna funkcje __set ?

[PHP] pobierz, plaintext 
<?php
function __set ($nazwa, $wartosc)
    {
        $this -> $nazwa = $wartosc;
    }
?>
[PHP] pobierz, plaintext

i pozniej juz tylko

[PHP] pobierz, plaintext 
<?php
$this -> tytul = "cos tam";
?>
[PHP] pobierz, plaintext

Crozin

22.02.2009, 23:30:37

Jeżeli __set() ma tak wyglądać to lepiej go nie robić.

ayeo

23.02.2009, 00:27:49

Przenoszę na PHP.

Pozdrawiam!

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.