mihaup
4.04.2007, 13:39:04
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
4.04.2007, 14:43:42
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.
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
4.04.2007, 18:25:47
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
Cytat(Turgon @ 4.04.2007, 17:25:47 )

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
5.04.2007, 10:52:27
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.
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.