0

テンプレートパターンスタイルでメソッドを書くのに問題があります。しかし、最初に私のコード：

私が使用している基本クラスは次のようになります。

public abstract class DbBase<T> where T : new()
{
    protected abstract Value CommandValue { get; }
    protected abstract CommandType CommandType { get; }
    protected abstract Mapper<T> GetMapper();
    protected abstract IDbConnection GetConnection();
    protected abstract Collection<IDataParameter> GetParameters(IDbCommand command);

    public Collection<IDataParameter> Paramaters { get; set; }

    #region Public Methods
    public T Single(int id)
    {
        return ExecuteReader(id).SingleOrDefault();
    }

    public Collection<T> All()
    {
        return ExecuteReader();
    }
    #endregion

    #region Private Methods
    private Collection<T> ExecuteReader(int? id = null)
    {
        var collection = new Collection<T>();

        using (var connection = GetConnection())
        {
            var command = connection.CreateCommand();
            command.Connection = connection;
            command.CommandType = CommandType;

            if (id.HasValue && id.Value > 0)
                command.CommandText = CommandValue.Single;
            else
                command.CommandText = CommandValue.All;

            var parameters = GetParameters(command);
            if (parameters != null)
            {
                foreach (var param in GetParameters(command))
                    command.Parameters.Add(param);
            }

            try
            {
                connection.Open();

                using (var reader = command.ExecuteReader())
                {
                    try
                    {
                        var mapper = GetMapper();
                        collection = mapper.MapAll(reader);
                        return collection;
                    }
                    finally
                    {
                        if (!reader.IsClosed)
                            reader.Close();
                    }
                }
            }
            catch (Exception ex)
            {
                throw new DbBaseException(ex.Message, ex);
            }
            finally
            {
                if (connection.State != ConnectionState.Closed)
                    connection.Close();
            }
        }
    }
    #endregion
}

したがって、変更される可能性のあるすべてのコードについて、継承している詳細クラスがあります。

public class UserDb : DbBase<User>
{
    private static readonly string ALL = "SELECT * FROM [USER]"; //don't use star!
    private static readonly string SINGLE = "SELECT * FROM [USER] WHERE USER_ID = @USER_ID";
    private static readonly CommandType commandType = CommandType.Text;

    protected override Value CommandValue
    {
        get
        {
            var value = new Value
            {
                Single = SINGLE,
                All = ALL
            };
            return value;
        }
    }

    protected override CommandType CommandType
    {
        get { return commandType; }
    }

    protected override Mapper<User> GetMapper()
    {
        return new UserMapper();
    }

    protected override Collection<IDataParameter> GetParameters(IDbCommand command)
    {
        var parameters = new Collection<IDataParameter>();
        var param = command.CreateParameter();
        param.ParameterName = "@USER_ID";
        param.Value = 2;
        parameters.Add(param);
        return parameters;
    }
}

市外局番： 

 var userDb = new UserDb();
 var user = userDb.Single(1);

 if (user != null)
     Console.WriteLine(string.Format("{0}, {1}, {2}", user.UserId, user.Username, user.Password));

ご覧のとおり、IDごとに1つの特定の行を取得するSingleというメソッドを実装しました。私の問題は、テンプレートパターンを壊さずにIDをExecuteReaderメソッドにプッシュするにはどうすればよいですか？

皆さんが私を助けてくれることを願っています。

どうも

4

1 に答える 1

2

のようなすべてのエンティティに同じ名前のパラメータを使用しないのはなぜですか@id。そうすれば、もう必要ありませんGetParameters。単に電話する

command.Parameters.AddWithValue("@id", id);

アップデート

異なる数のパラメーターを使用できるようにする場合は、paramsキーワードを使用できます。これにより、さまざまな数のパラメーター（ゼロを含む）を渡すことができます。

public T Single(params int[] id)
{ 
    return ExecuteReader(id).SingleOrDefault();         
}

と

private Collection<T> ExecuteReader(params int[] id)
{
    ...
    for (int i = 0; i < id.Length; i++) {
        command.Parameters.AddWithValue("@id" + i, id[i]);
    }
    ...
}

そして、あなたはあなたのパラメータに名前を付ける必要があります@id0, @id1, @id2, ...

var coll = ExecuteReader();
var coll = ExecuteReader(2);
var coll = ExecuteReader(5, 77);
...

var result = db.Single(1);
var result = db.Single(4, 13);
var result = db.Single(5, 100, 1);
...

更新＃2

SQLテキストからパラメータ名を抽出することもできます

private Collection<T> ExecuteReader(params object[] p)
{
    ...
    var matches = Regex.Matches(sql, @"@\w+");
    if (matches.Count != p.Length) {
        throw new ArgumentException("The # of parameters does not match ...");
    }
    for (int i = 0; i < p.Length; i++) {
        command.Parameters.AddWithValue(matches[i].Value, p[i]);
    }
    ...
}
于 2012-05-12T16:43:26.643 に答える