使用 foreach 循环时使用未赋值的局部变量

Question

我有一个代码块，可以从存储过程中获取一些数据。收到数据后，我想根据条件将值分配给"isExisting"。我不希望在声明时为"isExisting"赋值。

bool isExisting;
using (var conn = new SqlConnection(System.Configuration.ConfigurationManager.ConnectionStrings["ConnectionString"].ToString()))
{
    conn.Open();
    using (SqlCommand cmd = new SqlCommand("some_stored_procedure", conn))
    {
        cmd.CommandType = CommandType.StoredProcedure;
        cmd.Parameters.Add(new SqlParameter("input", value));
        using (var adapter = new SqlDataAdapter(cmd))
        {
            DataSet ds = new DataSet();
            adapter.Fill(ds);
            if (ds.Tables.Count > 0)
            {
                if (ds.Tables[0].Rows.Count == 0)
                    isExisting = false;
                else
                {
                    foreach (DataRow row in ds.Tables[0].Rows)
                    {
                        if (row["Key"].ToString() == ValueToCompareWith)
                        {
                            isExisting = true;
                            break;
                        }
                        else
                            isExisting = false;
                    }
                }
            }
            else
                isExisting = false;
        }
    }
}
if (!isExisting) //Step :getting error "use of unassigned local variable"
{
}

我想我已经涵盖了所有条件，变量"isExisting"在到达"步骤"时将有一个值，但仍然出现编译器错误。

问题开始是因为我的代码审查工具抛出评论

"当局部变量被分配一个未被任何后续指令读取的值时，就会发生死存储。计算或检索一个值只是为了覆盖它或丢弃它，可能表明代码中存在严重错误。即使这不是错误，充其量也是浪费资源。因此，应使用所有计算值。
不合规代码示例

void CalculateRate(int a, int b)
{
  int i;
  i = a + b; // Noncompliant; calculation result not used before value is overwritten
  i = DoSomething();  // Noncompliant; retrieved value not used
  for (i = 0; i < 10; i++)
  {
    //  ...
  }
  // ...
}

Answer 1

正如其他人指出的那样，编译器不够聪明，无法知道你是否真的在为变量赋值。

你可以通过使用一些 Linq 来解决这种歧义，并清理大量代码：

bool isExisting;
using (var conn = new SqlConnection(System.Configuration.ConfigurationManager.ConnectionStrings["ConnectionString"].ToString()))
{
    conn.Open();
    using (SqlCommand cmd = new SqlCommand("some_stored_procedure", conn))
    {
        cmd.CommandType = CommandType.StoredProcedure;
        cmd.Parameters.Add(new SqlParameter("input", value));
        using (var adapter = new SqlDataAdapter(cmd))
        {
            DataSet ds = new DataSet();
            adapter.Fill(ds);
            isExisting = ds.Tables
                .OfType<DataTable>()
                .Take(1)
                .SelectMany(t => t.Rows.OfType<DataRow>())
                .Any(r => r["Key"].ToString() == ValueToCompareWith);
        }
    }
}
if (!isExisting)
{
}

<小时 />

另一种方法是将isExisting检查分解为单独的方法：

bool CheckIsExisting(object value, string valueToCompareWith) 
{
    using (var conn = new SqlConnection(System.Configuration.ConfigurationManager.ConnectionStrings["ConnectionString"].ToString()))
    {
        conn.Open();
        using (SqlCommand cmd = new SqlCommand("some_stored_procedure", conn))
       {
            cmd.CommandType = CommandType.StoredProcedure;
            cmd.Parameters.Add(new SqlParameter("input", value));
            using (var adapter = new SqlDataAdapter(cmd))
            {
                DataSet ds = new DataSet();
                adapter.Fill(ds);
                if (ds.Tables.Count != 0 && ds.Tables[0].Rows.Count != 0)
                {
                    foreach (DataRow row in ds.Tables[0].Rows)
                    {
                        if (row["Key"].ToString() == ValueToCompareWith)
                        {
                            return true;
                        }
                    }
               }
           }
       }
    }
    return false;
}

然后做：

if (!CheckIsExisting(value, ValueToCompareWith))
{
    // do something...
}         

Answer 2

编译器是一个简单程序。它不会检查所有逻辑以确保变量始终获得值。那不是它的工作。只需给isExisting一个默认值，让编译器满意，你的真实代码无论如何都会设置正确的值(正如你已经说过的(。

Answer 3

编译器

无法确定是否每段代码都会被命中 - 例如，如果此循环中没有行：

foreach (DataRow row in ds.Tables[0].Rows)

因此，整个IF块不会被击中。

设置默认值。

Answer 4

Bool 的默认值为 false 。在您的代码中提供它。

bool isExisting=false;

用代码编写的整个业务逻辑应该注意

if (!isExisting) {
}

背后的主要逻辑是编译器几乎不打扰您的脚本。它只需要分配的值，然后再在任何条件下使用。

请参阅此处以了解更多信息。