function - 函数应该有多短？

Question

我在这里进行了一些搜索，但没有找到类似的东西，所以我将继续询问。这实际上更多是关于语义而不是实际的编程问题。我目前正在用 C++ 编写一些东西，但语言并不重要。

我很清楚，尽可能缩短函数/方法是一种很好的编程习惯。但是，您如何真正知道函数是否太长？或者，是否有可能过多地分解功能？

我学习的第一种编程语言(Applesoft BASIC 除外，这不算...)是 6502 汇编语言，其中速度和优化就是一切。如果几个循环计数搞砸了整个程序的时序，通常最好直接设置内存位置或寄存器，而不是跳转到另一个子程序。前者可能需要 3 或 4 个周期，而后者总共可能需要 2 或 3 倍。

虽然我意识到现在如果我什至向一些程序员提及循环计数，他们只会给我一个白眼，但这是一个很难改掉的习惯。

具体来说，假设(再次使用 C++)我们有一个类似于以下内容的私有(private)类方法:

int Foo::do_stuff(int x) {
    this->x = x;
    // various other operations on x
    this->y = this->x;
}

我已经看到一些论点，至少，每组操作都应该是它自己的函数。例如，do_stuff() 理论上应该命名为 set_x(int x)，应该为对类成员 x 执行的一组操作编写一个单独的函数，并且应该编写第三个函数来分配类成员的最终值x 到类成员 y。但是我看到了其他论点，即每个操作都应该有自己的功能。

对我来说，这似乎是错误的。再一次，我从内部的角度来看事情；每个方法调用都将一个地址压入堆栈，执行其操作，然后从子例程返回。对于相对简单的事情来说，这似乎是很多开销。

这种事情有最佳实践还是更多取决于个人判断？

Answer 1

自 6502 汇编以来，发生了两件事:计算机变得更快，编译器(在适当的情况下)变得更智能。

现在的建议是停止将所有时间都花在为各个周期烦恼上，直到你确定这是一个问题。你可以更明智地度过这段时间。如果您向我提及周期计数，我不会茫然地看着您，因为我不知道它们是什么。我会看着你想知道你是否在浪费你的努力。

相反，开始考虑让你的函数足够小:

• 可以理解，
• 可测试，
• 可在适当的地方重复使用。

如果稍后您发现某些代码运行速度不够快，请考虑如何优化它。

注意:优化可能是提示编译器将函数内联移动，因此您仍然可以获得上述优势，而不会影响性能。