Имея разногласия с коллегой, и мне на данный момент мне все равно, кто прав, мне более любопытно, какое решение лучше, чтобы я мог использовать его в дальнейшем.
У нас есть разные способы доступа к системе.
Вариант № 1:
Создает базу данных с кодом ниже.
using Microsoft.Practices.EnterpriseLibrary.Data;
namespace Ivans.Healthcare.CommercialAccess.Data
{
public abstract class DataAccess : DataHelperBase
{
public const int commandTimeout = 7200;
private static Database m_db = null;
public StringBuilder Status {get; set;}
public DataAccess()
{
this.Status = new StringBuilder();
if (m_db == null)
{
bool bIfRunsOnWebService = false;
try
{
if (DynamicConfigurationManager.AppSettings["WebService"] != null)
{
bIfRunsOnWebService = true;
}
}
catch {}
if (!bIfRunsOnWebService)
{
m_db = DatabaseFactory.CreateDatabase(DataAccessResource.IDS_DB_ALIAS);
}
else
{
m_db = CreateDatabase(DataAccessResource.IDS_WS_DB_ALIAS);
}
}
}
Тогда каждый раз, когда нужно вызвать хранимую процедуру, метод будет содержать что-то вроде этого:
public IEnumerable<InquiryServiceType> GetActive(bool is5010)
{
Database db = getDB();
DbCommand dbCmd = db.GetStoredProcCommand(DataAccessResource.IDS_SP_SEL_InquiryServiceTypeData_ListServiceTypes);
db.AddInParameter(dbCmd, DataAccessResource.IDS_SP_SEL_InquiryServiceTypeData_ListServiceTypes_Is5010Request, DbType.Boolean, is5010);
DataSet ds = new DataSet();
db.LoadDataSet(dbCmd, ds, new string[] { DataAccessResource.IDS_TBL_InquiryServiceTypeData });
return DataSetTranslator.TranslateInquiryServiceTypeDataSet(ds);
}
Вариант 2
Эта опция немного более модульная и пыталась создать общий метод базы данных.
private Database currentDB;
private const int commandTimeout = 7200;
public DataAccess(Common.Enums.ConnectionString currentConnection)
{
currentDB = DatabaseFactory.CreateDatabase(currentConnection.ToDescription());
}
public IEnumerable<T> SelectMany<T>(string spName, params Param[] parameters) where T : IDataPopulate<T>, new()
{
var storedProcedure = CreateStoredProcedureCommand(spName);
AddParameters(storedProcedure, parameters);
IDataReader myReader = null;
IList<T> listOfItems = new List<T>();
try
{
myReader = currentDB.ExecuteReader(storedProcedure);
if (myReader == null)
{
return listOfItems;
}
while (myReader.Read())
{
listOfItems.Add(new T().FillObject(myReader));
}
return listOfItems;
}
catch (Exception ex)
{
string message = string.Format("Error Message: {0}\r\nStored Procedure: {1}\r\n", ex.ToString(), spName);
throw new Exception(message);
}
finally
{
DataAccessDisposal.DataReader(myReader);
DataAccessDisposal.StoredProcedure(storedProcedure);
}
}
Тогда вызов базы данных будет выглядеть так:
public IEnumerable<InquiryServiceTypes> GetAll(int payerID)
{
Param payerIdParam = new Param("@payerID", DbType.Int32, payerID);
return dataAccess.SelectMany<InquiryServiceTypes>("dbo.proc_PayersInquiryServiceTypesSel", payerIdParam);
}
Заключение
Определенно есть вещи, которые неправильно закодированы в каждом из разделов. Я почти уверен, что есть середина, которая является наиболее эффективным кодом.
Код выше имеет две точки неэффективности. Во-первых, как он подключается к базе данных в первую очередь. Во-вторых, как только данные возвращаются, как вы справляетесь с этим. Я хотел бы обсудить оба, но чувствую, что первое более важно для этого пункта.
Спасибо,
C