podejrzane działanie is_uploaded_file(), wykorzystanie is_uploaded_file() z if Opcje

[yarexxx]

Witajcie,

może ktokolwiek już się spotkał z podobną sytuacją: za pomocą is_uploaded_file() robię upload pliku przez formularz (type='file' name ='attach').
Skrypt wygląda tak:

[PHP] pobierz, plaintext 
  public function upload_single_file(){
 
 
    if(is_uploaded_file($_FILES['attach']['tmp_name']))
 
      $this->file_path = "c:wamp/www/live/modules/hd/attachements/";
 
			$this->file_itself = substr($_FILES['attach']['name'], 0, strrpos($_FILES['attach']['name'], ".")) . "_" . time() . "." . substr($_FILES['attach'] ['name'], strrpos($_FILES['attach']['name'], ".")+1);
 
			move_uploaded_file($_FILES['attach']['tmp_name'], $this->file_path.$this->file_itself);
 
		 return $this->file_itself =  "'".$this->file_itself."'";      
    } else
 
    { 
      return $this->file_itself = "null";
 
      }
 
    }
[PHP] pobierz, plaintext 

Założenie jest takie, aby powyższa funkcja wywoływana przy SUBMIT formularza zwróciła nazwę pliczku jeżeli plik jest wysyłany lub null jeżeli pliku nie ma.
Problem polega na tym, że przy wskazaniu pliku (który na marginesie jest kopiowany do 'c:wamp/www/live/modules/hd/attachements/' i pojawia się tam) funkcja za każdym razem zwraca null - zarówno w przypadku kiedy plik ładujemy jak i w przypadku kiedy pliku nie ma.

Pls o spojrzenie 'świeżym okiem'.
(IMG:style_emoticons/default/smile.gif)

[melkorm]

a nie brakuje Tobie klamry za IF'em?

[yarexxx]

Przeoczenie.. ale to nie to. Poniżej poprawiony kod, ta sama sytuacja..

[PHP] pobierz, plaintext 
  public function upload_single_file(){
 
 
    if(is_uploaded_file($_FILES['attach']['tmp_name']))
    {
 
      $this->file_path = "c:wamp/www/live/modules/hd/attachements/";
 
			$this->file_itself = substr($_FILES['attach']['name'], 0, strrpos($_FILES['attach']['name'], ".")) . "_" . time() . "." . substr($_FILES['attach']['name'], strrpos($_FILES['attach']['name'], ".")+1);
 
			move_uploaded_file($_FILES['attach']['tmp_name'], $this->file_path.$this->file_itself);
 
		  return    $this->file_itself =  "'".$this->file_itself."'"; 
 
    }
 
      else {
              return   $this->file_itself = "null";
              }
 
    }
[PHP] pobierz, plaintext 

[Zyx]

Pierwsze pytanie, jakie się ciśnie, brzmi: a jaki to ma związek z programowaniem obiektowym?

Znając życie, pewnie masz skopany sam kod formularza, któremu brakuje atrybutu enctype="multipart/form-datas", co zmienia zupełnie działanie PHP. Inna sprawa, że funkcja is_uploaded_file() potrzebna Ci jest, jak piernik wiatrakowi. Jej funkcjonalność jest w całości wbudowana w move_uploaded_file(), zatem w Twoim przypadku nie ma żadnego racjonalnego powodu by jej używać. Koncentrujesz się na takich bzdetach, a nie robisz dość oczywistego testu na to czy zmienna $_FILES['attach'] w ogóle istnieje.

[yarexxx]

Związek ma taki, że jest to metoda w klasie do obsług plików.
Formularz nie zawiera błędów, po za tym enctype="multipart/form-data".
Z is_uploaded_file korzystam ponieważ potrzebuję aby metoda zwróciła mi tekst także wtedy jeżeli plik nie jest załączany.

PS.

tak jak pisałem plik jest przesyłany na serwer. Ale metoda zawsze zwraca null...

Ten post edytował yarexxx 20.10.2010, 13:36:03

[Zyx]

Twój problem nie ma żadnego związku z programowaniem obiektowym, a to, że używasz metod, nie ma tu nic do rzeczy. Ponadto dokładnie ten sam efekt osiągniesz, stosując move_uploaded_file(). Jak wspomniałem, te funkcje są tożsame i w Twoim przypadku nie ma ŻADNEGO racjonalnego powodu, by używać is_uploaded_file(), a testowanie czy plik był załączany, jak również wspomniałem, powinno być robione w inny sposób.

Pierwsze to zrób sobie test:

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

by zobaczyć czy coś w ogóle dostajesz.

[yarexxx]

var_dump zwraca:

[PHP] pobierz, plaintext 
array(1) { 
          ["attach"]=> array(5) 
            { 
            ["name"]=> string(13) "Diagram1.jpeg" ["type"]=> string(10) "image/jpeg" ["tmp_name"]=> string(23) "C:\wamp\tmp\phpFCD7.tmp" ["error"]=> int(0) ["size"]=> int(50377) 
            } 
         } 
[PHP] pobierz, plaintext