Question

我正在尝试计算一些循环复杂度，因此尝试绘制一个控制流图。首先，我正在尝试使用一种相当简单的方法来制作它。

首先，我尝试将其仅绘制为try部件，如下所示： Flow Graph

这里的方法：

    [HttpPost]
    public ActionResult GraphMethod([FromForm]string str)
    {
        try
        {
            int affectedRows = this._copyManager.CreateCopy(str);
            if (affectedRows < 1) return BadRequest("Error!");

            return Ok();
        }
        catch (Exception ex)
        {
            return BadRequest(ex.Message);
        }
    }

我将如何扩展它以包括整个方法和try部分？

这是我的第一个控制流图，因此，如果我搞砸了，我也想知道。

Answer 1

就我而言，我建议您使用此代码，更简单，更有效

[HttpPost]
public ActionResult GraphMethod([FromForm]string str)
{       
        if (this._copyManager.CreateCopy(str) < 1) 
            return BadRequest("Error!");

        return Ok();      
}

Answer 2

我将创建一个TryCreateCopy方法，并执行与@saya imad的答案非常相似的操作
像这样：

[HttpPost]
public ActionResult GraphMethod([FromForm]string str)
{ 
    // These two if statements can be concatenated into one, 
    // but that would be a bit hard to read
    if (this._copyManager.TryCreateCopy(str, out var affectedRows))
        if (affectedRows > 1)
            return Ok();

    return BadRequest("Error!");
}

// _copyManager Method, there's probably a better way for you
public bool TryCreateCopy(string str, out int affectedRows)
{
    try
    {
        affectedRows = CreateCopy(str);
    }
    // Please also don't do `catch (Exception)`, 
    // if you know which exception gets thrown always catch that
    catch (Exception e)
    {
        affectedRows = -1;
        return false;
    }

    return true;
}

创建副本时不抛出异常的情况下，TryCreateCopy方法返回true；如果抛出了副本，则返回false；如果存在受影响的行数，则返回false。

*可能有比我展示的方法更好的方法（例如，validate方法？），因为try / catch非常耗费资源

如何在控制流程图中表示“尝试/捕获”？

2 个答案: