Validation par email

audi68 Messages postés 626 Statut Membre -  
audi68 Messages postés 626 Statut Membre -
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

4 réponses

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
audi68 Messages postés 626 Statut Membre 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
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
audi68 Messages postés 626 Statut Membre 9
 
ok pas de souci
bonne nuit alors lol je pense bientot fini lol
je vais faire des recherhces apres
0