I need to optimize my code, creating an input property that'd get $_POST['count'] but I don't know how to do it... And use different functions to minimize the code. So the goal of this is to make an input where you'd type respectively : 3 (click button), 2 (click button), ... 0 using session variables ($_SESSION['count'] here).
I've tried to make a constructor and put the post in it using a property. But I don't think this is a good way. So I am asking you :').
index.php
<?php
session_start();
include_once 'test.php';
?>
<!DOCTYPE html>
<html>
<head>
<title>Countdown</title>
<meta charset="UTF-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
</head>
<body>
<form action="<?php echo $_SERVER['PHP_SELF']?>" method="POST">
<p>
<label for="count">Chiffre : </label>
<input type="text" id="count" for="count" name="count" required />
<input type="submit" value="Envoyer" />
</p>
</form>
<?php
$test = new Test();
if(isset($_POST['count'])) {
$test->validateInput($_POST);
echo $test->getMessage();
} else if(!isset($_SESSION['count'])) {
echo "Veuiller saisir 3 afin de commencer le compte à rebours.";
}
?>
</body>
</html>
test.php
<?php
class Test {
private $input;
private $message;
private $regex = '/^[0-3]{1}$/';
public function getInput()
{
return $this->input;
}
public function setInput($message)
{
$this->input = $input;
}
public function getMessage()
{
return $this->message;
}
public function setMessage($message)
{
$this->message = $message;
}
public function getRegex()
{
return $this->regex;
}
public function setRegex($regex)
{
$this->regex = $regex;
}
public function validateInput($input) {
//Vérification variable de session créée
if(!isset($_SESSION['count'])) {
//Vérification chiffre compris entre 0 et 3 et si saisie = 3
if(preg_match($this->getRegex(), $input['count'])) {
if($input['count'] == 3) {
$this->setMessage("Le chiffre suivant est ".($input['count']-1));
$_SESSION['count'] = $input['count'];
return true;
} else {
$this->setMessage("Vous devez commencer par le chiffre 3");
return false;
}
} else {
$this->setMessage("Saisie incorrecte, vous avez saisi '".$input['count']."' alors que le chiffre : ".($_SESSION['count']-1)." est attendu.");
return false;
}
} else {
if (preg_match($this->getRegex(), $input['count'])) {
//Vérification ancienne valeur-1 = valeur actuelle saisie
if (($_SESSION['count'] - 1) == $input['count'] && $input['count'] > 0) {
$this->setMessage("Le chiffre suivant est " . ($input['count'] - 1));
$_SESSION['count'] = $input['count'];
return true;
//Vérification si valeur saisie = 0
} else if (($_SESSION['count'] - 1) == $input['count'] && $input['count'] == 0) {
$this->setMessage("Fini ! - Pour recommencer, saisissez 3.");
session_destroy();
unset($_SESSION['count']);
return true;
} else {
$this->setMessage("La valeur saisie ne correspond pas à l'ordre du compte à rebours (3 2 1 0)<br>"
. "La valeur attendue est : " . ($_SESSION['count'] - 1));
return false;
}
} else {
$this->setMessage("Saisie incorrecte, vous avez saisi '" . $input['count'] . "' alors que le chiffre : " . ($_SESSION['count']-1) . " est attendu.");
return false;
}
}
}
}
?>
At the end, the form countdown should simply work correctly. Thank you in advance for your help, have a nice day !
