SQL DELETE door php

[gentillekes]

Ik heb met php/sql/html een website opgezet waar het voor mensen mogelijk is om een kleine CV te plaatsen welke een linkje bevat naar hun LinkedIN profiel. Alles werkt tot nu toe, ik heb zelfs een zoekfunctie toe weten te voegen, alle php kennis die ik tot nu toe heb, heb ik kunnen gebruiken om deze website in elkaar te zetten.
Het leren van PHP heb ik gedaan met behulp van het boek PHP en MYSQL voor dummies

Op deze website kan ik als beheerder inloggen, om zo uiteindelijk ook records te kunnen verwijderen uit de database.

Het enige struikelblok dat er nog heerst, is de verwijder mogelijkheid welke nog niet vlekkeloos werkt.

Ik heb een database met daarin één tabel met kandidaatgegevens (id, naam, email en linkedin-hyperlink ed.) deze informatie wordt mbv php weergegeven in een tabel:

code:

<?php 

$query      =    mysql_query("
SELECT * 
FROM kandidaat
ORDER BY regio, opleiding
");



echo "
<table border='1' bgcolor='fffffff'><tr>
<td><b>id</b></td>
<td><b>achternaam</b></td>
<td><b>voornaam</b></td>
<td><b>geslacht</b></td>
<td><b>regio</b></td>
<td><b>opleiding</b></td>
<td><b>vakgebied</b></td>
<td><b>uurloon</b></td>
<td><b>dienstverband</b></td>
<td><b>linkedin</b></td>
<td><b>email</b></td>
<td><b>datum inschrijving</b></td>
<td><b>#</b></td>
</tr>
";

while($result1 = mysql_fetch_assoc($query)){
    echo "<tr>";
    echo "<td>" .$result1['id']. "</td>";
    echo "<td>" .$result1['achternaam']. "</td>";
    echo "<td>" .$result1['voornaam']. "</td>";
    echo "<td>" .$result1['geslacht']. "</td>";
    echo "<td>" .$result1['regio']. "</td>";
    echo "<td>" .$result1['opleiding']. "</td>";
    echo "<td>" .$result1['vakgebied']. "</td>";
    echo "<td>" .$result1['uurloon']. "</td>";
    echo "<td>" .$result1['dienstverband']. "</td>";
    echo "<td><a href=".$result1['linkedin']." >Klik hier</a></td>";
    echo "<td>" .$result1['email']. "</td>";
    echo "<td>" .$result1['datum']. "</td>";
    echo "<td><input type=checkbox name=checkbox[] id=checkbox[] value=".$result1['id']."></td>";
    echo "</tr>";
}
echo "</table>";
echo "</br>";

?>

Achter elke rij gegevens heb ik tevens een checkbox toegevoegd, met als doelstelling dat ik deze uiteindelijk kan aanvinken en met meerdere tegelijk kan verwijderen.
Het script om ze te verwijderen ziet er als volgt uit:

code:

<input name="delete" type="submit" id="delete" value="verwijder">
<?

if($delete){
for($i=0;$i<$count;$i++){
$del_id = $checkbox[$i];
$sql = "DELETE FROM kandidaat WHERE id='$del_id'";

}

}
mysql_close();
?>

Kunnen jullie zien wat ik precies fout doe? voor jullie beeldvorming, wat het script zou moeten doen:

1. laat kandidaten zien (werkt prima)
2. vink checkboxes aan (werkt ook)
3. delete (met een knopje, om zo meerdere records tegelijk te kunnen verwijderen).

mijn website kunnen jullie bewonderen op http://test.rogermooren.nl
logindetails nodig? *Regel dat zelf maar, zet maar gewoon open neer volgende keer*

edit:

code:

<?php

ini_set ('display_errors', 1);

error_reporting(E_ALL);

?>
<?php
include("conf/dbconn.php");
include("header.php");
include("navigatie_beheer.php");
?>

Dit staat nog bovenaan mijn scriptje, oa, een include naar de gegevens van de database (namen, logingegevens etc.)

nog een edit: omdat ik error reporting aan heb staan, krijg ik de volgende melding:
Notice: Undefined variable: delete in /customers/rogermooren.nl/rogermooren.nl/httpd.www/test/beheer_membersdetail.php on line 80

[ Voor 8% gewijzigd door MueR op 07-01-2011 15:34 ]

[Spider.007]

Het kan aan mij liggen, maar moet je $sql ook niet nog naar je database sturen? En hopelijk werken $delete en $checkbox ook niet, zie http://php.net/manual/en/security.globals.php

[ Voor 13% gewijzigd door Spider.007 op 07-01-2011 14:58 ]

[SinergyX]

Waar wordt $count gevuld? Je gebruikt hem wel in je loop, maar ik zie nergens in script $count ook een waarde krijgen. (zolang 1<0 -> hij doet geen loop). Krijgt de checkbox wel een unieke name per regel? checkbox[] doet volgens mij niets.

Sterk punt ook van spider Moet dit niet met de POST values worden gedaan?
* SinergyX PHP skills zijn wat roestig.

[ Voor 41% gewijzigd door SinergyX op 07-01-2011 15:00 ]

[E-Vix]

Buiten de bovengenoemde punten, is het niet verstandiger om in plaats van een record te verwijderen, een extra kolom aan je records toe te voegen met daarin of een profiel wel of niet actief is? Zo kan je bijvoorbeeld makkelijk per ongeluk verwijderde profielen weer terug halen.

[gentillekes]

Hij doet inderdaad niets nee pak ik dit niet te complex aan? moet dit niet veel simpeler kunnen?

[Zoijar]

Even terzijde, check je wel je inputs goed. Volgens mij staat het open voor een sql injection. Even een handgemaakt form sturen met checkbox value van "0' OR id > '0" (hele tabel weg)

[ Voor 4% gewijzigd door Zoijar op 07-01-2011 15:08 ]

[gentillekes]

E-Vix schreef op vrijdag 07 januari 2011 @ 15:05:
Buiten de bovengenoemde punten, is het niet verstandiger om in plaats van een record te verwijderen, een extra kolom aan je records toe te voegen met daarin of een profiel wel of niet actief is? Zo kan je bijvoorbeeld makkelijk per ongeluk verwijderde profielen weer terug halen.

goed punt! daar had ik nog niet aan gedacht. het is voor mij echter wel de vraag of ik weet hoe ik dit moet toepassen, laat ik eerst eens kijken naar DELETE, op het moment dat dat lukt kijk ik graag verder naar non-actief functies.

[Meijuh]

Buiten de veiligheid om:
Als je $count veranderd door count($checkbox) in de for-loop, werkt het dan beter?

[NMe]

Ik heb deze al jaren niet meer hoeven linken maar volgens mij is 'ie hier weer nodig.
Programming FAQ - PHP: Mijn POST en GET forms werken niet (meer)

[gentillekes]

code:

if($delete){
for($i=0;$i<count($checkbox);$i++){
$del_id = $checkbox[$i];
$sql = "DELETE FROM kandidaat WHERE id='$del_id'";

zoiets?

veiligheid is idd nog een item waar ik nog even naar moet kijken. Dit doe ik na afloop van het hele verhaal. Dit omdat ik eerst moet kijken/leren hoe en óf het allemaal mogelijk is.

[Brons]

Achteraf naar de veiligheid kijken lijkt mij niet verstandig, je loopt dan het risico dat je zaken over het hoofd ziet. Ook leer je jezelf dan slechte gewoontes aan. SQL injection is eenvoudig te voorkomen maar ook erg gevaarlijk.

[Meijuh]

gentillekes schreef op vrijdag 07 januari 2011 @ 15:13:

code:

if($delete){
for($i=0;$i<count($checkbox);$i++){
$del_id = $checkbox[$i];
$sql = "DELETE FROM kandidaat WHERE id='$del_id'";

zoiets?

veiligheid is idd nog een item waar ik nog even naar moet kijken. Dit doe ik na afloop van het hele verhaal. Dit omdat ik eerst moet kijken/leren hoe en óf het allemaal mogelijk is.

Ja, en dan nog mysql_query($sql) or die(mysql_error()) er aan toevoegen.

[NMe]

Meijuh schreef op vrijdag 07 januari 2011 @ 15:17:
[...]

Ja, en dan nog mysql_query($sql) or die(mysql_error()) er aan toevoegen.

Ja, want het is in productie heel fijn om dat soort gegevens over je queries aan je gebruikers te geven...

[Meijuh]

NMe schreef op vrijdag 07 januari 2011 @ 15:26:
[...]

Ja, want het is in productie heel fijn om dat soort gegevens over je queries aan je gebruikers te geven...

Het leren van PHP heb ik gedaan met behulp van het boek PHP en MYSQL voor dummies

Hij begint net aan PHP, ik denk dat we hem genoeg gewaarschuwd hebben om dit niet in een productieomgeving te zetten. Maar iedereen moet toch op een manier php/mysql leren? Ik denk dat dit een goed begin is.

[MueR]

De helpdesk is gesloten. Als je een testcase online zet, willen we die op een publieke locatie hebben. We gaan niet wachtwoorden voor het testen bij je ophalen. Regel dat de volgende keer.

Dit topic is echt te basic. Ik mis elke vorm van debuggen. Dit is niet de eerste keer dat je hierop gewezen bent. Het wordt een beetje flauw om je te moeten blijven wijzen naar onze Het algemeen beleid #quickstart. Zorg nou eens eerst dat je zelf weet wat er met je data gedaan wordt, voordat je hier komt klagen.

[NMe]

Even een toevoeging na het slotje:

Meijuh schreef op vrijdag 07 januari 2011 @ 15:36:
[...]

[...]

Hij begint net aan PHP, ik denk dat we hem genoeg gewaarschuwd hebben om dit niet in een productieomgeving te zetten. Maar iedereen moet toch op een manier php/mysql leren? Ik denk dat dit een goed begin is.

Best hoor. Maar dan wel met de nuance dat je dat alleen tijdens het testen laat staan. Het werd hier gebracht als de beste permanente oplossing en dat is het pertinent niet.