CIO Tech Eksperten IT-JOB IT-Kurser Events Podcast Søg

Log ind eller opret profil

Du kan også logge ind via nedenstående tjenester

spil-eksperten Nybegynder

30. november 2010 - 14:18 Der er 15 kommentarer og
1 løsning

Åbenbart fejl i PHP... get image from mysql

Hej ...
Jeg troede at jeg sagtens kunne lave det her.. men den kommer frem med fejl:
Parse error: parse error in C:\wamp\www\dddd\viewimage.php on line 8
1 <?php
2 if(isset($_REQUEST['id']))
3 {
4 include "connect.php";
5
6
7
8 $id = $_ REQUEST['id']);
9 $query = mysql_query("SELECT 'img_name', 'img_type', 'img_size', 'img_data' FROM img_tbl WHERE id=$id");
10
11 $result = mysql_query($query) or die(mysql_error());
12 list($name, $type, $size, $content) = mysql_fetch_array($result);
13
14 header("Content-length: $size");
15 header("Content-type: $type");
16 print $content;
17
18 mysql_close($dbconn);
19 }
20 ?>

kan få lagt billeder op i mysql men kan ikke få dem ned igen!..
håber der er en som kan hjælpe..

Synes godt om

repox Seniormester

30. november 2010 - 14:23 #1

På linie 8: $id = $_ REQUEST['id'];

Synes godt om

danco Nybegynder

30. november 2010 - 14:23 #2

I linje 8 har du:

$id = $_ REQUEST['id']);

Skift det ud med:

$id = $_ REQUEST['id'];

får du stadig en fejl? i så fald, hvilken?

Synes godt om

spil-eksperten Nybegynder

30. november 2010 - 14:30 #3

rookie mistake... :)
men det har bare været en tastefejl herinde.. har ikke den ")" med i .php filen...
og ja den siger stadig det samme

Synes godt om

danco Nybegynder

30. november 2010 - 14:35 #4

Har du bestemt grund til at bruge $_REQUEST frem for $_GET og så bruge:
viewimage.php?id=213

Synes godt om

spil-eksperten Nybegynder

30. november 2010 - 15:04 #5

nej ikke rigtig..
har bare lært at bruge $_REQUEST...
men $_GET siger samme fejl...

det er jo heller ikke sikkert at det er linie 8 der er fejl i.. men kan simpelhen ikke gennemskue det..
har selvfølgelig et table som hedder

---img_tbl----
id
img_name
img_type
img_size
img_data

så vidt jeg kan forstå er der fejl med de 2 sætninger med at få fat i 'id' ...
altså..

2 if(isset($_REQUEST['id']))
8 $id = $_ REQUEST['id'];

er bare ikke så rutineret til at forstå hvorfor.. :(

forstår ikke hvad du mener med viewimage.php?id=213

Synes godt om

Slettet bruger

30. november 2010 - 15:52 #6

Husk også sikkerheden...
Hvis du sender en udefrakommende værdi direkte til din SQL-server via en query, er du fuldstændigt åben over for SQL injections. Hvis der er tale om heltal, kan du f.eks. sikre dig ved at gennemtvinge typen via casting med (int)$_REQUEST['id'].

Synes godt om

spil-eksperten Nybegynder

30. november 2010 - 16:03 #7

ja, jeg har tænkt over det...
men lige nu er det på WAMP .. og skal bare have det til at fungere..
har prøvet i mere end en uge at uploade et billede til mysql...
virker fint..
men at få det ned på en (anden end den man uploader fra) webpage kan åbenbart ikke lade sig gøre..(jeg kan ihvertfald ikke)

Synes godt om

repox Seniormester

30. november 2010 - 19:27 #8

er du sikker på du får de rigtige headers med ud? (læs: fortæl os hvilkere headere du præsenterer dit data med)

Synes godt om

cronaldo Nybegynder

30. november 2010 - 21:29 #9

Hvad med det mellemrum der er mellem $_ og REQUEST?

Synes godt om

danco Nybegynder

30. november 2010 - 22:09 #10

Jeg tænker at det kunne være relevant at vide hvad du overhovedet får ud fra databasen. Hvilken datatyper ligger dine data som?
Her tænker jeg om billederne ligger gemt ned som BLOB eller det er en varchar indeholdende filsti eller hvordan databasen ser ud.

Hvad jeg mente med $_GET tidligere var blot at du kunne hente din id fra URL'en vha. GET frem for REQUEST. Således at din url på et billede kunne være viewimage.php?id=4223

Det er korrekt at du skal være særdeles varsom med at bruge variable direkte i SQL forespørgsler uden at "vaske" dem først. Hvis dit billede har et id vil det altid være et tal og derfor kunne du fx bruge is_numeric() funktionen til det trick.

Synes godt om

cronaldo Nybegynder

30. november 2010 - 22:11 #11

1 <?php
2 if(isset($_REQUEST['id']))
3 {
4 include "connect.php";
5
6
7
8 $id = $_REQUEST['id'];
9 $query = mysql_query("SELECT 'img_name', 'img_type', 'img_size', 'img_data' FROM img_tbl WHERE id="'.mysql_real_escape_string($id)."'");
10
11 $result = mysql_query($query) or die(mysql_error());
12 list($name, $type, $size, $content) = mysql_fetch_array($result);
13
14 header("Content-length: $size");
15 header("Content-type: $type");
16 print $content;
17
18 mysql_close($dbconn);
19 }
20 ?>

prøv det der :)

Synes godt om

spil-eksperten Nybegynder

30. november 2010 - 23:13 #12

#cronaldo .. Thank u.. another rookie mistake :)
smid et svar
selvom jeg ikke er færdig med koden så var det den fejl jeg ledte efter...

er nu kommet frem til næste fejl.. så hvis der er nogen som vil hjælpe alligevel, ville det være cool..

Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given in on line 12

12 list($name, $type, $size, $content) = mysql_fetch_array($result);

#danco ..
id = int AI primary
img_name = varchar
img_type = varchar
img_size = int
img_data = blob

skulle være fint nok..

Synes godt om

cronaldo Nybegynder

30. november 2010 - 23:25 #13

Svar :)

