Witam, poniżej zamieszczam moje pierwsze klasy. Chcę zabrać się za artykuł o frameworkach z wortalu PHP.pl (właśnie go drukuje) i chce wiedzieć co robię źle w programowaniu obiektowym

SimpleRSSElement:
http://manual.babciastefa.boo.pl/download/...erss_class.phps
http://manual.babciastefa.boo.pl/module,vi...impleRSSElement - Manual

SimpleRSSBuilder:
http://manual.babciastefa.boo.pl/download/...lder_class.phps
http://manual.babciastefa.boo.pl/module,vi...impleRSSBuilder - Manual


Kodu nie jestem w stanie zamieścić na forum (za długi - ponad 20 kb)

Wiem że jest brak poprawnego kodowania w klasach (później to poprawie )
Piszcie wszystkie uwagi co do kodu...

Dziękuję, Babcia@Stefa

SimpleRSSElement - nawet dobra klasa, trzyma poziom,
Co do rssBuildera... nie lepiej użyć simplexml? Jakiś poziom abstrakcji by był. Tak to wygląda średnio.

Jest dobrze. Nie widzę żadnych rażących błędów.

To SimpleXML teæ tworzy dokumenty XML?
Ja nie wiedziaøem o tym...

Dziékujé, Babcia@Stefa

Pierwsza klasa wyglada ok. Drobna uwaga kosmetyczna - kilka (chyba 5) razy powtarza Ci sie

[PHP] pobierz, plaintext 
<?php
$xml = @simplexml_load_string($this->Contents);
?>
[PHP] pobierz, plaintext 

Mozna by jakos wyeliminowac te powtorzenia.

Jesli chodzi o Builder, to juz zostalo powiedziane o SimpleXML.

Acha - no i bledy jezykowe w wyjatkach ;]

Pzdr

Rozumiem, a więc wrzuce zmienną chronioną:
protected $XML;

Do konstruktora:
$this->XML = @simplexml_load_string($content);

I wszędzie wrzuce:
$xml = $this->XML // żeby nie przebudowywać całego kodu

I będzie dobrze ;]

Pozdrawiam, Babcia@Stefa