Discussion :

[Rayanea]

Bonjour,

Les 2 méthodes ci-dessous font la même chose, mais j'ai du en créer 2 car j'ai 2 appels de fonctions different : Func<int, double> et Func<int, DateTime, double>.

Existe-t-il une manière de réécrire cela de manière à ne garder plus qu'une seule procédure ?

Merci beaucoup.

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
        public static double SumItem(Func<int, double> method, int startPeriodIncluded, int endPeriodIncluded)
        {
            double sumItem = 0;
            for (int j = startPeriodIncluded; j < endPeriodIncluded + 1; j++)
            {
                sumItem += method(j);
            }
            return sumItem;
        }
 
        public static double SumItem(Func<int, DateTime, double> method, int startPeriodIncluded, int endPeriodIncluded, DateTime refinancingDate)
        {
            double sumItem = 0;
            for (int j = startPeriodIncluded; j < endPeriodIncluded + 1; j++)
            {
                sumItem += method(j, refinancingDate);
            }
            return sumItem;
        }

[popo]

Tout dépend de ce que font les méthodes passées en paramètre.
Mais, il est bizarre de voir une méthode statique attendre un pointeur de méthode.

[Rayanea]

Bonjour,

Vous trouverez ci-dessous les méthodes passées en paramètres. Ce sont de simples calculs de dates. Oui, j'apprends sur le tas et j'ai bien conscience que mon code doit contenir beaucoup de choses peu orthodoxes...

Bien à vous,

Code :

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

1
2
3
4
5
6
7
8
9
public virtual DateTime PeriodDate(int p)
        {
            return DateTimeUtils.PeriodDate(this, p);
        }
 
public virtual DateTime PeriodDate(int p, DateTime refinancingDate)
        {
           return DateTimeUtils.PeriodDate(this, p, refinancingDate);
        }

Elles appellent :

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
25
26
27
28
29
30
static public DateTime PeriodDate(FinancialInstrument financialInstrument, int p)
        {
            DateTime issueDate = DateOnly(financialInstrument.IssueDate);
            int fiFrequency = financialInstrument.FiFrequency; 
            return issueDate.AddMonths(p * fiFrequency);
        }
 
static public DateTime PeriodDate(FinancialInstrument financialInstrument, int p, DateTime refinancingDate)
        {
            int totalNbOfPeriods = financialInstrument.TotalNbOfPeriods;
            DateTime nDate = PeriodDate(financialInstrument, totalNbOfPeriods);
            DateTime pDate = PeriodDate(financialInstrument, p);
            DateTime refiDate = DateOnly(refinancingDate);
 
            if (refiDate <= nDate)
            {
                if (pDate < refiDate & pDate < nDate)
                {
                    return pDate;
                }
                else
                {
                    return refiDate;
                }
            }
            else
            {
                return pDate;
            }
        }

[popo]

En ne faisant qu'une seule méthode static PeriodDate prenant un Nullable<DateTime> tu devrais pouvoir n'avoir qu'une seul méthode virtuelle PeriodDate avec un Nullable<DateTime> et donc un seule méthode SumItem avec un Nullable<DateTime>.

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
25
26
27
28
29
30
31
32
33
static public DateTime PeriodDate(FinancialInstrument financialInstrument, int p, Nullable<DateTime> refinancingDate)
{
     if (refinancingDate.HasValue) 
    {
 
            int totalNbOfPeriods = financialInstrument.TotalNbOfPeriods;
            DateTime nDate = PeriodDate(financialInstrument, totalNbOfPeriods);
            DateTime pDate = PeriodDate(financialInstrument, p);
            DateTime refiDate = DateOnly(refinancingDate.Value);
 
            if (refiDate <= nDate)
            {
                if (pDate < refiDate & pDate < nDate)
                {
                    return pDate;
                }
                else
                {
                    return refiDate;
                }
            }
            else
            {
                return pDate;
            }
      }
      else
     {
            DateTime issueDate = DateOnly(financialInstrument.IssueDate);
            int fiFrequency = financialInstrument.FiFrequency; 
            return issueDate.AddMonths(p * fiFrequency);
     }
 }

Code :

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

1
2
3
4
public virtual DateTime PeriodDate(int p, Nullable<DateTime> refinancingDate)
{
     return DateTimeUtils.PeriodDate(this, p, refinancingDate);
}

Code :

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

1
2
3
4
5
6
7
8
9
10
 
        public static double SumItem(Func<int, Nullable<DateTime>, double> method, int startPeriodIncluded, int endPeriodIncluded, Nullable<DateTime> refinancingDate)
        {
            double sumItem = 0;
            for (int j = startPeriodIncluded; j < endPeriodIncluded + 1; j++)
            {
                sumItem += method(j, refinancingDate);
            }
            return sumItem;
        }

[Noxen]

Je ne sais pas exactement ce à quoi correspond ton code, mais je me demande si tu ne réécrits pas des choses qui existent déjà sur les dates.

[Rayanea]

@popo : merci je vais tester ça. Je pensais que le Delegate pouvait m'apporter une solution, mais de ce que j'en comprends, ça fonctionne comme un Func<>, donc a priori non

@Noxen : c'est possible, ça ne m'étonnerait pas. Comme dit, j'apprends sur le tas... Pourrais-tu expliciter que je comprenne l'erreur potentielle ?