C # - Quelles sont les meilleures pratiques / astuces pour ADO.NET?

Une erreur, je vois répété maintes et maintes fois:

L'instanciation et la configuration de tout (DbConnection, DbCommand, DbParameters) à l'intérieur d'une méthode qui est appelée à plusieurs reprises dans une boucle serrée.

La plupart du temps, vous pouvez restructurer ces variables locales, comme les membres de la classe de l'instanciation d'eux qu'une seule fois et de ne garder que les mises à jour de DbParameters à l'intérieur de la méthode.

---

Mis à JOUR pour inclure des exemples de code demandé dans les commentaires

Avertissement:
C'est un moyen rapide assemblés échantillon pour seul but de démontrer le point sur le mouvement répétitif des trucs hors de la boucle.
Les autres meilleures pratiques ne sont pas nécessairement mis en œuvre.

        public static void Show() {
            List people = new List();

            //Everything is done inside the loop
            PersonDal personDal = new PersonDal();
            foreach (Person person in people) {
                personDal.Insert(person);
            }

            //Things are setup once outside the loop.
            using (DbConnection connection = SqlClientFactory.Instance.CreateConnection()) {
                //setup the connection
                BetterPersonDal betterPersonDal = new BetterPersonDal(connection);
                connection.Open();
                foreach (Person person in people) {
                    betterPersonDal.Insert(person);
                }
            }
        }
    }

    class Person {
        public int Id { get; set; }
        public string Name { get; set; }
    }

Sur cette première mise en œuvre, chaque chose est mis en place chaque fois que la méthode est appelée:

class PersonDal {
    public int Insert(Person person) {
        DbProviderFactory factory = SqlClientFactory.Instance;

        using (DbConnection connection = factory.CreateConnection()) {
            connection.Open();

            connection.ConnectionString = "Whatever";

            using (DbCommand command = connection.CreateCommand()) {
                command.CommandText = "Whatever";
                command.CommandType = CommandType.StoredProcedure;

                DbParameter id = command.CreateParameter();
                id.ParameterName = "@Id";
                id.DbType = DbType.Int32;
                id.Value = person.Id;

                DbParameter name = command.CreateParameter();
                name.ParameterName = "@Name";
                name.DbType = DbType.String;
                name.Size = 50;
                name.Value = person.Name;

                command.Parameters.AddRange(new DbParameter[] { id, name });

                return (int)command.ExecuteScalar();
            }
        }
    }
}

Nous passons maintenant à la configuration d'objets de construction de la laisser sortir de la boucle:

class BetterPersonDal {
    private DbProviderFactory factory;
    private DbConnection connection;
    private DbCommand command;
    private DbParameter id;
    private DbParameter name;

    public BetterPersonDal(DbConnection connection) {
        this.command = connection.CreateCommand();
        this.command.CommandText = "Whatever";
        this.command.CommandType = CommandType.StoredProcedure;

        this.id = command.CreateParameter();
        this.id.ParameterName = "@Id";
        this.id.DbType = DbType.Int32;

        this.name = command.CreateParameter();
        this.name.ParameterName = "@Name";
        this.name.DbType = DbType.String;
        this.name.Size = 50;

        this.command.Parameters.AddRange(new DbParameter[] { id, name });
    }

    public int Insert(Person person) {
        this.id.Value = person.Id;
        this.name.Value = person.Name;

        return (int)command.ExecuteScalar();
    }
}

Être très intelligent à propos de la gestion de la connexion. Ouverture d'une connexion DB peut être très cher, essayez donc de garder cela à l'esprit lors de l'écriture de la base de données de la couche d'accès.

L'article Martin des liens vers est excellent. Je voudrais simplement ajouter que vous voulez absolument utiliser un DataReader au lieu d'un DataSet (j'adore les jeux de données, mais pas pour des raisons de performances).