一般返回写try里面还是外面

[-一个大坑]

有没有规范说放里面还是外面，是自己随意写？

public IActionResult DeleteContractTag(string tagName)
        {
            try
            {
                _service.Delete(tagName);
                _unitOfWork.Commit();
            }
            catch (Exception ex)
            {
                return Content("刪除失敗，原因：" + ex.Message);
            }

            return Content("OK");
        }

[exception92]

没有错误就return吧，一般都是这么写的。何必放到最后了

[mirrorspace]

呵呵 其实这是一种偷懒的写法. 不管里面多少异常.统一在最顶层搞个TRY CATCH. 这种是分成正常和异常两种情况, 能说明 程序正常和异常两种情况,但不能说明结果是否对错.

[-一个大坑]

引用 10 楼 mirror030 的回复:

呵呵 其实这是一种偷懒的写法. 不管里面多少异常.统一在最顶层搞个TRY CATCH. 这种是分成正常和异常两种情况, 能说明 程序正常和异常两种情况,但不能说明结果是否对错.

确实，这个是分层的basedata，没啥逻辑，就是调用BLL层，BLL再调用DAL框架进行增删改查，里面的程序没写try，主要也没啥写的

[非专业开发Five]

3
4
5
6
7
8
9
10
11
12
13
14
public IActionResult DeleteContractTag(string tagName)
        {
            string result="ok";
            try
            {
                _service.Delete(tagName);
                _unitOfWork.Commit();
            }
            catch (Exception ex)
            {
                result="刪除失敗，原因：" + ex.Message;
            }
            return result;
        }

我以前觉得这样可读性好些，后来觉得都差不多，这种合适捕捉错误但是返回ok+错误的那种多步骤多结果处理吧。

[shoppo0505]

你这种情况要放try里面，因为只有在没有error的时候，才会返回这个值

[维秀斯丢丢]

没有规范， try里面有return下面 的return就没有意义了。

[维秀斯丢丢]

引用 1 楼 shoppo0505 的回复:

你这种情况要放try里面，因为只有在没有error的时候，才会返回这个值

异常不做抛出，那么肯定是外面的return有效。怎么能放里面。 异常抛出的话，有没有返回值都没意义。

[zhanglong_longlong]

没有强制要求，看个人风格

[大鱼>]

看你自己怎么样舒服了，一般我是写在里面，代码看着整洁。

[shoppo0505]

引用 3 楼 twotuli_software 的回复:

[quote=引用 1 楼 shoppo0505 的回复:] 你这种情况要放try里面，因为只有在没有error的时候，才会返回这个值

异常不做抛出，那么肯定是外面的return有效。怎么能放里面。 异常抛出的话，有没有返回值都没意义。[/quote] 我说放在try里面，不是catch。 放外面肯定是不对的。

[大葡萄八块一斤]

尽早返回原则，你举例的是没啥差别

[-一个大坑]

引用 7 楼 chb345536638 的回复:

尽早返回原则，你举例的是没啥差别

拿一个简单的，麻烦的长了要往下拉。