[php+mysql] Zoeken met hele woorden

[TomPé]

Beste tweakers,

Ik heb voor me zelf het volgende script geschreven:

PHP:

$query = mysql_query("SELECT * FROM films WHERE Titel LIKE '%$search%'");

    echo"
    <table border='0' cellpadding='2' cellspacing='0'>
        <tr class='head'>
            <td width='60'>Code</td>            
            <td width='325'>Titel</td>
            <td width='85'>Genre</td>
            <td width='50'>Jaar</td>
        </tr>
    ";
while ($row = mysql_fetch_array($query))
    {
        $Code           = $row['Code'];
        $Titel          = $row['Titel'];
        $Genre          = $row['Genre'];
        $Jaar           = $row['Jaar'];
                
        echo"
        <tr class='row'>
            <td>". $Code ."</td>
            <td>". $Titel ."</td>
            <td>". $Genre ."</td>
            <td>". $Jaar ."</td>
        </tr>
        ";
    }
    echo"
    </table>
    ";

Als ik nu dus als zoekwoord topgun opgeef dan zou hij alle titels moeten laten zien waar topgun in voor komt. Maar ipv dat te doen laat hij alle titels zien waar een t een o een p enz inzit. Hoe kan ik zorgen dat hij naar het hele woord kijkt en niet naar afzonderlijke letters?

[Upsal]

Hmm, hij zou het gewoon moeten doen (mits $search een waarde bevat), probeer anders ook eens:

PHP:

$query = mysql_query("SELECT * FROM films WHERE Titel LIKE '%". $search ."%'");

[ Voor 9% gewijzigd door Upsal op 08-02-2007 19:41 ]

[CodeCaster]

Debug zelf eens. Print je query eens. Wat staat er in $search?

[PolarBear]

Tip voor de TS, gebruik de code tags, dat leest makkelijker voor de rest! Klik op om te kijken hoe dat voor elkaar krijgt.

PHP:

<?php

$query = mysql_query("SELECT * FROM films WHERE Titel LIKE '%$search%'");

echo"
<table border='0' cellpadding='2' cellspacing='0'>
<tr class='head'>
<td width='60'>Code</td> 
<td width='325'>Titel</td>
<td width='85'>Genre</td>
<td width='50'>Jaar</td>
</tr>
";
while ($row = mysql_fetch_array($query))
{
$Code = $row['Code'];
$Titel = $row['Titel'];
$Genre = $row['Genre'];
$Jaar = $row['Jaar'];

echo"
<tr class='row'>
<td>". $Code ."</td>
<td>". $Titel ."</td>
<td>". $Genre ."</td>
<td>". $Jaar ."</td>
</tr>
";
}
echo"
</table>
";

?>

[ Voor 9% gewijzigd door PolarBear op 08-02-2007 19:19 ]

[TomPé]

Raar inderdaad want zo werkt het beter

Heb ik eigenlijk nog een vraagje
Dit is het gedeelte waar ik de zoek opdracht in kan geven:

PHP:

<form method=POST name='search' action='index.php?page=search'>
<tr>
    <td>
    <input type=text name='search' style='width:125px'>
    </td>
</tr>
<tr>
    <td><input type=submit name='submit_search' value=' Zoek film '></td>
</tr>
<form>

Samen met dit natuurlijk:

PHP:

if(isset($_POST['submit_search']))
{
    $search = $_POST['search'];
}

Maar ik MOET de knop Zoek film gebruiken anders zoekt hij niet goed, ik kan niet in het input veld op enter drukken want dan laat hij gewoon weg alle films zien. Dat is toch ook wel tegen te gaan lijkt me ofniet?

[ Voor 5% gewijzigd door TomPé op 08-02-2007 19:25 ]

[PolarBear]

TomPé schreef op donderdag 08 februari 2007 @ 19:20:
</code>

Moet je natuurlijk wel blokhaken gebruiken

[Megamind]

Je moet een input hidden gebruiken om je form te processen, als je namelijk een je submit knop gebruikt dan wordt deze niet gepost als je op enter drukt

