row uitlezen van tabel

Gerrit Wiersma op 24/07/2010 22:51:13

$query = "SELECT * FROM uitslagen where jaartal = $_get['jaartal']";

Probeer eens:

$query = "SELECT * FROM uitslagen WHERE jaartal = '".$_GET['uitslag']."'";

is toch niet zo moeilijk?

if (isset($_GET['uitslag'])){
$query = "SELECT * FROM uitslagen WHERE jaar='$_GET[uitslag]'";
}else{
$query = "SELECT * FROM uitslagen";
}

Lieve god.
Alsjeblieft.
Ga allemaal eerst eens een handleiding lezen ofzo.
[php]mysql_Real_escape_string[/php].

Tip 1: gebruik de InnoDB engine ipv MyISAM
Tip 2: gebruik mysql_fetch_assoc ipv de _fetch_array
Tip 3: superglobals zijn gedefinieerd in hoofdletters $_get[] <> $_GET[]
Tip 4: pas op voor sql-injectie

<?php
$sWhereClause = '';

if (isset($_GET['jaartal']) && ctype_digit($_GET['jaartal']) {
$sWhereClause = "WHERE jaartal = ".mysql_real_escape_string($_GET['jaartal']);
}

$sql = "SELECT *
FROM uitslagen ".$sWhereClause;

if (($result = mysql_query($sql)) == false) {
echo mysql_error().'<br />'.$sql;
}
else {
while ($row = mysql_fetch_assoc($result)) {
print_r($row);
}
}
?>

Karl Karl op 24/07/2010 23:05:05

Lieve god.
Alsjeblieft.
Ga allemaal eerst eens een handleiding lezen ofzo.
[php]mysql_Real_escape_string[/php].

jij doelt om sql injections?

En selecteer dan ook nog eens wat je wilt hebben. Sterretje is een wildcard, dus de db mag net zo goed niks terug geven. Ook is het nog eens duidelijker, voor later. Dat is het mooie aan SQL het documenteert zichzelf, want je kunt zien wat je selecteerd.

Stefan Renne op 24/07/2010 23:07:18

[quote="Karl Karl op 24/07/2010 23:05:05"]
Lieve god.
Alsjeblieft.
Ga allemaal eerst eens een handleiding lezen ofzo.
[php]mysql_Real_escape_string[/php].

jij doelt om sql injections?
[/quote]
Ja, en om ongeldige data.

Bedankt heren voor de reacties maar ben nog geen streep verder, nog steeds een wit scherm met helemaal niks ;)

Selecteer ook wat je wilt hebben en plaats bovenaan in elk script:
error_reporting(E_ALL);
ini_set('display_errors', 1);

het is trouwens de bedoeling dat als ik op link: uitslag_uitlezen.php?jaartal=2010 klik, ik dan een scherm met linkjes van uitslagen in 2010 krijg, het uitlezen daarvan snap ik wel.

Ik zal het er eens in zetten Karl ;)

Dit kwam er uit :)

Notice: Undefined index: uitslag in /customers/gwmedia.nl/gwmedia.nl/httpd.www/degraaf/uitlezen_jaartal.php on line 48

en dan op regel 48 staat:

$query = "SELECT * FROM uitslagen WHERE jaartal = '".$_GET['uitslag']."'";