SCRIPT CONNEXION AJAX

Commentaire de JejeScript le 02/03/2013 12:38:41

Bonjour bebert2310,

Cette source fonctionne, mais et loin d'être idéal pour les débutants, pas de commentaires, pas de structures.

Utilise empty pour vérifier tes GET, car isset retourne une variable existante même si elle est vide.
Les structures de contrôle === sont mieux que == pour faire des comparatifs.

peut-être penser à faire des classes et des fonctions qui rendrais l'utilisation plus simple.

Cordialement,

JéjéScript.

Commentaire de nashouille le 04/03/2013 07:37:52

bonjour,

c'est quoi cette horreur, c'est la transcription sans réflexion d'une autoformation d'un mauvais livre

Commentaire de emilia123 le 05/03/2013 08:33:18

Bonjour,

Malgré le faible nombre de lignes il y a des problèmes.

1) IL NE FAUT PAS utiliser la comparaison simple "==" car cela réserve des surprises.

Tu peux essayer ce petit code :
------------------------------
$monCodeUltraSecret="mdp";
$variableUtilisateur=0;
if ($monCodeUltraSecret==$variableUtilisateur){
  echo "les accès de mon compte en banque sont : 123-456-789-0";
}
------------------------------
et tu auras une belle surprise.... oui oui 0(zéro) est bien égal à une chaine de texte pour PHP avec une comparaison simple.

Il faut donc TOUJOURS utiliser la comparaison strict "===".


2) Ensuite, tu utilises une variable intermédiaire entre ta variable en POST et ta comparaison, mais cela n'a aucune utilité, sauf une occupation inutile de la mémoire... autant faire la comparaison directement avec la variable "POST".


3) tu pourrais utiliser une classe PHP pour regrouper tes fonctionnalités.
Ta classe pourrait effectuer la vérification de la connexion et renvoyer TRUE ou FALSE. Ensuite le reste du code PHP se chargerait d'afficher OK ou FALSE (pourquoi pas TRUE/FALSE) en fonction de retour de cette classe.
Cela permet d'être plus propre et c'est plus simple si tu dois adapter pour retourner autre chose que OK/FALSE dans un cas précis.

Bonne continuation.

EM.