[PHP/MySQL] Afbeelding uit database halen

[Genosha]

Ik heb door middel van een standaard form een afbeelding in de database gezet, dit omdat een bestands uploadscript niet altijd even lekker werkt en dit mij de makkelijkste oplossing leek.

Nu heb ik alleen een probleem... ik kan hem niet uitlezen, dan krijg ik een foutmelding.

Mijn code is als volgt:

nieuw.php

code:

<div class="adminlabel">Foto:</div>
    <input name="foto" type="file" size="30" />

opslaan.php (verwerkingscript)

PHP: filename

$foto = base64_encode(file_get_contents($_FILES['foto']['tmp_name']));

$opslaan = "INSERT INTO producten (naamprod, omschrijving, foto) 
            VALUES ('".$naamprod."',
                    '".$omschrijving."',
                    '".$foto."')";

Nu wil ik hem dus laten tonen wanneer ik het item bewerk, maar

code:

<div class="adminlabel">Foto:</div>
    <img src="<?php echo $arr_foto['foto']; ?>" /><br />

Geeft de volgende foutmelding

code:

<img src="
Fatal error: Cannot use object of type stdClass as array in C:\xampp\htdocs\xampp\...\...\product_bewerken.php on line 36

Dus het gaat niet zo makkelijk als ik dacht. Hoe moet ik een afbeelding (gewoon *.jpeg) uitlezen uit de database? Hoop dat iemand mij hier mee kan helpen.

(Een goed upload script kan natuurlijk ook).

[TeeDee]

Bedenk eens waarom dit niet werkt? Je output nu een binary array in je html. Dat gaat over het algemeen niet goed. Je kan nog een base64 als image source gebruiken, maar dat wordt niet altijd gesupport.

Als je <img src="img.php?img=1"> waarbij img=1 correspondeert met het record in de database.
Vervolgens op img.php de img parameter uitlezen en echo-en.

Of je uploadscript goed werkt kan ik niet zeggen.

[Iska]

Je zet eerst neer 'file_get_contents', en daarna probeer je een url eruit te krijgen xD, gaat niet werken he
Zoals je het nu doet (wat ik trouwens bijzonder sterk afraadt maar oké) is de data van de afbeelding in de database zetten. Als je die dan wil weergeven moet je hem weer terugtoveren naar een afbeelding (je zult dan dus ook het afbeelding-type moeten opslaan).
Dat terugtoveren kan bijv. via 'imagejpeg' of 'imagepng'. Als eerste argument geef je wat er in de database stond en het resultaat is de afbeelding en dan komt je afbeelding tevoorschijn.

Maar nogmaals, probeer het echt anders te doen.. want je database gaat heel vol en langzaam worden + dat dit relatief veel rekenkracht kost


TeeDee was net eerder

[Kettrick]

Jasper91 schreef op woensdag 15 oktober 2008 @ 16:42:
Zoals je het nu doet (wat ik trouwens bijzonder sterk afraadt maar oké) is de data van de afbeelding in de database zetten. Als je die dan wil weergeven moet je hem weer terugtoveren naar een afbeelding (je zult dan dus ook het afbeelding-type moeten opslaan).
Dat terugtoveren kan bijv. via 'imagejpeg' of 'imagepng'. Als eerste argument geef je wat er in de database stond en het resultaat is de afbeelding en dan komt je afbeelding tevoorschijn.

Maar nogmaals, probeer het echt anders te doen.. want je database gaat heel vol en langzaam worden + dat dit relatief veel rekenkracht kost

Hier is onlangs nog een discussie over geweest, php en image caching

plaatjes in een database is niet per definitie een slecht idee

[FragFrog]

Genosha schreef op woensdag 15 oktober 2008 @ 16:22:
Geeft de volgende foutmelding

code:

<img src="
Fatal error: Cannot use object of type stdClass as array in C:\xampp\htdocs\xampp\...\...\product_bewerken.php on line 36

Lijkt me vrij evident wat hier mis gaat: je probeert een object uit te lezen alsof het een array is.

Kijken we naar de plek van de foutmelding: "<?php echo $arr_foto['foto']; ?>" - je probeert hier $arr_foto te benaderen alsof het een array is terwijl volgens PHP het een stdClass (ergo, object) is. Hoe kan dit?

