DIG Digital Corner
Alles wat je altijd al over computers, hardware, software, internet en elektronische gadgets had willen weten, maar niet op Tweakers.net durft te vragen.
hallo fokkers,
ik kom er niet uit met mijn opdracht voor PHP. Ik heb de delete query ergens van gekopieerd en probeer het te laten werken alleen mijn site geeft dus een error. Zelf denk ik dat ik een fout heb in de tabel, alleen krijg het niet opgelost. Geeft trouwens error aan in die regel dat er , of ; mist.
hier de code... alvast bedankt:
[ Bericht 71% gewijzigd door Fred op 12-04-2017 08:55:01 (PHP-tags toegevoegd) ]
ik kom er niet uit met mijn opdracht voor PHP. Ik heb de delete query ergens van gekopieerd en probeer het te laten werken alleen mijn site geeft dus een error. Zelf denk ik dat ik een fout heb in de tabel, alleen krijg het niet opgelost. Geeft trouwens error aan in die regel dat er , of ; mist.
hier de code... alvast bedankt:
| 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 | <?php $servername = "localhost"; $username = "root"; $password = ""; $dbname = "ArkEvolved"; // Create connection $conn = new mysqli($servername, $username, $password, $dbname); // Check connection if ($conn->connect_error) { die("Connection failed: " . $conn->connect_error); } if(isset($_POST['delete'])){ $deleteQuery="DELETE from arkevolved where id='$_post[hidden]'"; mysql_query($deleteQuery,$con); } //select data from database $sql = "SELECT id, name, map FROM arkevolved"; $result = mysqli_query($conn, $sql); if ($result->num_rows > 0) { echo "<table><tr><th>ID</th><th>Name</th><th>Diet</th></tr>"; while($row = $result->fetch_assoc()) { echo "<tr><td>".$row["id"]."</td><td>".$row["name"]." </td> <td>".$row["map"]."</td><td>".$row "<input type=submit name=delete value=delete" . "'</td></tr>"; } echo "</table>"; } else { echo "0 results"; } $conn->close(); } ?> |
[ Bericht 71% gewijzigd door Fred op 12-04-2017 08:55:01 (PHP-tags toegevoegd) ]
Ik krijg trouwens de error op elke pagina. op de inlog pagina, home etc.
het connecten met de database werkte in ieder geval, als ik weg haal wat ik erin heb gezet doet hij het weer zoals het was... alleen dan natuurlijk zonder delete knop.
het connecten met de database werkte in ieder geval, als ik weg haal wat ik erin heb gezet doet hij het weer zoals het was... alleen dan natuurlijk zonder delete knop.
De error geeft toch gewoon het regelnummer aan? Dan weet je waar het probleem zit.
Smile like you mean it
www.wefut.com
www.wefut.com
Koop aub een ander php boek of zoek wat recente tutorials, deze code lijkt wel uit 2002 te stammen. En verder gewoon op de regel kijken waar volgens de foutmelding de fout zit.
Op donderdag 2 juli 2009 22:41 schreef RTB het volgende:
als ik elk rap"liedje" een kans moest geven was ik aan het eind van dit millennium nog bezig met het tempo waarin die kotshoop uitgebraakt wordt.
👾
als ik elk rap"liedje" een kans moest geven was ik aan het eind van dit millennium nog bezig met het tempo waarin die kotshoop uitgebraakt wordt.
👾
Plaats eens een exit() om je querystring om te zien hoe die er precies uitziet bij het uitvoeren?
Ik vermoed dat je $_post moet vervangen door $_POST.
Maar ik zou me eerst eens verdiepen in beveiliging van PHP-applicaties voor je dit soort scripts in het wild loslaat. Hoe je bijvoorbeeld MySQL-injectie kunt voorkomen.
Prima om in het begin wat te copy-pasten, maar om nieuwe talen echt te leren en verantwoord te kunnen toepassen daar komt wel wat meer bij kijken.
Ik vermoed dat je $_post moet vervangen door $_POST.
Maar ik zou me eerst eens verdiepen in beveiliging van PHP-applicaties voor je dit soort scripts in het wild loslaat. Hoe je bijvoorbeeld MySQL-injectie kunt voorkomen.
Prima om in het begin wat te copy-pasten, maar om nieuwe talen echt te leren en verantwoord te kunnen toepassen daar komt wel wat meer bij kijken.
1) we hebben op fok een php tag: 
gebruik het..
2) ik ben er een hele tijd uit, maar ik zou zeggen dat je bij je delete query wat punten mist
gebruik het..
2) ik ben er een hele tijd uit, maar ik zou zeggen dat je bij je delete query wat punten mist
Er is al genoeg gezegd over de belabberde code maar pas de delete regel eens aan naar onderstaande..
| 1 | $deleteQuery="DELETE from arkevolved where id=".$_POST["hidden"]; |
Maak er dan op zijn minst (int)$_POST van...quote:Op woensdag 12 april 2017 01:37 schreef Ralphmeister het volgende:
Er is al genoeg gezegd over de belabberde code maar pas de delete regel eens aan naar onderstaande..
[ code verwijderd ]
Je moet gewoon een moderne db lib gebruiken, met ondersteuning voor sql placeholders. Niet zelf het wiel opnieuw gaan uitvinden. Want wat doe je dan als je een string als sql argument wil gebruiken? De meeste oplossingen die daarvoor zelf verzonnen worden zijn zo lek als een mandje. Doe dat niet.quote:Op woensdag 12 april 2017 06:49 schreef Barrah het volgende:
[..]
Maak er dan op zijn minst (int)$_POST van...
Zo bijv met PDO:
| 1 2 3 4 5 6 7 8 9 10 11 12 | <?php try { $dbh = new PDO($dsn, $user, $password); } catch (PDOException $e) { die( 'Connection failed: ' . $e->getMessage() ); } $sql = "DELETE from arkevolved where id = :id"; $sth = $dbh->prepare($sql); $sth->execute( array( ':id' => $_POST['hidden'] ) ); // etc. ?> |
Dit is inderdaad het echte antwoord. Maar gezien het beginnersniveau van TS zou ik niet teveel verwachten.quote:
[..]
Je moet gewoon een moderne db lib gebruiken, met ondersteuning voor sql placeholders. Niet zelf het wiel opnieuw gaan uitvinden. Want wat doe je dan als je een string als sql argument wil gebruiken? De meeste oplossingen die daarvoor zelf verzonnen worden zijn zo lek als een mandje. Doe dat niet.
Zo bijv met PDO:
[ code verwijderd ]
Nog beter is om ook gelijk een moderne taal te gebruiken en met een HELE grote boog om PHP heen te lopen.quote:
[..]
Je moet gewoon een moderne db lib gebruiken, met ondersteuning voor sql placeholders
It is impossible to live a pleasant life without living wisely and well and justly.
And it is impossible to live wisely and well and justly without living a pleasant life.
And it is impossible to live wisely and well and justly without living a pleasant life.
ben wel benieuwd welke moderne serverside taal je dan bedoeld?quote:
[..]
Nog beter is om ook gelijk een moderne taal te gebruiken en met een HELE grote boog om PHP heen te lopen.
Er zijn diverse mogelijkheden afhankelijk van wat je precies wilt doen.quote:
[..]
ben wel benieuwd welke moderne serverside taal je dan bedoeld?
Zo kan je bijvoorbeeld server-side Swift gebruiken. Een andere leuke optie is Scala met Akka. Of je kiest Go, etc. etc.
It is impossible to live a pleasant life without living wisely and well and justly.
And it is impossible to live wisely and well and justly without living a pleasant life.
And it is impossible to live wisely and well and justly without living a pleasant life.
Perl is als taal haast nog erger als PHPquote:
Op
It is impossible to live a pleasant life without living wisely and well and justly.
And it is impossible to live wisely and well and justly without living a pleasant life.
And it is impossible to live wisely and well and justly without living a pleasant life.
Dat is een misvatting, veroorzaakt door onwetendheid. Perl is pure kunst. Poezie van de hoogste orde!quote:
[..]
Perl is als taal haast nog erger als PHP
Hallo Promille,quote:
Ik zal even in m'n glazenbol kijken wat je bedoelt met "een error". Of kun je misschien ons helpen met wat voor error je krijgt?quote:ik kom er niet uit met mijn opdracht voor PHP. Ik heb de delete query ergens van gekopieerd en probeer het te laten werken alleen mijn site geeft dus een error.
Welke tabel? Hoe ziet die tabel eruit? Als jij denkt dat het aan die tabel ligt, dan lijkt het me vrij relevant om die gegevens ook met ons te delen. Hoe moeten wij anders helpen?quote:Zelf denk ik dat ik een fout heb in de tabel, alleen krijg het niet opgelost.
Welke regel? Welke error?quote:Geeft trouwens error aan in die regel dat er , of ; mist.
Oke, ik zal de sql-injection en het totale gebrek aan controle of degene die je pagina aanroept daadwerkelijk de delete zou mogen uitvoeren even negeren.quote:hier de code... alvast bedankt:
[ code verwijderd ]
Bedankt voor de code, maar zoals je zelf al aangeeft gaat er misschien iets mis met hoe je tabel is opgezet (geef die dan ook even!
) en krijg je schijnbaar een error (geef die dan ook even! )Succes met je opdracht...
Perl is een write-only taal. Je kan er wel dingen in schrijven maar probeer achteraf maar eens te achterhalen wat een Perl script doet.quote:
[..]
Dat is een misvatting, veroorzaakt door onwetendheid. Perl is pure kunst. Poezie van de hoogste orde!
Als je de shift knop ingedrukt houdt en je ramt 5 minuten lang op de bovenste rij van je toetsenbord is de kans groot dat het een geldig Perl script is.
Enige wat erger is is het config formaat van Sendmail.
It is impossible to live a pleasant life without living wisely and well and justly.
And it is impossible to live wisely and well and justly without living a pleasant life.
And it is impossible to live wisely and well and justly without living a pleasant life.
Ik vermoed zomaar dat je $_post[hidden] tussen { } moet zetten.
Verder helpt het inderdaad enorm om meer informatie te geven, we hebben geen glazen bol. Over het algemeen zijn programmeurs erg behulpzaam naar elkaar toe, maar alleen als "we" merken dat de ander er zijn best voor wil doen. Gevalletjes "hier is mijn code, fix het!" hebben we geen zin in.
Verder helpt het inderdaad enorm om meer informatie te geven, we hebben geen glazen bol. Over het algemeen zijn programmeurs erg behulpzaam naar elkaar toe, maar alleen als "we" merken dat de ander er zijn best voor wil doen. Gevalletjes "hier is mijn code, fix het!" hebben we geen zin in.
Ow kijk, gelukkig is dit ook het enige topic wat TS op Fok! heeft aangemaakt en is hij sindsdien niet meer online geweest...
Ik wilde het beroemde 99 bottles of beer script posten maar die compileert niet meer in moderne versies van perl.quote:
[..]
Perl is een write-only taal. Je kan er wel dingen in schrijven maar probeer achteraf maar eens te achterhalen wat een Perl script doet.
Als je de shift knop ingedrukt houdt en je ramt 5 minuten lang op de bovenste rij van je toetsenbord is de kans groot dat het een geldig Perl script is.
Enige wat erger is is het config formaat van Sendmail.
|
|
