Popularny post Lactriksen 108 Napisano 26 Stycznia Popularny post Udostępnij Napisano 26 Stycznia Darmowy skrypt na hosting zdjęć [PHP]. Free images hosting script. Demo: http://lactriksen.pl/hosting/ Download: Ukryta zawartość Przyznaj reakcję lub odpowiedz w temacie aby zobaczyć ukrytą zawartość. Pankrok, darkinzo kinzo, -n3veR i 8 innych dodali reakcje 8 1 2 Odnośnik do odpowiedzi Udostępnij na innych stronach
Ekspert Pankrok 127 Napisano 28 Stycznia Ekspert Udostępnij Napisano 28 Stycznia (edytowane) Fajnie ze kodzisz 🙂 co do skryptu nasuwa mi się kilka uwag: poczytaj o standardzie kodu PSR-2 albo lepiej PSR-12 pliki oraz nazwy zmiennych powinny być po angielsku, nie używaj zmiennych które nie wskazują jaka jest ich zawartość, nie ma sensu przypisywać zmiennej $folder do $katalog: Spoiler <?php // źle $katalog="$folder"; $kat=opendir($katalog); //lepiej $catalogDirectory = opendir($imgPath); zamiast wielu elseif lepiej użyć switch: Spoiler <?php // nope if($typfoto == 'png'){ header("content-type: image/png"); $img = imagecreatefrompng($imge); }elseif($typfoto == 'bmp' ){ header("content-type: image/bmp"); $img = imagecreatefrompng($imge); }elseif($typfoto == 'jpeg' || $typfoto == 'jpg'){ header("content-type: image/jpeg"); $img = imagecreatefromjpeg($imge); }elseif($typfoto == 'gif'){ header("content-type: image/gif"); $img = imagecreatefromgif($imge); } // prawda że ładniej? switch ($typfoto) { case 'png': header("content-type: image/png"); break; case 'bmp': header("content-type: image/bmp"); break; case 'jpeg': case 'jpg': header("content-type: image/jpeg"); break; case 'bmp': header("content-type: image/gif"); break; } $img = imagecreatefromgif($imge); poszukaj sobie jak korzystać z funkcji, klas i przestrzeni nazw dzięki temu podzielisz kod, będzie on bardziej czytelny i łatwiejszy w utrzymaniu, obecnie nie includuje się plików, zamiast tego poszukaj informacji o czymś takim jak composer i autoload PSR-4 (Wymaga znajomości klas, funkcji i przestrzeni nazw) złym nawykiem jest mieszanie logiki aplikacji i części wizualnej polecam np wzorzec MVC i skorzystanie np z szablonów TWIG, kiedy porównujesz wartości korzystaj z === a nie == jeśli coś ma być intem to zawsze ma być intem, powodzenia w nauce 😄 Edytowane 28 Stycznia przez Pankrok jeszcze jedna uwaga -n3veR, Lactriksen, ravenekse i 4 innych dodali reakcje 5 1 1 Odnośnik do odpowiedzi Udostępnij na innych stronach
esej 6 Napisano 28 Stycznia Udostępnij Napisano 28 Stycznia Odświeżając ekran przesłanego obrazu udało mi się w zaledwie 2 minuty skopiować plik kilka tysięcy razy 😛 SeNioR i Pankrok dodali reakcje 1 1 Odnośnik do odpowiedzi Udostępnij na innych stronach
Założyciel SeNioR 9 337 Napisano 29 Stycznia Założyciel Udostępnij Napisano 29 Stycznia Lactriksen przyznaj się kodziłeś Albicla? 😛Z 10 lat temu też tworzyłem takie rzeczy, dobry sposób do naukę PHP 😄 Jak trafnie zauważył esej wystarczy rozszerzenie na automatyczne odświeżanie co x sekund i przez niezabezpieczony formularz POST można załadować ten sam obraz tysiące razy. Pula unikalnej generowanej liczby jest za mała w końcu skrypt wygeneruje taką samą nazwę pliku i zostanie nadpisany albo skrypt wypluje błąd. Funkcja uniqid daje większe pole do zabawy. boleknowak dodał reakcję 1 Odnośnik do odpowiedzi Udostępnij na innych stronach
Rupert 9 Napisano 23 Lutego Udostępnij Napisano 23 Lutego http://karol-drag.eu/skrypty-php-i-js wersja v2 to jest ta którą dodałeś nie ładnie przywłaszczać sobie czyjejś pracy ;) paZZiuu dodał reakcję 1 Odnośnik do odpowiedzi Udostępnij na innych stronach
Rekomendowane odpowiedzi
Problem wciąż nierozwiązany? Dodaj swoją odpowiedź
Jeśli chcesz dodać odpowiedź, zaloguj się lub zarejestruj nowe konto. Jedynie zarejestrowani użytkownicy mogą komentować zawartość tej strony.
Zarejestruj nowe konto
Załóż nowe konto. To bardzo proste!
Zarejestruj sięZaloguj się
Posiadasz już konto? Zaloguj się poniżej.
Zaloguj się