Discussion :

[aspkiddy]

Bonjour,

Mon application Web donne un code de promo, par courriel, aux utilisateurs qui remplissent le formulaire.

Ce code est unique, utilisable une seule fois par l'utilisateur. Il est stocké dans une base de données (BDD).

Chaque fois qu'un utilisateur remplit le formulaire au 1er écran, l'application connecte à la bdd, et obtient la valeur du 1er champ, "cd_promo", du PREMIER ENGREGISTREMENT/ PREMIERE LIGNE de la table "client".

Code :

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

string mySqlReq = "SELECT cd_promo FROM client LIMIT 1";

Une fois que le code a été envoyé, par courriel, à l'utilisateur, le programme supprime le code dans la BDD, dans le 2e écran.

Code :

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

string mySqlReqSupr = "DELETE FROM  client WHERE cd_promo=@CodeEnvoye;";

Du coup, 2e utilisateur peut obtenir un autre code.

Ça marche très bien. Par contre, j'ai une idée qui peut causer beaucoup de problèmes*:

Mon problème est le suivant*: si par hasard, en même temps, 2 utilisateurs remplissent le formulaire au 1er écran, j'imagine qu'ils vont évidemment obtenir, tous les deux, le même code [QUI SE TROUVE DANS PREMIER ENREGISTREMENT] !!! Il faut éviter cela*:

Autrement dit, il ne faut pas qu'ils reçoivent le même code qui se trouve dans 1er enregistrement.

En plus de cela, j'imagine une autre chose qui irait se produire au 2e écran*: 1er utilisateur clique sur le bouton, l'email part dans sa boite de courriel et ce code est supprimé par la requête. Ensuite, 2er utilisateur clique sur le bouton, l'email part avec le même code de promo dans son boite de courriel et le code n'excite plus (puisque le 1er l'a supprimé avant lui) alors l'application peut afficher un message d'erreur.


Alors que je dois faire pour que ma requête (mySqlReq) ne donne pas la même valeur s'il y a plusieurs utilisateurs remplissent en même temps le formulaire. En plus, ils ont évidemment le droit de remplir le formulaire en même temps, et nous n'avons pas le droit de les empêcher.

Merci et bonne journée

[Graffito]

Une solution peut-être envisageable pour SQL Server serait d'ajouter un champ CDFLAG à la table (champ valant 0 lorsque le code est dispo et 1 lorsqu'il est vérouillé) et de de faire un ExecuteQuery avec :
"UPDATE client SET CDFLAG=1 WHERE CDFLAG=0 LIMIT 1 OUTPUT cd_Promo"

[aspkiddy]

Bonjour Graffito,

Est-ce que votre exemple marche avec MySql server ?
Si oui, comment je peux faire un champ CDFLAG dans MySql ?

J'ai regardé dans ma table, il n'y pas de de type 'CDFLAG'. Il y a possibilité de 39 types, mais pas CDFLAG :

voici les posibilités de type : TINYINT ; SMALLINT ; MEDIUMINT ; INT ; BIGINT ; DECIMAL ; FLOAT ; DOUBLE ; REAL ; BIT ; BOOL ; SERIAL ; DATE ; DATETIME ; TIMESTAMP ; TIME ; YEAR ; CHAR ; VARCHAR ; TINYTEXT ; TEXT ; MEDIUMTEXT ; LONGTEXT ; BINARY ; VARBINARY ; TINYBLOB ; MEDIUMBLOB ; BLOB ; LONGBLOB ; ENUM ; SET ; GEOMETRY ; POINT ; LINESTRING ; POLYGON ; MULTIPOINT ; MULTILINESTRING ; MULTIPOLYGON et EOMETRYCOLLECTION


Je pense que j'ai trouvé la solution :
faire une requête sql avec l’ordre aléatoire via Order by Rand ()
Donc s'il y a plusieurs consultations de formulaire en même temps par différents utilisateurs, le risque d'avoir plusieurs mêmes codes est très minimum...

Code :

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

string sqlBDD = "SELECT cd_promo FROM client order by rand() LIMIT 1";

Qu'en vous dites sur cette solution?

[ctxnop]

Salut,
C'est à ça que servent les transactions.

[Graffito]

CDFLAG était juste un nom pour un nouveau champ de type de INT (ou SMALLINT ou TINYINT) à ajouter à la table.

"OUTPUT" n'existe pas en MySql.

