Ezuker pour une meilleur lisibilité tu dois apprendre à parsé selon les règles imposé par PHP
pour une meilleur lisibilité si tu ne le fais pas les personnes qui sont reprendre ton code pour de la
maintenance vont ou se moqué de toi ou supprimer ton code et tout recommencer.
je sais, je sais.. je dis pas cela contre toi c'est plus comme les bonnes manière ou bonne pratique
je t'ai donc mis convenablement l'aspect esthétique pour que tu puisse bien voir dans le futur si
tu veux faire bien ça permet un grande lisibilité et c'est beaucoup plus simple pour le repérage des erreurs
faire ce petit effort petit à petit te permettra en transparence dans l'avenir de le faire sans même y penser
ça te sera naturel, évident et agréable à la lecture. je t'invite donc à comparer et voir les différences.
/**
* @name requete
* @version 1.0
* @param $forme Formule explication
* @param $valeur Valeurs explication
* @param $table Tableau explication
* @param $action Action explication
* @param $ordre Ordre explication
* @return bool|\mysqli_result|void
*/
function requete($forme, $valeur, $table, $action, $ordre)
{
$host = "localhost";
$base = "agenda";
$login = "root";
$passe = "";
$link = mysqli_connect($host, $login, $passe, $base) or die("connection impossible $base");
$select_base = mysqli_select_db($link, $base);
$result = mysqli_query($link, "SELECT DATABASE()");
if ($forme == "select") {
$sql = "SELECT $valeur FROM $table $action $ordre";
}
if ($forme == "delete") {
$sql = "DELETE FROM $table $action";
}
if ($forme == "update") {
$sql = "UPDATE $table SET $valeur $action";
}
if ($forme == "insert") {
$sql = "INSERT INTO $table VALUES ($valeur)";
}
$result = mysqli_query($link, $sql) or die('Erreur : ' . mysqli_error($result));
return $result;
}
et dernier info quand tu as plus de 3 if sur une variable à tester
utilise switch , 3 ou moins de 3 if des elseif et else parfois pour la lisibilité des elseif seront bien mieux "parfois".
je t'ai mis le code avec les switch. pourquoi un switch ? c'est simple tu as la zone defaut donc tu peux gerer
le cas de figurer imprévue ou standar et les exceptions avec case.
et aussi on utilise pas les "" à tous vas. les double cote en PHP sont là pour contenir du texte qui peut contenir des variables donc il y a une préanalyse du contenu alors si on fait d'autre language comme java je sais c'est chiant. mais
en php le mieux est de privilégier les simple cote '' quand tu as du texte sans variable l'interpréteur ne perdra pas de temps à vérifier s'il y a un contenu à interpréter comme une variable. donc simple cote c'est une chaine de caractère pure et double cote une chaine de caractère exotique pouvant contenir des éléments à interpréter donc on évite mais plus bas tu utilise correctement les double cote avec : $sql = "SELECT $valeur FROM $table $action $ordre"; <<- gg
et ici $host = "localhost"; <<- c'est pas une erreur mais ce n'est pas nécessaire les doubles cote.
/**
* @name requete
* @param $forme Formule explication
* @param $valeur Valeurs explication
* @param $table Tabkeau explication
* @param $action Action explication
* @param $ordre Ordre explication
* @return bool|\mysqli_result|void
* @version 1.0
*/
function requete($forme, $valeur, $table, $action, $ordre)
{
$host = "localhost";
$base = "agenda";
$login = "root";
$passe = "";
$link = mysqli_connect($host, $login, $passe, $base) or die("connection impossible $base");
$select_base = mysqli_select_db($link, $base);
$result = mysqli_query($link, "SELECT DATABASE()");
switch ($forme) {
case 'select':
$sql = "SELECT $valeur FROM $table $action $ordre";
break;
case 'delete':
$sql = "DELETE FROM $table $action";
break;
case 'update':
$sql = "UPDATE $table SET $valeur $action";
break;
case 'insert':
$sql = "INSERT INTO $table VALUES ($valeur)";
default:
// si il y a un cas de figure inconnu
// on peut le gerer ici ou pas
}
$result = mysqli_query($link, $sql) or die('Erreur : ' . mysqli_error($result));
return $result;
}
mise à part c'est petits détails ton code est propre visuellement.