Mijn gok: je doet niet mysql_fetch_assoc maar mysql_fetch_object ergens - helaas laat je de essentiele code nou weer net niet zien

_{Overigens ben ik het in zoverre niet met bovenstaande poster eens dat plaatjes in je DB bijna altijd een slecht plan zijn, performance wise en qua DB handelbaarheid, maar dat is hier nu net niet de discussie}

[Verwijderd]

Wat ik altijd doe is de bestandsverwijzing in een database zetten, en het plaatje in een uploads mapje zetten. Dat is volgens mij ook beter voor de performance van je database en het uitlezen van de array is dan ook geen probleem. Mocht je het per se in een database willen hebben, moet je dacht ik de binary data in een blob veldje gooien en uitlezen in een php scriptje dat met GD een plaatje vouwt. Dat scriptje spreekje dan aan in je <img> tag.

Zoiets dus:

code:

<img src="bakjpg.php?ID=<?=$row['plaatjeID']?>" alt="<?=$row['plaatjeAlt']?>" />

Dat bak jpg script maakt de connectie, haalt de data eruit en maakt een jpg met GD.

Maar het is imho veel gemakkelijker om de bestanden gewoon als een bestand weg te schrijven in httpdocs/img/uploads oid en de bestandslocatie op te slaan in een varchar veldje.

[Cartman!]

Je hebt helemaal geen GD nodig, je spuugt gewoon die binaire data uit met de juiste header. Overigens moet je wel weten wat voor image t is, GIF, JPG, PNG of misschien BMP of noem maar op want die hebben andere headers

[Genosha]

Ik gebruik inderdaad mysql_fetch_object om het foto-veld uit mijn database te halen. $arr_ is gewoon de standaard die ik gebruik zodat ik niet 100 varianten heb die allemaal eigenlijk query results zijn, misschien verwarring hierdoor.

Waarom database? Omdat ik geen goed uploadscript kon vinden en de database mij een goed alternatief leek, vooral omdat de afbeeldingen niet zo heel erg groot in KB worden, worden dus geen foto's van 1MB. Ook moet het systeem noob-proof zijn, niet elke gebruiker gaat snappen dat hij/zij de afbeelding moet uploaden via de FTP en dan de locatie moet vermelden in dat veldje.

Het lijkt me natuurlijk mogelijk om een restrictie op alleen .JPG te maken zodat ik maar met één type header rekening moet houden.

Anyway... nog geen idee hoe ik het moet oplossen. Hoe lees je dat nu precies uit?

[Kettrick]

Genosha schreef op donderdag 16 oktober 2008 @ 10:02:
Waarom database? Omdat ik geen goed uploadscript kon vinden en de database mij een goed alternatief leek,

Ondanks mijn eerdere verhaal over plaatjes in databases is nou niet bepaald de juiste werkwijze .

vooral omdat de afbeeldingen niet zo heel erg groot in KB worden, worden dus geen foto's van 1MB.

Elke normale database heeft er geen problemen mee om een bestand van 1MB op te slaan, wat dat betreft maakt het opslaan op disk of in een db weinig uit.

Ook moet het systeem noob-proof zijn, niet elke gebruiker gaat snappen dat hij/zij de afbeelding moet uploaden via de FTP en dan de locatie moet vermelden in dat veldje.

Dat is ook niet nodig, je kan met php de file upload afhandelen en in een dir op de server neerzetten, of je het op filesystem of database niveau opslaat maakt voor de gebruiker niet uit. Googlen op php file upload moet je daar genoeg over kunnen vertellen

Het lijkt me natuurlijk mogelijk om een restrictie op alleen .JPG te maken zodat ik maar met één type header rekening moet houden.

Anyway... nog geen idee hoe ik het moet oplossen. Hoe lees je dat nu precies uit?

Kijk naar de eerder genoemde oplossingen, je kan niet simpelweg de image data ergens in je html neer zetten. Ik raad je overigens wel aan om je blob data buiten je normale data neer te zetten, dus niet een zootje binaire data tussen je prijzen en je producten. Ik gebruik zelf voor een project een koppeltabel tussen huizen en blobs, de blobdata wordt dus alleen opgehaald door getblob.php, verder nergens.

