Ter illustratie de onderstaande simplistische code

Code:

<?php if($_POST['submit']) { $process = true;
    
    // ingevuld of niet
    if(!$_POST['name']){ $process = false; $_SESSION['fill-name'] = true; }else{ unset($_SESSION['fill-name']); }
    // redirect
    if($process){ header('Location: stap_2.php'); }else{ header('Location: stap_1.php'); }
    
} ?>

<form method="post">
<input type="text" name="name" value="" class="<?php if($_SESSION['fill-name']){ echo 'error'; } ?>" />
<input type="submit" name="submit" value="verwerk" />
</form>

Ik heb een eenvoudig formulier met een veld 'naam' dat verplicht in te vullen is. Is het ingevuld, dan gaat men naar de tweede stap. Is het veld leeg, dan blijft men op dezelfde pagina ( met redirect ) en kleurt het veld rood met de klasse .class

Ik werk met een header redirect naar dezelfde pagina om de popup bij het vernieuwen na een post te voorkomen. Om het veld toch rood te laten kleuren gebruik ik een sessie. Het idee is uitgewerkt en operationeel. Ik heb echter mijn twijfels of dit niet beter gemaakt kan worden? Zelf dacht ik aan javascript om de velden te valideren, maar dat kan men uitzetten ... dus niet ideaal lijkt mij ... iemand een beter idee dan wat ik momenteel doen?

Je zou het met jQuery kunnen oplossen :)
Zo doe ik het zelf ook altijd.

Bedankt voor uw reactie. Met jQuery zou het inderdaad lukken ... maar als men dan Javascript gaat uitzetten, dan kan men zonder validatie naar de volgende stap gaan? Of begrijp ik het verkeerd?

Origineel gepost door R van Breugel

Je zou het met jQuery kunnen oplossen :)
Zo doe ik het zelf ook altijd.

Je kunt met PHP het veld toch ook valideren, zonder naar de volgende stap te moeten gaan. Met jQuery zou je dan stap 2 kunnen verbergen en pas weergeven wanneer stap 1 succesvol is ingevuld. Eventueel kan ik het formulier wel voor je maken.

Ik zou het gewoon met javascript (jquery) doen. Mensen die tegenwoordig js uitzetten in de browser kunnen amper een site fatsoenlijk zien, dus het percentage dat dit daadwerkelijk doet is volgens mij te verwaarlozen.

Checken aan de php kant moet je tenslotte toch altijd doen, of het nu met js gechecked is of niet.

Origineel gepost door WesleyVN

Bedankt voor uw reactie. Met jQuery zou het inderdaad lukken ... maar als men dan Javascript gaat uitzetten, dan kan men zonder validatie naar de volgende stap gaan? Of begrijp ik het verkeerd?

Javascript gebruik je om het makkelijk te maken voor de bezoeker (zoals in dit geval). Maar je moet altijd de invoer ook aan de serverkant controleren (php). Alles wat van de browser komt is namelijk te manipuleren en mag je niet vertrouwen.

Je kunt ook dubbel controleren (javascript en daarna php) voorkomen door het formulier via AJAX naar de server te sturen en daar te controleren.

Iets in deze trend:

Javascript:

PHP Code:

$("form input[type='submit']").on("click", function(e){
    e.preventDefault();
    $.ajax({
        type: 'POST',
        url: "register.php",   //Op register.php controleer je dan de invoer
        cache: false,
        data: $("form").serialize(),
        success: function (data) {
            //data is wat register.php terug stuurt. Dit kan bijvoorbeeld "true" of "false" zijn.
            if (data == "true") {
                $("input[name='name']").addClass("error"); // invoerveld gekleurd maken
                alert("De invoer is onjuist");
            } else {
                window.location = "gelukt.php"; //doorsturen naar andere pagina.
            }
        }
    });   
}); 


Uw code heb ik werkende ( was ook niet moeilijk vermits ik volledig copy/paste kon doen )
De enige vraag die ik momenteel heb is hoe ik de koppeling van register.php naar het script maak.
Ik kom steeds in de else uit ... hoe stel ik die data == true in op register.php? :-)

