Webshop > Etalage-pagina

Door Marieke Violet op 27-11-2010 21:02 gewijzigd op 27-11-2010 21:04

7.860 views

Beste Mede programmeurs..

Zelf zit ik in 6VWO, en we moeten een webshop programmeren..
Tot nu toe gaat dat redelijk, maar ik ben de laatste puntjes op de i aan het zetten,
en nu loop ik tegen 2 grote problemen aan.

1. Ik krijg een foutmelding, en weet niet waarom.. Wat ik eraan moet doen, ect.
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ' ORDER BY productnaam' at line 3
in file C:\xampp\htdocs\Php\HOOG5281\categorie_pagina.php on line 90
In regel 90 staat: $query = mysql_query($sql) or die (mysql_error()."<br>in file ".__FILE__." on line ".__LINE__);

Wie kan mij hiermee helpen?

2. Ik moet in de etalage nog de afbeelding weergeven...
Dit is mijn code tot nu toe..
<?php
include "verbinding.inc.php";
$sql = "SELECT product_ID, productnaam, kunstenaar, prijs_per_week, afbeelding, info
FROM producten WHERE
<?php
kunstenaar = $_GET[$kunstenaar];
>
ORDER BY productnaam";
$query = mysql_query($sql) or die (mysql_error()."<br>in file ".__FILE__." on line ".__LINE__);
$num = mysql_num_rows($query);

if(empty($num)) {
echo "<p>Er zijn geen producten gevonden.</p>\n";
} else {

while($product = mysql_fetch_object($query)) {
// plaatjes weergeven met een lusje? (formaat aangeven?) ?>
<img src="Pics/Pollock_Key.jpg">
<?php
echo "<form action=\"toevoegen.php\" method=\"POST\">\n";
echo "<input type=\"hidden\" name=\"productnummer\" value=\"".$product->product_ID."\" />\n";
echo "<p><strong>".$product->naam."</strong> - ".$product->productnaam."<br />\n";
echo "Informatie: ".$product->info."<br />\n";
echo "Prijs per week: EUR ".$product->prijs_per_week."<br />\n";
echo "Uitleenweek: <input type=\"text\" name=\"uitleenweek\" size=\"2\" maxlength=\"2\" value=\"1\" />\n";
echo "Inleverweek: <input type=\"text\" name=\"inleverweek\" size=\"2\" maxlength=\"2\" value=\"1\" />\n";
echo "<input type=\"submit\" value=\"Plaats in Winkelmandje\" /></p>\n";
echo "</form>\n";
}
}
?>

De afbeeldingen staan in de database van phpMyAdmin opgeslagen, via een link naar een map op de server..

Degene die mij denkt hiermee te kunnen helpen,
is een held, echt waar.
Ben er al uren mee bezig, en kom er maar niet uit..

Post een berichtje, en dan kunnen we communiceren over dit forum of de email, wat geliefd is.

Alvast bedankt!
xx
Marieke

Niels K

27-11-2010 21:15

Goh, wat ze hedendaags voor het vwo al niet moeten doen.
Maar goed, jouw probleem:

In jouw querie open je nogmaals php? Is dat nieuw? Geloof het niet he? Gewoon weghalen dus;) ( line 5 en 7 )

Verder op line 17 sluit je php tag weer? Word wel niet uitgevoerd omdat je die line becommentarieerd hebt maar toch.

Op line 19 open je hem weer?

Line 31 sluit je hem, kijk is goed maar is niet nodig. Dan kan je er ook geen problemen mee ondervinden :) Verder op line 31 ook nog [/i]

is line 33 tot 44 ook php?

Verder mis je nog wat foutenafhandeling en is je script gevoelig voor SQL injection.

Nou succes he;)

Milo S

27-11-2010 21:17 gewijzigd op 27-11-2010 21:23

Weet niet wat je hebt gedaan, maar veel soeps is het niet geworden.


<?php
include "verbinding.inc.php";
	
$sql = "SELECT
			product_ID,
			productnaam,
			kunstenaar,
			prijs_per_week,
			afbeelding,
			info
		FROM
			producten
		WHERE
			kunstenaar = '".mysql_real_escape_string( $_GET['kunstenaar'] )."'
		ORDER BY
			productnaam";
					