Les solutions seront donc plus complexes; En voici une évitant de recourir aux transactions :
- on fait un SELECT ... LIMIT 1
- on récupère ainsi un code promo et on l'inverse,
- on fait un UPDATE ... SET codePromo=Code_promo_inversé WHERE codePromo=CodePromo_récupéré
- on teste le retour de l'update pour voir le nombre d'enregistrements mis à jour,
- si ce nombre est nul, on reboucle sur le SELECT.

Qu'en vous dites sur cette solution?

Non, noon, noooon
On ne peut pas accepter une bug identifiée quelque soit son degré de probabilité. Si on tolère de telles dérives à tous les niveaux d'un projet, on va additioner les probabilités, et au final bonjour les dégats ...

[aspkiddy]

CDFLAG était juste un nom pour un nouveau champ de type de INT (ou SMALLINT ou TINYINT) à ajouter à la table.

"OUTPUT" n'existe pas en MySql.

Les solutions seront donc plus complexes; En voici une évitant de recourir aux transactions :
- on fait un SELECT ... LIMIT 1
- on récupère ainsi un code promo et on l'inverse,
- on fait un UPDATE ... SET codePromo=Code_promo_inversé WHERE codePromo=CodePromo_récupéré
- on teste le retour de l'update pour voir le nombre d'enregistrements mis à jour,
- si ce nombre est nul, on reboucle sur le SELECT.

Merci pour vos explications, Graffito
--- Alors j'ai ajouter une autre champs "cd_promo_vide" [ varchar(5) DEFAULT NULL]

--- Je fait ma requête

Code :

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

string sqlBDD = "SELECT cd_promo FROM client WHERE "cd_promo_vide"  IS NULL order by rand() LIMIT 1;

Donc j'ai choisi un engresistrement par hasard du champ cd_promo et le champ cd_promo_vide est vide

--- En fait, il faut faire une changement...

Code :

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

UPDATE client SET code_test_vide = cd_promo WHERE  cd_promo = vChiffre;

ou bien :

Code :

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

UPDATE client SET code_test_vide = vChiffre WHERE  cd_promo = vChiffre;

ou bien mieux encore :

Code :

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

UPDATE client SET code_test_vide = '1' WHERE  cd_promo = vChiffre;

N'est ce pas ?
Par contre, je ne sais pas comment je peux faire 2 requêtes en même temps dans une seule connexion avec une variable locale "vChiffre" ?
Il faut 2 connexions ?
Voici le code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
string mySqlReq = "SELECT cd_promo FROM client WHERE cd_promo_vide IS NULL  order by rand() LIMIT 1";
 
MySqlCommand cmdCDpomo = new MySqlCommand(mySqlReq, connCDpomo);
 
MySqlDataReader myReaderCDpomo;
 
myReaderCDpomo = cmdCDpomo.ExecuteReader();
string vChiffre;
 // Always call Read before accessing data.
while (myReaderCDpomo.Read()) 
	{
 
	vChiffre = myReaderCDpomo.GetString(0);
 
	ecranApageState.CodePromo = vChiffre;
 
	// Faut il faire la 2e requête ici pour récupérer la valeur de vChiffre?
	// il faut fermer la connections? donc il faut que je sorte de ma boucle while mais dans ce cas là, je ne peux pas récupérer la valeur de vChiffre
	}
 
// always call Close when done reading.
myReaderCDpomo.Close();
// Close the connection when done with it.
connCDpomo.Close();

Bonne soirée

[aspkiddy]

Salut,
C'est à ça que servent les transactions.

C'est très intéressant, cet article...

par contre, on ne peut pas appliquer, si je ne m'en trompe pas, dans mon cas puisqu'il s'agit d'une action qui commence dans un écran et qui se termine dans un autre écran*:

Code :

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

1
2
3
4
START TRANSACTION;
SELECT @A:=SUM(salary) FROM table1 WHERE type=1;
UPDATE table2 SET summmary=@A WHERE type=1;
COMMIT;

Merci pour cet article,ctxnop.

[Bluedeep]

Je pense que j'ai trouvé la solution :
faire une requête sql avec l’ordre aléatoire via Order by Rand ()
Donc s'il y a plusieurs consultations de formulaire en même temps par différents utilisateurs, le risque d'avoir plusieurs mêmes codes est très minimum...

Code :

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

string sqlBDD = "SELECT cd_promo FROM client order by rand() LIMIT 1";

Qu'en vous dites sur cette solution?

Comme dit plus haut, tu ajoute un champs CDFLAG (ou tu l'appelle comme tu veux), que tu mets à jour dans une transaction et tu retourne le numéro mis à jour après le COMMIT; ainsi tu es sur de l'absence de collision.