Origineel gepost door Wilbert E.

Javascript gebruik je om het makkelijk te maken voor de bezoeker (zoals in dit geval). Maar je moet altijd de invoer ook aan de serverkant controleren (php). Alles wat van de browser komt is namelijk te manipuleren en mag je niet vertrouwen.

Je kunt ook dubbel controleren (javascript en daarna php) voorkomen door het formulier via AJAX naar de server te sturen en daar te controleren.

Iets in deze trend:

Javascript:

PHP Code:

$("form input[type='submit']").on("click", function(e){
    e.preventDefault();
    $.ajax({
        type: 'POST',
        url: "register.php",   //Op register.php controleer je dan de invoer
        cache: false,
        data: $("form").serialize(),
        success: function (data) {
            //data is wat register.php terug stuurt. Dit kan bijvoorbeeld "true" of "false" zijn.
            if (data == "true") {
                $("input[name='name']").addClass("error"); // invoerveld gekleurd maken
                alert("De invoer is onjuist");
            } else {
                window.location = "gelukt.php"; //doorsturen naar andere pagina.
            }
        }
    });   
}); 


# EDIT

Wanneer ik in register.php simpelweg TRUE zet, dan kom ik in de IF terecht.
Kan ik ook meerdere resultaten van de register gebruiken voor meerdere velden.
Bijvoorbeeld : validmail=true - validname = true - ... die dan op de een of
andere manier gebruiken zodat ik meerdere velden kan kleuren? :-)

Ja je kunt alles terug sturen wat je wilt.
Als je weet hoe het moet kun je een XML genereren.
Geen idee hoe dat moet dan kun je inderdaad doen wat jij voorstelt. Er zijn legio mogelijkheden.
Je krijgt een string terug waarmee je in Javascript alles kunt doen wat je wilt.

Stuur je bijvoorbeeld dit terug:

validmail=true&validname=true

dan doe je in javascript:

PHP Code:

success: function (data) {
//---------

var arr = data.split("&"); //splitten alles op het & teken

//string bijhouden om alert te laten zien
var errMessage = "";

//loop door de array
for(i=0;i<arr.length;i++){

     var item = arr[i].split("="); //splitten elk array item op het = teken.
     var name = item[0];
     var value = item[1];

    if(name == "validmail" && value =="false") { 
        $("input[name='mail']").addClass("error"); 
        errMessage += "- Het e-mail adres is onjuist\n";
    }
    if(name == "validname" && value =="false") { 
        $("input[name='name']").addClass("error"); 
        errMessage += "- De naam adres is onjuist\n";
    }
}

//controleren of er een melding is anders laten zien
if(errMessage != "") alert(errMessage); 

//-----
} 


Dit is uit mijn hoofd, net als de vorige dus ik weet niet of het copy paste proof is.

Wat ook kan:

dataType ="json" opgeven als parameter bij je ajax call en je register.php dan een json encoded array laten teruggeven:

$ret = array ('a' => true, 'b' = 'false');
echo json_encode($ret);

dan kun je in de javascript weer checken met data.a == true, data.b == false of whatever je ermee wilt doen.

Er zijn dus twee mogelijkheden, ofwel JSON ofwel via mijn eerste gedacht hierboven ( validmail=true&validname=true ) - Mijn eerdere idee lijkt een stuk sneller te maken dan de optie JSON - is dit ook effectief beter dan een string met de resultaten doorsturen? De string ziet men toch nergens verschijnen, of heb ik het mis?

Jouw methode werkt, zonder twijfel. Er zijn zoveel methodes hoe je dit werkend kunt krijgen.
Maar een betere methode is JSON of XML.

Het probleem met jouw methode komt opspelen zodra een scheidingsteken (bijvoorbeeld "&" of "=") voorkomt in de naam van de variabele.
Met XML of JSON heb je dat probleem niet. Mocht je weten hoe je in PHP een JSON of XML string terug kunt sturen dan zou ik daar zeker de voorkeur aan geven.