Discussion :

[magiczouf]

Bonjour, je débute c# .net et objet

Voilà mon problème :

J'ai un objet. La valeur d'une propriété de cet objet est calculée en fonction de la valeur de 2 autres. (un peu comme si vous aviez une trigramme en fonction du nom et prenom par ex.)

J'ai aussi une classe dao instancie l'objet et fait la relation avec le dataset fortement typé.

enfin j'ai l'écran en lui même qui affiche les données (gridview, detailview etc .)


Je me pose la question de la meilleure façon de faire, lorsque je crée un nouvel enregistrement :

1) Faire le calcul dans le Get de la propriété
2) le faire dans l'update / insert de la classe Dao (ou même avant)

Pour l'instant j'utilise le Get. Est-ce la meilleure méthode ?

Merci

[Elrilmathiel]

Si la valeur de ta propriété résulte toujours du même calcul, il vaut mieux le faire dans le Get, c'est beaucoup moins lourd

[tomlev]

Si la valeur de ta propriété résulte toujours du même calcul, il vaut mieux le faire dans le Get, c'est beaucoup moins lourd

Ben non, justement c'est plus lourd : ça fait que le calcul est refait à chaque appel de la propriété. Il vaut mieux le faire dans le set des 2 autres propriétés, comme ça elle est recalculée seulement quand les valeurs des propriétés dont elle dépend sont mises à jour

[magiczouf]

Ok! Merci pour l'info . je n'avais pas pensé à ça.

[Elrilmathiel]

Ben non, justement c'est plus lourd : ça fait que le calcul est refait à chaque appel de la propriété. Il vaut mieux le faire dans le set des 2 autres propriétés, comme ça elle est recalculée seulement quand les valeurs des propriétés dont elle dépend sont mises à jour

Je viens de me rendre compte que j'avais mal compris sa seconde proposition.
J'ai cru qu'il voulait faire un acces chaque fois qu'il voulait calculer la valeur de sa propriété

Ca m'apprendra, je lirais moins vite la prochaine fois, et désolé d'avoir raconté des betises du coup

[magiczouf]

ça reste quand même une modification de la propriété :

- au chargement du gridview
- à l'ouverture du detail view
- lors du passage en mode édition ()
- lors de l'enregistrement

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
 
        public string a       
      {
            get { return _a; }
            set {
                resultat =  calcul de b en fonction de value
                _b =  resultat du calcul;
               _a = value; 
               }
        }
 
        public string b   
        {
            get { return _b; }
            set { _b  = value; }
        }

Je dois oublier quelquechose. Je comparerais bien value à l'ancienne valeur, mais je ne sais pas ou la trouver .

Merci pour votre aide

[tomlev]

Je comparerais bien value à l'ancienne valeur, mais je ne sais pas ou la trouver .

ben c'est _a...

[Elrilmathiel]

Au lieu de

Code :

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

1
2
3
4
5
6
 
public string b   
        {
            get { return _b; }
            set { _b  = value; }
        }

tu peux déjà écrire

Code :

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

public string b { get; set; }

le compilo fait le reste tout seul.

Et pour ton ancienne valeur, c'est tout simplement _a

Edit : pris de vitesse par tomlev

[Guulh]

Ben non, justement c'est plus lourd : ça fait que le calcul est refait à chaque appel de la propriété. Il vaut mieux le faire dans le set des 2 autres propriétés, comme ça elle est recalculée seulement quand les valeurs des propriétés dont elle dépend sont mises à jour

Ca dépend de la lourdeur du calcul.

Code :

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

1
2
3
4
5
6
class Ghjdvgjkfhgk
{
  public int A {get; set; }
  public int B {get; set; }
  public int APlusB {get { return A + B; } }
}

Dans ce cas, certes la somme sera faite à chaque appel, mais du coup la classe est plus légère.

On retombe dans un bon vieux cas d'optimisation du nombre d'instructions versus optimisation de l'empreinte mémoire Et encore, uniquement dans le cas où les perfs sont critiques, parce que sinon les deux solutions se valent, mais j'ai un faible pour celle que je viens d'écrire parce qu'elle est bien plus concise.