C# .NET：嵌套的 try catch 块未正确处理异常

Question

为了演示异常处理，我故意将错误的表名和 null 实例放到SqlDataReader dr中。 内部 try 块捕获错误的表引用异常SqlClient.SqlException但外部 try 块未能捕获dr.hasRows上的NullReferenceException 。

using System;
using System.Collections.Generic;
using System.Data.SqlClient;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
namespace Exp_Handling
{
    internal class Program
    {
        static void Main(string[] args)
        {
            SqlConnection conn;
            SqlCommand cmd;
            //SqlDataReader dr;
            try
            {
                conn = new SqlConnection(@"data source=ITEM-S132712\MSSQLSERVER2019;initial catalog=Hotel;integrated security=true");
                cmd = new SqlCommand("Select * from Room_Entry1", conn);

                conn.Open();
                SqlDataReader dr = null;
                try
                {
                    dr = cmd.ExecuteReader();
                }
                catch (SqlException ex)
                {
                    Console.WriteLine("****" + ex.Message);
                }
                if (dr.HasRows)
                {
                    Console.WriteLine("DATA FOUND");
                    while (dr.HasRows)
                    {
                        Console.WriteLine(dr.GetString(0));
                    }
                }
            }
            catch (System.NullReferenceException nullExp)
            {

                Console.WriteLine("##try1###" + nullExp.Message);
                Console.WriteLine("Instance is Null for DataReader");
            }
            catch (Exception out_Ex)
            {
                Console.WriteLine("##try2###" + out_Ex.Message);
            }

            Console.ReadKey();
            }
        }
    }

来自参考https://www.mssqltips.com/sqlservertip/6055/net-exception-handling-for-database-calls-to-sql-server-with-try-catch-and-finally/的演示

Answer 1

这段代码有很多问题，很难知道从哪里开始：

• 捕获NullReference是荒谬的：除非ExecuteReader失败并且您捕获了SqlException ，否则您不会得到一个，在这种情况下，您应该将catch (SqlException移到外部，然后您可以删除catch (NullReferenceException 。
• conn cmd和dr需要using .
• while (dr.HasRows)没有意义，应该是while (dr.Read())
• 不要使用SELECT * ，仅使用 select 您需要的列。
• 不要使用GetString(0) ，而是使用列名。
• 不要对连接字符串进行硬编码，而是将其存储在设置文件中

static void Main(string[] args)
{
    try
    {
        using (var conn = new SqlConnection(YourConnString))
        using (var cmd = new SqlCommand("Select SomeColumn from Room_Entry1", conn))
        {
            conn.Open();
            using (var dr = cmd.ExecuteReader())
            {
                if (dr.HasRows)
                {
                    Console.WriteLine("DATA FOUND");
                    while (dr.HasRows)
                    {
                        Console.WriteLine(dr["SomeColumn"]);
                    }
                }
            }
        }
    }
    catch (SqlException ex)
    {
        Console.WriteLine("****" + ex.Message);
    }
    catch (Exception out_Ex)
    {
        Console.WriteLine("##try2###" + out_Ex.Message);
    }

    Console.ReadKey();
}