Parser HTML, Prosty, acz zasobożerny parser HTML Opcje

[b4rt3kk]

Witam,
przedstawiam do oceny parser HTML mojego autorstwa.

Plik do pobrania:
http://www.filedropper.com/htmlparserclassphp

Opis:
Zasada działania jest prosta, wystarczy zaincludować plik z klasą, która znajduje się w powyższym pliku. Następnie utworzyć nowy obiekt, np.

[PHP] pobierz, plaintext 
$parser = new HTMLparser();
[PHP] pobierz, plaintext 

Dostępne są 3 metody pobierania HTML, który będziemy parsować:
- z pliku

[PHP] pobierz, plaintext 
$parser->parseFile($dir);
[PHP] pobierz, plaintext 

- ze stringa

[PHP] pobierz, plaintext 
$parser->parseString($string);
[PHP] pobierz, plaintext 

- z URL

[PHP] pobierz, plaintext 
$parser->parseUrl($url);
[PHP] pobierz, plaintext 

Następnie wywołanie parsera:

[PHP] pobierz, plaintext 
$resource = $parser->find('div');
[PHP] pobierz, plaintext 

Jako wynik zmienna $resource będzie przechowywać tablicę obiektów DIV-ów znalezionych w parsowanym źródle, bądź jeden obiekt, jeśli w metodzie find jako drugi argument podamy liczbę (wtedy zwróci n-ty DIV z parsowanego źródła).

Odczyt informacji:
Odczyt jest bardzo prosty, każdy wynik zawiera obiekty text oraz html, ponadto jeśli DIV posiada atrybuty, np. class bądź id zostaną one zwrócone również w postaci obiektów. Przykład:

[PHP] pobierz, plaintext 
foreach ($resource as $div) {
  echo $div->text;
  echo $div->html;
  echo $div->class;
  // itd.
}
[PHP] pobierz, plaintext 

To tak po krótce tytułem wstępu. Jednak parser wymaga optymalizacji, ponieważ czas parsowania średniej wielkości źródła jest dosyć długi. Proszę o oceny, a także o wskazanie dalszej drogi rozwoju owego parsera (zastanawiałem się jeszcze nad wyszukiwaniem elementów HTML np. po class, bądź po ID, jednak w tym momencie nie mam jeszcze pomysłu jak to zrealizować, bo już teraz parser w obecnej formie jest dość "ciężki"). Pozdrawiam i dzięki za ewentualne sugestie.

EDIT:
Zapomniałem dodać, możemy również szukać dzieci naszych wynikowych obiektów, np.

[PHP] pobierz, plaintext 
foreach ($resource as $div) {
  echo $div->find('a', 0)->href;
}
[PHP] pobierz, plaintext 

Możliwość zagnieżdżania jest nieograniczona.

Ten post edytował b4rt3kk 4.10.2013, 13:31:07

[mstraczkowski]

Na pierwszy rzut oka:

1. Brak modyfikatorów dostępu przy metodach
2. Dwie klasy w jednym pliku (tak nie powinno się pisać)
3. Mało komentarzy w kodzie oraz mało poprawne tagi phpdoc (@param, a nie @params)
4. Projektując klasę nie powinno się już raczej używać kończącego tagu php ?>
5. Słaba estetyka kodu

Ten post edytował mstraczkowski 4.10.2013, 15:41:02

[b4rt3kk]

Na pierwszy rzut oka:

1. Brak modyfikatorów dostępu przy metodach
2. Dwie klasy w jednym pliku (tak nie powinno się pisać)
3. Mało komentarzy w kodzie oraz mało poprawne tagi phpdoc (@param, a nie @params)
4. Projektując klasę nie powinno się już raczej używać kończącego tagu php ?>
5. Słaba estetyka kodu

Kolego, przepraszam bardzo, ale jak to się ma do zasadności mojego pytania, czyli optymalizacji? To że poprawie komentarz z params na param nie wpłynie w żaden sposób na wydajność klasy. W wersji skompresowanej nawet nie będzie tych komentarzy. Nie oceniasz mojego zeszytu do polskiego, żeby patrzeć na estetykę, czy przypisy na marginesie... Czy mogę liczyć na jakieś sensowne porady? Preg_match nie jest tam znów tak wiele, ale byłbym wdzięczny jakby mi ktoś wskazał choć jeden który mogę zastąpić inną, szybszą funkcją. Lub wykazał błędność mojego toku rozumowania. Ogólnie rzecz biorąc, nie chodzi o estetykę, a o użyteczność.