Vermijden dat een veld met alleen spaties wordt weggeschreven
Ik heb voor de Naam een pattern ingestoken (minstens 2 tekens en verder de gebruikelijke letters en spaties.
Code (php)
1
<input name="Naam" required pattern="[A-Za-z']+[A-Za-z'. ]{2,}" />
Maar dat werkt blijkbaar niet. Het is nog altijd mogelijk om een naam weg te schrijven die enkel uit 2 spaties bestaat. Dus eigenlijk leeg.
Ik heb al geprobeerd dit af te vangen via mijn code om het weg te schrijven naar mijn database. Maar ook dit lukt niet.
Er wordt nog altijd een leeg veld weggeschreven. Waar zit ik fout in mijn redenering.
Code (php)
1
2
3
4
5
6
2
3
4
5
6
if (!empty($naam)
|| !empty($vnaam)
|| !empty($email)
|| !empty($paswoord)) {
$sql = "INSERT INTO Leden(Naam,Voornaam,mailadres,Paswoord)
VALUES ('$naam','$vnaam','$email','$paswoord')
|| !empty($vnaam)
|| !empty($email)
|| !empty($paswoord)) {
$sql = "INSERT INTO Leden(Naam,Voornaam,mailadres,Paswoord)
VALUES ('$naam','$vnaam','$email','$paswoord')
Kortom wat kan ik doen om geen veld met alleen spaties weg te schrijven. Een naam zoals Van de Bossche moet wel kunnen. Daar zitten spaties in. Ik kan natuurlijk ook het minimum aantal in te voeren tekens verhogen naar 6 ofzo maar een naam als De Bo of Debo lukt dan niet meer.
Code (php)
1
2
3
2
3
echo '<p>lengte : ' . strlen($_POST['Naam']) . '</p>';
$leeg = trim($_POST['Naam']);
echo '<p>lengte : ' . strlen($leeg) . '</p>';
$leeg = trim($_POST['Naam']);
echo '<p>lengte : ' . strlen($leeg) . '</p>';
Toevoeging op 05/01/2022 21:44:59:
En je kan bij de input een min en max lengte opgeven.
Code (php)
1
<input minlength="10" maxlength="50" type="text" id="name" placeholder="Enter your login name" onkeyup="checkInputLength();">
Toevoeging op 05/01/2022 21:49:19:
En om het maar gelijk compleet te maken.
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
<script>
function checkInputLength()
{
document.getElementById("submit").disabled = true;
var nameMinLength = document.getElementById("name").minLength;
var nameValue = document.getElementById('name').value;
if ( nameValue.length >= nameMinLength )
{
nameValueC = true;
}
else
{
nameValueC = false;
}
if ( nameValueC )
{
document.getElementById("submit").disabled = false;
}
}
</script>
function checkInputLength()
{
document.getElementById("submit").disabled = true;
var nameMinLength = document.getElementById("name").minLength;
var nameValue = document.getElementById('name').value;
if ( nameValue.length >= nameMinLength )
{
nameValueC = true;
}
else
{
nameValueC = false;
}
if ( nameValueC )
{
document.getElementById("submit").disabled = false;
}
}
</script>
Toevoeging op 05/01/2022 21:58:56:
Nou, nog eentje dan.
https://www.w3schools.com/jsref/jsref_trim_string.asp
Gewijzigd op 05/01/2022 21:50:35 door Adoptive Solution
Ook weer jammer dan dat
Jan-Willem,
Zoë
René
ðalbjörn (ik hoop dat phphulp deze IJslandse naam kan weergeven)
en Þór
niet mee mogen doen.
Gewijzigd op 06/01/2022 10:48:53 door Ivo P
En hopelijk schrijf je het niet meteen weg zoals in je voorbeeld!
Maar in mijn browser (firefox) wordt een input van 2 spaties geweigerd
En dan voor de verwerking doet dit het ook niet:
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
2
3
4
5
6
7
8
9
10
11
12
$naam = trim(mysqli_real_escape_string($conn, $_POST['Naam']));
$vnaam = trim(mysqli_real_escape_string($conn, $_POST['Voornaam']));
$email = trim(mysqli_real_escape_string($conn, $_POST['email']));
$paswoord= trim(md5(mysqli_real_escape_string($conn, $_POST['paswoord'])));
if (empty($naam)
|| empty($vnaam)
|| empty($email)
|| empty($paswoord)){
//header("Location: fout01.php")
else{
$sql = "INSERT INTO Leden(Naam,Voornaam,mailadres,Paswoord)
$vnaam = trim(mysqli_real_escape_string($conn, $_POST['Voornaam']));
$email = trim(mysqli_real_escape_string($conn, $_POST['email']));
$paswoord= trim(md5(mysqli_real_escape_string($conn, $_POST['paswoord'])));
if (empty($naam)
|| empty($vnaam)
|| empty($email)
|| empty($paswoord)){
//header("Location: fout01.php")
else{
$sql = "INSERT INTO Leden(Naam,Voornaam,mailadres,Paswoord)
Het blijft maar wegschrijven.
En paswoord moet je niet escapen: md5() zorgt al dat je een string overhoudt met alleen 0-9A-F erin.
Of de invoer van md5() nu alleen maar bestaat uit ' of uit de hele tekst van de handleiding van je koffiemachine.
(afgezien van het zwaar verouderd zijn van een password op deze manier te hashen.)
Probleem is, dat als je straks je password weer controleert en dan alleen md5() gebruikt, het werkt zolang iemand geen ' in zijn password heeft staan.
Toevoeging op 06/01/2022 11:32:09:
Ik denk trouwens dat je beter af bent met een filter op een minimale lengte (2?) en tekens die je beslist niet in een naam verwacht.
En dan denk ik aan cijfers en tekens als !@#$%^&*
Probeer niet om alle letters te vangen die je wél wilt zien, want A-Z dekt de lading niet, maar dan zit je ook nog met 10 verschillende accenten die al dan niet in combinatie op de raarste letters gezet kunnen worden.
Ik kom er wel uit en ondertussen leer ik heel veel.
Omgekeerd: in een password zou je aan een minimum aantal eisen moeten voldoen.
Dingen als :
* minstens 8 tekens
* Minstens 1 hoofd en 1 kleine letter
* een of meer cijfers
* en of meer "andere" tekens.
En zou geen enkele reden moeten zijn om een teken als $, ' of " te verbieden in een password of te eisen dat het hooguit 8 tekens lang is.
Zeker aangezien het password gehasht wordt, is er geen enkel veiligheidsissue mbt het opslaan van het password.
En later bij het inloggen wordt weer een vergelijk getrokken tussen de hash van het password bij het inloggen en wat in de database gehasht staat opgeslagen.
Uit praktisch oogpunt zou een password van 1024 tekens misschien lastig worden ivm een max aantal bytes dat de webserver kan aanpakken van de browser bij een POST, maar technisch moet dat geen probleem zijn.
Toevoeging op 07/01/2022 12:30:04:
Dit Pattern zorgt er blijkbaar voor dat je niet kan beginnen met spaties:
Code (php)
1
pattern="[A-Za-z'][A-Za-z\s']{2,}"
Om in te loggen hebben ze enkel hun emailadres en paswoord nodig. Maar het mailadres kunnen we nog eens laten bevestigen, dus een vals of onbestaand kunnen we er dan zo uithalen. Misbruik kunnen we nooit helemaal uitsluiten, het moeilijk maken kunnen we wel.
Ik ben tevreden met het resultaat.
Ignace Verschaeve op 06/01/2022 11:22:33:
Ik weet niet wat er aan de hand is maar met dit pattern="[A-Za-z']+[A-Za-z'. ]" worden nog altijd 2 spaties aanvaard. Zowel in Chrome, als in Firefox.
En dan voor de verwerking doet dit het ook niet:
Het blijft maar wegschrijven.
En dan voor de verwerking doet dit het ook niet:
Code (php)
1
2
3
4
5
6
7
8
9
10
11
12
2
3
4
5
6
7
8
9
10
11
12
$naam = trim(mysqli_real_escape_string($conn, $_POST['Naam']));
$vnaam = trim(mysqli_real_escape_string($conn, $_POST['Voornaam']));
$email = trim(mysqli_real_escape_string($conn, $_POST['email']));
$paswoord= trim(md5(mysqli_real_escape_string($conn, $_POST['paswoord'])));
if (empty($naam)
|| empty($vnaam)
|| empty($email)
|| empty($paswoord)){
//header("Location: fout01.php")
else{
$sql = "INSERT INTO Leden(Naam,Voornaam,mailadres,Paswoord)
$vnaam = trim(mysqli_real_escape_string($conn, $_POST['Voornaam']));
$email = trim(mysqli_real_escape_string($conn, $_POST['email']));
$paswoord= trim(md5(mysqli_real_escape_string($conn, $_POST['paswoord'])));
if (empty($naam)
|| empty($vnaam)
|| empty($email)
|| empty($paswoord)){
//header("Location: fout01.php")
else{
$sql = "INSERT INTO Leden(Naam,Voornaam,mailadres,Paswoord)
Het blijft maar wegschrijven.
Ik weet 100% zeker dat de trim functie in php een spatie voor en na de input weghaalt.
Dus ik neem aan dat 2 spaties geen probleem is.
Als je eerst de trim functie gebruikte en dan pas de escape.
Code (php)
1
2
2
$naam = trim($_POST['Naam'];
$naam = mysqli_real_escape_string($conn, $naam));
$naam = mysqli_real_escape_string($conn, $naam));
Daarnaast gebruik je een if statement die alleen naar OR kijkt terwijl het zou moeten zijn EN.
Code (php)
1
2
3
4
5
6
2
3
4
5
6
if (!empty($naam)
&& !empty($vnaam)
&& !empty($email)
&& !empty($paswoord)) {
$sql = "INSERT INTO Leden(Naam,Voornaam,mailadres,Paswoord)
VALUES ('$naam','$vnaam','$email','$paswoord')
&& !empty($vnaam)
&& !empty($email)
&& !empty($paswoord)) {
$sql = "INSERT INTO Leden(Naam,Voornaam,mailadres,Paswoord)
VALUES ('$naam','$vnaam','$email','$paswoord')
Ik zou aanraden om de allemaal appart te checken zodat je kan aangeven wat ze niet ingevuld hebben.
Of foutief ingevuld.
En kijk aub niet naar wat Leden invoeren maar wat je zelf doorzet naar de database.
Gewijzigd op 08/01/2022 01:32:35 door Ralf d
Bij het verzenden van je formulier bekijk je of die array waardes van foute invoer bevat (count($errors) >0). In dat geval laat je zien aan de gebruiker welke velden er missen of fout ingevoerd zijn. Als alles ingevuld is mag de database zijn werk doen.
Of..... je gebruikt gewoon een kant en klaar product, zoals deze class :P
https://github.com/SandroMiguel/verum-php
Dan kan je gewoon met een klein stukje code en een array een validatie maken.
De Nederlandse vertaling is aan mijzelf te danken. ;-)
Gewijzigd op 08/01/2022 01:43:17 door - Ariën -
@Ralf d. Die If die je voorstelt werkt al zeker niet want als er eentje ingevuld is dan schrijft het weg. Ik had het daarom al omgedraaid. Als Naam Of Voornaam of email of paswoord leeg is dan terug naar invulformulier in het ander geval wegschrijven. Maar om de ene of andere reden lukt dit niet. Ik denk dat het eigenlijk een XOR zou moeten zijn.
Ignace Verschaeve op 08/01/2022 15:23:07:
@Ariën. We gaan dat laatste eens proberen.
@Ralf d. Die If die je voorstelt werkt al zeker niet want als er eentje ingevuld is dan schrijft het weg. Ik had het daarom al omgedraaid. Als Naam Of Voornaam of email of paswoord leeg is dan terug naar invulformulier in het ander geval wegschrijven. Maar om de ene of andere reden lukt dit niet. Ik denk dat het eigenlijk een XOR zou moeten zijn.
@Ralf d. Die If die je voorstelt werkt al zeker niet want als er eentje ingevuld is dan schrijft het weg. Ik had het daarom al omgedraaid. Als Naam Of Voornaam of email of paswoord leeg is dan terug naar invulformulier in het ander geval wegschrijven. Maar om de ene of andere reden lukt dit niet. Ik denk dat het eigenlijk een XOR zou moeten zijn.
Dat die niet werkt dat zou zeker kunnen!
Zou dat zelf namelijk nooit doen.
Geef daarna ook aan dat je deze beter appart in een if statement kunt zetten.
En bij iedere if een fout afhandeling.
Pas bij de laatste naar de database schrijven.
het wordt hierboven al duidelijk uit gelegd.
Gewijzigd op 09/01/2022 00:42:10 door Ralf d