如果我有一个返回某些东西的方法,比如
public DataTable ReturnSomething() { try { //logic here return ds.Tables[0]; } catch (Exception e) { ErrorString=e.Message; } }这会产生编译器错误,显然是因为catch{}块不返回任何内容。
所以当我有返回值的方法时,我不使用try-catch块,这是一种不好的做法。 如果有错误,我想将错误字符串设置为该错误。 但是我还需要一个返回值。 建议吗?
将您的返回值存储在一个临时变量中,如下所示:
public DataTable ReturnSomething() { DataTable returnValue = null; try { //logic here returnValue = ds.Tables[0]; } catch (Exception e) { ErrorString=e.Message; } return returnValue; }您应该在catch块中引发/抛出exception并在调用方法中处理它。
public void invokeFaultyCode() { try { DataTable dt = ReturnSomething(); } catch(Exception e) { // Print the error message, cleanup, whatever } } public DataTable ReturnSomething() throws Exception { try { //logic here return ds.Tables[0]; } catch (Exception e) { ErrorString=e.Message; throw; } }PS:很抱歉任何语法错误,我对C#有点生疏。
ErrorString变量看起来像一个错误代码变量。 建议的做法是使用exception在必要时直接传递错误信息,而不是将内容存储到错误代码中。
如果您只是让调用者捕获exception,您实际上正在使用ErrorString执行相同的操作:从方法本身中删除响应错误的责任。 这是一个很好的目标。 但是使用错误字符串并不能获得任何使用exception的信息。 事实上,你会以这种方式丢失信息。 可能会发生任意类型的错误,并且许多错误都与它们相关联,并且具有自己的特殊属性来保存有关失败的上下文信息。 只需将消息存储在String中,就会丢失此信息。
因此,除非您的目标是专门隐藏调用者发生的错误类型,否则您只能通过让exception通过来获得。
另一件需要考虑的事情是这是否真的是一个错误场景。 如果是,那么你的调用方法几乎不会关心返回值。 在这种情况下,您只需让exception进行而不返回任何内容就无需担心。 如果它不是一个真正的错误场景,并且调用者只是继续并做其他事情,那么,这是由调用者决定的,对吧? 通过返回错误字符串和虚拟DataTable或null来获取仍然没有多少好处,而不是抛出所有上下文失败信息的exception。
你应该用try catch来包装调用者在被调用的例程中发生的任何exception都会冒出来给调用者,你可以在那里捕获它们。
就个人而言,我认为在这个例程中有一个try catch是不合适的,因为你应该让调用者处理exception。
对于我的例子,这将编码如下
private void DoSomething() { try { DataTable dt = ReturnSomething(); } catch (Exception ex) { } } public DataTable ReturnSomething() { DataTable dt = new DataTable(); // logic here return dt; }如果您打算“不要抛出exception路线”(我不一定会推荐),您可以按照MS使用的TryParse方法进行操作。
就像是:
private string FillDataTable(out DataTable results) { try { results = new DataTable(); //something like this; return String.Empty; } catch (Exception ex) { results = null; return ex.Message; }}
这取决于你的应用程序。 您可以返回null ,空DataTable或任何适合的环境。
我假设你仍然可以设置消息,然后返回null或c#等价物
public DataTable ReturnSomething(){ try { //logic here return ds.Tables[0]; } catch (Exception e) { ErrorString=e.Message; return null; } }这个怎么样 :
public DataTable ReturnSomething(out string errorString) { errorString = string.Empty; DataTable dt = new DataTable(); try { //logic here dt = ds.Tables[0]; } catch (Exception e) { errorString = e.Message; } return dt; }我认为你的代码是在一个足够高的调用堆栈级别运行的,它与UI代码混合在一起。 如果确实如此,您可以在catch块中return null 。 但是,如果您正在编写可重用代码,则应重构它以使其不包含UI操作并在调用堆栈中的更高级别处理exception。
由于您在示例中对exception进行了处理(而不是再次抛出),因此外部代码假设每个都没问题,因此您应该返回一些有用的东西。
如果你需要在那里捕获exception并做一些好的事情,但是如果它仍然是一个错误的情况你应该抛出它,或者一个不同的exception,也许你刚刚捕获的那个作为InnerException。
您可以像下面的示例代码那样执行此操作。
上述就是C#学习教程:try-catch块具有返回类型分享的全部内容,如果对大家有所用处且需要了解更多关于C#学习教程,希望大家多多关注编程笔记
public DataTable ReturnSomething(out string OutputDesc) { try { //logic here OutputDesc = string.Format("Your Successful Message Here..."); return ds.Tables[0]; } catch (Exception e) { OutputDesc =e.Message; return null; } }