Zmienne z wewnątrza klasy, Jak użyć zmiennej na zewnątrz klasy Opcje

[aniol]

Orientuję się już co nieco w PHP a ostatnio zainteresowałem się tematem
klas przy okazji znalezienia i próby wykorzystania gotowej klasy
imageresizer.class.php z www.phpclasses.org
Dość prosta i fajna klasa do resizowania uploadowanych obrazków.

Nie wiem czy jest sens cytowania całej klasy, zrobię to jeśli będzie konieczne.
Podam tylko sposób wywołania klasy:

[HTML] pobierz, plaintext 
<?php
if (isset ($_POST["submit"])) {
 
include ("imageresizer.class.php");
			$image = new Image($_FILES['file']['name']);		
			$image -> PicDir = "foto_upload/";
			$image -> PicURL = "http://www.domena.pl/";
			$image -> TmpName 	 = $_FILES['file']['tmp_name'];
			$image -> FileSize   = $_FILES['file']['size'];
			$image -> FileType   = $_FILES['file']['type'];
		//$image -> Save(); //use this if you wish images without resizing
		$image -> Resize ();
 
// W tym miejscu chciał bym mieć dostępną zmienna z wnętrza klasy zawierającą numer błędu
 
	}
	?>
<form name="form" action="<?php echo htmlspecialchars($_SERVER['PHP_SELF']); ?>" method=post enctype="multipart/form-data">
<input type="file" name="file"  size="40"><br>
<br>
<input type="submit" name="submit" value="Prześlij >>">
</form>
[HTML] pobierz, plaintext 

Moim problemem jest to, że chciałem użyć zmiennej $error występującej w fonkcji wewnątrz klasy.
Zmienna ta zawiera numer błędu zwyczajnie cyfry od 0-4 symbolizujące typ błędu.
Klasa zawiera funkcję, która wyświetla (echo) odpowiedni komunikat błędu w zależności od cyferki
w zmiennej $error. I ładnie mi to działa ale ja po wykonaniu (wywołaniu) klasy i wyjściu z niej
chciał bym wykonać stosowne akcje w zależności od wartości błędu $error. A ta zmienna nie jest
dostępna już poza klasą, jedynie wewnątrz niej.

Proszę o jakąś ogólną zasadę jak to można zrobić.

Uzupełnienie - treść klasy:
(Od siebie dodałem jedną poprawkę błędu i kilka modyfikacji w sekcji wyświetlającej informacje o obrazku.
Oryginalna klasa dostępna na: www.phpclasses.org)

Należy zwrócić uwagę na funkcję GetError.

[PHP] pobierz, plaintext 
<?php
 
class Image {
        var $FileName;
        var $FileSize;
        var $FileType;
        var $AllowedExtentions = array ("image/png", "image/gif", "image/jpeg", "image/jpg"); 
        var $newWidth = 155; // new width 
        var $newHeight = 116; //new height
        var $TmpName;
        var $PicDir;  //store uploaded images
        var $PicURL;  //URL bazowy
        var $MaxFileSize =  1258291.2;  // 1,2 MB  w kilobitach
            var $ImageQuality = 85;  // image compression (max value 100)
 
        function Image($FileName) {
            $this->FileName=$FileName;
        }
        
        function GetInfo() {
            $newfilepath = $this->PicDir . '' .$this->FileName;
            echo $newfilepath."<br>";
            $out='  <br><br>Obrazek przesłany!<br>
                        Nazwa plik: <b>'.$this->FileName.'</b><br>
                    Oryginalny rozmiar: <b>'.round(($this->FileSize/1024)/1024, 2).' MB</b><br>
                    Rozmiar po obróbc: <b>'.round(((filesize($newfilepath)/1024)/1024), 4).' MB</b> czyli <b>'.round((filesize($newfilepath)/1024), 2).' KB</b><br>
                    Typ pliku graficznego: <b>'.$this->FileType.'</b><br><br>
                    <img border=1 src='. $this->PicURL .''. $this->PicDir .  $this->FileName . '><br><br>';
                    
            return $out;        
        }
        
         
         function GetFileExtention ($FileName) {
            if (in_array ($this->FileType, $this -> AllowedExtentions)) {
                return true;         
            }     else {
               return false;            
            }             
         
         }             
         
