isset is wel mooier, anders krijg je een warning bij error_reporting(E_ALL);quote:Op zondag 15 april 2012 19:38 schreef cablegunmaster het volgende:
[..]
Antwoord staat er al tussen om post met standaard bericht te voorkomen.
De Isset is overbodig inderdaad. kan beter !empty zijn .
quote:Op zondag 15 april 2012 20:15 schreef drumminggod het volgende:
Oke. Ik heb nu iets. Alleen als ik nu mijn naam NIET heb ingevuld. Of een bericht niet. Dan moet er eigenlijk een foutmelding komen... Alleen nu plaatst hij gewoon lege vakken in mijn db.. Zal wel iets kleins zijn maar voor mij onvindbaar op ww3schools
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 | <?php If(!empty($_POST['VELDNAAM'])) { $error = "Er is geen BLABLABLABLA ingevuld"; } //Om je query heen invullen. if(!isset($error)){ // Insert Query uitvoeren. } //bij je formulier. if(!empty($error)){ echo $error; } ?> |
quote:
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 | <?php $errorlog = array(); if ( empty($_POST['veldnaam']) === false ) { $errorlog[] = "<li>error omschrijving</li>"; } if ( count($errorlog) == 0 ) { // do database work } else { echo "<ul>" . join("", $errorlog) . "</ul>"; } ?> |
Een string met alleen een spatie is niet empty.quote:Op zondag 15 april 2012 21:44 schreef totalvamp het volgende:
Ik zou geen empty gebruiken. Dan werkt een spatie alleen ook Doe een betere check met strlen
Nee dat is mijn punt. Je moet dus een adequate check doen op je POST. strlen() + trim() + weetikveel()quote:Op maandag 16 april 2012 21:17 schreef Light het volgende:
[..]
Een string met alleen een spatie is niet empty.
Maar voordat je iets gaat doen met een post-variable, moet je wel controleren of de variable bestaat. Dat voorkomt notices, en ook die moet je niet willen.quote:Op maandag 16 april 2012 23:53 schreef totalvamp het volgende:
[..]
Nee dat is mijn punt. Je moet dus een adequate check doen op je POST. strlen() + trim() + weetikveel()
Daarvoor gebruik je isset() niet empty. Zoals eerder aangegeven, maar alleen een isset is natuurlijk niet voldoende, net als empty dat niet is.quote:Op dinsdag 17 april 2012 00:13 schreef Light het volgende:
[..]
Maar voordat je iets gaat doen met een post-variable, moet je wel controleren of de variable bestaat. Dat voorkomt notices, en ook die moet je niet willen.
Yep. Volgens mij bedoelen we hetzelfde maar omschrijven we het anders.quote:Op dinsdag 17 april 2012 00:23 schreef totalvamp het volgende:
[..]
Daarvoor gebruik je isset() niet empty. Zoals eerder aangegeven, maar alleen een isset is natuurlijk niet voldoende, net als empty dat niet is.
Dit is inderdaad een belangrijk punt, tevens kun je controleren of je formulier wel goed over komt. Je kunt dus andere acties ondernemen als ze niet ge-set zijn. Dit kan betekenen dat er met het formulier is gerommeld.quote:Op dinsdag 17 april 2012 00:13 schreef Light het volgende:
[..]
Maar voordat je iets gaat doen met een post-variable, moet je wel controleren of de variable bestaat. Dat voorkomt notices, en ook die moet je niet willen.
Bedankt! Het werkt!quote:Op zondag 15 april 2012 20:47 schreef Pakspul het volgende:
[..]
[ code verwijderd ]
Zo kun je meerdere velden checken
1 | <input type="submit" value="Toevoegen" onClick=<a href="javascript:location.reload(true)" /> |
Wat is er mis mee? Dat je een tool nodig bent?quote:Op maandag 16 april 2012 13:58 schreef wipes66 het volgende:
Weet iemand nog een goede tool om php code mee te analyseren? Bv om ongebruikte variable/functies, foutieve opmaak etc mee te detecteren. Het liefst eentje met een webinterface (geen cli).
Na het schrijven naar de database de gebruiker doorsturen naar de goede pagina header("Location xxx"); ?quote:Op dinsdag 17 april 2012 10:31 schreef drumminggod het volgende:
[..]
Bedankt! Het werkt!
nog 1 resterend probleempje... Bij het herladen van mijn pagina verstuurd hij mijn berichtje net zo vaak als dat f5 wordt ingedrukt. Ik heb deze code:
[ code verwijderd ]
maar als ik op toevoegen heb geklikt en druk op f5 komt hij dus twee keer! (of meerdere malen)
Wat is er fout? Ik dacht aan buffer legen... alleen geen idee hoe dat netjes moet.
1. Simpelste oplossing is een redirect . Dan stuurt hij door naar de volgende pagina en dan mogen ze zo veel F5 drukken als ze willen maar dan herhaalt hij niet.quote:Op dinsdag 17 april 2012 10:31 schreef drumminggod het volgende:
[..]
Bedankt! Het werkt!
nog 1 resterend probleempje... Bij het herladen van mijn pagina verstuurd hij mijn berichtje net zo vaak als dat f5 wordt ingedrukt. Ik heb deze code:
[ code verwijderd ]
maar als ik op toevoegen heb geklikt en druk op f5 komt hij dus twee keer! (of meerdere malen)
Wat is er fout? Ik dacht aan buffer legen... alleen geen idee hoe dat netjes moet.
1 2 3 4 5 6 7 8 9 10 11 12 13 | <?php function esc($string){ return mysql_real_escape_string($string); } //Dit eronder ff opzoeken hoe je het aanpakt maar het generale idee is er wel :). $test = $db->fetchAssoc($db->query("Select * from table where text = ".esc($_POST['text'])."")); if(empty($test[0]['text'])) { // execute insert query. } redirect("pagina.php"); ?> |
Headers mogen geen output daarvoor hebben dus een reeks html + vervolgens de header kan tot fouten leiden.quote:Op dinsdag 17 april 2012 11:39 schreef Pakspul het volgende:
[..]
Na het schrijven naar de database de gebruiker doorsturen naar de goede pagina header("Location xxx"); ?
1 2 3 4 5 6 7 8 9 10 11 12 13 14 | <?php //==== Redirect... Try PHP header redirect, then Java redirect, then try http redirect.: function redirect($url){ if (!headers_sent()){ //If headers not sent yet... then do php redirect header('Location: '.$url); exit; }else{ //If headers are sent... do java redirect... if java disabled, do html redirect. echo '<script type="text/javascript">'; echo 'window.location.href="'.$url.'";'; echo '</script>'; echo '<noscript>'; echo '<meta http-equiv="refresh" content="0;url='.$url.'" />'; echo '</noscript>'; exit; } }//==== End -- Redirect |
Liever gewoon code waarbij er niks wordt getoond tot op het einde als alles verwerkt is dan heb je die problemen ook nietquote:Op dinsdag 17 april 2012 11:40 schreef cablegunmaster het volgende:
[..]
1. Simpelste oplossing is een redirect . Dan stuurt hij door naar de volgende pagina en dan mogen ze zo veel F5 drukken als ze willen maar dan herhaalt hij niet.
2. een Check inbouwen dat als de text precies zo al voorkomt dat hij niet meer een Insert uitvoert maar doorgaat met redirecten.
[ code verwijderd ]
[..]
Headers mogen geen output daarvoor hebben dus een reeks html + vervolgens de header kan tot fouten leiden.
[ code verwijderd ]
Prefereer daarom liever Redirect
Dit, tussen door stukken al in elkaar zetten en op het laatste even layoutje er omheen rammen en er uit gooien. Dan kun je er zelf voor kiezen of er uberhaupt iets geprint moet worden en kun je veilig omgaan met header functies.quote:Op dinsdag 17 april 2012 12:09 schreef totalvamp het volgende:
[..]
Liever gewoon code waarbij er niks wordt getoond tot op het einde als alles verwerkt is dan heb je die problemen ook niet
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 | <?php //pseudo code. om +1 te krijgen. function Switch_ID($id) { $sourceid = $db->Assocfetch($db->query("select * from categorie where id = " '.esc($id) .' " limit 1 " )); $id++; $target = $db->Assocfetch($db->query("select * from categorie where id = " '.esc($id) .' " limit 1" )); if(!isset($target)) { return false; }else{ $db->query("Update categorie set Variables = $sourceid[0] where id = " '.$id.' " limit 1" ); $id--; $db->query("Update categorie set Variables = $target[0] where id = " '.esc($id).' " limit 1" ); } } ?> |
Is dat niet het probleem opschuiven? Want dan zit je met een orderkolom met 1,2,3,4quote:Op dinsdag 17 april 2012 17:51 schreef Pakspul het volgende:
Gebruik een order kolom. Haal eerst oude order nummer op en zet die in de nieuwe nummer kun je berekenen door de richting.
opschuiven maakt toch niet uit, dan sorteer je toch gewoon op die kolom en heb je het allemaal goed. Als je ID nummers gaat veranderen dan begrijp je het idee van een primary key niet helemaal.quote:Op dinsdag 17 april 2012 17:57 schreef cablegunmaster het volgende:
[..]
Is dat niet het probleem opschuiven? Want dan zit je met een orderkolom met 1,2,3,4
en als je een groot aantal hebt. dan heb je dat probleem alsnog.
ff nadenken
Daarom moet je pas output richting browser sturen nadat alle logica is geweest. Dan heb je daar geen last van.quote:Op dinsdag 17 april 2012 11:40 schreef cablegunmaster het volgende:
[..]
1. Simpelste oplossing is een redirect . Dan stuurt hij door naar de volgende pagina en dan mogen ze zo veel F5 drukken als ze willen maar dan herhaalt hij niet.
2. een Check inbouwen dat als de text precies zo al voorkomt dat hij niet meer een Insert uitvoert maar doorgaat met redirecten.
[ code verwijderd ]
[..]
Headers mogen geen output daarvoor hebben dus een reeks html + vervolgens de header kan tot fouten leiden.
[ code verwijderd ]
Prefereer daarom liever Redirect
Forum Opties | |
---|---|
Forumhop: | |
Hop naar: |