Scripts - Tutorials - Forum - Downloads - Showcase - Contact

Artikels

VPN vergelijken

Algemeen

Beginpagina

FAQ

Grafische worm (243)

Links

Nieuwsartikels

Nieuwsarchief

Boeken programmeren

Overzicht

Samenwerken

Webhosting

Zoek op Sitemasters

Leden

Registreren

Ledenlijst

Ons team

Links

webhostingtop10.be

Sociale media

Follow @sitemasters

Sitemasters

Adverteren op Sitemasters?

Contacteer ons

RSS

Link naar ons

Donaties

Poll

Je moet ingelogd zijn om te stemmen.

Statistieken

Linkpartners

Forum

Categorieën > PHP

Kwaliteit na resize, even logica snappen (Opgelost)

JeroenI - 11/03/2014 22:28 (laatste wijziging 11/03/2014 22:48)

PHP interesse

Goedenavond,

Doormiddel van "imagejpeg($tn, $save,100);" wordt het bestand weggeschreven als TN. Ik neem aan dat 100 staat voor de kwaliteit? Waarom is de kwaliteit dan nog zo slecht van een thumbnail na uploaden? Kan dit komen omdat hij van heel groot naar betrekkelijk klein moet verkleinen?

Alvast mijn dank.

Klein deeltje code, misschien zien jullie in een oogopslag waar de resize mis gaat...

php code - Bekijk de code zonder highlighting - Klap code in

$file = ''.$pad.''.$album.'-'.$subalbum.'-'.$imagename.''; 
$save = ''.$pad.'thmb/'.$imagename.''; 

list($width, $height) = getimagesize($target);

$modheight = 100;
$diff = $height / $modheight;
$modwidth = $width / $diff;

$tn = imagecreatetruecolor($modwidth, $modheight) ;
$image = imagecreatefromjpeg($file) ;
imagecopyresampled($tn, $image, 0, 0, 0, 0, $modwidth, $modheight, ImageSX($image), ImageSY($image)) ;

imagejpeg($tn, $save,100);

$file = ''.$pad.''.$album.'-'.$subalbum.'-'.$imagename.''; 
$save = ''.$pad.'thmb/'.$imagename.''; 
 
list($width, $height) = getimagesize($target);
 
$modheight = 100;
$diff = $height / $modheight;
$modwidth = $width / $diff;
 
$tn = imagecreatetruecolor($modwidth, $modheight) ;
$image = imagecreatefromjpeg($file) ;
imagecopyresampled($tn, $image, 0, 0, 0, 0, $modwidth, $modheight, ImageSX($image), ImageSY($image)) ;
 
imagejpeg($tn, $save,100);

7 antwoorden

Gesponsorde links

vinTage - 11/03/2014 23:05
Nieuw lid		Op het eerste gezicht ziet je code er normaal uit, behalve dat je waar je ImageSX(..) en ImageSY(..) gebruikt, daar de waardes van de orginele image moet gebruiken. Ik weet uit mijn hoofd niet wat die list teruggeeft, dus misschien is het wel goed wat je daar gebruikt, maar ik zou die waardes ter controle toch even dumpen.
Bedankt door: JeroenI

JeroenI - 11/03/2014 23:09 (laatste wijziging 11/03/2014 23:20)
PHP interesse		Bedankt voor je reactie. Ik had eerst ipv ImageSX... en SY... $width en $height gebruikt, uitgelezen waardes uit het orgineel. Dit gaf hetzelfde resultaat. google kwam met het idee voor ImageSX en SY... maar helaas.. [edit] en $target is gewoon de orginele afbeelding [edit2] $target inmiddels gewijzigd naar $file, zag dat hij er onnodig dubbel stond deze ook gewijzigd php code - Bekijk de code zonder highlighting - Klap code in imagecopyresampled($tn, $image, 0, 0, 0, 0, $modwidth, $modheight, ImageSX($image), ImageSY($image)); imagecopyresampled($tn, $image, 0, 0, 0, 0, $modwidth, $modheight, ImageSX($image), ImageSY($image)); naar php code - Bekijk de code zonder highlighting - Klap code in imagecopyresampled($tn, $image, 0, 0, 0, 0, $modwidth, $modheight, $width, $height); imagecopyresampled($tn, $image, 0, 0, 0, 0, $modwidth, $modheight, $width, $height); $width en $height geven gewoon de correcte waardes

Thomas - 11/03/2014 23:39
Moderator		Bovenstaande thumb is volgens mij wel volgens de regels der (thumbcreatie) kunst gemaakt, behalve $modWidth, dit is waarschijnlijk een float. Misschien gooit dit roet in het eten? Probeer deze eens te typecasten naar int, of af te ronden met round().

JeroenI - 12/03/2014 00:26
PHP interesse		Zou je je nader willen verklaren of een voorbeeld kunnen geven? Waarom zou dit roet in het eten kunnen gooien? De afmetingen kloppen namelijk gewoon wel.

marten - 12/03/2014 08:44
Beheerder		Waar ik zelf altijd voorstander van ben is het gebruik van PHP.net: bcdiv en PHP.net: bcmul En dan afronden op 5 cijfers achter de komma. Verder kan je sterke verbeteringen verwachten bij het gebruik van bijvoorbeeld http://www.imagemagick.org/

