Discussion :

[moimp]

Bonjour,
Lorsqu'un utilisateur demande son inscription comme membre, je l'enregistre provisoirement dans une table dat_users_query. Lorsqu'il valide son inscription, je l'enregistre dans dat_users et j'efface la ligne correspondante dans dat_users_query.
Lors de sa demande d'inscription, je vérifie que son pseudo et son adresse mail n'existent dans aucune des deux tables avec cette fonction... qui ne marche pas:

Code php :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
function checkIfDataExists($col, $value) {
	global $db;
	var_dump($col, $value);
 
	$query = "
		SELECT COUNT(*) AS counter FROM dat_users WHERE $col=:value 
		UNION ALL 
		SELECT COUNT(*) AS counter FROM dat_users_query WHERE $col=:value
	";
	$result=$db->prepare($query);
	$result->bindValue(':value', $value);
	$result->execute();
	return $result->fetchColumn();
}

[Toufik83]

Salut,
as-tu essayé de mettre des apostrophes autour de la variable $col ?
comme

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

WHERE '$col'=:value

[moimp]

Sur ton indication, j'ai essayé mais j'ai déjà mis des variables pour les noms de colonnes ou de tables et ça marche très bien.
Je suis presque sûr que c'est la requête qui n'est pas bonne.

[Toufik83]

Quand tu exécute la requête dans phpmyadmin, ça te renvois bien quelque chose ?

sinon essaie :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
 
$query = "
		SELECT COUNT(*) AS counter FROM dat_users WHERE :colonne=:value 
		UNION ALL 
		SELECT COUNT(*) AS counter FROM dat_users_query WHERE :colonne=:value
	";
	$result=$db->prepare($query);
	$result->bindValue(':value', $value);
        $result->bindValue(':colonne',$col);
	$result->execute();

[badaze]

"Ca ne marche pas" ne veut rien dire. Il faut toujours préciser ce qui te fait dire que ça ne marche pas.

[CosmoKnacki]

Personnellement, j'utiliserai une seule table agrémentée d'une colonne avec la date d'inscription et d'une colonne de type boolean pour signifier si le compte est confirmé ou non: de cette manière il est facile d'automatiser la suppression des comptes non confirmés au delà d'une certaine période.

J'ajouterai une contrainte d'unicité sur la colonne pseudo, ainsi je n'aurai qu'à me fier à l'erreur émise par MySQL (erreur 1062) lors de la tentative d'insert, pour connaître le problème et afficher le message adéquate.

[badaze]

Personnellement je ne ferais pas comme CosmoKnacki. Les contrôles doivent être séparés de la mise à jour des données.

[CosmoKnacki]

Pour contrôler les données provenant de l'extérieur, je suis d'accord, mais là tout ce que tu fais c'est une requête inutile la plupart du temps pour vérifier que le pseudo n'existe pas déjà. Donc autant y aller carrément et tenter l'insert, tu économiseras un requête.

[moimp]

Je vais essayé de réunir les deux tables en une seule selon les indications de CosmoKnacki. En ce qui concerne les doublons, je pense que, côté client, il est bon d'indiquer à l'internaute si le pseudo est déjà utilisé avant de l'enregistrer ce qui n'empêche pas de faire le contrôle côté serveur.
Je marque provisoirement le sujet comme résolu. Je le rouvrirai si nécessaire.

[Celira]

Pour information : cette requête ne fonctionnera pas :

sinon essaie :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 
$query = "
		SELECT COUNT(*) AS counter FROM dat_users WHERE :colonne=:value 
		UNION ALL 
		SELECT COUNT(*) AS counter FROM dat_users_query WHERE :colonne=:value
	";

On ne peut pas paramétrer les noms des tables et des colonnes, car PDO va échapper les valeurs, ce qui donnera :

Code sql :

Sélectionner tout - Visualiser dans une fenêtre à part

SELECT COUNT(*) AS counter FROM dat_users WHERE 'nom_colonne'='valeur'

au lieu de :

Code sql :

Sélectionner tout - Visualiser dans une fenêtre à part

SELECT COUNT(*) AS counter FROM dat_users WHERE nom_colonne='valeur'

[Toufik83]

oui Celira, tu as raison.(+1)
j'ai été trop vite dans ma réponse oui , en plus j'avais le même problème auparavant et j'ai finis par comprendre qu'on ne peut pas passer les noms des tables et les colonnes en tant que paramètres.