Validation par email

audi68 Messages postés 567 Date d'inscription   Statut Membre Dernière intervention   -  
audi68 Messages postés 567 Date d'inscription   Statut Membre Dernière intervention   -
Bonjour a tous,
je viens de faire une activation par email sur mon espace membre

cela fonctionne mais j aimerais avoir votre avis sur mon code le voici :

<?
require("conf.php3");

// CONNEXION MYSQL
$db_link = @mysql_connect($sql_serveur,$sql_user,$sql_passwd);
if(!$db_link) {echo "Connexion impossible à la base de données <b>$sql_bdd</b> sur le serveur <b>$sql_server</b><br>Vérifiez les paramètres du fichier conf.php3"; exit;}

// SELECTION DE L'ENREGISTREMENT CONTENANT L'ID EN COURS
$requete=mysql_db_query($sql_bdd,"select * from annuweb_membre where id=\"$id\"",$db_link) or die(mysql_error());

// Modification de la colonne compte acitve en 1

$sql ='UPDATE annuweb_membre SET Compte_Active = "1" WHERE id="'.$id.'"';
mysql_query($sql) or die('Erreur SQL !'.$sql.'<br />'.mysql_error());
// SI L'ID N'EXISTE PAS
if(mysql_num_rows($requete)==0)
{
// REDIRECTION PAGE ERREUR
header("Location:$url_erreur");
exit;
}

// DECONNEXION MYSQL
mysql_close($db_link);

echo "<meta http-equiv='refresh' content='2;URL=membres.php3?id=$id'>"; ?>

Merci pour vos infos
Configuration: Windows Vista
Internet Explorer 7.0

4 réponses

  1. gaerebut Messages postés 1060 Statut Membre 171
     
    Salut,

    déjà à la première lecture, on voit que tu n'utilise pas de classe donc pas d'objet.
    Je te conseille de te mettre à PDO.
    Cela remplace mysql_connect et tout le bazard.
    De plus, je te conseil d'utiliser des try Catch pour générer des exception en cas d'erreur (log ...) et ne pas afficher une belle erreur PHP à l'utilisateur si c'est le cas :p ( ==> Donc erreur personnalisée).

    Si tu veux, je peux t'aider en t'expliquant commentçamarche avec PDO et le class. lol

    C'est toi qui vois !

    A+

    Gaerebut
    0
  2. audi68 Messages postés 567 Date d'inscription   Statut Membre Dernière intervention   9
     
    Merci de ta reponse gaerebut,

    Oui je suis preneur de nouvel explication

    je suis la pour apprendre alors oui dit moi ce qui n est pas bon ou plus d actualité

    Merci de ton aide
    0
  3. gaerebut Messages postés 1060 Statut Membre 171
     
    Ok mais demain là je me couche (au canada il est 2h40 du mat :p)
    Fais des recherche en attendant si t'es motivé ! lol

    A+

    Gaerebut
    0
  4. audi68 Messages postés 567 Date d'inscription   Statut Membre Dernière intervention   9
     
    ok pas de souci
    bonne nuit alors lol je pense bientot fini lol
    je vais faire des recherhces apres
    0