0

我有一个从存储过程中获取一些数据的代码块。收到数据后，我想根据条件将值分配给“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”在达到“Step”时会有一个值，但我仍然得到编译器错误。

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

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

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++)
  {
    //  ...
  }
  // ...
}
4

4 回答 4

1

编译器很简单。它不会检查所有逻辑以确保变量始终获得值。那不是它的工作。只需提供isExisting一个默认值以使编译器满意，并且您的实际代码无论如何都会设置正确的值（正如您已经说过的）。

于 2017-02-10T04:30:57.800 回答
1

编译器不能确定是否每段代码都会被命中——例如，如果这个循环中没有行：

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

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

设置一个默认值。

于 2017-02-10T04:32:07.323 回答
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...
}
于 2017-02-10T04:41:32.330 回答
0

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

bool isExisting=false;

您在代码中编写的整个业务逻辑应该注意

if (!isExisting) {
}

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

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

于 2017-02-10T04:45:54.490 回答