abonnementen ibood.com bol.com Gearbest
pi_170159087
registreer om deze reclame te verbergen
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:
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 FredvZ op 12-04-2017 08:55:01 (PHP-tags toegevoegd) ]
pi_170159185
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.
pi_170159210
exploits_of_a_mom.png
  dinsdag 11 april 2017 @ 23:34:39 #4
363995 Reemi
Zeg maar Remi.
pi_170159280
registreer om deze reclame te verbergen
De error geeft toch gewoon het regelnummer aan? Dan weet je waar het probleem zit.
Smile like you mean it
www.wefut.com
  dinsdag 11 april 2017 @ 23:40:52 #5
91039 mstx
2x1/2 = 1/2 x 1/2
pi_170159347
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.
👾
  dinsdag 11 april 2017 @ 23:43:57 #6
422610 Barrah
Komt niet uit Zevenbergen
pi_170159394
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.
pi_170159584
registreer om deze reclame te verbergen
1) we hebben op fok een php tag: php.png
gebruik het..

2) ik ben er een hele tijd uit, maar ik zou zeggen dat je bij je delete query wat punten mist
pi_170160199
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"];
pi_170160485
Waar is de error?
  woensdag 12 april 2017 @ 06:49:03 #10
422610 Barrah
Komt niet uit Zevenbergen
pi_170160711
quote:
0s.gif 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 ]

Maak er dan op zijn minst (int)$_POST van...
pi_170162532
quote:
1s.gif Op woensdag 12 april 2017 06:49 schreef Barrah het volgende:

[..]

Maak er dan op zijn minst (int)$_POST van...
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:
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.
?>
  woensdag 12 april 2017 @ 11:06:16 #12
422610 Barrah
Komt niet uit Zevenbergen
pi_170163246
quote:
0s.gif Op woensdag 12 april 2017 10:14 schreef Farenji het volgende:

[..]

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 ]

Dit is inderdaad het echte antwoord. Maar gezien het beginnersniveau van TS zou ik niet teveel verwachten. ;)
  woensdag 12 april 2017 @ 11:48:18 #13
2671 Aaargh!
Gebruik op eigen risico.
pi_170163930
quote:
0s.gif Op woensdag 12 april 2017 10:14 schreef Farenji het volgende:

[..]

Je moet gewoon een moderne db lib gebruiken, met ondersteuning voor sql placeholders
Nog beter is om ook gelijk een moderne taal te gebruiken en met een HELE grote boog om PHP heen te lopen.
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.
pi_170165552
quote:
0s.gif Op woensdag 12 april 2017 11:48 schreef Aaargh! het volgende:

[..]

Nog beter is om ook gelijk een moderne taal te gebruiken en met een HELE grote boog om PHP heen te lopen.
ben wel benieuwd welke moderne serverside taal je dan bedoeld?
  woensdag 12 april 2017 @ 13:42:00 #15
2671 Aaargh!
Gebruik op eigen risico.
pi_170165660
quote:
0s.gif Op woensdag 12 april 2017 13:34 schreef mschol het volgende:

[..]

ben wel benieuwd welke moderne serverside taal je dan bedoeld?
Er zijn diverse mogelijkheden afhankelijk van wat je precies wilt doen.

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.
pi_170165858
quote:
0s.gif Op woensdag 12 april 2017 13:42 schreef Aaargh! het volgende:

[..]

Er zijn diverse mogelijkheden afhankelijk van wat je precies wilt doen.

Zo kan je bijvoorbeeld server-side Swift gebruiken. Een andere leuke optie is Scala met Akka. Of je kiest Go, etc. etc.
Of Perl in combinatie met PSGI!
  woensdag 12 april 2017 @ 14:51:05 #17
2671 Aaargh!
Gebruik op eigen risico.
pi_170166871
quote:
7s.gif Op woensdag 12 april 2017 13:55 schreef Farenji het volgende:

[..]

Of Perl in combinatie met PSGI!
Perl is als taal haast nog erger als PHP
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.
pi_170167142
quote:
0s.gif Op woensdag 12 april 2017 14:51 schreef Aaargh! het volgende:

[..]

Perl is als taal haast nog erger als PHP
Dat is een misvatting, veroorzaakt door onwetendheid. Perl is pure kunst. Poezie van de hoogste orde! _O_
pi_170167145
quote:
1s.gif Op dinsdag 11 april 2017 23:17 schreef Promille het volgende:
hallo fokkers,
Hallo Promille,

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.
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:
Zelf denk ik dat ik een fout heb in de tabel, alleen krijg het niet opgelost.
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:
Geeft trouwens error aan in die regel dat er , of ; mist.
Welke regel? Welke error?

quote:
hier de code... alvast bedankt:
[ code verwijderd ]
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. :o

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...
  woensdag 12 april 2017 @ 16:14:33 #20
2671 Aaargh!
Gebruik op eigen risico.
pi_170168464
quote:
0s.gif Op woensdag 12 april 2017 15:05 schreef Farenji het volgende:

[..]

Dat is een misvatting, veroorzaakt door onwetendheid. Perl is pure kunst. Poezie van de hoogste orde! _O_
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.
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.
pi_170168645
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.
pi_170168694
Ow kijk, gelukkig is dit ook het enige topic wat TS op Fok! heeft aangemaakt en is hij sindsdien niet meer online geweest...
pi_170168779
quote:
1s.gif Op dinsdag 11 april 2017 23:17 schreef Promille het volgende:
hallo fokkers,

Opdracht voor de Hanze hogeschool, CMD?
pi_170169206
quote:
0s.gif Op woensdag 12 april 2017 16:14 schreef Aaargh! het volgende:

[..]

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.
Ik wilde het beroemde 99 bottles of beer script posten maar die compileert niet meer in moderne versies van perl. ;(
abonnementen ibood.com bol.com Gearbest
Forum Opties
Forumhop:
Hop naar:
(afkorting, bv 'KLB')