Discussion :

[icexplorer]

Bonjour,

je souhaiterais avoir votre avis sur le problème suivant :
je suis entrain de créer un site contenant une liste de produits que les internautes peuvent noter. Une note moyenne est calculée pour chaque produit, à partir de l'ensemble des notes des internautes.

Question : à quel moment faut-il que je calcule cette moyenne ? Au moment où l'internaute met sa note, ou via un batch qui tournerait toutes les nuits par exemple ?
Le problème est que si je calcule la moyenne au moment de l'enregistrement de la note, il y a le risque que 2 internautes notent au même moment, et que le calcul soit erroné.

J'ai déjà réfléchi aux solutions suivantes, pouvez-vous me dire quelle est la plus adaptée :
- mettre le mot clé "synchronized" sur la méthode qui sert à calculer la moyenne, afin d'éviter les accès simultanés.
- créer un batch (comme dit précédemment) qui va calculer les moyennes toutes les nuits.
- au moment d'enregistrer la note, faire un traitement asynchrone pour calculer la moyenne en arrière-plan (en mettant toujours un "synchronized")
- mettre des locks en base de données

Quelle est selon vous la meilleure solution ?

Merci beaucoup !

[dinobogan]

Dans l'hypothèse ou les notes et la moyenne sont stockées en base de données et que ta base de données accepte les transactions, alors aucune hésitation : tu ouvres une transaction, tu ajoutes la nouvelle note et tu calcules la nouvelle moyenne puis tu commit la transaction.
Le batch est une mauvaise idée car il ne résout pas le problème d'accès concurrent si deux nouvelles notes sont ajoutées en même temps. De plus, c'est du code à maintenir dans un traitement distinct. Enfin, il n'y a plus l'aspect "affichage immédiat suite à action de l'utilisateur", ça pourrait être déroutant.

[icexplorer]

Merci pour ta réponse
Toutefois les transactions ne s'appliquent pas à mon problème pour ce cas précis. Admettons que j'ai la table "produit" et "note" comme ceci :

Code :

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

1
2
produit : idProduit, nomProduit, noteMoyenneProduit
note : idNote, idInternaute, idProduit, noteProduit

(bon évidemment ça sera un peu plus complexe, d'ailleurs pour recalculer la moyenne il me faudra la somme des notes et leur nombre total pour pouvoir faire somme/nombre = moyenne)

L'internaute met donc sa note, qui elle apparaîtra tout de suite dans la liste des notes du produit (en dessous du produit on affiche la note attribuée par chaque internaute). Donc jusque-là pas de problème.

En revanche, il faut calculer la moyenne à partir de toutes les notes. Pour le calcul pas de soucis, je récupère le produit, je recalcule la moyenne et je le ré-enregistre. Donc le problème est le suivant :
si je recalcule la moyenne du produit au moment où l'internaute met sa note, il faut que je fasse le traitement en 2 temps : 1) j'enregistre la note dans la table "note" 2) je recalcule la moyenne et je l'enregistre dans la table "produit". Mais si 2 internautes enregistrent leur note au même moment (au millième de seconde près) le recalcul de la moyenne ne sera pas correct puisqu'il y a le risque que l'une des 2 notes ne soit pas prise en compte dans le calcul (accès concurrent).


Je pense que le problème est un peu plus clair ainsi ?

[dinobogan]

La transaction s'applique tout à fait à ton problème et est faite pour ça :

1. ouvrir transaction
2. placer la nouvelle note dans la table "note"
3. calculer la nouvelle moyenne et l'enregistrer dans la table "produit"
4. commit de transaction
5. afficher la note de l'utilisateur
6. afficher la moyenne

[icexplorer]

Je ne comprends pas en quoi une transaction peut m'aider, étant donné que plusieurs utilisateurs peuvent ouvrir des transactions simultanément et faire des commit en base de données simultanément également.

A moins qu'il y ait quelque chose qui m'échappe

[dinobogan]

A moins qu'il y ait quelque chose qui m'échappe

Une transaction est une action atomique du point de vue extérieur à la base de données, peu importe le nombre d'actions réalisées dans cette transaction. La base de données assure de conserver l'intégrité technique (et fonctionnelle si ton programme est correcte) de tes données.
Si tu effectues le calcul de moyenne comme expliqué précédemment, tu seras assuré d'avoir une moyenne toujours correcte et des données en base correctes également.
Je t'encourage vivement à lire de la documentation sur les transactions appliquées à la base de données que tu utilises

[icexplorer]

J'ai commencé à lire ce document sur Hibernate :
http://docs.jboss.org/hibernate/core...nsactions.html
qui explique notamment comment gérer les accès concurrents au niveau des transactions.

Donc maintenant j'ai une meilleure idée sur la solution à adopter.
Passage à résolu