if( $res = mysql_query( $sql ) )
{
	if( mysql_num_rows( $res ) > 0 )
	{
		while( $rec = mysql_fetch_assoc( $res ) )
		{
			echo '<img src="Pics/Pollock_Key.jpg">
            
            <form action="toevoegen.php" method="post">
            	<input type="hidden" name="productnummer" value="' . $rec['product_ID'] . '" />
                <p><strong>' . $rec['naam'] . '</strong> - ' . $rec['productnaam'] . '<br />
                Informatie: ' . $rec['info']. '<br />
                Prijs per week: EUR ' . $rec['prijs_per_week'] . '<br />
                Uitleenweek: <input type="text" name="uitleenweek" size="2" maxlength="2" value="1" />
                Inleverweek: <input type="text" name="inleverweek" size="2" maxlength="2" value="1" />
                <input type="submit" value="Plaats in Winkelmandje" />
            </form>';
		}
	}
	else
	{
		echo 'Er zijn nog geen items gevonden.';	
	}
}
else
{
	echo 'Er is een fout in de query opgetreden.';
}
?>

- SanThe -

27-11-2010 21:23

@Milo: Puntkomma op regel 14?

Niels K

27-11-2010 21:23 gewijzigd op 27-11-2010 21:25

@Milo

Leuk dat je ze helpt, maar zo leert ze er niks van. Want als ze zelf goed had gekeken had ze achter mijn puntjes zelf ook kunnen komen?

Of help je nu gewoon omdat het iemand uit het vrouwelijk geslacht is?

Verder heeft milo nog last van XSS injection.

[offtopic]
Je weet wie het zegt :P
[/offtopic]

Milo S

27-11-2010 21:25

@ SanThe; Dankje SanThe, die had ik gemist.

@ Niels; Hmmm, ik heb het ook zo geleerd? tis nog best goed gekomen al zeg ik het zelf... En je 2e punt, misschien wel, maar Gelderland is best ver weg, heb ik niks an. Jaja ik weet het :P.

Niels K

27-11-2010 21:29

Haha, wat zijn we weer lekker bezig vanavond :-)

Hmmm, ik heb het ook zo geleerd?

Appart geval.

tis nog best goed gekomen al zeg ik het zelf

Oh? En OO dan?

misschien wel, maar Gelderland is best ver weg, heb ik niks an. Jaja ik weet het :P.

Haha, ach beter ver weg dan nooit toch? Want zeg zelf, jij bent toch de ideale schoonzoon? Haha.

Maar goed, succes marieke.

Milo S

27-11-2010 21:34 gewijzigd op 27-11-2010 21:35

[offtopic]

Haha, wat zijn we weer lekker bezig vanavond :-)

Vind je?

Appart geval.

Klopt :D

Oh? En OO dan?

Nou, vergelijken bij wat het was dan. Bijdehandjes gegeten? ikke wel :D.

Haha, ach beter ver weg dan nooit toch? Want zeg zelf, jij bent toch de ideale schoonzoon? Haha.

Wie zegt dat het nooit is? Ja dat klopt wel ja, en me pak is al binnen whahaha.

[/offtopic]

Niels K

27-11-2010 21:39

Haha, we zijn beiden goed op dreef vanavond. Maar goed, laten we ontopic gaan:)

Vind trouwens dat het database model beter kan. Jij niet milo?

[offtopic]

Post een berichtje, en dan kunnen we communiceren over dit forum of de email, wat geliefd is.

Jouw beurt milo.
[/offtopic]

Milo S

27-11-2010 21:41 gewijzigd op 27-11-2010 21:45

Ja kan wel eigenlijk, maar misschien kan madame beter even posten wat ze allemaal precies wil, dan kunnen we verder helpen.

[offtopic]

Jouw beurt milo.

Ik ga voor een beurt :$ :P XD

6VWO, das 2 jaar ouder als ik man... Val niet op oudere vrouwen dan ik, bah.
[/offtopic]

Niels K

27-11-2010 21:47

Wat is trouwens zie is dat afbeelding wel wordt geselecteerd maar niet wordt gebruikt?

Ik ga voor een beurt :$ :P XD

6VWO, das 2 jaar ouder als ik man... Val niet op oudere vrouwen dan ik, bah.

En het is weer weekend, whottefock.
Even weer on topic man, is alleen maar slap gelul dit, als je daar behoefte aan hebt de zeg je maar weer wat op msn.

Reageren

Inloggen om te reageren