如果只使用一次，使用局部函数有什么意义吗？

Question

想象一下我有这个代码：

public void Foo()
{
    // Do bar work
    // Do baz work
    // Do foobar work
}

我意识到我可以（并且应该因为它做不止一件事）将其重构为：

public void Foo()
{
    bar();
    baz();
    foobar();
}

private void bar()    { /* do bar work */ }
private void baz()    { /* do baz work */ }
private void foobar() { /* do foobar work */ }

但后来我意识到我永远不会在Foo() 之外使用这些功能，所以这些功能只会使主页和自动完成功能变得混乱。我可以摆脱这个：

public void Foo()
{
    bar();
    baz();
    foobar();

    void bar()    { /* do bar work */ }
    void baz()    { /* do baz work */ }
    void foobar() { /* do foobar work */ }
}

这会让事情变得更整洁、更简洁，但我现在真正所做的只是让方法变得更长而不是更短。

Answer 1

这会让事情变得更整洁，更简洁，但我现在真正所做的只是让方法变得更长而不是更短。

不，你没有。您基本上是在说类似的东西，一个类具有一个执行大量工作的方法与一个执行相同工作但具有多个更短且更易于维护的方法的类之间没有任何区别。

您的本地函数就像方法一样，它们包含在另一个方法中这一事实并不排除整体更容易维护；功能被封装在明确定义的范围内。

Answer 2

我很喜欢@Mark Benningfield 使用部分文件的想法（当我的课程太大并且有一两个超级方法时，我会这样做）

我对局部函数的唯一问题是它们可以捕获变量，并且并不总是很清楚它们是否正在这样做。因此，通过将“真实”方法“推广”到“本地”，您正在扩大它的知名度。

Answer 3

本地函数比匿名函数更有优势，因为匿名函数只能通过委托调用，除了为委托分配内存之外，调用成本更高。

本地函数可以递归，而无需委托所需的技巧：

int f(int i) => i >= 1 ? i * f(i - 1) : 1;

Func<int,int> d = null;
d = (int i) => i >= 1 ? i * d(i - 1) : 1;

与匿名委托一样，与顶级方法不同，局部函数可以捕获局部变量。而且因为 thy 是本地的，所以它们不能被其他函数调用。

Answer 4

使用匿名函数：

public void Foo()
{
    Action bar = delegate () { /* do bar work */ };
    Action baz = delegate () { /* do baz work */ };
    Action foobar = delegate () { /* do foobar work */ };

    bar();
    baz();
    foobar();
}

或 lambda 表达式语法：

public void Foo()
{
    Action bar = () => { /* do bar work */ };
    Action baz = () => { /* do baz work */ };
    Action foobar = () => { /* do foobar work */ };

    bar();
    baz();
    foobar();
}