Hallo,

ik vond op wmcity volgend script: http://www.wmcity.nl/scripts.php?actie=bekijk&id=1468

nu gebruik ik het zo..

PHP Code:

$text = preg_replace('/\[img](.+?)\[\/img]/is', '<a href="javascript:popUp(\'\\1\')">'.imageresizer('\\1').'</a>', $text); 


de code:

PHP Code:

function imageresizer( $image, $alt="" )
{
    $gewenste_breedte = 50;
    
    if( empty( $image ) )
    {
        $uitpoep = "U heeft geen bestand opgegeven.";
    }
    elseif( !@fopen( $image, "r" ) )
    {
        $uitpoep = "Het bestand dat u opgaf bestaat niet.";
    }
    else
    {
        if( $afmetingen = @getimagesize( $image ) )
        {
            $breedte = $afmetingen[0];
            $hoogte = $afmetingen[1];
            if( $breedte > $gewenste_breedte )
            {
                $breedte_n = $gewenste_breedte;
                $hoogte_n = round( ( ( $hoogte*$gewenste_breedte ) / $breedte ), 0 );
                $uitpoep = "<img src=\"" . $image . "\" alt=\"" . $alt . "\" style=\"width: " . $breedte_n."px; height: " . $hoogte_n . "px;\" />";
            }
            else
            {
                $uitpoep = "<img src=\"" . $image . "\" alt=\"" . $alt . "\" />";
            }
        }
        else
        {
            $uitpoep = "Het bestand dat u opgaf is geen afbeelding.";
        }
    }
    return $uitpoep;
} 


Nu krijg ik steeds, bij elke url die ik tussen de img tags zet:

het bestand dat u opgaf bestaat niet.




iemand enig idee?

mvg,
thijs v

Gebruik je directe URLs (http://www.site.com/image/test.jpg) of URLs direct naar je hosting? (image/test.jpg),.. in het tweede geval kan een fout met je directory voorkomen misschien.

Wat je anders eens kunt proberen is het op te delen in stukjes. Eerst de IMG-tag(s) opzoeken, en dan per tag de resize functie laten returnen in een var ($resizedImage = imageresizer($link); ---> en de imageresizer() laten returnen en niet echo'en!). Deze var weer gebruiken in de uiteindelijke replace functie.

Ik heb zelf ook al eens een dergelijke functie gemaakt, ik zal eens even zoeken voor je, dan plaats ik hem hier.

Hier is ie, de functie:

PHP Code:

function imgSize($file, $maxWidth, $maxHeight) {
        if(!empty($file)) {
            list($width, $height, $type, $attr) = getimagesize($file);
                                   
            $xRatio = $maxWidth / $width;
            $yRatio = $maxHeight / $height;
                        
                if ($width <= $maxWidth && $height <= $maxHeight) {
                    $newWidth = $width;
                    $newHeight = $height;
                }
                else if (($xRatio * $height) < $maxHeight) {
                    $newHeight = ceil($xRatio * $height);
                    $newWidth = $maxWidth;
                }
                else {
                    $newWidth = ceil($yRatio * $width);
                    $newHeight = $maxHeight;
                }
        
            return "<a href='".$file."' target='_blank'><img src=".$file." width=".$newWidth." height=".$newHeight." border=0></a>";
        
        }
        else {
            
            return "<img src='".$file."' border='0'>";
        
        }
} 


En het stuk UBB Code replacer:

PHP Code:

function ubb_code($string) {
   // hele meuk andere replacers hier (smileys e.d.)
   $string = preg_replace("/\[img\](.*?)\[\/img\]/e", "imgSize('\\1', 300, 250)", $string);
    return $string;
} 


Enige verschil wat ik zo 1,2,3 zie is dat ik de imgSize (de functie dus) niet 'uitgequote' heb hier. Aan de andere kant controleerd mijn functie wel niet of de file bestaat.

Code:

if($file != "") {

Bah.. ;) Hier hebben we trim( ) en empty( ) voor.

Deze code is 4 jaar oud, don't blame me :) Herinner me alleen dat ik deze code dus wel werkend heb gekregen omdat hij ingebouwd was in een nieuwssysteem en een forumpje wat ik destijds gebouwd heb.

Origineel gepost door Martijn Dwars

Code:

if($file != "") {

Bah.. ;)

Vrij nutteloos ook om er verder op in te gaan. Zoals ik al zei is dit 4 jaar oud en ik plaats het hier alleen omdat ik weet dat het werkte zoals ik het wilde en zoals de topicstarter probeert het werkend te krijgen.

Deze standaard van programmeerstijl ben ik allang voorbij ;)

Code:

$maxWidth = $maxw;
$maxHeight = $maxh;

Ook bah.. herschrijven van variabelen is het nuttelooste wat er maar bestaat en kost alleen maar meer geheugen. Bericht is overigens niet om jou te beledigen, maar om de TS te adviseren de code te herschrijven omdat hij niet helemaal correct is.

Ja, en jou reacties zijn ook bah. Echt een toegevoegde waarde hebben ze ook niet. Als je toch niks kunt toevoegen waar de topicstarter wat aan heeft kun je wat mij betrefd je reacties wel weglaten. Ik probeer de topicstarter alleen maar te helpen door te laten zien wat ik gebruikte (en wat voor mij werkte).

De kwaliteit van het stukje code doet er in mijn ogen niet echt toe.. maargoed, ik zal het voor je aanpassen, anders doe je misschien geen oog dicht vannacht ;)

Post voor de verandering eens een oplossing die wel werkt, want tot nu toe hebben we nog geen oplossing die de controle of de file wel bestaat oplost, en daar ligt het knelpunt. Niet of de functie empty() wordt gebruikt of dat vars worden herschreven.

Ik probeer enkel de TS te wijzen op wat slordigheidjes in jou geposte snipper en geef daarbij nog aan dat het totaal geen aanval op jou is. Dat er dan meteen zo'n sarcastische opmerking mijn kant op moet worden gemaakt vind ik erg jammer.. En trouwens, ik ging er vanuit dat jou snipper wel zou werken.. dan is het toch alleen maar mooi dat ik de TS wijs op deze slordigheidjes?

Zal deze discussie maar meteen staken, geen zin in geflame in dit topic. Kennelijk zijn mijn verbeterpunten aan de geposte oplossing niet gewend, dus dit zal ik voortaan niet meer doen.

Gaat er niet om dat je verbeterpuntjes gaat posten. Maar de manier waarop je dit doet, in eerste instantie plaats je ze in één post en daarna maak je er 2 losse posts van - dat was opzich al vreemd. Maar verder dan het het beetje 'bah' roepen en dan één regel tekst gaat het ook niet in je reactie.

Het is een oud script, niets met slordigheidjes te maken. Ik begon toen pas ''echt'' met PHP, dus dit zal eerder te maken hebben met iets wat ik toen nog niet wist dan een slordigheid :)

Maargoed, het scriptje is aangepast zodat de TS wat beters heeft.