PHP: find længste sidde i billede

Du kan finde ud af, hvilken side at billedet, der er længst, og så gøre den til 350px, derefter kan du tilpasse den anden side, så forholdet passer.

Var det forståeligt? :)

Udskift med denne:
function resizeImage($src_file, $dest_file, $newWidth = '350')

Yeah I know, men denne kode er i forbindelse med at jeg uploader et billede. Derfor skal scriptet selv finde den længste side. Nu er spørgsmålet hvordan...?

Hov. Her istedet:

resizeImage("Images/7.jpg,Images/7-5.jpg,350");

Bare glem det jeg skrev før

fant0mas: tager den kode også højde for, hvis det nu er længden på billedet der er længst og generere den så længden som værende 350 px i stedet for bredden?

Nej undskyld. Jeg sover.
Den klarer vi lige..

Altså den måde, du finder den længste side, kan gøre med if:

if ($srcWidth > $srcHeight) {
// Her er bredden størst
}
elseif ($srcHeight > $srcWidth) {
// Her er højden størst
elseif ($srcHeight == $srcWidth) {
// Her er højde og bredde ens
}
else {
// Fejl, $srcHeight eller $srcWidth er nok ikke et tal
return false;
}

Hmm, er det en kode, du selv har lavet? For i din funktion burde du nok ikke have $newWidth, da du her fortæller koden, at den skal gøre bredden af billedet til det, den modtager fra resizeImage()

Forstår jeg dig ret, vil du have scriptet til at ændre bredde og højde, så den længste side er maks 350 px

tobias: yes præcis

Ikke testet:

<?
function resizeImage($src_file, $dest_file, $newWidth, $newHeight)
{
$imginfo = @getimagesize($src_file);
if ($imginfo == NULL)
  return false;

// height/width
$srcWidth = $imginfo[0];
$srcHeight = $imginfo[1];


$wscale = $newWidth / $srcWidth;
$hscale = $newHeight / $srcHeight;
$scale = ( $hscale > $wscale ? $hscale : $wscale );
$nw = round( $srcWidth * $scale, 0 );
$nh = round( $srcHeight * $scale, 0 );
$src_img = imagecreatefromjpeg($src_file);
$dst_img = imagecreatetruecolor( $nw, $nh );
ImageCopyResampled($dst_img, $src_img, 0, 0, 0, 0, $nw, $nh, $srcWidth, $srcHeight);
imagejpeg($dst_img, $dest_file, 100);
imagedestroy($src_img);
imagedestroy($dst_img);
return $dest_file;
};
resizeImage("Images/7.jpg,Images/7-5.jpg,350,350");
?>

Men har man i dit script ikke allerede defineret at BÅDE højde og bredde skalvære 350 px?

Jo de vil altid uanset størrelse blive max h/b 350.
Ellers:

<?
function resizeImage($src_file, $dest_file, $newWidth, $newHeight)
{
    $imginfo = @getimagesize($src_file);
    if ($imginfo == NULL)
    return false;

    // height/width
    $srcWidth = $imginfo[0];
    $srcHeight = $imginfo[1];


    $wscale = $newWidth / $srcWidth;
    $hscale = $newHeight / $srcHeight;
    $scale = ( $hscale > $wscale ? $hscale : $wscale );
    $nw = round( $srcWidth * $scale, 0 );
    $nh = round( $srcHeight * $scale, 0 );
   
    if($srcWidth < $newWidth && $srcHeight < $newHeight){
        $nw = $srcWidth;
        $nh = $srcHeight;       
    }
   
    $src_img = imagecreatefromjpeg($src_file);
    $dst_img = imagecreatetruecolor( $nw, $nh );
    ImageCopyResampled($dst_img, $src_img, 0, 0, 0, 0, $nw, $nh, $srcWidth, $srcHeight);
    imagejpeg($dst_img, $dest_file, 100);
    imagedestroy($src_img);
    imagedestroy($dst_img);
    return $dest_file;
};
resizeImage("Images/7.jpg,Images/7-5.jpg,350,350");
?>

Dette burde kunne gøre det.
Og her er der taget højde for, at max-længden er 350, men billedet KAN være mindre, dvs. den ikke strækker et lille billede..

Dog er scriptet ikke testen

<?
function resizeImage($src_file, $dest_file, $newWidth)
{
$imginfo = @getimagesize($src_file);
if ($imginfo == NULL)
  return false;

// height/width
$srcWidth = $imginfo[0];
$srcHeight = $imginfo[1];

$ratioWidth = $srcWidth/$newWidth;
$ratioHeight = 0;
$destWidth = $newWidth;
$destHeight = $srcHeight / $ratioWidth;
$src_img = imagecreatefromjpeg($src_file);
$dst_img = imagecreatetruecolor($destWidth,$destHeight);
ImageCopyResampled($dst_img, $src_img, 0, 0, 0, 0, $destWidth,(int)$destHeight, $srcWidth, $srcHeight);
imagejpeg($dst_img, $dest_file, 100);
imagedestroy($src_img);
imagedestroy($dst_img);
return $dest_file;
});
resizeImage("Images/7.jpg,Images/7-5.jpg,20");
?>