Kijk ook nog even naar http header / cache control, zorg er dan voor dat je niet de hele dag blobs loopt te maken maar dat deze door de client gecached worden.

[Equator]

Genosha schreef op donderdag 16 oktober 2008 @ 10:02:
Waarom database? Omdat ik geen goed uploadscript kon vinden

Zo moeilijk is dat anders niet.

PHP:

if(!empty($_FILES['poelaatje']['name']))
{
    if (move_uploaded_file($_FILES['poelaatje']['tmp_name'], "/path/to/dir/" . $_FILES['poelaatje']['name'] ))
    {
        //upload sucessfull
    }
    else
    {
        //upload unsuccessfull
        echo "upload error";
        die();
    }
}

Voor error checking zien: http://www.php.net/features.file-upload

[Verwijderd]

Cartman! schreef op donderdag 16 oktober 2008 @ 09:46:
Je hebt helemaal geen GD nodig, je spuugt gewoon die binaire data uit met de juiste header. Overigens moet je wel weten wat voor image t is, GIF, JPG, PNG of misschien BMP of noem maar op want die hebben andere headers

Klopt natuurlijk, dom.

Genosha schreef op donderdag 16 oktober 2008 @ 10:02:
Het lijkt me natuurlijk mogelijk om een restrictie op alleen .JPG te maken zodat ik maar met één type header rekening moet houden.

Dit is niet nodig. Gewoon kijken wat voor Content Type je plaatje is, vergelijken met de extentie en vervolgens de juiste header zetten.

Uitlezen van de database is geen probleem toch? RoeLz says it pretty much all. je moet de data niet direct in je <img> tag gooien, maar in een php script dat een plaatje maakt, en die aanroepen in je <img> tag. De data is namelijk (nog) geen plaatje, maar is binary data. Dat van die Cache control is een hele goeie tip, dat scheelt je weer een hoop performance.

[Equator]

Verwijderd schreef op donderdag 16 oktober 2008 @ 10:35:
Uitlezen van de database is geen probleem toch? RoeLz says it pretty much all. je moet de data niet direct in je <img> tag gooien, maar in een php script dat een plaatje maakt, en die aanroepen in je <img> tag. De data is namelijk (nog) geen plaatje, maar is binary data. Dat van die Cache control is een hele goeie tip, dat scheelt je weer een hoop performance.

Op het feit na dat mysql_fetch_object() geen array returned, maar een mysql object.
Hij kan veel beter naar mysql_fetch_array / mysql_fetch_assoc kijken.

[Genosha]

Equator schreef op donderdag 16 oktober 2008 @ 10:33:
[...]
Voor error checking zien: http://www.php.net/features.file-upload

Altijd geklooi met dat filepath, alles geprobeerd en helemaal niets.

[Kettrick]

Genosha schreef op donderdag 16 oktober 2008 @ 11:12:
[...]


Altijd geklooi met dat filepath, alles geprobeerd en helemaal niets.

Equator heeft al een hele opzet gegeven, wat gaat er precies mis ?
Let er wel op dat, afhankelijk van je hoster, je aan de slag moet met chmod e.d., maar in dat geval zou je iets als permission denied moeten krijgen.

error_reporting(E_ALL) is iig je vriend in dit soort situaties

[Genosha]

Ik draai momenteel locaal d.m.v. Xampp op een Windows systeem, geen CHMOD dus.

code:

Warning: move_uploaded_file(/htdocs/xampp/iwood/final/fotos/lowl.JPG) [function.move-uploaded-file]: failed to open stream: No such file or directory in C:\xampp\htdocs\xampp\iwood\final\admin\product_nieuw_opslaan.php on line 18

Warning: move_uploaded_file() [function.move-uploaded-file]: Unable to move 'C:\xampp\tmp\php31C.tmp' to '/htdocs/xampp/iwood/final/fotos/lowl.JPG' in C:\xampp\htdocs\xampp\iwood\final\admin\product_nieuw_opslaan.php on line 18
upload errorArray ( [foto] => Array ( [name] => lowl.JPG [type] => image/pjpeg [tmp_name] => C:\xampp\tmp\php31C.tmp [error] => 0 [size] => 27192 ) )

