[php]in_array i problem z uploadem, in_array, niezrozumiały komunikat błędu Opcje

[felixk]

Witam, piszę upload plików i chciałbym aby uploadowany plik miał sprawdzane rozszerzenie i dopiero jeśli będzie ono takie jak sobie życzę dokonywany był upload. Napisałem to w następujący sposób:

[PHP] pobierz, plaintext 
$allow = array("jpg", "jpeg", "gif", "png", "bmp");  // linijka 48
 
$rozszerzenie_pliku = substr( $file_name, strrpos( $file_name, '.' ) +1 );  
 
    if (in_array($rozszerzenie_pliku, $allow))  
{  
 
    $result  =  move_uploaded_file($temp_name, $file_path);  
}  
 
else  
{  
        echo "<script language=\"javascript\">alert('Dozwolone tylko pliki graficzne')</script>";  
 
}
[PHP] pobierz, plaintext 

Jednak w trakcie pojawił się błąd o treści:

Warning: in_array() [function.in-array]: Wrong datatype for second argument in /home/c/h/o/chojnowski/www/test/index.php on line 48

i bladego pojęcia nie mam jak to rozwiązać, jakieś pomysły

Pozdrawiam i z góry dziękuję za pomoc

[darko]

zamień

[PHP] pobierz, plaintext 
$rozszerzenie_pliku = substr( $file_name, strrpos( $file_name, '.' ) +1 ); 
[PHP] pobierz, plaintext 

na

[PHP] pobierz, plaintext 
$tmp = pathinfo($file_name);
$rozszerzenie_pliku = $tmp['extension'];
[PHP] pobierz, plaintext 

Ten post edytował darko 24.12.2009, 14:23:44

[wookieb]

Tak jeden magiczny.
Przed ifem daj

[PHP] pobierz, plaintext 
var_dump($allow);
[PHP] pobierz, plaintext 

I zobacz czy rzeczywiście jest tablicą.

[felixk]

Tak jeden magiczny.
Przed ifem daj
[PHP] pobierz, plaintext

1. var_dump($allow);

[PHP] pobierz, plaintext
I zobacz czy rzeczywiście jest tablicą.

Chyba klops, zwraca wartość NULL. Jakiś pomysł

[darko]

Co po drodze do warunku if (in_array($rozszerzenie_pliku, $allow)) robisz ze zmienną $allow ? To kompletny kod?

[felixk]

Co po drodze do warunku if (in_array($rozszerzenie_pliku, $allow)) robisz ze zmienną $allow ? To kompletny kod?

Nie, tu jest tylko samo sprawdzenie pliku. Kompletny kod całego skryptu wygląda tak:

[PHP] pobierz, plaintext 
 
$site_name = $_SERVER['HTTP_HOST'];
$url_dir = "http://".$_SERVER['HTTP_HOST'].dirname($_SERVER['PHP_SELF']);
$url_this =  "http://".$_SERVER['HTTP_HOST'].$_SERVER['PHP_SELF'];
 
$upload_dir = "upload_files/";
$upload_url = $url_dir."/upload_files/";
$message ="";
$allow = array("jpg", "jpeg", "gif", "png", "bmp");
$rozszerzenie_pliku = substr( $file_name, strrpos( $file_name, '.' ) +1 );
$ext = substr( $file_name, strrpos( $file_name, '.' ) +1 );
 
//create upload_files directory if not exist
//If it does not work, create on your own and change permission.
if (!is_dir("upload_files")) {
    die ("Folder nie istnieje");
}
 
if ($_FILES['userfile']) {
    $message = do_upload($upload_dir, $upload_url);
}
 
print $message;
 
function do_upload($upload_dir, $upload_url) {
 
    $temp_name = $_FILES['userfile']['tmp_name'];
    $file_name = $_FILES['userfile']['name']; 
    $file_type = $_FILES['userfile']['type']; 
    $file_size = $_FILES['userfile']['size']; 
    $result    = $_FILES['userfile']['error'];
    $file_url  = $upload_url.$file_name;
    $file_path = $upload_dir.$file_name;
 
    //File Name Check
    if ( $file_name =="") { 
        $message = "Brak nazwy pliku";
        return $message;
    }
    //File Size Check
    else if ( $file_size > 500000) {
        $message = "Plik powyżej 500KB. Proszę zmniejszyć rozmiar do 500 KB.";
        return $message;
    }
    //File Type Check
 
    var_dump($allow); //zwróciło wartość NULL
 
    if (in_array($rozszerzenie_pliku, $allow)) //linijka o którą czepia się parser
{
    $result  =  move_uploaded_file($temp_name, $file_path);
 
}    
else
{
        echo "<script language=\"javascript\">alert('Dozwolone tylko pliki graficzne')</script>";
}
    return $message;
}
 
 
[PHP] pobierz, plaintext 

[darko]

To zmienia postać rzeczy, zasięg zmiennych, podstawy, funkcja do_upload nie widzi zmiennej $allow. Daj na początku w funkcji do_upload: global $allow, albo przekaż jej tablicę $allow parametrem.

Ten post edytował darko 24.12.2009, 14:42:58

[felixk]

To zmienia postać rzeczy, zasięg zmiennych, podstawy, funkcja do_upload nie widzi zmiennej $allow. Daj na początku w funkcji do_upload: global $allow, albo przekaż jej tablicę $allow parametrem.

Działa, tylko mi menda wyświetla całą tablicę jako wynik, oto efekt

[PHP] pobierz, plaintext 
array(5) {   [0]=>   string(3) "jpg"   [1]=>   string(4) "jpeg"   [2]=>   string(3) "gif"   [3]=>   string(3) "png"   [4]=>   string(3) "bmp" } 
[PHP] pobierz, plaintext 

średnio to wygląda, jak się mogę tego pozbyć To już efekt po weryfikacji


Dziękuję, znalazłem rozwiązanie. Jeszcze raz dziękuję za pomoc

Ten post edytował felixk 24.12.2009, 14:59:06

[darko]

zakomentuj var_dump($allow);