java - 我的代码是否过于程序化？

Question

最近有人看了我的代码并评论说它太程序化了。需要明确的是，他们看到的代码并不多——只是清楚地概述了在应用程序中采取的逻辑步骤的部分。

if(downloadFeeds(ftpServer, ftpUsername, ftpPassword, getFtpPathToLocalPathMap())) {
    loadDataSources();
    initEngine();
    loadLiveData();
    processX();
    copyIds();
    addX();
    processY();
    copyIds();
    addY();
    pauseY();
    resumeY();
    setParameters();
}

然后这些不同的方法创建了一大堆不同的对象，并根据需要在这些对象上调用不同的方法。

我的问题是 - 是否有一段代码明确驱动您的应用程序(例如此代码)是否表明过程编程，如果是这样，哪种更面向对象的方法可以实现相同的结果？

非常感谢所有评论!

Answer 1

嗯，这段代码所在的类显然承担了太多责任。我不会把所有的东西都隐藏在一个外观中，而是将所有与某些 ftp 引擎、数据源和其他实体相关的东西都放在一个单一的上帝对象(反模式)中，应该有一个业务流程，它有所有这些类型的实体。

所以代码看起来更像这样:

if(downloadFeeds(ftpServer, ftpUsername, ftpPassword, getFtpPathToLocalPathMap())) {
    datasource.load();
    engine.init();
    data.load();
    engine.processX(data);
    data.copyIds()
    foo.addX();
    engine.processY();
    // ...
}

数据源、引擎和所有其他组件可能会被注入(inject)到您的业务流程中，因此 a) 测试变得更加容易，b) 交换实现得到简化，c) 代码重用成为可能。

请注意，一段程序化的代码并不总是糟糕的:

class Person {
    public void getMilk() 
    {
        go(kitchen);
        Glass glass = cupboard.getGlass(); 
        fridge.open(); 
        Milk milk = fridge.getMilk(); 
        use(glass, milk);
        drink(glass);
    } 
    // More person-ish stuff
}

虽然该代码显然看起来是程序性的，但它可能没问题。它完全清楚这里发生了什么并且不需要任何文档(来自马丁的干净代码鼓励这样的代码)。只需牢记单一责任原则和所有其他基本 OOP 规则。