Witam,

Ostatnio do mojego formularza kontaktowego dopisałem funkcje dodawania screenów i dopiero teraz zauważyłem że nie działa ona porawnie, a konkretniej chodzi oto że nawet gdy screen nie jest dodany, skrypt wyrzuca błąd formatu pliku :

Struktura kodu wygląda mniejwięcej tak :

[PHP] pobierz, plaintext 
  if (isset($_FILES['picture'])) {
 
$ext = end(explode(".", $_FILES["picture"]["name"]));
if ((($_FILES["picture"]["type"] == "image/gif") {
Tutaj funkcja bla bla bla
} else {
 
    echo 'Możesz wysłać tylko pliki graficzne';
 
}
} elseif (empty($_FILES['picture'])) {
$screen = FALSE;
}
 
[PHP] pobierz, plaintext 

I problem polega na tym iż gdy pliku nie ma, zamiast przejść od razu do elseif empty to skrypt wykonuje pierwszego if'a i wyrzuca błąd "Możesz wysać tylko pliki tekstowe"

Kolejność powinna być if, elseif, else

Edit: Sorki teraz zauważyłam zagnieżdżonego if-a

A weź sobie daj na samym początku var_dump($_FILES). I co się wtedy okaże? Zresztą Twoja pętla nie ma racji bytu. Zobacz:

[PHP] pobierz, plaintext 
if (isset($_FILES['picture'])) { // gdy istnieje tablica FILES
} elseif (empty($_FILES['picture'])) { // gdy tablica FILES nie istnieje i gdy jest jest pusta. Jak może być pusta skoro nie istnieje?
[PHP] pobierz, plaintext 

Czyli zamiast tego końcowego elseif wystarczy że dam po prostu if ?
czyli wtedy będzie jeśli tablica FILES jest pusta, wykonaj to co w klamrach.

Zgadza się ?

b4rt3kk

Php to magiczny język i nie sprawdza w empty czy zmienna istnieje wiec jak nie istnieje to jest równoznaczne z tym, że jest pusta...
http://php.net/manual/en/function.empty.php

problem tkwi własnie w isset dlatego iż nawet jeśli zmienna jest pusta to ona istnieje i dlatego wykonuje się ten kod, wiec zamiast tego daj

[PHP] pobierz, plaintext 
  if (!empty($_FILES['picture'])) { 
          ...
}
[PHP] pobierz, plaintext 

Czyli dla pewności, usuwam te elseif i daje samego if'a i nad if (isset($_FILES['picture'])) { daje var_dump($_FILES);

to elseif nie gra roli daj po prostu tak

[PHP] pobierz, plaintext 
  if (!empty($_FILES['picture'])) { 
          ...
} elseif (empty($_FILES['picture'])) {
$screen = FALSE;
}
[PHP] pobierz, plaintext 

To samo... cały czas wyrzuca tego elsa w pierwszym if'ie...

to w takim razie $_FILES['picture'] nie jest pusta, a co wygenerował var_dump?

Wyrzuca :
array(1) { ["picture"]=> array(5) { ["name"]=> string(0) "" ["type"]=> string(0) "" ["tmp_name"]=> string(0) "" ["error"]=> int(4) ["size"]=> int(0) } } Możesz wysłać tylko pliki graficzne

no tak czyli z tego wynika ze $_FILES['picture'] istnieje... to spróbuj może

[PHP] pobierz, plaintext 
 if (!empty($_FILES["picture"]["name"])) { ... }
[PHP] pobierz, plaintext 

Nie o to idzie, jest to logiczna implikacja. Jeśli coś nie istnieje, to nie może być puste, bo tego nie ma.




var_dump miałeś sobie dać przed tymi if-ami. Chciałem żebyś zobaczył, że przesłanie pustego formularza i tak powoduje utworzenie tablicy $_FILES. Sprawdzanie isset ma tylko rację bytu jeśli do podanej strony użytkownik może się dostać bez konieczności wysyłania formularza.

b4rt3kk

owszem, z poziomu logiki masz rację, ja temu nie przeczę, bo to jest oczywiste, że jak coś nie istnieje to wiadomo, że nie może być puste... ale tak jak powiedziałem php się tym nie przejmuje bo dla niego coś jest puste w tych przypadkach:
co zresztą wyraźnie pieszą w manualu...

zresztą prosty przykład :
http://ideone.com/6TWHbz