mysql query gaat fout

[Verwijderd]

heya,

Ik heb een script gemaakt dat links in een database plaats. deze werkt perfect maar in het script dat de links weer op de webpagina moet krijgen gaat iets fout. Ik krijg de volgende error:

Error1064 : 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 's' AND dir ='1' ORDER BY naam' at line 1.

Ik heb al zitten kijken in de query maar ik kan niet zien wat ik fout doe
In de onderstaande code zit het probleem, ziet 1 van jullie misschien zo snel het probleem?

PHP:

$category=$groep["cat"];
// voor elk blok de links selecteren & html code schrijven
                $sql="SELECT * FROM joriso_sknil WHERE cat = '$category' AND dir ='$dir_id' ORDER BY naam ";
                if (!($result =  mysql_query($sql,$connection)))
                showerror();

                $count++;
                $counttotal++;                              

                //categorie kop
                $linkcode.="<div class=\"list1\"><h4>".$category."</h4></div><div class=\"list2\">";
                
                while ($row = mysql_fetch_array($result) )
                {
                $count++;
                $counttotal++;                              
                //link
                $linkcode.="<a target=\"_blank\" href=\"".$row["link"]."\">".$row["naam"]."</a><br>";


                if ($count >= $splitvalue)
                    {$makebreak=1;}

                }
                
                if ($makebreak==1)
                { 
                // einde?
                    if ($counttotal == $totalrows)
                        {$linkcode.= "</div><br>";}
                //nog een kolom     
                    else { 
                    $linkcode.= "</div></div><div class=\"linkcontainer\">";
                    $makebreak=0;
                    $count=0;
                    $breaksmade++;}
                }
                
                else
                {
                //einde categorie
                $linkcode.= "</div><br>";
                }

[ Voor 0% gewijzigd door moto-moi op 13-10-2010 12:12 . Reden: ff onze mooie php parser aangezet voor je stuk code ;) ]

[Janoz]

Druk je query eens volledig af en kijk eens waarom mysql vindt dat je velden ergens anders ophouden dan de bedoeling is. Ga daarna even googlen op SQL injections.

[ThinkPad]

Voer de query eens uit in PHPmyAdmin, dan is het waarschijnlijk iets duidelijker waarom het niet werkt.

Moet je voor 2 variabelen wel zelf even wat invullen.

en zoals hierboven wordt gezegd is je query echo'en ook wel handig

[ Voor 31% gewijzigd door ThinkPad op 13-10-2010 12:17 ]

[Crayne]

Is $dir_id wel een string?

[Verwijderd]

@crayne: Als het goed is wel. Dit is de complete code. als het ongewenst is verwijder ik hem wel weer?

code:

<?php

// met deze 2 variabelen kun je het aantal kolommen en de kolombreedte wijzigen:

$aantalkolommen=3;
$kolombreedte="180px";

//db connectie
require('admin/connectie.php');

// gekozen link-directory uit url halen 
if ($_GET['dir'])
{$dirnaam=$_GET['dir'];}

// alle link-directories selecteren en het menu bouwen
$q="SELECT d.did, d.naam, d.titel, s.uid FROM joriso_linkdirs d LEFT JOIN joriso_sknil s ON d.did = s.dir GROUP BY d.naam";

if (!($tmp =  mysql_query($q,$connection)))
showerror();

while ($xmenu = mysql_fetch_array($tmp))
{
//default directory - indien geen linkdirectory is aangegeven in de url -  de eerste met links kiezen
if (empty($dirnaam))
    {
    if ($xmenu['uid'] != NULL)
        {
        $dir_id=$xmenu['did'];
        $dirnaam=$xmenu['naam'];
        $kop = $xmenu['titel'];
        }
    }

// Linkdirectory menu-opbouw
if ($xmenu['naam'] == "$dirnaam")
    { 
    $dir_id = $xmenu['did'];
    $kop = $xmenu['titel'];
$xmenuitems.="<span class=\"menuactief\"><b>".$xmenu['naam']."</b></span>";
    }
else{
$xmenuitems.="<span class=\"menunonactief\"><a href=\"links.php?dir=".$xmenu['naam']."\"><b>".$xmenu['naam']."</b></a></span>";
    }
}
$xmenuitems ="<div align=\"right\" >".$xmenuitems."</div>";

//start de opbouw van de linkpagina

$linkcode="<h1>".$kop."</h1><div class=\"linktank\"><div class=\"linkcontainer\">";

//  link aantallen en afbreekwaarden bepalen
$breaksmade=0;
$makebreak=0;
$count=0;
$counttotal=0;

$sqllll="SELECT COUNT(uid) as total FROM joriso_sknil WHERE dir ='$dir_id'";
if (!($temp_2 =  mysql_query($sqllll,$connection)))
showerror();
$aantallen = mysql_fetch_array($temp_2);