Thomas - 12/03/2014 10:19 (laatste wijziging 12/03/2014 11:08)

Moderator

marten schreef:

Waar ik zelf altijd voorstander van ben is het gebruik van PHP.net: bcdiv en PHP.net: bcmul En dan afronden op 5 cijfers achter de komma.

Overkill much? Daarnaast verwacht imagecreatetruecolor integers, waar het mij eigenlijk om begonnen was.

EDIT: Het onderstaande test-script doet precies hetzelfde, de kwaliteit van de gegenereerde thumbnails lijkt gewoon goed? Het enige verschil is dus regel 30 (afgeronde breedte).

php code - Bekijk de code zonder highlighting - Klap code in

<?php
function dump($a) {
    echo '<pre>'.print_r($a, true).'</pre>';
}

if ($_SERVER['REQUEST_METHOD'] == 'POST') {
    // Sanity checks.
    $errors = array();
    if ($_FILES['file']['error'][0] > 0) {
        $errors[] = 'upload error '.$_FILES['file']['error'][0];
    }
    $extension = array_pop(explode('.', $_FILES['file']['name'][0]));
    if (!in_array($extension, array('jpg'))) {
        $errors[] = 'unsupported file type';
    }
    if (($dimensions = @getimagesize($_FILES['file']['tmp_name'][0])) === false) {
        $errors[] = 'could not determine image dimensions';
    }
    if (count($errors)) {
        dump($errors);
        dump($_POST);
        dump($_FILES);
        exit;
    }

    // Image calculations.
    list($width, $height) = $dimensions;
    $resizedHeight = 100;
    $ratio = $height / $resizedHeight;
    $resizedWidth = round($width / $ratio);

    // Image creation.
    $imageSource = imagecreatefromjpeg($_FILES['file']['tmp_name'][0]);
    $imageDestination = imagecreatetruecolor($resizedWidth, $resizedHeight);
    imagecopyresampled($imageDestination, $imageSource, 0, 0, 0, 0, $resizedWidth, $resizedHeight, $width, $height);
    imagedestroy($imageSource); // memory cleanup

    // Send image to output.
    header('Content-Type: image/jpeg');
    imagejpeg($imageDestination, null, 100);

    imagedestroy($imageDestination); // memory cleanup
} else {
    header('Content-Type: text/html; charset=UTF-8');
?>
<!DOCTYPE html>
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
<title>upload test</title>
</head>

<body>
<form action="<?php echo htmlspecialchars($_SERVER['PHP_SELF'], ENT_QUOTES, 'UTF-8') ?>" method="post" accept-charset="UTF-8" enctype="multipart/form-data">
<input type="file" name="file[]" />
<button type="submit">submit</button>
</form>
</body>
</html>
<?php
}
?>

<?php
function dump($a) {
    echo '<pre>'.print_r($a, true).'</pre>';
}
 
if ($_SERVER['REQUEST_METHOD'] == 'POST') {
    // Sanity checks.
    $errors = array();
    if ($_FILES['file']['error'][0] > 0) {
        $errors[] = 'upload error '.$_FILES['file']['error'][0];
    }
    $extension = array_pop(explode('.', $_FILES['file']['name'][0]));
    if (!in_array($extension, array('jpg'))) {
        $errors[] = 'unsupported file type';
    }
    if (($dimensions = @getimagesize($_FILES['file']['tmp_name'][0])) === false) {
        $errors[] = 'could not determine image dimensions';
    }
    if (count($errors)) {
        dump($errors);
        dump($_POST);
        dump($_FILES);
        exit;
    }
 
    // Image calculations.
    list($width, $height) = $dimensions;
    $resizedHeight = 100;
    $ratio = $height / $resizedHeight;
    $resizedWidth = round($width / $ratio);
 
    // Image creation.
    $imageSource = imagecreatefromjpeg($_FILES['file']['tmp_name'][0]);
    $imageDestination = imagecreatetruecolor($resizedWidth, $resizedHeight);
    imagecopyresampled($imageDestination, $imageSource, 0, 0, 0, 0, $resizedWidth, $resizedHeight, $width, $height);
    imagedestroy($imageSource); // memory cleanup
 
    // Send image to output.
    header('Content-Type: image/jpeg');
    imagejpeg($imageDestination, null, 100);
 
    imagedestroy($imageDestination); // memory cleanup
} else {
    header('Content-Type: text/html; charset=UTF-8');
?>
<!DOCTYPE html>
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
<title>upload test</title>
</head>
 
<body>
<form action="<?php echo htmlspecialchars($_SERVER['PHP_SELF'], ENT_QUOTES, 'UTF-8') ?>" method="post" accept-charset="UTF-8" enctype="multipart/form-data">
<input type="file" name="file[]" />
<button type="submit">submit</button>
</form>
</body>
</html>
<?php
}
?>

Je hebt overigens imagesx() en imagesy() (inderdaad) niet nodig, je hebt deze waarden al in $width en $height.

Bedankt door: JeroenI

JeroenI - 14/03/2014 21:16
PHP interesse		Toppie, opgelost. Bedankt!

Gesponsorde links

Je moet ingelogd zijn om een reactie te kunnen posten.

Actiefste leden van de maand

Actieve forumberichten