Klasa do uploadu plikow, Przedstawiam swoje wypociny, prosze o konstruktywna krytyke, rady, pop Opcje

[mihaup]

Witam.
http://cpaste.com/195 <- Pełny kod klasy, niestety za dużo znaków aby go tu umieścić. Szczególy w komentarzach.

[Zeman]

tak jak przejrzałem - czy jeśli zdefiniuje dozwolone rozszerzenia na 'jpg|png|gif' i wgram plik o rozszezreniu .GIF (z duzych liter) to klasa go przyjmie ?

Pozwiodronka,
Zeman.

[Prph]

Dlaczego wbudowales w ta klase obsluge obrazkow? Zakladasz, ze sluzy ona do uploadowaia plikow graficznych? Ladnie byloby, gdyby klasa miala pluginy i np. okreslasz, ze dla plikow JPEG szerokosc nie moze byc wieksza nic 350px;

Pozdrawiam, Adrian.

[Turgon]

Przejrzałem klasę i w mojej skromnej opinii klasa powinna zajmować się tylko uploadowanie plików i tyle.
Prph: Myślę, że pluginów nie powinno być. Np. klasa Image wtedy by kontrolowała co i jak, a klasa Upload tylko umieszczała na serwerze te pliki i nimi zarządzała.
Mam nadzieję, że w miarę jasno wyraziłem o co mi chodzi.

Pozdrawiam
Łukasz

[Prph]

Prph: Myślę, że pluginów nie powinno być.

Jakis powod? Pluginy to akurat swietny sposob na budowanie elastycznych obiektow. Klasa do uploadu tez by je mogla miec. Jezeli jestes takim przeciwnikiem, skomentuj Request w Rapide - http://rapideframework.googlecode.com/svn/...quest.class.php (pluginy w http://rapideframework.googlecode.com/svn/...apide/Request/).

@mihaup: Jak wspomnialem - ogolnie calkiem dobra klasa, ale walidacja image to nie jej broszka

Adrian.

[Zeman]

Ja bym proponował raczej żeby wyrzucić z klasy obsługę obrazków i wtedy niech to b będzie klasa bazowa, a klasa uploadująca obrazki niech dziedziczy po tej bazowej.

No ale co głowa to inne techniki programowania, jak dla mnie to za mały pierdół żeby pluginy robić.

W sumie tak jak teraz jest jest też dobrze jak dla mnie.