Elektroda.pl
Elektroda.pl
X
Proszę, dodaj wyjątek www.elektroda.pl do Adblock.
Dzięki temu, że oglądasz reklamy, wspierasz portal i użytkowników.

[php] - Multiupload - prośba o sprawdzenie skryptu

piottro1 18 Lut 2014 14:13 9327 2
  • #1 18 Lut 2014 14:13
    piottro1
    Poziom 10  

    Witam,
    w założeniu miał powstać formularz umożliwiający wysłanie kilku obrazków na serwer. Można zwiększać ilość pól maksymalnie do sześciu (+).
    Zabezpieczenia:
    - sprawdzanie czy dodano plik,
    - dozwolone formaty plików: jpg, png, jpeg, gif,
    - sprawdzanie czy taki plik już istnieje,
    - limit wagi pliku: 500KB
    Po zatwierdzeniu formularza są wyświetlane informacje o pliku:
    - numer dodawanego pliku,
    - rozmiar pliku,
    - ścieżka dostępu,
    - nazwa pliku,
    - typ pliku,
    - miniatura obrazka

    Pliki zapisują się poprawnie w wybranej lokalizacji. W przypadku błędów, oddawane są komunikaty:
    - brak pliku,
    - plik zbyt duży,
    - niedozwolony format,
    - taki plik już istnieje

    index.php

    Kod: php
    Zaloguj się, aby zobaczyć kod


    upload.php
    Kod: php
    Zaloguj się, aby zobaczyć kod



    Dodaję kilka zrzutów:
    Spoiler:

    A. pusty formularz, (+) dodaje kolejne pola
    [php] - Multiupload - prośba o sprawdzenie skryptu

    B. dodane obrazki do formularza
    [php] - Multiupload - prośba o sprawdzenie skryptu

    C. wynik po wysłaniu
    [php] - Multiupload - prośba o sprawdzenie skryptu

    D. dodane obrazy (istniejący[1], nowy[2], zbyt duży[3]), dodane niedozwolone formaty plików[4, 5], brak dodanego pliku[6]
    [php] - Multiupload - prośba o sprawdzenie skryptu

    E. komunikaty błędów po wysłaniu
    [php] - Multiupload - prośba o sprawdzenie skryptu


    Wydaje mi się, że wszystko działa, ale ze względu na małe doświadczenie nie mam pewności co do poprawności kodu. Zapewne można go poprawić a może nawet rozwinąć (myślę o dodaniu opcji zmiany nazwy wysyłanych plików).

    Osoby chętne i bardziej doświadczone proszę o sprawdzenie tego co napisałem.
    Z góry dziękuję.

    (w załączniku dodaję cały skrypt)

    0 2
    Załączniki:
  • Pomocny post
    #2 18 Lut 2014 22:52
    Maskara
    Poziom 12  

    Przede wszystkim komentuj kody, nawet przy tak prostym skrypcie, kiedy przyjdzie coś poprawić po dłuższym czasie głupio brzmiący komentarz może sporo pomóc.
    Plik upload.php ma strukturę z grubsza:

    Kod: php
    Zaloguj się, aby zobaczyć kod

    Moim zdaniem jest to nieeleganckie i na dłuższą metę może być to zły nawyk utrudniający czytanie kodu.

    Ogólnie nie jestem zwolennikiem wielokrotnie pozagnieżdzanych ifów, jeśli użyłbyś instrukcji continue wyglądałoby to czytelniej i elegancko.
    Kod jest OK, ale popracuj nad czytelnością kodu.

    0
  • #3 27 Lut 2016 20:43
    piottro1
    Poziom 10  

    Dzięki za sugestie.

    0