PHPhulp.nl
35k+ leden Over PHPhulp Offerte opdracht Contact
P

Ineens werkt SELECT niet!

Door Patrick Helsloot op 20-10-2010 15:29
1.848 views
Ik heb het volgende PHP-script:

<?php

function getDir($iDir)
{
$aDirs = explode('/', $_SERVER['REQUEST_URI']);
return $aDirs[$iDir];
}
$id = getDir(2);
$dbhost="localhost";
$dbuser="";
$dbpass="";
$db="content";

$mysql_id=mysql_connect("$dbhost","$dbuser","$dbpass");
$sql="SELECT trailer FROM film WHERE id=$id";

$res=mysql_db_query("$db","$sql",$mysql_id);
$fout=mysql_errno($mysql_id);
if ($fout != 0) {
$fout=mysql_error($mysql_id);
echo '<font color="#FF2306">' . $fout . '</font>';
mysql_close($mysql_id);
echo '</body></html>';
exit;
}


echo "$row[0]";
?>

Als ik dit script uitvoer krijg ik geen resultaat.. Hij geeft zelfs geen fouten weer. Als ik vraar om $id dan krijg ik netjes de ID van de pagina, maar waarom kan hij de gegevens niet uit de database halen?

Met vriendelijke groet,
Patrick
T
Teun Hesseling
20-10-2010 15:37 gewijzigd op 20-10-2010 15:39
geen flauw idee of het uitmaakt maar dit:<?php WHERE id=$id ?> is <?php WHERE id= '".$id."' ?> en dit <?php $res=mysql_db_query("$db","$sql",$mysql_id) ?> kleur verschil zegt al genoeg
P
Patrick Helsloot
20-10-2010 15:39
Helaas werkt niet... Snap er niets meer van! Volgens mij zou het gewoon moeten werken!
Jel le
20-10-2010 15:41
Ik zie ook nergens waar $row word aangemaakt.
P
Patrick Helsloot
20-10-2010 15:46
Heey! Dat is een heldere opmerking... Die regel was ik vergeten. Soms zit ik me zo blind te staren... Super bedankt.

$row=mysql_fetch_row($res);

toegevoegd en nu werkt het...
T
Teun Hesseling
20-10-2010 15:48
haha ik dacht die row een kopieer fout was lol
Niels K
20-10-2010 15:53 gewijzigd op 20-10-2010 15:54
Ga eens netjes scripte jonge!

Wat voor je veranderd.. En nee, script werkt nog steeds niet.. Lees commentaar:)

<?php

/**
* Functie om een specifieke map te verkrijgen
*
* @var integer $dir
* @return Void
*/
function getDir( $dir ) {
$dirs = explode( '/', $_SERVER['REQUEST_URI'] );
return $dirs[$dir];
}

// Specifiek commentaar
$id = getDir(2);

// Array met database configuratie.. Geen appart bestand?
$db = array(
'host' => 'localhost',
'user' => '',
'password' => '',
'database' => 'content'
);

// Connecteren met MySQL database
$mysql = mysql_query( $db['host'], $db['user'], $db['password'], $db['database'] );

// Controleren of connectie is gelukt
if( !$mysql ) {
echo 'Error blaadiebla';
}

// Database selecteren doen we niet meer???

// Query
$query =
"
SELECT
trailer
FROM
film
WHERE
id = '" .(int)$id. "'
";

// mysql_db_query ?? WTF hoe oud, en al die foute foutenafhandeling??
$result = mysql_query( $query );

// Controleren of query gelukt is
if( !$result ) {
echo 'Error blaadiebla';
}

// Row wordt niet gedefinieerd? while loopje?

?>
P
Patrick Helsloot
20-10-2010 15:55
Super bedankt allemaal... Het werkt nu en ben er helemaal gelukkig mee. Haha!
Tobias Tobias
20-10-2010 15:55
wat ik zelf vaak doe is een echo mysql_num_rows($sql), dan weet je in ieder geval of er iets gevonden is. Zo ja, dan gaat er iets fout in de afhandeling (zoals nu het geval is), zo niet, dan zit er iets fout in de query
- Ariën -
20-10-2010 15:58
En SQL injection is ook mogelijk als men de URL aanpast.

Niels K
20-10-2010 16:35
Je bedoeld zijn eerste versie neem ik aan?

Reageren

Inloggen om te reageren