如何重用稍有不同的代码

Question

关于如何正确设计和实现两种方法，我遇到了一些困难。

我有2种方法，A和B。方法A做2件事情，方法B只做1件事情。 方法A所做的事情之一与方法B所做的事情相同，因此从方法A内调用方法B是非常合理的。

现在的问题是，这两种方法都需要向用户发送一封电子邮件。 当我调用A时，我希望收到1封电子邮件，而当我调用B时，我也希望收到1封电子邮件。 这意味着，如果我在A内调用方法B，现在我将在执行1个操作（=调用A）的同时收到2封邮件。 为了使在B中简单包含B更具诱人性，这两种方法的设置过程实际上都是相同的，因此除了重做B中的设置之外，我还可以简单地使用A中的这些设置数据来调用方法B。但是，当直接调用B时，我没有提供设置数据，因此在这种情况下，我仍然需要进行设置。

解决此类问题的最佳方法是什么？ 我是不是该：

1. 在方法B中添加一个参数，在该方法中，我将说出是否应该发送电子邮件以及设置数据的另一个参数
2. 将两个方法分开，因为方法B在两个上下文中做的事情并不完全相同
3. ...其他建议？

PS。 我不确定stackoverflow是否真的适合这样的问题，请让我知道是否有更好的stackexchange平台。

谢谢你的任何想法

Answer 1

我也不知道这是否真的是话题。 那个...

在我看来，“发送电子邮件”是方法A需要完成的方法B工作中包含的内容之一。 因此，为什么不仅仅在方法B中实现它呢？ 然后方法A在调用方法B时免费获得它。

如果电子邮件的确切内容因是否调用方法A而有所不同，请确定…您可以向方法B添加参数以某种方式自定义电子邮件。

最后，您在细节上含糊不清。 通过从方法A调用方法B可以节省多少代码还不太清楚。如果有意义，那么我强烈支持这样的代码共享。 但是，如果我们只说一条语句，那……似乎就不那么值得打扰了； 也许只是将相同的单个语句放入每个方法中会更好。

抱歉，答案含糊。 GIGO。 :)

Answer 2

您可以将方法的返回用于我们的电子邮件内容：

private String funcA() {
    // foo
    return "Mail from funcA";
}

private String funcB() {
    funcA();      // ignoring the return value here, we later return our on
    // foo
    return "Mail from funcB";   // our own return
}

然后，您在以下功能之外发送电子邮件（仅示例代码）：

private void mainFunc() {
    String mail = condition ? funcA() : funcB();

    sendMail(mail);
}

Answer 3

我有两个想法：

• 创建表示A和B的通用功能的C方法，并使用不同的参数调用A和B中的C
• 在A内部调用B，但仅提取邮件功能以分离方法C

为了阐明第一个想法：

methodA() {
        firstAction();
        secondAction(Parameter parameterFromA);
        }

methodB() {
        secondAction(Parameter parameterFromB);
        }

Answer 4

我建议您进一步分离功能。

当前，您可以使用两种方法来描述：

void MethodA()
{
    DoThing1();
    DoThing2();
    SendEmail();
}

void MethodB()
{
    DoThing1();
    SendEmail();
}

因此，一个相当简单的答案是将两种方法中的实际功能位提取为它们自己的方法，并留下类似于上述内容的外壳。 每个DoThingX方法都可以返回构建电子邮件等所需的内容。

当然，如果DoThingX方法真的很小（例如， DoThingX两行），那么以这种方式将它们分开可能没有意义。