         function ExistFile () {
            $fileexist = $_SERVER['DOCUMENT_ROOT']  . 
                          dirname($_SERVER['PHP_SELF']) . 
                          '/' . $this->PicDir .  
                                $this->FileName;
                if (file_exists($fileexist)) { return true; }
             }
         
function GetError ($error) {
            
switch ($error) {            
case 0 <span style="color: #007700":
echo "Error: Błędny typ pliku <b>$this->FileType</b>! Dozwolone typ: .jpg, .jpeg, .gif, .png  <b>$this->FileName</b><br>";
break;
                
case 1 <span style="color: #007700":
echo "Error: Rozmiar plik: <b>".round(($this->FileSize/1024)/1024, 2)." MB</b> jest za duży!<br>Możesz załadować plik max. do <b>".round(($this->MaxFileSize/1024)/1024, 2)." MB</b> (Megabajta)<br>";
break;                
                
case 2 <span style="color: #007700":
echo "Error: Proszę wskazać plik do przesłania ze swojego dysku lokalnym!<br>";
break;
                                    
case 3 <span style="color: #007700":
echo "Error: Plik o nazwie <b>$this->FileName</b><br>już istnieje w katalogu docelowym !<br>";
break;                
    }
            
}
 
        
         function Resize () {
            if (empty  ($this->TmpName))                                         {echo $this -> GetError (2);}
                else if ($this->FileSize > $this->MaxFileSize)                    {echo $this -> GetError (1);}                        
                else if ($this -> GetFileExtention ($this->FileName) == false)     {echo $this -> GetError (0);} 
                else if ($this -> ExistFile())                                     {echo $this -> GetError (3);}
                    
                    else {
                
            $ext = explode(".",$this->FileName);
            $ext = end($ext);
            $ext = strtolower($ext);
            
            // Get new sizes
            list($width_orig, $height_orig) = getimagesize($this->TmpName);
 
            $ratio_orig = $width_orig/$height_orig;
 
                if ($this->newWidth/$this->newHeight > $ratio_orig) {
               $this->newWidth = $this->newHeight*$ratio_orig;
                } else {
               $this->newHeight = $this->newWidth/$ratio_orig;
                }
 
            $normal  = imagecreatetruecolor($this->newWidth, $this->newHeight);
 
            if              ($ext == "jpg") { $source = imagecreatefromjpeg($this->TmpName);  }
            else if     ($ext == "gif") { $source = imagecreatefromgif ($this->TmpName);  }
            else if     ($ext == "png") { $source = imagecreatefrompng ($this->TmpName);  }
 
            imagecopyresampled($normal, $source,    0, 0, 0, 0, $this->newWidth, $this->newHeight, $width_orig, $height_orig);
 
 
            if              ($ext == "jpg") { 
                                //ob_start();
                                imagejpeg($normal, "$this->PicDir/$this->FileName", "$this->ImageQuality"); 
                                //$binaryThumbnail = ob_get_contents(); 
                                //ob_end_clean(); 
                                }
            else if     ($ext == "gif") { imagegif ($normal, '', "$this->ImageQuality");  }
            else if     ($ext == "png") { imagepng ($normal, '', "$this->ImageQuality");  }
 
            imagedestroy($source);
             
             echo $this -> GetInfo();
                        
         }    
    
    }                 
                             
        function Save() {        
                if (empty  ($this->TmpName))                                     {echo $this -> GetError (2);}
                else if ($this->FileSize > $this->MaxFileSize)                    {echo $this -> GetError (1);}                        
                else if ($this -> GetFileExtention ($this->FileName) == false)     {echo $this -> GetError (0);} 
                else if ($this -> ExistFile ())                                 {echo $this -> GetError (3);}         
                    
                    else {
            
                copy($this->TmpName,$this->PicDir.$this->FileName);
            
                echo $this -> GetInfo();
                      
             }
         }
     }
 
?>
[PHP] pobierz, plaintext 

Ten post edytował aniol 29.09.2008, 11:48:17

[LBO]

Przydała by się jednak ta klasa tutaj. Bez Tego tobie nie powiemy jak się do tej zmiennej dostać.
Może ma metodę getErrorNumber(); Może atrybu Error jest publiczny; Może rzuca wyjątkiem z tym numerem.

edit:

Przejrzałem ta klasę. Nie da rady wyciagnąć tego błędu. On jest tylko wewnętrznie wykorzystywany (przekazywany przez parametr w odpowiedniej funkcji) i tyle.

Ten post edytował LBO 29.09.2008, 11:47:05

[kallosz]

mozesz utworzyc kolejna zmienna ktorej bedziesz przypisywac numer bledu.

[PHP] pobierz, plaintext 
<?php
public $error = null;
?>
[PHP] pobierz, plaintext 

nastepnie w

