Drukowana wersja tematu

Napisany przez: b4x 27.05.2013, 16:18:44

Witam.

Udostępniam movieDownloader - wyciąganie linków z serwisów hostujących filmy.

Na chwilę obecną zwraca tylko linki - będę troszkę rozwijał o np. zapis (pobieranie na HDD)

Obsługiwane serwisy: Anyfiles.pl, * Maxvideo.pl, Sprocked + niedługo kolejne.

* Wymaga konta premium.

Dostępny na Githubi'e pod adresem: https://github.com/nSolutionsPL/movieDownloader

Napisany przez: Crozin 27.05.2013, 16:43:59

1. Brak wsparcia dla Composera.
2. Niby wykorzystujesz przestrzenie nazw, a mimo to korzystasz ze Starej_Konwencji_Nazewnicta.
3. Ten "prosty" autoloader jest zbędny.
4. Interfejs Downloadera jest źle napisany. W żaden sposób nie umożliwia przekazania linku do filmu, który chcemy pobrać.

Sam interfejs dla użytkownika też do najwygodniejszych nie należy. Trzeba ręcznie na podstawie filmu określić z czego skorzystać, a to mogłoby być zrobione automatycznie. Wystarczyłaby jakiś obiekt skupiający wszystkie Downloadery, gdzie każdy z nich implementując interfejs Downloader ma metodę supports($url) zwracająca TRUE/FALSE. W użyciu wyglądałoby to już całkiem sensownie:

[PHP] pobierz, plaintext 
$downloader = ...;
 
http://www.php.net/echo $downloader->get('http://sprocked.com/movie/show/ORGa')->getLink();
http://www.php.net/echo $downloader->get('http://video.anyfiles.pl/videos.jsp?id=32364')->getLink();
[PHP] pobierz, plaintext 

Wnętrze metody get():

[PHP] pobierz, plaintext 
public function get($url) {
  $urlParts = http://www.php.net/parse_url($url);
 
  // rzuc wyjatkiem gdy $urlParts nie jest poprawne
 
  foreach ($this->downloaders as $downloader) {
    if ($downloader->supports($urlParts)) { // albo samo $urlParts['host']
      return $downloader->get($urlParts);
    }
  }
 
  // rzuc wyjatkiem, nie ma czego zwrocic
}
[PHP] pobierz, plaintext 

Napisany przez: b4x 27.05.2013, 17:14:19

@Crozin - dzięki za krytykę :-)

Faktycznie troszkę pomieszałem, ale później trochę zmienie...