php5 oop class vurdering

Du kunne f.eks. også tilføje support for GIF og BMP.

synes bare jeg have læst et sted at gif ikke er muligt i v. 2+ pga. copyright? :-)

Note: GIF support was removed from the GD library in Version 1.6, and added back in Version 2.0.28. This function is not available between these versions.

ahh så har jeg misledt mig selv. Self skal klassen understøtte gif. :)

så er gif også understøttet ;>

Denne kode:
if (stristr(strtolower($this->file),'.jpg') || stristr(strtolower($this->file),'.jpeg'))
        {
            $this->format = "JPG";
        }
        else if (stristr(strtolower($this->file),'.png'))
        {
            $this->format = "PNG";
        }
        else if (stristr(strtolower($this->file),'.gif'))
        {
            $this->format = "GIF";
        }
        else
        {
            echo "Unknown file format.";
            exit;
        }

Ville jeg nok hellere bygge op over en switch/case ala min egen artikel her: :o)

http://www.eksperten.dk/artikler/1132

Og så ville jeg også fange FIL EXTENSION med min funktion.

Og BMP bruges ikke meget, men du kunne jo overveje at smide det med alligevel:
http://www.techzine.nl/scripts/425/PHP-functies-imagecreatefrombmp-en-imagebmp.html

Så er det rettet :-)

Du kan erstatte:
if ($this->format)
            {
                switch($this->format)
                {
                    case 'JPG':
                        $this->image[oldImage] = ImageCreateFromJpeg($this->file);
                    break;
   
                    case 'PNG':
                        $this->image[oldImage] = ImageCreateFromPng($this->file);
                    break;

                    case 'GIF':
                        $this->image[oldImage] = ImageCreateFromGIF($this->file);
                    break;
                }
            }
med:
switch($this->format)
{
    case 'JPG':
        $this->image[oldImage] = ImageCreateFromJpeg($this->file);
        break;
   
    case 'PNG':
        $this->image[oldImage] = ImageCreateFromPng($this->file);
        break;
   
    case 'GIF':
        $this->image[oldImage] = ImageCreateFromGIF($this->file);
        break;
    default:
        return 'ERROR, WRONG EXTENSION';
}

Og jeg tror også jeg ville bruge exceptions og ikke
echo 'ERROR fejl...';
exit();

Så er det rettet.

Hvad er fordelen ved at benytte exceptions?

har opdateret getformat så den sætter this->format til hvilket som helst format. Så tjekker vi med din case on formatet er tilladt eller ej.

exceptions er smart når du koder oop, da du kan lave en klasse til at håndtere alle exceptions og dermed præsentere fejl for brugeren, sende email til admin, logge i en fil osv...

Jo det kan du have ret i.

Jeg har læst lidt om det her: http://dk2.php.net/exceptions men jeg tror ikke det er særlig relevant i en enkelt klasse :-)

Jeg kunne evt. lave en fejl-handler som laver et billede med fejl-beskeden istedet så hvis man benytter scriptet i en img src="" så får man ikke bare et tomt billede ved fejl.

Men tak for dine svar. læg venligst et svar ;>

Et svar.

Takker for point.