データベースに格納されている文字列 ("(x + y) * 4" など) を取り込み、データベースから x と y の値を取得し、計算を実行して結果をデータベース。これには時間がかかりすぎているようで、Linq の落とし穴に足を踏み入れてしまったのではないかと心配しています。これを改善する方法があれば教えてください：
public Nullable<decimal> CalculateFormulaByYearDistrict(int formulaId, int fundingYearId, int districtId)
{
string formulaText = "";
decimal? retValue = null;
using (STARSEntities context = new STARSEntities())
{
var formulaItems = from fi in context.STARS_FormulaItem
where fi.FormulaId == formulaId
select fi;
STARS_Formula formula = formulaItems.FirstOrDefault().STARS_Formula;
formulaText = formula.FormulaText;
foreach (STARS_FormulaItem formulaItem in formulaItems)
{
int accountingItemId = formulaItem.AccountingItemId;
var itemValue = (from iv in context.AccountingItemValues
join di in context.STARS_DistrictInputData
on iv.DomainSpecificId equals di.DistrictInputDataId
where (di.DistrictId == districtId || di.DistrictId == -1) //District -1 is the invalid and universal district for coefficients
&& di.DomainYearReportingPeriod.FundingYearId == fundingYearId
&& iv.AccountingItemId == accountingItemId
select iv).SingleOrDefault();
//If no value exists for the requested Assessment Item Value, then force an error message into the formula text
//to be thrown during calculate.
if (itemValue != null)
formulaText = Regex.Replace(formulaText, @"\b" + formulaItem.ItemCode + @"\b", itemValue.Amount.ToString());
else
formulaText = Regex.Replace(formulaText, @"\b" + formulaItem.ItemCode + @"\b", "No Value Exists for " + formulaItem.ItemCode);
}
switch (formula.FormulaTypeId)
{
case (int)FormulaType.CALC:
retValue = Calculate(formulaText);
break;
case (int)FormulaType.EXPONENT:
// pull the number directly after e and and calculate the Math.Exp(value) and then push that into the calculation.
retValue = Calculate(ReplaceExponent(formulaText));
break;
case (int)FormulaType.IFTHEN:
// evaluate the If statements and pass any math to the calculator.
retValue = Calculate(EvaluateIf(formulaText));
break;
default:
break;
}
}
return retValue;
}
public bool CalculateAndSaveResults(DistrictDataCategory category, List<int> districtIds, int fundingYearId, int userId)
{
//Optimization Logic
DateTime startTime = DateTime.Now;
Debug.WriteLine("Starting Calculate and Save at:" + startTime.ToString());
using (STARSEntities context = new STARSEntities())
{
var formulas = from f in context.STARS_FormulaCategory
where f.DistrictDataCategoryId == (int)category
select f.STARS_Formula;
foreach (var districtId in districtIds)
{
Debug.WriteLine("District: " + districtId.ToString());
DateTime districtStartTime = DateTime.Now;
foreach (var formula in formulas)
{
var itemValue = (from iv in context.AccountingItemValues
join di in context.STARS_DistrictInputData
on iv.DomainSpecificId equals di.DistrictInputDataId
where (di.DistrictId == districtId)
&& di.DomainYearReportingPeriod.FundingYearId == fundingYearId
&& iv.AccountingItemId == formula.ResultAccountingItemId
select new { iv, di }).SingleOrDefault();
itemValue.iv.Amount = CalculateFormulaByYearDistrict(formula.FormulaId, fundingYearId, districtId);
//Update Actual Amount Record
itemValue.iv.LastUpdated = DateTime.Now;
itemValue.iv.UpdatedBy = userId;
//Update District Data Import Record
itemValue.di.LastUpdated = DateTime.Now;
itemValue.di.UpdatedBy = userId;
}
Debug.WriteLine("District Calculation took: " + ((TimeSpan)(DateTime.Now - districtStartTime)).ToString() + "for " + districtId.ToString());
}
context.SaveChanges();
}
Debug.WriteLine("Finished Calculate and Save at:" + ((TimeSpan)(DateTime.Now - startTime)).ToString());
return true;
}
基礎となるデータ構造に関する情報が必要な場合はお知らせください。重要と思われることは、数式テキストを格納する数式テーブル間に関連エンティティがあることです。これにより、特定の地区の特定のタイプのすべての計算を実行できます。格納される実際の値は AccountingItemValue テーブルにありますが、アカウンティング項目の値に関する位置情報を持つ DistrictInputData という関連テーブルがあります。
どうもありがとうございました。