Bien que toutes les suggestions précédentes soient également bonnes, j'ajouterais une troisième façon. Désactivez ces avertissements et ignorez-les. Bien que J'apprécie les tentatives de Microsoft pour que tout le monde code efficacement et proprement, ce n'est pas une bonne suggestion à mon avis et cela produit en fait du code difficile à lire et à modifier.

Tout d'abord, cela transforme essentiellement l'initialisation de l'objet en une seule ligne de code, et toutes les erreurs sont signalées comme telles. Si vous aviez 20 bits de données en cours de chargement dans un objet vous seriez présenté avec une erreur sur la première ligne et ne pas dire quelle propriété a commis une erreur. Le débogage n'aidera pas car on vous montre tout le bloc de code comme erreur.

Deuxièmement, si vous devez à l'avenir développer le code et ajouter du code supplémentaire pour une propriété spécifique, vous devrez maintenant le faire dans un code séparé. Cela ajoute à la fragmentation et sépare les bits de code connexes (peut-être, c'est discutable).

Ces deux problèmes peuvent sembler très mineurs, mais l'avertissement suggère un correctif qui est également une chose très mineure. Par souci de bracketing votre initialisation, vous avez rendu votre code plus difficile à déboguer et à modifier. C'est un mauvais compromis à mon avis.

Vous pouvez désactiver l'avertissement en cliquant avec le bouton droit sur l'avertissement et en sélectionnant "Supprimer", ou allez dans Outils > Options > Éditeur de texte > Style de Code > Général > initialiseur D'objet préféré > et définissez L'avertissement sur aucun.

J'ai eu un problème similaire avec ce code:

        Customer oCust = new Customer();
        oCust.Address = txtAddress.Text;
        oCust.City = txtCity.Text;
        oCust.State = txtState.Text;

Et résolu avec ce code:

        Customer oCust = new Customer()
        {
           Address = txtAddress.Text,
           City = txtCity.Text,
           State = txtState.Text
        };

Sooo...to désactivez le message D'avertissement (IDE0017) (dans VS 2017):
Cliquez sur L'ongletoutils . Ensuite, descendez à Options...
Alors / TextEditor / C# / CodeStyle / Général /
En vertu de Expressoin préférences modifier Préfèrent Initialiseur d'Objet à Non.

Je suppose que ces messages veulent juste que vous utilisiez la syntaxe floowing:

var select = new OleDbCommand
{
   Connection = cnDTC,
   CommandText = string.Format("SELECT MAX(VERSION_NO) AS MAX_VERSION FROM ({0})", strSQL2)
};

Ceci est pour votre 3ème cas.

J'aime @tonyenkiducx répondre , mais j'ai l'impression qu'il y a des idées plus grandes qui devraient être discutées.

D'après mon expérience, les suggestions de refactor fournies par Visual Studio ne sont pas utiles. Je pense que la plus grande question à considérer est de savoir si la conception du code est correcte ou non. Dans la programmation orientée objet, définir les propriétés les unes après les autres mai violer l'encapsulation. L'idée est que l'objet doit toujours être dans un état valide après tout membre est accessible / appelé jusqu'au moment où l'objet est détruit. Dans ce cas, l'État doit être valide après que chaque propriété est définie. Une bonne encapsulation conduira à une application logicielle globale améliorée parce que vous augmentez sa cohésion.

Les messages object initialization can be simplified peuvent être utiles pour détecter des points dans votre code où vous pouvez utiliser un modèle de création , en cas de violation de l'encapsulation:

• motif D'usine abstrait
• constructeur motif
• modèle de méthode D'usine
• modèle Prototype

Cela nous permet de répondre aux préoccupations soulevées par @tonyenkiducx :

Tout d'abord, cela transforme essentiellement l'initialisation de l'objet en une seule ligne de code, et toutes les erreurs sont signalées comme telles. Si vous aviez 20 bits de données en cours de chargement dans un objet, une erreur vous serait présentée sur la première ligne et ne vous indiquerait pas quelle propriété a commis une erreur. Le débogage n'aidera pas comme vous l'êtes montré le bloc entier de code comme l'erreur.

Deuxièmement, si vous devez à l'avenir développer le code et ajouter du code supplémentaire pour une propriété spécifique, vous devrez maintenant le faire dans un code séparé. Cela ajoute à la fragmentation et sépare les bits de code connexes (peut-être, c'est discutable).

Donc, au lieu d'intégrer l'instanciation au point où l'objet est consommé, ce qui est souvent suggéré par Visual Studio, je vous suggère d'utiliser un modèle de création. Cela peut ne pas supprimer le message de simplification, mais à ce stade, vous avez soigneusement considéré cet indicateur de message et pouvez le supprimer en toute sécurité.