[Equator]

C:\xampp\tmp\php31C.tmp' to '/htdocs/xampp/iwood/final/fotos/lowl.JPG'

Ik heb nog nooit php op een windows systeem gehad, maar bovenstaande lijkt me vrij 'logisch'

2e hit @Google: http://php.meetup.com/93/messages/boards/thread/1950602
Dat gaat welliswaar over een persission probleem, maar je ziet wel de correcte syntax met pad namen op Windows

[ Voor 35% gewijzigd door Equator op 16-10-2008 11:28 ]

[FragFrog]

Genosha schreef op donderdag 16 oktober 2008 @ 10:02:
Ik gebruik inderdaad mysql_fetch_object om het foto-veld uit mijn database te halen. $arr_ is gewoon de standaard die ik gebruik zodat ik niet 100 varianten heb die allemaal eigenlijk query results zijn, misschien verwarring hierdoor.
[..]

Anyway... nog geen idee hoe ik het moet oplossen. Hoe lees je dat nu precies uit?

Ik gaf al aan wat er fout ging: je leest een object uit alsof het een array is, daarover geeft PHP een error. De PHP manual is dan vrij simpel te vinden: http://php.net/mysql_fetch_object

Op die pagina staat een uitstekend voorbeeld hoe je dan wel data uit zo'n object haalt. Alternatief is, zoals ik al zei, mysql_fetch_assoc te gebruiken in plaats van mysql_fetch_object.

//edit @ hierboven
Onder windows mag je met PHP zowel de dubbele forward-slash als backslash gebruiken. Ie, zowel /pad/naar/folder als \\pad\\naar\\folder zijn goed. Die temp name wordt door Apache gegenereerd en is (in theorie) ook gewoon goed. Relatieve paden mogen voor zover ik weet ook gewoon, dus de error is volgens mij helemaal niet zo evident.

Ik gok persoonlijk dat er een folder niet bestaat danwel het script in een subfolder aan het zoeken is ernaar, wat je uiteraard wel weer op kan lossen door daar een absoluut pad vanaf de root van te maken (eventueel met $_SERVER['script_name'] / $_SERVER['script_filename']). Zou echter niet moeten hoeven

//edit
Bedenk me net trouwens een minder voor de hand liggend probleem: je gebruikt Xampp. Dit is in mijn ervaring zo'n brak pakket dat random dingen die op een normale apache + mysql + php install perfect werken daar ineens stuk gaan. Dingen als driedubbele configfiles die op volslagen debiele locaties worden opgeslagen, vage security-defaults, etc. Als je dan toch een all-in-one pakket wilt hebben ben je beter af met AppServ, levert bij mij veel minder vaak dit soort problemen op.

[ Voor 45% gewijzigd door FragFrog op 16-10-2008 11:42 ]

[Verwijderd]

Equator schreef op donderdag 16 oktober 2008 @ 10:39:
[...]

Op het feit na dat mysql_fetch_object() geen array returned, maar een mysql object.
Hij kan veel beter naar mysql_fetch_array / mysql_fetch_assoc kijken.

Klopt maar

PHP:

echo $row->fotodata;

Kan toch ook gewoon neem ik aan?

En die foutmelding lijkt me nogal self explainatory, een bestand naar een windows machine wegschrijven met een linux bestandslocatie gaat niet werken

[Cartman!]

Ik kan melden dat fileuploads op xampp geen probleem hoeft te zijn, hier werkt het namelijk uitstekend. Als je je paden maar goed weet in te stellen

[Genosha]

Ik heb het script nu draaien op een Linux webhost waar doormiddel van een andere pagina al een upload script gebruikt wordt, het zal dus moeten werken, maar helaas. Een foutmelding

Warning: move_uploaded_file(/public_html/iwood/fotos/rick_astley-gal-goodweek.jpg) [function.move-uploaded-file]: failed to open stream: No such file or directory in /home/(site)/domains/(site).com/public_html/iwood/admin/product_nieuw_opslaan.php on line 18

