Discussion :

[popo]

Bonjour,

J'ai un problème avec un constructeur primaire (C#12).

Mon point de départ est le code suivant :

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
internal class AnonymousDataProvider
{
    public AnonymousDataProvider(Configuration configuration)
    {
        faker = new Faker(configuration.Generation.Language.ToString().ToLowerInvariant());
    }
 
    public string LastName()
    {
        return faker.Name.LastName();
    }
 
    public string PhoneNumber()
    {
        return faker.Phone.PhoneNumber("0# ## ## ## ##");
    }
 
    private readonly Faker faker;
}

Visual studio affiche le message "IDE0290 : Use primary constructor".
Le [Ctrl]+[;] me propose cette correction.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
internal class AnonymousDataProvider(Configuration configuration)
{
    public string LastName()
    {
        return faker.Name.LastName();
    }
 
    public string PhoneNumber()
    {
        return faker.Phone.PhoneNumber("0# ## ## ## ##");
    }
 
    private readonly Faker faker = new (configuration.Generation.Language.ToString().ToLowerInvariant());
}

Cela enlève bien le message mais affiche désormais cet avertissement :
S3604 : Remove the member initializer, all constructors set an initial value for the member

Cet avertissement me parait bizarre mais si je comprend bien, ça me dit que mon Faker n'a pas besoin d'être initialisé.
D'ailleurs le [Ctrl]+[;] me propose l'option "Remove redundant initializer".

J'essaie donc cette option.
Le code devient :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
internal class AnonymousDataProvider(Configuration configuration)
{
    public string LastName()
    {
        return faker.Name.LastName();
    }
 
    public string PhoneNumber()
    {
        return faker.Phone.PhoneNumber("0# ## ## ## ##");
    }
 
    private readonly Faker faker;
}

Mais du coup, j'ai les avertissements suivants :
S3459 : Remove unassigned field 'faker', or set its value
CS9113 : Parameter 'configuration' is unread.

Bref, il y a quelque chose qui m'échappe mais je ne parviens pas à savoir quoi.
Quelqu'un a-t-il une solution ?

Merci.

[popo]

Le code suivant ne provoque plus de messages ou d'avertissements.
Mais, je trouve que cela fait un montage un peu trop complexe juste pour initialiser mon faker.
Je laisse la discussion ouverte pour le moment.

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
internal class AnonymousDataProvider(Configuration configuration)
    : AbstractDataProvider(configuration)
{
    public string LastName()
    {
        return Faker.Name.LastName();
    }
 
    public string PhoneNumber()
    {
        return Faker.Phone.PhoneNumber("0# ## ## ## ##");
    } 
}
 
internal abstract class AbstractDataProvider(Configuration configuration)
{
    protected Faker Faker => faker ??= new Faker(configuration.Generation.Language.ToString());
    private Faker? faker;
}

[umfred]

Est-ce que ça ne fonctionne pas ? (dans le 2nd code, il n'y a pas l'appel au constructeur Faker, une erreur de copier/coller ou de la correction proposée ?)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
internal class AnonymousDataProvider(Configuration configuration)
{
    public string LastName()
    {
        return faker.Name.LastName();
    }
 
    public string PhoneNumber()
    {
        return faker.Phone.PhoneNumber("0# ## ## ## ##");
    }
 
    private readonly Faker faker =new Faker(configuration.Generation.Language.ToString().ToLowerInvariant());
}

[popo]

Est-ce que ça ne fonctionne pas ? (dans le 2nd code, il n'y a pas l'appel au constructeur Faker, une erreur de copier/coller ou de la correction proposée ?)

Il n'y a pas d'erreur dans ce code mise à part le warning cité.
L'unique différence entre ta version et la mienne est que j'utilise l'expression new simplifiée.
Avec ta proposition, j'obtiens donc le même warning plus une suggestion pour simplifier le new.

[Gluups]

Bonjour,

J'ai le même message dans une application MVC qui doit recevoir un argument de l'URL, et qui donc doit transmettre cet argument par le constructeur du contrôleur.

Dans ces conditions, je me demande bien quel sens peut avoir un constructeur sans arguments.

Je me demande si ils ne poussent pas un peu loin le bouchon.

[popo]

J'ai trouvé.
En réalité il n'y a pas de problème.
Ou plutôt il y a avait un faux problème, enfin deux.

Il y avait un souci dans .Net 8.0.0 donc dans le SDK 8.0.100 où ça ne fonctionnait pas à cause du readonly.
Ce problème a été corrigé par Microsoft dans la version 8.0.5 et donc dans le SDK 8.0.300.

Et ma version de Sonar Lint générait un faux positif.
https://github.com/SonarSource/sonar...g/9.14.0.81108


Au final, mon code est devenu beaucoup plus simple :

Code :

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

1
2
3
4
5
6
7
8
 
    internal class AnonymousDataProvider(Configuration configuration)
    {
        public string LastName() => faker.Name.LastName();
        public string PhoneNumber() => faker.Phone.PhoneNumber("0# ## ## ## ##");
 
        private readonly Faker faker = new (configuration.Generation.Language.ToString().ToLowerInvariant());
    }