Discussion :

[alexking2005]

Bonjour,

j'ai une appli C# qui se connecte à une bdd MySQL.
Dans mon appli, j'ai une liste qui affiche des salles de conférence. Quand je clique sur une salle, une autre liste se charge avec les disponibilités de cette salle (par exemple lundi 8h00, lundi 14h00, lundi 16h00, mardi 8h00...).
Je clique alors sur la disponibilité qui me convient et je clique sur "Réserver". TOUT FONCTIONNE!!!!

Seulement mes utilisateurs n'utilisent pas l'appli comme je le pensais. En fait, ils sélectionnent la salle, la liste des dispo est donc affichée. Puis ils restent sur cette fenêtre quelques secondes ou minutes, puis seulement clique sur "Réserver". Seulement entre temps une autre utilisateur a peut être réservé la salle et cette dispo. Je mets à jour 2 tables pour la réservation.

J'ai voulu utilisé SELECT FOR UPDATE mais j'ai pas l'impression que ce soit fait pour moi. J'ai fait une procédure pour la mise à jour des 2 tables, car parfois, je mettais à jour une table et pas l'autre car ça plantait au milieu. Mais ça ne règle pas ce problème d'accès concurrent. Que me conseillez vous? J'imaginais vérifier sur la dispo est toujours dispo avant de mettre à jour.

Par avance, merci.

[DotNetMatt]

Il faut que tu vérifies que la disponibilité soit toujours valable avant de vouloir enregistrer la réservation. Tu peux faire ça avec une simple requête SELECT. Ensuite tu auras deux scénarios à gérer :

1. Soit la dispo est toujours présente, donc tu enregistres la réservation
2. Soit elle n'est plus valide, et tu affiches un message à l'utilisateur pour lui indiquer qu'il doit en choisir une autre.

[alexking2005]

C'est bien ce que j'imaginais. Merci.

[Graffito]

Pour traiter techniquement le probleme, il suffit de mettre dans la requete UPDATE une condition WHERE sur le champ indiquant la disponibilité.
Si aucun enregistrement n'a été modifié (Count retourné par la requête egal à 0), tu affiches un message à l'utilisateur.

[Pol63]

et idéalement rafraichir la fenetre vu qu'elle reste ouverte longtemps (soit faire un refresh toutes les secondes ou faire un système de notification de mise à jour)

[meziantou]

Il faut que tu vérifies que la disponibilité soit toujours valable avant de vouloir enregistrer la réservation. Tu peux faire ça avec une simple requête SELECT. Ensuite tu auras deux scénarios à gérer :

1. Soit la dispo est toujours présente, donc tu enregistres la réservation
2. Soit elle n'est plus valide, et tu affiches un message à l'utilisateur pour lui indiquer qu'il doit en choisir une autre.

Il faut faire cela dans une transaction, sinon tu ne résouds pas le problème des réservations concurrentes.

La méthode proposée par Graffito est plus simple et comme l'a dit Pol63 l'utilisateur appreciera que la fenêtre se rafraichisse.

[Pol63]

avec la méthode de faire un select pour voir si c'est toujours dispo, et si c'est le cas faire un update dans la foulée est potentiellement une source de bug
car entre le select et l'update il y a pu avoir un update (enfin un autre select suivi d'un update vu que le code sera le même)
avec une transaction ca va laisser un verrou entre les 2 interdisant quelqu'un d'autre de faire un update pendant ce temps là (bien qu'il faille vérifier qu'un select dans une transaction laisse son verrou)

avec un update where not déjà pris il n'y a pas de problème, les update posent des verrous exclusifs empechant un autre update simultanné et quand le 1er sort pour le 2ème la condition ne sera plus vraie

[DotNetMatt]

Ah oui effectivement, au temps pour moi.. A force de faire 36 choses à la fois je suis passé un peu à côté du sujet

Merci Pol63 et meziantou pour m'avoir fait remarquer ce point.