<?
function resizeImage($src_file, $dest_file) {
$maxSize = 350;
$imginfo = @getimagesize($src_file);
if ($imginfo == NULL) return false;
$srcWidth = $imginfo[0];
$srcHeight = $imginfo[1];
if (($srcWidth > $maxSize) || ($srcHeight > $maxSize)) {
  if ($srcWidth => $srcHeight) {
  $scale = $srcWidth/$maxSize;
  }
  elseif ($srcWidth < $srcHeight) {
  $scale = $srcHeight/$maxSize;
  }
  else {
  return false;
  }
}
$newWidth = round($srcWidth*$scale, 0);
$newHeight = round($srcHeight*$scale, 0);
$src_img = imagecreatefromjpeg($src_file);
$dst_img = imagecreatetruecolor($newWidth, $newHeight);
ImageCopyResampled($dst_img, $src_img, 0, 0, 0, 0, $newWidth, $newHeight, $srcWidth, $srcHeight);
imagejpeg($dst_img, $dest_file, 100);
imagedestroy($src_img);
imagedestroy($dst_img);
return $dest_file;
}

Hov, der kom for meget med. Kig på dette bare:

<?
function resizeImage($src_file, $dest_file) {
$maxSize = 350;
$imginfo = @getimagesize($src_file);
if ($imginfo == NULL) return false;
$srcWidth = $imginfo[0];
$srcHeight = $imginfo[1];
if (($srcWidth > $maxSize) || ($srcHeight > $maxSize)) {
  if ($srcWidth => $srcHeight) {
  $scale = $srcWidth/$maxSize;
  }
  elseif ($srcWidth < $srcHeight) {
  $scale = $srcHeight/$maxSize;
  }
  else {
  return false;
  }
}
$newWidth = round($srcWidth*$scale, 0);
$newHeight = round($srcHeight*$scale, 0);
$src_img = imagecreatefromjpeg($src_file);
$dst_img = imagecreatetruecolor($newWidth, $newHeight);
ImageCopyResampled($dst_img, $src_img, 0, 0, 0, 0, $newWidth, $newHeight, $srcWidth, $srcHeight);
imagejpeg($dst_img, $dest_file, 100);
imagedestroy($src_img);
imagedestroy($dst_img);
return $dest_file;
}
?>

Efter rettelse af nogle fejl, virker dette:

<?
function resizeImage($src_file, $dest_file) {
$maxSize = 350;
$imginfo = @getimagesize($src_file);
if ($imginfo == NULL) return false;
$srcWidth = $imginfo[0];
$srcHeight = $imginfo[1];
if (($srcWidth > $maxSize) || ($srcHeight > $maxSize)) {
  if ($srcWidth >= $srcHeight) {
  $scale = $srcWidth/$maxSize;
  }
  elseif ($srcWidth < $srcHeight) {
  $scale = $srcHeight/$maxSize;
  }
  else {
  return false;
  }
}
else {
  $scale = 1;
}
$newWidth = round($srcWidth/$scale, 0);
$newHeight = round($srcHeight/$scale, 0);
$src_img = imagecreatefromjpeg($src_file);
$dst_img = imagecreatetruecolor($newWidth, $newHeight);
ImageCopyResampled($dst_img, $src_img, 0, 0, 0, 0, $newWidth, $newHeight, $srcWidth, $srcHeight);
imagejpeg($dst_img, $dest_file, 100);
imagedestroy($src_img);
imagedestroy($dst_img);
return $dest_file;
}
?>

tobias: den siger
Parse error: syntax error, unexpected T_DOUBLE_ARROW in /web/www/frac/users/j/jimages/try.php on line 9

Fantomas: Den siger følgende ved dit script:

Warning: Missing argument 2 for resizeImage(), called in /web/www/frac/users/j/jimages/try.php on line 32 and defined in /web/www/frac/users/j/jimages/try.php on line 2

Warning: Missing argument 3 for resizeImage(), called in /web/www/frac/users/j/jimages/try.php on line 32 and defined in /web/www/frac/users/j/jimages/try.php on line 2

Warning: Missing argument 4 for resizeImage(), called in /web/www/frac/users/j/jimages/try.php on line 32 and defined in /web/www/frac/users/j/jimages/try.php on line 2

Tobias: den siger
Warning: Missing argument 2 for resizeImage(), called in /web/www/frac/users/j/jimages/try.php on line 32 and defined in /web/www/frac/users/j/jimages/try.php on line 2 til din reviderede version

