Question

我有多个foreach循环，通过它我可以从Facebook获取工作历史记录，用户的教育历史记录并将其插入到数据库中：

Result soap = serializer.Deserialize<Result>(ser);
foreach (var item in soap.data)
{
    int length = item.education_history.Length;
    int lenght2 = item.work_history.Length;

    foreach (var edu in item.education_history)
    {
        length--; 
        ftr = ftr + "," + edu.name.ToString();
    }

    foreach (var wrk in item.work_history)
    {
        lenght2--;
        ftr1 = ftr1 + "," + wrk.company_name.ToString();
    }

    string str1 = "Insert into [Snaps] (Loc_city,Loc_state,Loc_country,Edu_Hist1,Work_Hist1) values ('" + item.current_location.city.ToString() + "','" + item.current_location.state.ToString() + "','" + item.current_location.country.ToString() + "','" + ftr + "','" + ftr1 + "')";
    SqlCommand cmd1 = new SqlCommand(str1, con);
    cmd1.ExecuteNonQuery();
}

PS：此代码仅用于测试目的，因此未使用参数化sql查询

我遇到的问题是，当我计算父foreach循环中第一个项目的ftr和ftr1的值时

foreach (var item in soap.data)

当我将这些值插入数据库时，它会在item中附加第二个数组索引并将其保存在数据库中。此外，ftr和ftr1的值未刷新，它会将新值附加到先前的值

我在这里做错了什么？

谢谢

Answer 1

问题是ftr和ftr1变量在主循环中的每次迭代后都没有被重置，因此这些值会一直与它们连接。

一种解决方案是将ftr和ftr1设置为null或在主循环结束时清空字符串：

foreach (var item in soap.data)
{
    foreach (var edu in item.education_history)
    {
        // ...
    }

    foreach (var wrk in item.work_history)
    {
        // ...
    }

    // ...

    ftr = null;
    ftr1 = null;
}

但是，更简单的方法是使用LINQ中的Enumerable.Select方法从education_history和work_history数组中提取值列表，并使用String.Join进行连接他们分为两个字符串：

foreach (var item in soap.data)
{
    var educationNames = item.education_history.Select(i => i.name.ToString());
    var ftr = String.Join(",", educationNames);

    var previousCompaniesNames = item.work_history.Select(i => i.company_name.ToString());
    var ftr1 = String.Join(",", previousCompaniesNames);

    var statement = "INSERT INTO [Snaps] (Loc_city,Loc_state,Loc_country,Edu_Hist1,Work_Hist1) VALUES (@city, @state, @country, @educationNames, @previousCompaniesNames)";
    var command = new SqlCommand(statement, con);
    command.Parameters.Add("@city", SqlDbType.VarChar, 255).Value = item.current_location.city.ToString();
    command.Parameters.Add("@state", SqlDbType.VarChar, 255).Value = item.current_location.state.ToString();
    command.Parameters.Add("@country", SqlDbType.VarChar, 255).Value = item.current_location.country.ToString();
    command.Parameters.Add("@educationNames", SqlDbType.VarChar, 4000).Value = ftr
    command.Parameters.Add("@previousCompaniesNames", SqlDbType.VarChar, 4000).Value = ftr1
    command.ExecuteNonQuery();    
}

这样ftr和ftr1变量在每次迭代时声明为，因此无需手动重置其值。

Answer 2

ftr和ftr1没有刷新，它会附加新值以前的那些

是的，你没有刷新它，如果你想为每次迭代使用新值，那么使用

ftr = "";
ftr1 = "";

迭代多个foreach循环的问题

2 个答案: