amélioration du code / suggestion

xTG · 15-02-2010 15:02:19

1) ok pour les erreurs de connexion

2) ligne 18 : si une erreur survient, tu affiches à n'importe quel visiteur la requête et l'erreur ! Tu le renseignes donc très bien sur comment fonctionne ton script et il n'est guère difficile de tricher les requêtes par la suite. Si tu veux absolument afficher l'erreur et la requête fais en sorte que seuls les administrateurs de ton site les voient.

3) ligne 25 : si tu n'as rien dans ta base de donnée ta requête renvoie un tableau vide. Donc l'indice 0 n'existe pas et cela te générera un warning => utilisation de isSet sur $data[0] pour éviter le warning. C'est la même remarque pour la ligne 32.

david7 · 15-02-2010 17:15:59

ok j'ai remplacé la ligne 18 :

$req = mysql_query($sql) or die('Erreur SQL !<br />'.$sql.'<br />'.mysql_error());

PAR :

$req = mysql_query($sql) or die('Erreur SQL !');

c'est bon ?

sinon pour la 25/32 je cherche mais je réussi pas a comprendre comment faire.. tu pourrais me faire un exemple si sa te derange pas ?

xTG · 15-02-2010 18:21:53

2) ok

3)

if( isSet($data[0]) && $data[0] == 0 )

Pas plus compliqué que cela.

david7 · 15-02-2010 19:23:02

ok mais je rajoute le { a la fin ? comme ceci :

    // si on obtient une réponse, alors l'utilisateur est un membre

    if (isset($data[0]) && $data[0] == 0 ) {

      session_start();

      $_SESSION['login'] = $_POST['login'];

      header('Location: membre.php');

      exit();

    }

    // si on ne trouve aucune réponse, le visiteur s'est trompé soit dans son login, soit dans son mot de passe

    elseif (isset($data[0]) && $data[0] == 0 ) {

      $erreur = 'Compte non reconnu.';

    }

    // sinon, alors la, il y a un gros problème :)

    else {

      $erreur = 'Probème dans la base de données : plusieurs membres ont les mêmes identifiants de connexion.';

    }

xTG · 15-02-2010 19:51:40

session_start() est à mettre en début de script et non au milieu. Sinon il va te générer des erreurs.

PS : ta première comparaison est à faire avec la valeur 1 et non 0.

david7 · 15-02-2010 22:22:52

"session_start() est à mettre en début de script et non au milieu. Sinon il va te générer des erreurs."
en haut de la page ? ok

"PS : ta première comparaison est à faire avec la valeur 1 et non 0."
? hein ou ?

xTG · 15-02-2010 23:49:34

Relis ton code. Tu vérifies que la valeur soit 0 et dans ton else tu vérifies aussi qu'elle soit à 0.
Et cela parce que tu as fait un bête copier/coller de ma réponse. ^^

david7 · 16-02-2010 03:56:38

^^ je look a chaque fois mon code (meme t'es réponses je regarde bien) mais parfois je comprend pas trop les "termes" donc j'ai du mal mais sinon je suis pas du genre parresseu cher collaborateur je me donne du mal pour apprendre, je demande sur les forums, je cherche sur des sites a l'aide de google, etc...

mais sinon j'ai un script la sous la main, un espace membre avec biensur une page login.php j'ai regardé l'histoire du 0 dedans pour la connexion la etc... c'est pas la meme chose enfin c'est different non ? c'est plus propre non ? je prend lequel le mien ou celui la ? (je parle juste de cette portion) regarde :

<?php

if (isset($_POST["action"]) && $_POST["action"] == "send")

{

$login = postInput($_POST["login"]);

$password = postInput($_POST["password"]);

  $r = mysql_query("select idmbr from membres where login='$login' and password='".md5($password)."' and etat=1;");

  if (mysql_num_rows($r)==0)

  {

  $_GET["msg"] = "Veuillez vérifier vos informations";

  }

  else

  {

    $idmbr = mysql_result($r,0,"idmbr");

    $_SESSION["membre"]["idmbr"] = $idmbr;

    $_SESSION["membre"]["login"] = $login;

    $_SESSION["membre"]["password"] = $password;

    header("location: index.php");

    exit();

  }

}

else

{

$login = "";

$password = "";

}

?>

biensur je parle PAS du isset en haut ainsi que de l'erreur en GET j'ai pigé la leçon l'autre jour

moogli · 16-02-2010 04:01:53

yop,

je te conseil d'utiliser le tiens, car tu le comprendra surement mieux

@+

david7 · 16-02-2010 14:10:50

le mien.. le code juste en dessus avec les 31 lignes ?

Dernière modification par david7 (16-02-2010 14:13:09)

moogli · 17-02-2010 11:41:40

non l'idée est qu'un code fait par toi même te sera plus compréhensible, plus adapté et à mon avis mieux fait

@+

david7 · 17-02-2010 12:40:59

oui mais bizarrement.. je comprend mieu l'autre
il ma lair plus fuide, plus facile a lire d'un seul coup d'oeil toi t'en pense quoi niveau qualité du code je veux dire

PHP|Débutant :: Forums

Advertisement

#26 15-02-2010 15:02:19

Re : amélioration du code / suggestion

#27 15-02-2010 17:15:59

Re : amélioration du code / suggestion

#28 15-02-2010 18:21:53

Re : amélioration du code / suggestion

#29 15-02-2010 19:23:02

Re : amélioration du code / suggestion

#30 15-02-2010 19:51:40

Re : amélioration du code / suggestion

#31 15-02-2010 22:22:52

Re : amélioration du code / suggestion

#32 15-02-2010 23:49:34

Re : amélioration du code / suggestion

#33 16-02-2010 03:56:38

Re : amélioration du code / suggestion

#34 16-02-2010 04:01:53

Re : amélioration du code / suggestion

#35 16-02-2010 14:10:50

Re : amélioration du code / suggestion

#36 17-02-2010 11:41:40

Re : amélioration du code / suggestion

#37 17-02-2010 12:40:59

Re : amélioration du code / suggestion

Pied de page des forums