resizeImage('Images/7.jpg','Images/7-5.jpg','350','350')

Prøv det

Dette var hvad jeg kaldte funktionen: resizeImage("Images/21.jpg, Images/21-ggg.jpg");

Hvad skriver du, når du kalder funktionen?

Du skal skrive resizeImage($tmp_img, $dest_img);

$tmp_img er billedestien fra det uploadede billede, det som sikkert ligger i Apaches tmp-mappe, som vist nok kan findes ved $_FILES["fil_navn"]["tmp"];
$dest_img er billedestien, hvor billedet skal gemmes, fx /web/www/frac/users/j/jimages/uplimg/billedenavn.jpg

Fantomas: Den laver højden til at være 350 skønt det er længden der er længst..

Altså man skal skrive gåseøjnene rundt om hvert argument, altså
resizeImage("Images/21.jpg", "Images/21-ggg.jpg");

og ikke
resizeImage("Images/21.jpg, Images/21-ggg.jpg");

kan du se forskellen? :)

Hvis du iøvrigt skal bruge dette til thumps, behøver du ikke gemme billedet, men bare udskrive det. Er det ønsket, kan jeg nok godt flette lidt sammen hurtigt

Tobias: Jeg indsatte hele dette script. Jeg indbygger den først i upload når jeg kan se den virker, derfor bruger jeg faste billedstier til afprøvning..

<?
function resizeImage($src_file, $dest_file) {
$maxSize = 350;
$imginfo = @getimagesize($src_file);
if ($imginfo == NULL) return false;
$srcWidth = $imginfo[0];
$srcHeight = $imginfo[1];
if (($srcWidth > $maxSize) || ($srcHeight > $maxSize)) {
  if ($srcWidth >= $srcHeight) {
  $scale = $srcWidth/$maxSize;
  }
  elseif ($srcWidth < $srcHeight) {
  $scale = $srcHeight/$maxSize;
  }
  else {
  return false;
  }
}
else {
  $scale = 1;
}
$newWidth = round($srcWidth/$scale, 0);
$newHeight = round($srcHeight/$scale, 0);
$src_img = imagecreatefromjpeg($src_file);
$dst_img = imagecreatetruecolor($newWidth, $newHeight);
ImageCopyResampled($dst_img, $src_img, 0, 0, 0, 0, $newWidth, $newHeight, $srcWidth, $srcHeight);
imagejpeg($dst_img, $dest_file, 100);
imagedestroy($src_img);
imagedestroy($dst_img);
return $dest_file;
}
$tmp_img = "Images/7.jpg";
$dest_img = "Images/7-heyyyyyy.jpg";
resizeImage($tmp_img, $dest_img);
?>

tobias_louv:
Der er jo ingen grund til at have en funktion, hvis du smadrer den som ved at fjerne mulighederne for at genbruge den til evt. at lave thumbs.

Fantomas: i dit tilfælde, vil den så resize, hvis højde og bredde er lige store? Og hvad hvis billedet er under 350 på både højde og bredde, vil den så forstørre til 350?

Hehe, fik ikke lige læst din sidste, altså der skal foretages en lille ændring, hvis den skal ikke skal "smadres":

<?
function resizeImage($src_file, $dest_file) {
$maxSize = 350;
$imginfo = @getimagesize($src_file);


skal ændres til

<?
function resizeImage($src_file, $dest_file, $maxSize) {
$imginfo = @getimagesize($src_file);


Og så skal man bare kalde den via: resizeImage("Images/21.jpg", "Images/21-ggg.jpg", 350);

DET VIRKER! Tobias jeg tror det var dit script jeg fik til at virke, men jeg vil gerne give jer begge point fordi i var tålmodige og hjalp mig igennem!

Det lyder helt fint med mig :-)

Det vigtigste er jo, det virker!! Så her får du et svar.
Fik du set den sidste ændring, altså den med, at du kan benytte resize lidt mere dynamisk? Det som Fantomas fik påpeget?

Giv dem til tobias. God weekend :D

Yes :-) Fantomas: Så mangler jeg bare et officielt "svar" fra dig også :-)
Hav en god aften

Det var vist jer begge der gjorde det muligt så det er fair du, fantomas også får points ;-)

Når folk har sat sig i hovedet ikke at få points, er det svært at få dem til at poste et svar - faktisk umuligt..!

Men nu er du jo i den heldige situation, at du faktisk HAR et svar fra fant0mas.
Dvs. du kan godt dele pointsne, benyt dig af denne heldige mulighed!! ;)
(jeg ved godt, at lige netop dét svar er forkert, men fant0mas har dog svaret videre - så bare klø på)

Tak for hjælpen gutter