表达总是真实的,冗余的捕获？

Question

我有以下方法,Resharper告诉我,这if(drivers != null)将永远是真的,但我不知道为什么,它告诉我catch块是多余的,但有人可以解释为什么？这是代码:

public List<Driver> GetDrivers(int id)
        {
            if (_context != null)
            {
                try
                {
                    var drivers = _context.Drivers.Where(x=> x.id == id).ToList();


                    //Always true
                    if (drivers != null)
                    {
                        //code
                    }
                    else
                    {
                        //Heuristically unreachable
                        throw new Exception("No Driver");
                    }
                }
                catch (Exception ex)
                {
                    throw;
                }
            }

            return drivers;
        }

1. 为什么if(drivers != null)总是如此？驱动程序不能为空吗？如果它是正确的,我假设有一个非空的驱动程序的默认值.
2. 我在假设它告诉else语句是无法访问的,因为它认为驱动程序永远不会为空,但是这样的情况呢？
3. 它告诉我这catch是多余的,但除了是null之外,哪个resharper说它不能,是不是还有另一个可能导致catch执行的异常？

Answer 1

你正在使用.Where,它返回一个集合.如果没有匹配则它将是一个空集合,因此不为null.

我想你想使用.SingleOrDefault而不是.Where.

Answer 2

好吧,捕获确实是多余的,你没有做任何事情,只是重新抛出完全相同的异常:

catch (Exception ex)
{
   // would make more sense if for example you're writing to log file

   // otherwise this will be thrown anyway (even without the catch)
   throw;
}

此外,这永远不会返回null,它可能有0个条目,但它不会为null:

_context.Drivers.Where(x=> x.id == id).ToList();