Error echo array

Door Lorenzo Sipkema op 09-04-2015 14:44

2.272 views

Hi,

Ik probeer een loginsysteem te maken maar de bedoeling is dat alle errors die komen zichtbaar zijn

[qoute]
<?php
if (!isset($_POST["username"])) {
echo "Vul wel een gebruikersnaam in";
die();
}
if (!isset($_POST["password"])) {
echo "Vul wel een wachtwoord in";
die();
}
?>
[/qoute]

Moet dan de username empty error te laten zien & password wanneer beide velden leeg zijn

Rick B

09-04-2015 15:07 gewijzigd op 09-04-2015 15:10

[CODE]
<?php
$errors = array();
if (!isset($_POST["username"])) {
$errors[] = "Vul wel een gebruikersnaam in";
}
if (!isset($_POST["password"])) {
$errors[] = "Vul wel een wachtwoord in";
}
if (!empty($errors)) {
foreach($errors as $error) {
echo $error."<br>";
}
die();
}
else {
// Rest van je code
}
?>

Lorenzo Sipkema

09-04-2015 15:25

Wanneer ik gebruik maak van dit form werkt het niet ;s

[qoute]
<form method="POST" action="./login.php">
<input type="text" placeholder="Gebruikersnaam" name="username" /><br />
<input type="password" placeholder="Wachtwoord" name="password" /><br />
<input type="submit" value="Submit" />
</form>
[/qoute]

Ariën

09-04-2015 15:30

Gebruik a.u.b. de code-tag voor scripts.

Controleer ook eerst of je het formulier hebt verstuurd:


<?php
if($_SERVER['REQUEST_METHOD']=="POST") {
	// het formulier is verstuurd
}
?>

Verder controleer ik of iets niet ingevuld is met:

<?php
if(trim($_POST["password"])=='') {
	// veld is leeg!
}
?>

Dan strip je ook enkele spaties alleen.

Lorenzo Sipkema

09-04-2015 15:35 gewijzigd op 09-04-2015 15:36

Aangepast naar aar dr script.

Wanneer ik nu bijv het input veld username invult krijg ik alleen de error dat username leeg is terwijl die wel gevuld is en password niet,

script


<?php
if ($_SERVER["REQUEST_METHOD"] == "POST") {
	$errors = array();
	if (trim($_POST["username"])) {
		$errors[] = "Vul wel een gebruikersnaam in";
	}
	if (trim($_POST["password"])) {
		$errors[] = "Vul wel een wachtwoord in";
	}
	if (!empty($errors)) {
		foreach($errors as $error) {
			echo $error."<br>";
		}
		die();
	} else {
		echo "Rest van de code.";
	}
} else { 
	echo "Geen formulier verstuurd.";
}
?>

Ariën

09-04-2015 15:38

Je moet niet alleen trim() doen, maar ook vergelijken op een lege string.
Zie http://php.net/trim

Rick B

09-04-2015 15:42 gewijzigd op 09-04-2015 15:42

Je controleert op dit moment nog niet of de velden leeg zijn.

Verander regel 4 en 7

 if (trim($_POST["username"]))

naar

 if(trim($_POST["username"])=='')

Lorenzo Sipkema

09-04-2015 15:42

In principe zou ik dan alleen moeten filteren op username met trim omdat sommige wachtwoorden andere chars hebben, nou is de vraag wat zou ik dan moeten filteren bij usernames?

Ariën

09-04-2015 15:45

Wachtwoorden kunnen nooit leeg zijn, dus een controle daarop kan geen kwaad tijdens het inloggen of registreren. Die trim() kan je daar dan achterwege laten.

Lorenzo Sipkema

09-04-2015 15:49

Ik maak nu gebruik van trim zoals je zei. alleen als ik nu spaties intoets bij het login form kom ik uit bij die echo van rest van de code..


<?php
if ($_SERVER["REQUEST_METHOD"] == "POST") {
	$errors = array();
	if (trim($_POST["username"])==" ") {
		$errors[] = "Vul wel een gebruikersnaam in";
	}
	if (trim($_POST["password"])==" ") {
		$errors[] = "Vul wel een wachtwoord in";
	}
	if (!empty($errors)) {
		foreach($errors as $error) {
			echo $error."<br>";
		}
		die();
	} else {
#Hier kom ik dan uit
		echo "Rest van de code.";
	}
} else { 
	echo "Geen formulier verstuurd.";
}
?>

Ariën

09-04-2015 15:51 gewijzigd op 09-04-2015 15:52

- Controleer op een lege string, zie mijn voorbeeld nog eens.
- die() is niet nodig. Geen enkel script hoort 'dood' te gaan.
- Wat nu als mijn wachtwoord met een spatie begint?

Reageren

Inloggen om te reageren