Synes godt om

alphabits Nybegynder

01. december 2010 - 01:48 #14

Ang. din nye fejl, skyldes det at du kalder mysql_query to gange, og så mener jeg også at mysql brokker sig når man bruger ' omkring kolonnenavne. Istedet kan du bruge ` eller ingen ting.

<?php

    if(isset($_REQUEST['id']))
    {
        include "connect.php";

        $id = $_REQUEST['id'];
        $query = "SELECT img_name, img_type, img_size, img_data 
            FROM img_tbl WHERE id="'.mysql_real_escape_string($id)."'";

        $result = mysql_query($query) or die(mysql_error());
        list($name, $type, $size, $content) = mysql_fetch_array($result);

        header("Content-length: $size");
        header("Content-type: $type");
        print $content;

        mysql_close($dbconn);
    }

Synes godt om

alphabits Nybegynder

01. december 2010 - 01:52 #15

Der er en lille fejl i mit forrige indlæg. Kom til at bytte om på anførselstegnene. Istedet for:


$query = "SELECT img_name, img_type, img_size, img_data 
    FROM img_tbl WHERE id="'.mysql_real_escape_string($id)."'";

skulle der stå


$query = "SELECT img_name, img_type, img_size, img_data 
    FROM img_tbl WHERE id='".mysql_real_escape_string($id)."'";

Synes godt om

spil-eksperten Nybegynder

01. december 2010 - 02:37 #16

ahh tak..
men nu viser den blank side...
ved du hvorfor den gør det?.. burde der ikke komme et billede frem?

Synes godt om

Ny bruger Nybegynder

Din løsning...

Tilladte BB-code-tags: [b]fed[/b] [i]kursiv[/i] [u]understreget[/u] Web- og emailadresser omdannes automatisk til links. Der sættes "nofollow" på alle links.

Følg dette spørgsmål

Opret Preview

Webudvikling kurser

Vi tilbyder markedets bedste kurser inden for webudvikling

Se alle Webudvikling kurser

Flere spørgsmål fra PHP kategorien

Titel	Indlæg	Oprettet	Seneste aktivitet
Datafordeler Af Lsk i PHP	2	I går 22:00	I dag 09:30
Datoformat ved eksport til Excel Af dane022 i PHP	8	17/09/202414:48	30/09/202419:55
Hjælp til PHP ( mener jeg det hedder) Af lurup i PHP	0	12/09/202423:39	-
wp_delete_user virker ikke i Wordpress Af Ravnsen i PHP	2	21/08/202417:53	21/08/202422:03
php driller Af moscov i PHP	1	19/08/202415:09	20/08/202421:24

Se alle spørgsmål i kategorien Opret spørgsmål

Log ind eller opret profil

Hov!

For at kunne deltage på Computerworld Eksperten skal du være logget ind.

Det er heldigvis nemt at oprette en bruger: Det tager to minutter og du kan vælge at bruge enten e-mail, Facebook eller Google som login.

Du kan også logge ind via nedenstående tjenester

Alle kategorier på Eksperten

Seneste artiklerRSS

01/11

Test af Apple Watch Series 10: Apples jubilæums-model er blevet lidt af en mester af det hele

01/11

Google Clouds netop fratrådte topchef har fundet sit nye job: Bliver en af de mest magtfulde personer hos ServiceNow

01/11

Nu går OpenAI efter struben på Google: ChatGPT fungerer nu også som en AI-søgemaskine

01/11

Nørgaard: Agenterne kommer, og jeg er klar: Måske bliver min gamle drøm snart virkelighed

01/11

Staten vil selv udvikle kæmpe it-system til 650 millioner kroner: Nu tvinges regeringen til at lægge vigtige dokumenter frem

01/11

Det går fantastisk på det danske Dynamics-marked: NNIT-selskabet Scales kan vælge og vrage mellem opgaver

01/11

Linus Torvalds giver russiske Linux-udviklere sparket: Vil ikke have noget med dem at gøre

01/11

Skelsættende GDPR-sag ændrer definitionen af 'følsomme oplysninger'

01/11

Google indlemmer sin AI Gemini i Google Maps, Google Earth og Waze: Det kommer det til at betyde

01/11

Microsoft: Tvungen skift til Windows 11 kan udskydes med et år, hvis du betaler os 200 kroner

01/11

Sådan blev Margrethe Vestager ’Silicon Valleys største fjende’

Vis flere artikler

IT-JOB

Det Kongelige Teater

Applikationsudvikler med skarpt overblik

Unik System Design A/S

Proceskonsulent / OKR Lead

Jyske Bank

Product Owner til investering

Udviklings- og Forenklingsstyrelsen

Økonomipartner til økonomistyring i en spændende it-organisation

SDC A/S

Procurement Specialist med kommercielt mindset

Vis flere jobs

Seneste spørgsmål Seneste aktivitet

I dag 06:10	Excel åbner fil i kæmpe format Af Aske i Excel
I går 22:00	Datafordeler Af Lsk i PHP
I går 12:37	Summere beløb pr. dato Af TTA i Excel
31/1022:44	Tilslutte chassic fans Af viking69 i PC
31/1020:28	LED lysstofrør Af ErikHg i Fri debat