[Basti]

@"FloB":
Den Typ würde ich nie anhand der Dateiendung festmachen, sondern immer mit getimagesize().

Und auch Dotfiles würde ich nicht rausnehmen, wozu denn? Und warum den Links nicht folgen, was spricht dagegen? Ich meinte bei dem Punkt nur, dass der Variablenname $file ein bischen eng ist. $entry würde z.B. besser passen.

Mit dem !== false meinte ich, folgendes:

PHP-Code:

if (($a && $b) !== false)

ist das gleiche, wie

if ($a && $b) 


Ich benutze übrigens folgende Methode für eine proportionale Verkleinerung auf gegebene Maximalmaße:

PHP-Code:

        $iMaxWidth  = 100;
        $iMaxHeight = 200;

        if ($this->width <= $iMaxWidth
            && $this->height <= $iMaxHeight)
            return;

        $fRelOrig  = $this->width/$this->height;
        $fRelMax   = $iMaxWidth/$iMaxHeight;

        if ($fRelOrig >= $fRelMax) {

            $iWidth  = $iMaxWidth;
            $iHeight = round($iWidth * (1/$fRelOrig));

        } else {

            $iHeight = $iMaxHeight;
            $iWidth  = round($iHeight * $fRelOrig);
        }

        $this->width  = $iWidth;
        $this->height = $iHeight; 


Und von wegen Konstanten wäre halt sowas möglich.

PHP-Code:

define('IMAGE_JPEG', 1 << 0);
define('IMAGE_PNG', 1 << 1);
define('IMAGE_GIF', 1 << 2);
define('IMAGE_TIFF', 1 << 3);

$ImageDir->createThumbnails($TargetDir, 100, 200, IMAGE_JPEG | IMAGE_PNG | IMAGE_GIF); 


[EDIT]Ich seh gerade, dass es bereits diese Konstanten gibt. Sie tragen das Präfix "IMG_". Tiffs sind allerdings nicht dabei [/EDIT]

Basti

[FloB]

Hm, wieso geht das jetz an mich?

[Basti]

Ich meinte "dago", sorry. Ich bozog mich mit meinem ersten Satz auf diese RegExp-Prüfung in seinem Code.

Basti

[dago]

An der Stelle finde ich es schon so richtig @Basti. Es sollen ja nur bestimmte Dateien aussortiert werden. Ob es sich dann wirklich um ein JPEG handelt, tut ja die Klasse überprüfen und falls nicht eine Exception werfen, auf die reagiert werden kann.
Natürlich währe es mir auch lieber, wenn an der Stelle eine Funktion aufgerufen würde, die den Mimetype der Datei zurückgibt/überprüft. Ich kenne allerdings keine, die in der Standard-Bibliothek von PHP ist.

[Basti]

Ist einfach eine Frage der Anforderungen. Wenn diese lauten, dass der Bildtyp mit angegeben werden kann, dann muss man den prüfen, wenn diese sind, dass bestimmte Dateiendungen angegeben werden können, dann eben diese oder eben beides zusammen.

Und hier ging es ja um den Bildtyp und nicht darum, wie die Bilder benannt wurden, daher mein Einwand. Aber klar, kann man natürlich mit einbauen - ist eben nur die Frage, was man will.

Basti

[Jann Hendrik]

so - ich habe mich mal gerade hieran gemacht:

Zitat von dago

Hallo Jann, dass was mich etwas stört ist wie du die Größe bestimmst:

PHP-Code:

      // neue Größe bestimmen
      if($src_width >= $src_height)
      {
        $new_image_width  = $ThumbnailMaxX;
        $new_image_height = $src_height * $ThumbnailMaxX / $src_width;
      }
      if($src_width < $src_height)
      {
        $new_image_height = $ThumbnailMaxY;
        $new_image_width  = $src_width * $ThumbnailMaxX / $src_height;
      } 


Mag sein, dass es für deine Zwecke funktioniert. Aber wenn man schon maximale Größe vergibt, dann soll sich die Funktion auch an die Vorgaben halten.

Beispiel:

PHP-Code:

// Vorgabe
$max_width = 140; $max_height = 100;
// Bildausmasse
$width = 800; $height = 600;

// Ergebnis new_width: 140, new_height: 105 


Du siehst, dass die Höhe bei deiner Funktion bei dieser Vorgabe falsch berechnet wird.

Deiner Theorie kann ich nur bestimmen:

wenn man schon maximale Größe vergibt, dann soll sich die Funktion auch an die Vorgaben halten.

Also - das ganze ausprobiert - und klar - damit hast du natürlich vollkommen Recht.

Ich habe mir mal den minimalen Punkt, der dann aber doch entscheidend ist angeschaut.

Danke dafür!


Der Teil sieht bei mir nun so aus - also wie bei dir

PHP-Code:

          // calculate new size
          if ($src_width >= $src_height)
          {
            $new_image_height = $src_height / $src_width * $this->ThumbnailMaxX;
            $new_image_width  = $this->ThumbnailMaxX;

            if ($new_image_height > $this->ThumbnailMaxY)
            {
              $new_image_width  = $new_image_width / $new_image_height * $this->ThumbnailMaxY;
              $new_image_height = $this->ThumbnailMaxY;
            }
          }
          else
          {
            $new_image_width  = $src_width / $src_height * $this->ThumbnailMaxY;
            $new_image_height = $this->ThumbnailMaxY;

            if ($new_image_width > $this->ThumbnailMaxX)
            {
              $new_image_height = $new_image_height / $new_image_width * $this->ThumbnailMaxX;
              $new_image_width  = $this->ThumbnailMaxX;
            }
          } 


// Nachtrag:
Ich habe festgestellt, dass ein Problem dann auftritt wenn das Originalbild kleiner als das zu erzeugende Bild ist.
Dann würde das Vorschaubild vergrößert werden - was unschön aussieht.
Daher habe ich das entsprechend erweitert:

PHP-Code:

          // calculate new size
          if ($src_width >= $src_height)
          {
            $new_image_height = $src_height / $src_width * $this->ThumbnailMaxX;
            $new_image_width  = $this->ThumbnailMaxX;

            if ($new_image_height > $this->ThumbnailMaxY)
            {
              $new_image_width  = $new_image_width / $new_image_height * $this->ThumbnailMaxY;
              $new_image_height = $this->ThumbnailMaxY;
            }
          }
          else
          {
            $new_image_width  = $src_width / $src_height * $this->ThumbnailMaxY;
            $new_image_height = $this->ThumbnailMaxY;

            if ($new_image_width > $this->ThumbnailMaxX)
            {
              $new_image_height = $new_image_height / $new_image_width * $this->ThumbnailMaxX;
              $new_image_width  = $this->ThumbnailMaxX;
            }
          }

         // for the case that the thumbnail would be bigger then the original picture
         if($new_image_height > $src_height)
          {
            $new_image_width  = $new_image_width * $src_height/$new_image_height;
            $new_image_height = $src_height;
          }

          if($new_image_width > $src_width)
          {
            $new_image_height = $new_image_height * $new_image_width/$src_width;
            $new_image_width  = $src_width;
          }