if ($aantallen['total'] == 0)
{echo "<strong>Directory is leeg</strong>.";
exit;
}


// categorien in de geselecteerde linkdir vinden
$sqll="SELECT DISTINCT cat FROM joriso_sknil WHERE dir ='$dir_id' GROUP BY cat ";
    if (!($temp =  mysql_query($sqll,$connection)))
        showerror();


$aantalcats=mysql_num_rows($temp);

//kolommen

$totalrows= $aantallen['total'] + $aantalcats;

$splitvalue= ceil($totalrows/$aantalkolommen);


// voor elke categorie een apart blok bouwen
    while ($groep = mysql_fetch_array($temp))
            {
            
                $category=$groep["cat"];
// voor elk blok de links selecteren & html code schrijven
                 $sql="SELECT * FROM joriso_sknil WHERE cat = '$category' AND dir ='$dir_id' ORDER BY naam ";
                if (!($result =  mysql_query($sql,$connection)))
                showerror();

                $count++;
                $counttotal++;                              

                //categorie kop
                $linkcode.="<div class=\"list1\"><h4>".$category."</h4></div><div class=\"list2\">";
                
                while ($row = mysql_fetch_array($result) )
                {
                $count++;
                $counttotal++;                              
                //link
                $linkcode.="<a target=\"_blank\" href=\"".$row["link"]."\">".$row["naam"]."</a><br>";


                if ($count >= $splitvalue)
                    {$makebreak=1;}

                }
                
                if ($makebreak==1)
                { 
                // einde?
                    if ($counttotal == $totalrows)
                        {$linkcode.= "</div><br>";}
                //nog een kolom     
                    else { 
                    $linkcode.= "</div></div><div class=\"linkcontainer\">";
                    $makebreak=0;
                    $count=0;
                    $breaksmade++;}
                }
                
                else
                {
                //einde categorie
                $linkcode.= "</div><br>";
                }


    }
    
    //niet genoeg kolommen maar wel genoeg  categorien correctie:
    if ($breaksmade < $aantalkolommen && $aantalcats >=$aantalkolommen)
    {   function str_replace_count($search,$replace,$subject,$times) {
        
           $subject_original=$subject;
          
           $len=strlen($search);   
           $pos=0;
           for ($i=1;$i<=$times;$i++) {
               $pos=strpos($subject,$search,$pos);
              
               if($pos!==false) {               
                   $subject=substr($subject_original,0,$pos);
                   $subject.=$replace;
                   $subject.=substr($subject_original,$pos+$len);
                   $subject_original=$subject;
               } else {
                   break;
               }
           }
          
           return($subject);
        
        }
    
    $extrabreaks =$aantalkolommen-$breaksmade;

    $linkcode = str_replace_count("</div><br>",  "</div></div><div class=\"linkcontainer\">",  $linkcode, ($extrabreaks-1));

}
// hieronder volgt de html en css code  voor de link-pagina. Aanpassen naar eigen voorkeur. 

?>

[Brummetje]

Crayne schreef op woensdag 13 oktober 2010 @ 12:24:
Is $dir_id wel een string?

Kunt prima een int in een string doen hoor

Zoals hierboven is gezegd kun je hem beter eerst even testen in PHPMyAdmin op te kijken of het daar wel werkt.

[ajakkes]

In plaats van:
if (!($result = mysql_query($sql,$connection)))
showerror();

print "Dit moet in phpmyadmin: -> ".$sql." <- Dit niet meer";

En dan het resultaat op je pagina tussen >< bekijken en in phpmyadmin testen.

[ Voor 9% gewijzigd door ajakkes op 13-10-2010 12:39 ]

[CodeCaster]

Verwijderd schreef op woensdag 13 oktober 2010 @ 12:11:
Error1064 : 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 's' AND dir ='1' ORDER BY naam' at line 1.

PHP:

"WHERE cat = '$category' AND dir ='$dir_id' ORDER BY naam "

Heb je soms een categorie auto's? Of programma's?

Ga nou eens escapen, da's PHP- en SQL-tutorial 101.

[NMe]

CodeCaster schreef op vrijdag 15 oktober 2010 @ 10:31:
[...]

Heb je soms een categorie auto's? Of programma's?

Ga nou eens escapen, da's PHP- en SQL-tutorial 101.

Inderdaad, hier is meer dan genoeg over te vinden op Google en GoT. Kom je daar met je eigen zoekwoorden niet uit, dan kun je deze proberen: sql injection.

Overigens werd je gevraagd om je query af te drukken, niet om dan maar je hele code te posten. Dat laatste is eigenlijk nooit echt nuttig of gewenst hier.

Aangezien dit probleem al vaak genoeg voorbij is gekomen en je er dus met de search ook wel uit komt gaat dit topic dicht.