Matthias Apsel: Neuer Anlauf

Beitrag lesen

Om nah hoo pez nyeetz, Enrico!

ein Umkopieren der Werte ist nicht notwendig.

Du solltest auch prüfen, ob in $_GET['Bild'] was sinnvolles steckt.

$Bild = "../GRAFIKEN/SORTIMENT/" . $_GET['Bild'] . ".png";  
  
$Bild = imagecreatefrompng($Bild);  
  
$faktor = 1;  
  
if (isset ($_GET['Breite'])) {  
  if (intval($_GET['Breite']) >= 100) {  
    $faktor = intval($_GET['Breite'])/imagesx($Bild);  
  }  
  else {  
    $faktor = 100/imagesx($Bild);  
  }  
elseif (isset ($_GET['Hoehe'])) {  
  if (intval($_GET['Hoehe']) >= 100) {  
    $faktor = intval($_GET['Hoehe'])/imagesy($Bild);  
  }  
  else {  
    $faktor = 100/imagesy($Bild);  
  }  
}  
  
$NeueHoehe  = round(imagesx($Bild) * $faktor);  
$NeueBreite = round(imagesy($Bild) * $faktor);

Wenn nichts eingegeben wird, soll das Bild so bleiben, also Faktor = 1.

Du möchtest sicher nicht alle möglichen Werte zulassen. So solltest du sicherstellen, dass ein bestimmtes Maß nicht unterschritten wird. (Im Beispiel 100)

Außerdem soll das Bild sicher nicht verzerrt werden, wenn doch, musst du einen einzelnen Streckungsfaktor für die Höhe und einen für die Breite haben.

Eine maximale Höhe wäre sicher auch toll.

Platzsparend würde ich den ternären Operator verwenden.

Matthias

--
1/z ist kein Blatt Papier.