Wat is er fout aan deze query?

PHP Code:

$sql = "SELECT * FROM yourls_url WHERE ip ='$ip' ORDER BY timestamp DESC  limit '.($page*$eachPage).','.$eachPage.'"; 


Je LIMIT doe je niet goed (de enkele aanhalingstekens moeten dubbele worden). Mijn tip: ga werken in een programma die je code highlight, dan zie je de fout meteen.

Zo dus?

PHP Code:

$sql = 'SELECT * FROM yourls_url WHERE ip =\'$ip\' ORDER BY timestamp DESC  limit ".($page*$eachPage).",".$eachPage."'; 


Aanhalingstekentechnisch wel ja.

Zo:

PHP Code:

$sql = "SELECT * FROM yourls_url WHERE ip ='".$ip."' ORDER BY timestamp DESC  limit ".($page*$eachPage).",".$eachPage; 


Bedankt Jopie! Het script werkt nu helemaal! Bedankt!
Ik ga even uitvogelen wat ik nu fout deed.

Je escaped single quotes met dubbele en daarna escape je dubbele quotes met enkele. Terwijl je dubbele met dubbele escaped en enkel met enkel:

Stel je dit voor:

PHP Code:

$goed = "prima"; $fout = "verkeerd"; 


Met de volgende code:

PHP Code:

echo "Dit is ".$goed." gedaan"; 


PHP Code:

echo "Dit is '.$fout.' gedaan"; 


PHP Code:

echo 'Dit is '.$goed.' gedaan'; 


PHP Code:

echo 'Dit is ".$fout." gedaan'; 


1 geeft als output: Dit is prima gedaan
2 geeft als output: Dit is '.$fout.' gedaan (en jij verwacht 'Dit is verkeerd gedaan')
3 geeft als output: Dit is prima gedaan
4 geeft als output: Dit is ".$fout." gedaan (en jij verwacht 'Dit is verkeerd gedaan')

Hoi Joshua de Gier,
Bedankt voor je uitleg! Is zeker erg nuttig! Moet bekennen dat ik dit nog niet wist :)
Maar hem het overgeschreven dus ga dit onthouden.Thanks!

Wat belangrijk is om te onthouden is dat je het type aanhalingsteken gebruikt waar je ook de echo mee opent/sluit. Wat is dan het verschil? Tussen dubbele aanhalingstekens, ", worden variabelen gelezen, tussen enkele, ', niet. Dus goed is ook:

PHP Code:

$naam = 'Peter';
echo "Ik heet $naam en ik ben een man."; 


Het is gemakkelijk om bij het typen van HTML-code enkele quotes te gebruiken zodat je de dubbele quotes in de HTML zonder te escapen kunt gebruiken:

PHP Code:

//dit probleem hen je niet meer...
echo "<div class=\"header\">";

//...als je het zo doet
echo '<div class="header">'; 


PHP Code:

$naam='Peter';
echo"Ikheet$naamenikbeneenman."; 


Het werkt maar het is niet bepaald netjes.
Bovendien wanneer de programmeurs van php dit gedrag uit toekomstige php versies haalt dan kun je alle je code aanpassen dus doe het gewoon direct goed dus de variable uit de qoutes (lees uit de string) te halen

Verder lees ik nergens over de sql injection die de topicstarter in zijn query heeft

Origineel gepost door Raymond Nijland

Verder lees ik nergens over de sql injection die de topicstarter in zijn query heeft

Het zijn allemaal variabelen, jij weet toch niet wat ts voor de query met die variabelen allemaal gedaan heeft?

Las vanmorgen met slaperig ogen in trein id in plaats van ip