[PHP] pobierz, plaintext 
<?php
function GetError ($error) {
$this->error = $error;
?>
[PHP] pobierz, plaintext 

[LBO]

mozesz utworzyc kolejna zmienna ktorej bedziesz przypisywac numer bledu.

[PHP] pobierz, plaintext 
<?php
public $error = null;
?>
[PHP] pobierz, plaintext 

nastepnie w

[PHP] pobierz, plaintext 
<?php
function GetError ($error) {
$this->error = $error;
?>
[PHP] pobierz, plaintext 

Wiesz, żeby korzystać z zalet OOP ta klasa powinna być napisana przynajmniej względnie dobrze. A ponieważ nie jest - radziłbym koledze znalezienie innego narzędzia, a nawet napisanie własnego i to w PHP5. Nauczy się więcej a nie podpatruje takie badziewia.

Pozdrawiam, Alan

[aniol]

Wiesz, żeby korzystać z zalet OOP ta klasa powinna być napisana przynajmniej względnie dobrze. A ponieważ nie jest - radziłbym koledze znalezienie innego narzędzia, a nawet napisanie własnego i to w PHP5. Nauczy się więcej a nie podpatruje takie badziewia.

Pozdrawiam, Alan

Pomijając twoją ocenę, czy masz może inny lepszy pomysł, bo rozwiązanie kolegi którego cytowałeś
nie zadziałało u mnie, nie wiem czemu.

Anioł

Ten post edytował aniol 29.09.2008, 13:11:46

[kallosz]

Wiesz, żeby korzystać z zalet OOP ta klasa powinna być napisana przynajmniej względnie dobrze. A ponieważ nie jest - radziłbym koledze znalezienie innego narzędzia, a nawet napisanie własnego i to w PHP5. Nauczy się więcej a nie podpatruje takie badziewia.

Pozdrawiam, Alan

@LBO: tak wiem...
@anioł: a w jaki sposób wyciągasz $error na zwenątrz?

BTW:
Do uploadu obrazkow oraz innych rozmaitosci stosuje http://www.verot.net/php_class_upload.htm
bardzo fajna klasa

Ten post edytował kallosz 29.09.2008, 13:21:30

[melkorm]

w funkcji resize czyli tam gdzie sparwdzasz i wyświetlasz błedy zamaist wywoływać funkcje możesz dać

[PHP] pobierz, plaintext 
<?php
$this->error = cyfra;
?>
[PHP] pobierz, plaintext 

no i później możesz się normalnie do tego odwołać i użyć :

[PHP] pobierz, plaintext 
<?php
$image->GetError($image->error);
?>
[PHP] pobierz, plaintext 

ogólnie w tej funkcji geterror chyba lepiej dać return niż echo i później robić echo do wywoływanej funkcji

Ten post edytował melkorm 29.09.2008, 13:24:15

[LBO]

Proszę bardzo. Zacznijmy, że z założenia nie modyfikuje się zewnetrznych klas.

[PHP] pobierz, plaintext 
<?php
class MyImage extends Image {
     // w ujeciu bardziej proceduralnym
     public $error = null;
 
     protected function getError($error) { // nie uzywamy tego
         $this->error = $error;
         return ''; // przez te głupie echo w resize() i save()
     }
 
     public function hasError() {
         return !is_null($this->error);
     }
 
     public function getErrorCode() {
         return $this->error;
     }
 
     // lub na wyjatkach!!!!!!!!!!
 
     protected function getError($error) { // nie uzywamy tego
         throw new Exception('An error occurred.', $error);
     }
}
?>
[PHP] pobierz, plaintext 

Pozdrawiam, Alan

edit:

poprawiłem kilka błędów.

Ten post edytował LBO 29.09.2008, 13:26:31

[Sedziwoj]

Jak błąd to błąd, czyli wyjątek, różny kod lub typ (ale dziedziczące z jednej dla danej klasy).
Zresztą to jest tak fatalnie napisane... powtarzanie tego samego kodu w paru miejscach, błędne nazewnictwo...

[aniol]

BTW:
Do uploadu obrazkow oraz innych rozmaitosci stosuje http://www.verot.net/php_class_upload.htm
bardzo fajna klasa
[/quote]

Po zapoznaniu się z w/w adresem, sprowadzony na ziemię, nie chcę już więcej oglądać tego czym rozpocząłem ten wątek. www.verot.net/php_class_upload.htm to poprostu doskonałe kompleksowe rozwiązanie. Tak więc moje dotychczasowe zmagania straciły sens.

Zamykam wątek.

Dzięku "kallosz".

Ten post edytował aniol 30.09.2008, 09:40:18