Warning: move_uploaded_file() [function.move-uploaded-file]: Unable to move '/tmp/phpS0YGSr' to '/public_html/iwood/fotos/rick_astley-gal-goodweek.jpg' in /home/(site)/domains/(site).com/public_html/iwood/admin/product_nieuw_opslaan.php on line 18
upload errorArray ( [foto] => Array ( [name] => rick_astley-gal-goodweek.jpg [type] => image/pjpeg [tmp_name] => /tmp/phpS0YGSr [error] => 0 [size] => 32083 ) )

De CHMOD op de map foto's is 777

De map locatie is /public_html/iwood/fotos/ zoals mijn FTP client ook aangeeft. Heb geen idee wat er mis gaat.

[Verwijderd]

post heel je serveruploadcode eens. Want zo tasten we in het duister

[Genosha]

Heb het scriptje wat Equator hier poste gebruikt maar is als volgt;

PHP:

<?php 
// header("Location: admin.php?pagina=producten&id=product_overzicht"); 

include 'config.php'; 

$naamprod= $_POST['naamprod'];
$omschrijving= $_POST['omschrijving'];
//$foto = base64_encode(file_get_contents($_FILES['foto']['tmp_name']));
$foto= $_POST['foto'];
$video = $_POST['video'];
$datum_ingang = $_POST['datum_ingang'];
$datum_stop = $_POST['datum_stop'];
$kliks = $_POST['kliks'];

 
if(!empty($_FILES['foto']['name'])) 
{ 
    if (move_uploaded_file($_FILES['foto']['tmp_name'], "/public_html/iwood/fotos/" . $_FILES['foto']['name'] )) 
    { 
        //upload sucessfull 
    } 
    else 
    { 
        //upload unsuccessfull 
        echo "upload error"; 
        print_r($_FILES);
        
        print
        
        die(); 
    } 
} 


$opslaan = "INSERT INTO iwood_producten (naamprod, omschrijving, foto, video, datum_ingang, datum_stop, kliks) 
            VALUES ('".$naamprod."',
                    '".$omschrijving."',
                    '".$foto."',
                    '".$video."',
                    '".$datum_ingang."',
                    '".$datum_stop."',
                    '".$kliks."')";

if (!($temp = mysql_query($opslaan))) 

showerror(); 

?> 

[Kettrick]

Verander de 2e param op regel 18 eens in :

code:

$_SERVER['DOCUMENT_ROOT']."/iwood/fotos/" . $_FILES['foto']['name'] ))

Je probeert de fofo's op de verkeerde plaats op te slaan.

[Genosha]

WOW! Hij upload!
Maar nu ik geen locatie van mijn afbeelding in mijn database staan, kan ik dit combineren of uitroepen uit het script?

[Verwijderd]

Genosha schreef op vrijdag 17 oktober 2008 @ 14:42:
WOW! Hij upload!
Maar nu ik geen locatie van mijn afbeelding in mijn database staan, kan ik dit combineren of uitroepen uit het script?

ofwel sla je dat mee op, ofwel stopt je dat in een config-file, ofwel stop je dat in een configtabel, ofwel verzin je nog iets anders. De keuze is aan jou.

[ Voor 4% gewijzigd door Verwijderd op 17-10-2008 14:45 ]

[Genosha]

Verwijderd schreef op vrijdag 17 oktober 2008 @ 14:45:
[...]

ofwel sla je dat mee op, ofwel stopt je dat in een config-file, ofwel stop je dat in een configtabel, ofwel verzin je nog iets anders. De keuze is aan jou.

Als ik niet weet hoe zal ik het niet vragen natuurlijk...

[Cartman!]

Je weet toch de locatie waar die image wordt opgeslagen? Die kun je opslaan als varchar in je tabel...

[Genosha]

'".$_SERVER['DOCUMENT_ROOT']$_FILES['name']."',

Dit werkt natuurlijk niet om de locatie en bestandnaam op te halen en wegteschrijven naar me VARCHAR(255) veldje.

[Cartman!]

Een beetje zelf nadenken mag best hoor... verzin gewoon dat je het deel moet opslaan in je database zoals je t vanaf web moet benaderen bijv.

[Bolukan]

Waarom zou je $_SERVER['DOCUMENT_ROOT'] opslaan ??