Pomoc - Szukaj - Użytkownicy - Kalendarz
Pełna wersja: Klasa do uploadu plikow
Forum PHP.pl > Inne > Oceny
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
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 winksmiley.jpg

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.
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.
Invision Power Board © 2001-2025 Invision Power Services, Inc.