小编nar*_*aru的帖子

我有两种在我的应用程序中保存数据的方法：保存到数据库和保存到文件。由于我不希望客户端代码处理对象的构造，因此我创建了一个类（据我所知）是带有工厂方法的简单工厂。代码如下：

public static DataPersister createDataPersister(Boolean saveToDb, Session session, String filename) {
    if (saveToDb) {
        return new DatabaseDataPersister(session);
    } else {
        return new FileDataPersister(filename);
    }
}

使用此设置，客户端代码不必处理构建任何内容或决定是否保存到数据库或文件 - 它只需调用save()工厂返回的对象上的方法，如下所示：

DataPersister dataPersister = DataPersisterSimpleFactory.createDataPersister(this.savetoDb, this.session, this.filename);
dataPersister.save(this.data);

我的问题是 - 这个解决方案是否违反了 SOLID 原则？例如，为了创建一个DatabaseDataPersister客户端代码需要传递一个filename参数，并且这个实现DataPersister不会使用它。我觉得它与类似于接口隔离原则的东西并不相符，但并非如此。

如果解决方案确实是代码异味 - 我该如何清理它？