如何简化 c#.net 中的 if 条件和 foreach 条件？

Question

我的控制器中有一个重复的 foreach 条件。我怎样才能简化它？ 因此，我几乎达到了 500 行。我一直在 x8 的每个条件下使用它。

List<jewelry_dashboard_view_per_month> transactionmonthlynewloan = dashboardmanager.Get_MonthlyTransaction(search_branch, (monthlyonly + "01"), "N-", (monthlyonly + no_of_items), no_of_items, monthlyonly);
myNewLoanMontlyList.Add(transactionmonthlynewloan);

List<jewelry_dashboard_view_per_month> transactionmonthlyrenewal = dashboardmanager.Get_MonthlyTransaction(search_branch, (monthlyonly + "01"), "R-", (monthlyonly + no_of_items), no_of_items, monthlyonly);
myRenewalMontlyList.Add(transactionmonthlyrenewal);

这是if条件

if (myNewLoanMontlyList[0].Count != 0)
{
    foreach (var internal_monthly_newloan_data in myNewLoanMontlyList[0].SelectMany(c => c.id_data))
    {monthly_newloan_data_ID.Add(internal_monthly_newloan_data);}

    foreach (var internal_monthly_newloan_data in myNewLoanMontlyList[0].SelectMany(c => c.debit_data))
    {monthly_newloan_data_debit.Add(internal_monthly_newloan_data);}
}
else
{
    monthly_newloan_data_ID.Add(0);
    monthly_newloan_data_debit.Add(0);
};

这是 foreach 条件

//newloan
int newloan_data_id = 0;
DateTime newloan_data_transdate = DateTime.Parse((DateTime.Today).ToString());
decimal newloan_data_debit = 0;
string newloan_data_txnname = "";
string newloan_data_branchID = "";
foreach (var newloan_data in newloan)
{
    newloan_data_id = newloan_data.ID;
    newloan_data_transdate = DateTime.Parse((newloan_data.Transdate).ToString());
    newloan_data_debit = Decimal.Parse((newloan_data.Debit).ToString());
    newloan_data_txnname = newloan_data.TransactionName;
    newloan_data_branchID = newloan_data.BranchID;
};
datanewloan = new transaction_details()
{
    ID = newloan_data_id,
    Transdate = DateTime.Parse(newloan_data_transdate.ToString("yyyy-MM-dd")),
    Debit = Decimal.Parse(newloan_data_debit.ToString()),
    TransactionName = newloan_data_txnname,
    BranchID = newloan_data_branchID
};

Answer 1

你可以使用Linq，

SelectMany: 将序列的每个元素投影到IEnumerable<T>。您无需再次迭代并将其添加到单独的列表中

对于你的 if 条件，

if (myNewLoanMontlyList[0].Any())
{
    monthly_newloan_data_ID = myNewLoanMontlyList[0].SelectMany(c => c.id_data);
    monthly_newloan_data_debit = myNewLoanMontlyList[0].SelectMany(c => c.debit_data);
}
else
{
    monthly_newloan_data_ID.Add(0);
    monthly_newloan_data_debit.Add(0);
}

---

Select: 将序列的每个元素投影到新的形式中。在你的 案例新形式是transaction_details的实例

使用 Linq .Select() 代替 for 循环，

var result = newloan.Select(x => new transaction_details(){
          ID = x.ID,
          Transdate = DateTime.Parse(x.Transdate.ToString("yyyy-MM-dd")),
          Debit = Decimal.Parse((x.Debit).ToString()),
          TransactionName = x.TransactionName,
          BranchID =  x.BranchID
}).LastOrDefault();

为了获取最后一个元素，我使用了LastOrDefault()。您可以通过索引或条件获取单个元素。

Answer 2

在您的if-else 条件下：

if (myNewLoanMontlyList[0].Count != 0)
{
    foreach (var internal_monthly_newloan_data in myNewLoanMontlyList[0].SelectMany(c => c.id_data))
    {monthly_newloan_data_ID.Add(internal_monthly_newloan_data);}

    foreach (var internal_monthly_newloan_data in myNewLoanMontlyList[0].SelectMany(c => c.debit_data))
    {monthly_newloan_data_debit.Add(internal_monthly_newloan_data);}
}
else
{
    monthly_newloan_data_ID.Add(0);
    monthly_newloan_data_debit.Add(0);
};

如果方法中if-else之后没有其他进程，则可以只使用if条件，不带else部分。

    if (myNewLoanMontlyList[0].Count == 0)
    {
       monthly_newloan_data_ID.Add(0);
       monthly_newloan_data_debit.Add(0);
    }

    foreach (var internal_monthly_newloan_data in myNewLoanMontlyList[0].SelectMany(c => c.id_data))
    {monthly_newloan_data_ID.Add(internal_monthly_newloan_data);}

    foreach (var internal_monthly_newloan_data in myNewLoanMontlyList[0].SelectMany(c => c.debit_data))
    {monthly_newloan_data_debit.Add(internal_monthly_newloan_data);}