小编Ctr*_*l-C的帖子

如何分组功能没有副作用？

我有一个带有几个辅助函数的函数.这是相当普遍的情况.我想将它们分组在一个共同的上下文中以便于阅读,我想知道如何正确地做到这一点.

他们需要~15行
只有主函数从其他地方调用
没有计划在不久的将来重用辅助功能

简化示例:

def create_filled_template_in_temp(path, values_mapping):
    template_text = path.read_text()
    filled_template = _fill_template(template_text, values_mapping)
    result_path = _save_in_temp(filled_template)
    return result_path

def _fill_template(template_text, values_mapping):
    ...

def _save_in_temp(filled_template):
    _, pathname = tempfile.mkstemp(suffix='.ini', text=True)
    path = pathlib.Path(pathname)
    path.write_text(text)
    return path

...
create_filled_template_in_temp(path, values_mapping)

Run Code Online (Sandbox Code Playgroud)

请注意,我不希望模块级别的辅助方法,因为它们只属于一种方法.想象一下,在同一模块中有如上所述的几个这样的例子.模块级别的Maany非公共功能.一团糟(这种情况多次发生).另外,我想给它们上下文并使用上下文的名称来简化内部命名.

解决方案#0:一个模块

把它放在另一个模块中:

template_fillers.create_in_temp(path, values_mapping)

Run Code Online (Sandbox Code Playgroud)

问题:

添加文件的代码太少,特别是当已经有很多文件时(这会造成混乱)
这是一个动作,现在我被迫为模块创建一个基于名词的名称(或打破模块命名规则).而且简化它会使它太宽泛(在这种情况下创建一个真正是单身的集合).

最后,这只是为它添加模块的代码太少.

解决方案#1:一堂课

创建一个没有__init__且只有一个公共(通过命名约定)方法的类:

class TemplateFillerIntoTemp:
    def run(self, path, values_mapping):
        template_text = path.read_text()
        filled_template = self._fill_template(template_text, values_mapping)
        result_path = self._save_in_temp(filled_template)
        return result_path

    def _fill_template(self, template_text, values_mapping):
        ...

    def _save_in_temp(self, filled_template):
        _, …

Run Code Online (Sandbox Code Playgroud)

python

Ctr*_*l-C

2018 05-09

11
推荐指数

1
解决办法

221
查看次数