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

search bar geen geen juiste resultaten

Door Frank Thijssen op 21-10-2016 10:12
2.344 views
Hallo allemaal,

ALs beginnent programmeur ben ik op een klein probleempje gesteut omdat mijn zoekfunctie niet goed werkt.
Mijn zoekfunctie bestaat uit twee pagina's één is de zoekpage en twee is de resultpage.

Na testen maakt het niets uit wat ik intype in mijn zoekveld. Krijg de volgende resultaten.
name="q" Geeft alle resutaten uit de database
Sleutelend aan de '%%' kom ik er ook niet uit.

Wat doe ik fout?
Hieronder mijn code

Invulpage:

<form name="reactie_formulier" method="GET" action="html/zoekresultaten.php">
<table class="table">
<tr>
<td><p>Voornaam:</p></td>
<td><input type="text" name="q" placeholder="Zoeken..." class="form-control" /></td>
</tr>
<tr>
<td></td>
<td><input type="submit" value="Zoeken" /></td>
</tr>
</table>
</form>


resultpage

<?php
if($_GET['q'] == 'Zoeken...') {
header('Location: zoeken.php');
}
if (isset($_GET['q']) && $_GET['q'] !== '') {


$con = mysqli_connect("localhost","root","");
mysqli_select_db($con, "manuela")or die(mysqli_error("$connection"));


$search = mysqli_query( $con, "SELECT * FROM klantenbestand WHERE voornaam LIKE '%$q%'");
$num_rows = mysqli_num_rows($search);

while ($row = mysqli_fetch_array($search)){
$voornaam = $row['voornaam'];
$achternaam = $row['achternaam'];
$telefoon = $row['telefoon'];
$opmerking = $row['opmerking'];
$adres = $row['adres'];

echo '<h3>' . $voornaam. ' ' . $achternaam . '</h3><p>' . $adres . ' ' . '<br />' . $telefoon . ' ' . '<br />' . $opmerking . '</p><br />';

}
} else {

echo 'Er zijn geen klanten gevonden.';
}
?>
P
Pg Vincent
21-10-2016 10:17
Begin met lezen over "prepared statements" want je huidige script is bijzonder gevoelig voor SQL injectie en dat is echt gevaarlijk. Als je toch nog aan het leren bent dan zou ik ook meteen overstappen op PDO, dat is een wat meer gestandaardizeerde manier van werken met databases.

Verder doe je nog niets aan foutcontrole, dus zelfs als de query hopeloos faalt, dan kom jij dat nooit te weten. De handleiding van mysqli legt uit hoe je kunt controleren of de query gelukt is, maar, wederom, PDO doet dat net even wat netter.

Had je de query al handmatig uitgeprobeerd via iets als phpmyadmin?
F
Frank Thijssen
21-10-2016 10:40 gewijzigd op 21-10-2016 10:42
Hallo Pg Vincent,

Bedankt voor je reactie. Wellicht dat ik nu de verkeerde manier hanteer, daar moet ik nog een gevoel bij creeren. de SQL infectie wil ik op later moment gaan toevoegen. Hier heb ik nog wat oefening voor nodig.
Bedankt voor deze feedback.

De query $search daarentegen doet het in de MySQL database gewoon.
Aangezien ik voor mijn gevoel de juiste linken aanmaak snap ik niet wat er verkeerd gaat.
Als ik '%$q%' gebruik krijg ik een 'unknown varibale q in ....' warning.
B
Ben van Velzen
21-10-2016 10:46
>> Als ik '%$q%' gebruik krijg ik een 'unknown varibale q in ....' warning.
En dat klopt, want $q komt uit de lucht vallen. Je maakt hem nergens aan.
P
Pg Vincent
21-10-2016 10:59
"de SQL infectie wil ik op later moment gaan toevoegen. Hier heb ik nog wat oefening voor nodig."

Dat moet je niet doen. Begin meteen met de juiste manier van werken, want nu leer je jezelf allerlei dingen aan die je straks weer moet afleren.

bv. met PDO komt jouw query er zo uit te zien; volledig veilig tegen injectie:


$sth = $dbh->prepare('SELECT * FROM klantenbestand WHERE voornaam LIKE :q');
$sth->execute(array(':q' => '%' . $_GET['q'] . '%'));
$red = $sth->fetchAll();


Simpel; overzichtelijk en veilig.
F
Frank Thijssen
21-10-2016 11:00
>> <td><input type="text" name="q" placeholder="Zoeken..." class="form-control" /></td>

In de searchpage schrijf ik de 'q' weg. Deze pak ik op op de resultpage. Maar gaat het dan daar fout?
Ramon van Dongen
21-10-2016 11:18
In de searchpage schrijf ik de 'q' weg. Deze pak ik op op de resultpage. Maar gaat het dan daar fout?
Correct!

Wat je in je formulier in <input name="q"> in vult, komt niet standaard in $q naar in $_POST['q'] of $_GET['q'], afhankelijk van of je method post of get is.
F
Frank Thijssen
21-10-2016 13:08
Oke! die begrijp ik niet helemaal Ramon. nu gebruik ik de GET method met de input name='q'
Ik ga er vanuit dat als ik iets invul deze waarde de 'q' vertegenwoordigd en wordt doorgegeven naar mijn tweede page resultpage.

Ik ben nu even bezig geweest met de tips van boven.

Toegevoegd heb ik de code $sth van Pg Vincent. hieruit kom ik voor de volgende uitdaging, namelijk:
"Notice: Undefined variable: dbh in"
. Deze warning gaf als antwoord dat ik de
$sth = $dbh->prepare('SELECT
moest veranderen in
$this->dbh
. Maar dat hielp ook niet.



B
Ben van Velzen
21-10-2016 13:21
Het eerste punt is eenvoudig: eerder op de pagina controleer je netjes $_GET['q'], die moet je ook in je query gebruiken. $q bestaat niet.
Het tweede punt: wat Vincent gaf was een voorbeeld, welke je zelf moet uitwerken dmv het maken van een databaseverbinding met PDO etc. Een vraag die ik wel heb, is dat je aangeeft dat "Notice: undefined variable: dbh" in zou houden dat je $this->dbh moet gebruiken. Waarom zou dat zo zijn/wie heeft je dat wijsgemaakt?
P
Pg Vincent
21-10-2016 13:27
Wat ik postte was inderdaad alleen een voorbeeldje van hoe het er uit zou zien met PDO, geen copy/paste oplossing.

Voor meer informatie over hoe je met PDO werkt kun je beter een tutorial doornemen. zie b.v. http://www.phptuts.nl/view/27/5/
maar ook de PHP handleiding, waar mijn stukje uit komt, bevat alle informatie die je nodig hebt (zij het in een wat minder hapklare-brokken formaat).

Reageren

Inloggen om te reageren