如何为“浮动方法”返回null-错误处理

Question

我想在代码中添加一些错误处理。 对于以下示例，我不知道如何执行此操作：

public class DataPoints
{
   public PointF[] RawData {get; set;} //raw measurement pairs
   public float xMax; //max value on X axis
   public float yMax; //max value on Y axis

   public float GetMaxX()
   {
       if(RawData == null)
       {
          throw new NullReferenceException();
          return null; //THIS does not compile! I want to exit the method here
       }

     //DO other stuff to find max X
     return MAX_X; //as float
   }
}

因此，想法是，我需要检查RawData是否已设置，然后在GetMaxX()方法中进行其余工作。 这是个好习惯吗？ 在这种情况下您会怎么做？

Answer 1

此代码有两个问题，

首先，您将引发一个异常，然后引发一个return-永远不会命中return语句，因为该异常将停止执行该方法的其余部分，从而使return语句变得多余。

其次，当返回类型为float时，不能返回null。 您必须将返回类型更改为float？ （请参阅： 可为空的类型 ）

因此，如果这是一个真正的错误情况，那么除了例外，您无能为力：

   public float GetMaxX()
   {
       if(RawData == null)
          throw new NullReferenceException();

     //DO other stuff to find max X
     return MAX_X; //as float
   }

或者，返回null并删除异常：

   public float? GetMaxX()
   {
       if(RawData == null)
          return null; 

     //DO other stuff to find max X
     return MAX_X; //as float
   }

就个人而言，如果RawData为null是一个错误情况/特殊情况，这种情况永远都不会发生，那么我会说抛出异常，并在调用代码中抛出异常时进行处理。

另一种方法是强制通过构造函数初始化RawData ，将RawData私有（或至少是setter），然后在其中RawData异常。 在类中保留任何其他逻辑以清除任何RawData /空检查的异常，因为它可以假定RawData之前已被设置。

结果如下：

public class DataPoints
{
    private readonly PointF[] rawData; //raw measurement pairs
    public float xMax; //max value on X axis
    public float yMax; //max value on Y axis

    public DataPoints(PointF[] rawData)
    {
        if (rawData == null)
            throw new ArgumentNullException("rawData");

        this.rawData = rawData;
    }

    public float GetMaxX()
    {
        //DO other stuff to find max X
        return MAX_X; //as float
    }
}

Answer 2

如果您引发异常，则无论如何将不会执行return语句，因此您所尝试的版本将是正确的

  public float GetMaxX()
  {
      if(RawData == null)
      {
         throw new NullReferenceException();
      }

      //DO other stuff to find max X
      return MAX_X; //as float
   }

return语句不会编译，因为float是一个值类型，除非您使用可空类型float，否则它永远不能为null？

就您个人提供的代码示例而言，由于您当前通过公共设置器公开RawData对象，因此我会引发异常，因此您无法保证调用GetMaxX时该对象不会为null。 然后可以将异常传播到堆栈并在任何级别上捕获，而通过使返回类型为空，则必须向调用代码添加其他检查，以查看您的方法是否返回了null并进行了适当的处理。

Answer 3

如果发生错误，我不清楚您要做什么。 您是否要引发异常或返回null？

一般来说，当调用者应该更了解并且您不想恢复时，抛出异常是让调用者清理混乱。

当调用者可能有充分的理由不初始化RawData时，返回null，这对您而言很有意义。 为此，您需要

public float? GetMaxX()

如果您真的想在不进行任何初始化的情况下崩溃，请将RawData作为参数放入构造函数中。

Answer 4

您可以删除throw exception语句下方的return语句。 每当抛出异常时，方法中就不会执行其他语句（最终，块是对此的异常，但在此上下文中是无关紧要的）。

除了这个函数的问题之外，我还有一个关于RawData属性可以公开访问的论点。 打开这样的集合通常不是一个好主意。 正如@ sq33G所建议的那样，您可以通过传递RawData作为构造函数参数来保证具有有效的对象。 当传递无效数组时，您可能会在构造函数的早期失败（空，可能大小为零？）。

private PointF[] _rawData;

public DataPoints(PointF[] rawData)
{
    if(rawData == null || rawData.Length == 0)
        throw new ArgumentException("RawData should not be null and should contain at least one element");
    this._rawData = rawData;
}

如果需要从类外部访问RawSata，我建议您以一种既不能更改数组本身（因此也不能更改设置器）或更改其内容的方式进行操作。 使用IEnumerable是正确的方法。

public IEnumerable<PointF> RawData
{
    get { return _rawData; }
}

Answer 5

并不总是需要一个方法来返回一个值。 特别是，它还可以通过引发异常（在这种情况下不返回任何值）而退出。 您可以在此处检查规则

在您的示例中，您可以

（a）引发NullReferenceException-这将中断流程并返回

（b）如果RawData为null，则返回默认值-这将中断流程并返回默认值。

float f()
    {
        if (RawData == null)
        {
            throw new NullReferenceException();
            return default(float);
        }
        return doOtherOperation(RawData);
    }

    float doOtherOperation(PointF[] RawData)
    {
        //do what you wanted to do
        return default(float);
    }