Probleme conception , ergonomie

Version imprimable

25/10/2009, 21h59
LEK

Probleme conception , ergonomie

Bonsoir,
j'ai une question concernant un petit problème d'ergonomie : je développe une webform présentant de multiple informations sur une entité "contact" (provient en base d'une table contact) : sur ma page j'ai un onglet "adresse(s)" (table adress_contact), un autre onglet "suivi" (table historique_contact),etc...
Lorsque l'utilisateur clique sur le bouton sauvegarder je dois d'abord enregistrer mon contact en base puis avec son identifiant renseigner les différentes infos provenant des onglets en base.... Là j'ai déjà un problème car cet identifiant m'est donné par une séquence oracle où je ne dois pas avoir de trou or, si un problème se produit en cours de route je dois annuler ma transaction...
Je voulais savoir comment en général vous traitiez ce genre de problème.

Merci de vos conseils,
Lek.
25/10/2009, 23h13
Graffito

Citation:

une séquence oracle où je ne dois pas avoir de trou

Mauvais plan, c'est à éviter absolument car :
- l'annulation d'une transaction ORACLE n'annule pas les éventuelles incrémentations du numéro de séquence,
- et, même si c'était le cas, tu pourrais avoir un problème si plusieurs nouveaux contacts sont traités en même temps.
26/10/2009, 01h30
tomlev
Une séquence Oracle n'est pas la bonne solution, pour la raison expliquée par Graffito.

Il vaudrait mieux gérer manuellement les numéros de séquence dans une table genre ça :
Code:

1 2 3 4 5 CREATE TABLE CHRONO ( CHRONO_ID VARCHAR2(30) NOT NULL PRIMARY KEY, CHRONO_VALUE NUMBER NOT NULL );
Quand tu veux récupérer la prochaine valeur du chrono "CUSTOMER_ID" par exemple, tu fais ça :
Code:

1 2 3 4 SELECT CHRONO_VALUE + 1 FROM CHRONO WHERE CHRONO_ID = 'CUSTOMER_ID' FOR UPDATE
La clause "FOR UPDATE" verrouille la ligne jusqu'à la fin de la transaction, donc tu peux tranquillement mettre à jour ta base sans t'inquiéter qu'un autre thread utilise le même ID (si un autre thread exécute la même requête avec "FOR UPDATE", son exécution sera bloquée jusqu'à ce que la ligne soit libérée)
En cas de problème => ROLLBACK, et ton chrono n'est pas modifié (contrairement à une séquence).

Par contre, n'oublie pas d'incrémenter le chrono avant de valider la transaction :
Code:

1 2 3 UPDATE CHRONO SET CHRONO_VALUE = CHRONO_VALUE + 1 WHERE CHRONO_ID = 'CUSTOMER_ID'
27/10/2009, 22h20
LEK

Merci pour votre aide, cela répond à mon problème pour la partie liaison avec la bdd... Mais côté asp.net, je voulais savoir comment vous géreriez ce type de problématique : dois-je par exemple implémenter une méthode save dans chacun de mes user control (par onglet) afin de leur fournir la séquence à utiliser pour qu'il puisse en contrôle responsable mettre à jour la base... Ou bien depuis ma page asp, je dois contrôler l'ensemble des opérations en effectuant de manière centralisé toute les opérations de sauvegarde : à ce moment les user control se contentent de rester de gentil conteneur opaque qui m'expose deux ou trois propriété utiles...

Merci d'avance pour vos réponses,
Lek.

Clairement, il vaudrait mieux gérer ça de manière centralisée dans une couche d'accès aux données (DAL), pour éviter de dupliquer inutilement du code.

La méthode que j'utiliserais pour gérer ces chronos reposerait sur un objet IDisposable, donc utilisable dans un bloc using pour être sûr de ne pas oublier d'incrémenter le chrono après usage :

(désolé c'est du C#, trop la flemme d'écrire du VB... mais ça se traduit facilement ;))

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
public class DbChrono : IDisposable
{
    private string _chronoID;
    private DbConnection _connection;
    private DbTransaction _transaction;
 
    public long Value { get; private set; }
 
    public DbChrono(string chronoID, DbConnection connection, DbTransaction transaction)
    {
        _chronoID = chronoID;
        _connection = connection;
        _transaction = transaction;
 
        GetChronoValue();
    }
 
    private void GetChronoValue()
    {
        using (DbCommand command = _connection.CreateCommand())
        {
            command.CommandText =  @"
                SELECT CHRONO_VALUE + 1
                FROM CHRONO
                WHERE CHRONO_ID = :CHRONO_ID
                FOR UPDATE";
 
            DbParameter parameter = command.CreateParameter;
            parameter.ParameterName = "CHRONO_ID";
            parameter.DbType = DbType.String;
            parameter.Value = _chronoID;
 
            command.Transaction = _transaction;
 
            this.Value = Convert.ToInt64(command.ExecuteScalar());
        }
    }
 
    public void Dispose()
    {
        UpdateChronoValue();
    }
 
    private void UpdateChronoValue()
    {
        using (DbCommand command = _connection.CreateCommand())
        {
            command.CommandText =  @"
                UPDATE CHRONO
                SET CHRONO_VALUE = CHRONO_VALUE + 1
                WHERE CHRONO_ID = :CHRONO_ID";
 
            DbParameter parameter = command.CreateParameter;
            parameter.ParameterName = "CHRONO_ID";
            parameter.DbType = DbType.String;
            parameter.Value = _chronoID;
 
            command.Transaction = _transaction;
 
            command.ExecuteNonQuery();
        }
    }
}

Et pour l'utiliser (en supposant que tu aies déjà une connexion ouverte et une transaction en cours) :

Code:

1
2
3
4
5
6
7
8
using (DbChrono chrono = new DbChrono("CUSTOMER_ID", connection, transaction))
{
 
     long customerID = chrono.Value;
     // Etc...
 
 
} // Le chrono est automatiquement mis à jour ici

J'ai écrit ça à la volée, donc c'est évidemment pas testé, et je décline toute responsabilité en cas de dysfonctionnement :aie:

28/10/2009, 19h49
LEK

OK, merci pour le code :

Citation:

(désolé c'est du C#, trop la flemme d'écrire du VB... mais ça se traduit facilement )

Pas de problème, j'écris aussi en C#, je n'avais tout simplement pas fais gaffe que j'étais dans le forum VB.
Merci encore pour votre implication ;-)