[Voutloos]

TomPé schreef op donderdag 08 februari 2007 @ 19:20:
Samen met dit natuurlijk:

if(isset($_POST['submit_search']))
{
$search = $_POST['search'];
}

En $search komt letterlijk in de query en zo natuurlijk is dat niet. Lees je aub wat in over sql injection.

Verder een tip:
Een LIKE clause beginnend met een wildcard (de % ) wil je niet in een redelijk omvangrijke tabel, als je verder geen goede index hebt om aantal rijen mee te beperken. Verder is ook SELECT * iets dat je wil vermijden, selecteer expliciet de kolommen die je nodig hebt.

Kortom: doe wat aan je sql stijl, want je gaat anders echt heel hard balen van securityleaks en slechte performance.

Overigens ben je ook inconsistent in je naamgeving, $query en $row met kleine letter, maar die 4 vars die je uit de row haalt (welke bij zo'n kleine loop body ook geen extra variabele nodig hebben ) met een hoofdletter.

[TomPé]

Megamind schreef op donderdag 08 februari 2007 @ 19:29:
Je moet een input hidden gebruiken om je form te processen, als je namelijk een je submit knop gebruikt dan wordt deze niet gepost als je op enter drukt

Kan ik niet helemaal volgen. Ik kan mijn input toch niet op hidden zetten want dan kan ik immers geen zoek woorden meer opgeven. En bij de meeste zoek functies ziet altijd wel een knop maar werkt enter ook nog is.

[Upsal]

TomPé schreef op donderdag 08 februari 2007 @ 19:20:
...

PHP:

if(isset($_POST['submit_search']))
{
    $search = $_POST['search'];
}

Maar ik MOET de knop Zoek film gebruiken anders zoekt hij niet goed, ik kan niet in het input veld op enter drukken want dan laat hij gewoon weg alle films zien. Dat is toch ook wel tegen te gaan lijkt me ofniet?

Vervang de eerste regel eens door:

PHP:

if($_SERVER['REQUEST_METHOD'] == 'POST')

[ Voor 1% gewijzigd door Upsal op 08-02-2007 19:39 . Reden: Typo ]

[TomPé]

Dan werkt de gehele zoek functie niet meer

[Depress]

Dit even buiten je zoekfunctie om.

Waarom gebruik je geen fulltext search?
Is wellicht iets sneller en geeft betere resultaten.

[TomPé]

Dat heb ik eerder een keer gehoord, schijnt goed te werken icm php maar daar heb ik helemaal geen kaas van gegeten. Kan me er in de toekomst natuurlijk wel is in gaan verdiepen.

Misschien heb je een suggestie om ergens te beginnen?

[mithras]

TomPé schreef op vrijdag 09 februari 2007 @ 09:10:
Dat heb ik eerder een keer gehoord, schijnt goed te werken icm php maar daar heb ik helemaal geen kaas van gegeten. Kan me er in de toekomst natuurlijk wel is in gaan verdiepen.

Misschien heb je een suggestie om ergens te beginnen?

Wat dacht je van de manual http://dev.mysql.com/doc/refman/5.0/en/fulltext-search.html

Daarnaast lijkt het me handig om iets te lezen over SQL injection

[ Voor 14% gewijzigd door mithras op 09-02-2007 09:17 ]

[NMe]

Megamind schreef op donderdag 08 februari 2007 @ 19:29:
Je moet een input hidden gebruiken om je form te processen, als je namelijk een je submit knop gebruikt dan wordt deze niet gepost als je op enter drukt

Dan liever de oplossing die Upsal in "\[php+mysql] Zoeken met hele woorden" noemt.

Verder maak ik aan dit topic ook even een einde. De vragen die hier gesteld worden zijn namelijk met een beetje moeite ook wel zelf op te lossen; hoe LIKE-query's werken en hoe formuliertjes verwerkt moeten worden zijn vrij basale dingen die in elk boek en een hoop tutorials wel behandeld worden.

TomPé, zie ook even Programming Beleid - De "quickstart".