scvo0ne | Un début de solution :
Code :
- switch ($_POST['tonChampSelect']) {
- case 1:
- maPremiereFonction();
- break;
- case 2:
- maSecondeFonction();
- break;
- }
- function maPremiereFonction() {
- // code la premiere fonction
- }
- function maSecondeFonction() {
- // code la seconde fonction
- }
|
Sinon ton code est effectivement cradingue... tu m'étonnes que les codeurs PHP ont mauvaise réputation
Code :
- $test=($_POST['NameAppl']); -> t'as pas besoin des parenthèses
- NameAppl, Type, MDP -> t'aurais pas pu faire moins cohérent ? Soit tu mets des majuscules, soit des minuscules, soit du camelCase, mais faut rester cohérent. Ou aimer le débogage
- $test01=($_POST['Type']);
- echo $test01; -> Tu déclare une variable juste pour l'afficher - tu t'en ressert pas après.
- echo $_POST['Type']; fait la même chose en 1 ligne
- Encore mieux, tes 5 lignes tu peux les résumer comme ça :
- printf("NameAppl %s - Type %s - MDP %s , $_POST['NameAppl'], $_POST['Type'], $_POST['MDP']);
- Ou encore plus simplement avec print_r($_POST);
- $Insert="Insert into opi values ('66','".$_POST['NameAppl']."', '".$_POST['Type']."', '".$_POST['MDP']."')";
- En général, on met les mots clé SQL en majuscules. C'est plus lisible
- mysqli_query($conn, $Insert); -> Injection sql, requêtes préparées ...
|
|