[MySQL][PHP] Problem z uploadem zdjecia na serwer Opcje

[Ravik]

Witam,

Otóż mam klasę, która odpowiada za uploadowanie zdjęcia na serwer i przesłaniu do mysqla o tym informacji, za to odpowiada klasa napisania poniżej, ale niestety nie działa, kiedy przesyłam plik za pomocą formularza, wyskakuje po prostu białe tło, plik nie został dodany, a w mysqlu nie pojawiła się informacja. Proszę o nakierowanie na to co zrobiłem źle.

class.Uploader.php

[PHP] pobierz, plaintext 
<?php
class ImageUploader {
 
   var $name;
   var $type;
   var $size;
   var $temp;
   var $image;
   var $blad;
   var $max_rozmiar;
   var $category;
 
   function init() {
      global $_FILES;
 
      $this -> blad = $_FILES['userfile']['error'];
      if($this -> blad != 0)
      {
         $error = 'PHP zwrócił następuący błąd: '.$this -> blad;
         return FALSE;
      }
      $this -> name = $_FILES['userfile']['name'];
      $this -> type = $_FILES['userfile']['type'];
      $this -> size = $_FILES['userfile']['size'];
      $this -> temp = $_FILES['userfile']['tmp_name']; /* 1 */
      $this -> max_rozmiar = 8192;
      return TRUE;
   }
 
   function validate() {
 
      if($this -> type !='image/gif' && $this -> type !='image/pjpeg' && $this->type !='png') /* 2 */ 
      {
         $error ="To nie jest poprawny format obrazka (dopuszczalne rozszerzenia gif,jpg,jpeg,png)";
         return FALSE;
      }
 
      if($this -> size >= $max_rozmiar)
      {
         $error= "Plik jest za duży. Dopuszczalny rozmiar to: $max_rozmiar";
         return FALSE;
      }
 
 
      $rozmiar = getimagesize($this -> temp);
      if($rozmiar[1]>2560 || $rozmiar[2]>1280)
      {
         $error = 'Wgrywanie tego zdjecia może trochę potrwać... Czekaj...';
         return TRUE;
      }
      return TRUE;
   }
 
   function savefile($id) {
 
      global $forum;
      global $db;
      if (!$fh = fopen($this-> temp, "r"))
      {
         $error = 'Nie można otworzyć pliku z katalogu tymczasowego';
         return FALSE;
      }
      $this -> image = addslashes(fread($fh, $this -> size));
      $db -> query("INSERT INTO zdjecia ('byte', 'image_type', 'category') VALUES('".$this -> image."','".$this -> type."','".$this -> category."'");
      return TRUE;
   }
}
 
 
?>
[PHP] pobierz, plaintext 

upload.php

[PHP] pobierz, plaintext 
<?php 
require_once ("config.php");
require_once ("class.Uploader.php");
 
$db = @new db($host, $nazwabazy, $user, $pass);
 
if($img -> init())
   {
      if($img -> validate())
      {
          if($img -> savefile())
          {
            echo 'Twój obrazek został pomyślnie zapisany.';
 
      }
   }
  }
?>
[PHP] pobierz, plaintext 

dla formalnośći index.html

[PHP] pobierz, plaintext 
<form action="upload.php" method="post" enctype="multipart/form-data" accept="image/gif, image/jpeg, image/png">
<input type="hidden" name="MAX_FILE_SIZE" value="8192">
Dodaj zdjecie: <input name="userfile" type="file">
<br /><input type="submit" value="Wgraj">
</form>
[PHP] pobierz, plaintext 

Pozdrawiam,
Ravik

[erix]

Strzelam, że wrzucasz plik większy niż limit obiektów do uploadu ustawiony w php.ini.

Chyba że wrzucasz mniejsze, ale tego nie wiemy.

[Ravik]

Wrzucam dużo mniejsze pliki, ale one w ogóle nie ładują się na serwer. Błąd pewnie jest gdzieś w kodzie, proszę o pomoc.

[erix]

Nie masz błędów w składni? Sprawdź, czy na innym serwerze dzieje się to samo.

[muniekw]

Na pierwszy rzut oka ta linika wydała mi się dziwna:

[PHP] pobierz, plaintext 
   if($this -> type !='image/gif' && $this -> type !='image/pjpeg' && $this->type !='png') /* 2 */ 
[PHP] pobierz, plaintext 

a szczególnie:

[PHP] pobierz, plaintext 
image/pjpeg
[PHP] pobierz, plaintext 

[erix]

To nie jest dziwne - Internet Explorer wysyła JPEG z takim MIME.

[muniekw]

A to przepraszam. Nie wiedziałem o tym.

[Ravik]

Błędów w składni nie ma, coś jest z logiką nie tak, tylko nie wiem co. Jest ktoś w stanie mi z tym pomóc ? Dodam że nie działa na każdym serwerze.

Potrafi mi ktoś pomóc ?

Ten post edytował Ravik 20.02.2010, 13:51:32

[Armstrong]

[PHP] pobierz, plaintext 
if($img -> init())
   {
      if($img -> validate())
      {
          if($img -> savefile())
          {
            echo 'Twój obrazek został pomyślnie zapisany.';
 
      }
   }
  }
[PHP] pobierz, plaintext 

Nie powinno być czasem na początku:

[PHP] pobierz, plaintext 
$img = new  ImageUploader();
[PHP] pobierz, plaintext 

Chyba, że wcześniej deklarujesz.

Ten post edytował Armstrong 21.02.2010, 09:32:17

[Fifi209]

Kurde, piszesz jak kura pazurem.

To mnie wręcz przeraziło:

[PHP] pobierz, plaintext 
global $_FILES;
[PHP] pobierz, plaintext 

Pierwsza sprawa nie pisz w php4 tylko zgodnie z php5 - na pewno lepiej na tym wyjdziesz.

Po drugie po co robić globalną zmienną $db jak można ją przekazać do konstruktora klasy upload? (swoją drogą global jest złe)

A to:

[PHP] pobierz, plaintext 
if($this -> type !='image/gif' && $this -> type !='image/pjpeg' && $this->type !='png')
[PHP] pobierz, plaintext 

Znacznie lepiej napisać tak:

[PHP] pobierz, plaintext 
$types = array('image/gif', 'image/pjpeg', 'png');
 
if (!in_array($this->type, $types)) {
// error
}
[PHP] pobierz, plaintext 

IMHO nie widzę potrzeby pisania klasy dla czegoś takiego

Ten post edytował fifi209 21.02.2010, 09:42:37

[Ravik]

dziękuję za pomoc, poradziłem sobie w inny sposób, do zamknięcia