将upload_to函数与模型类相关联的首选pythonic方式？

Question

我有一个带有Django ImageField 的类，我一直在努力在存储该字段的upload_to 函数的两种选择之间做出决定。第一种方法非常简单。该函数在模块级别定义（c.f. https://stackoverflow.com/a/1190866/790075, https://stackoverflow.com/a/3091864/790075）：

def get_car_photo_file_path(instance, filename):
    ext = filename.split('.')[-1]
    filename = "%s.%s" % (uuid.uuid4(), ext) # chance of collision <1e-50
    return os.path.join('uploads/cars/photos', filename)

class CarPhoto(models.Model):
    photo = models.ImageField(upload_to=get_car_photo_file_path)

这简单易懂，但是通过添加一个实际上只与 CarPhoto 类相关的函数来污染模块范围。

在第二种方法中，我使用可调用类模式将函数与 CarPhoto 类更紧密地关联起来。这会将upload_to 函数移出模块范围，但感觉不必要地复杂。

class CarPhoto(models.Model):
    class getCarPhotoFilePath():
        # Either use this pattern or associate function with module instead of this class
        def __call__(self, instance, filename):
            ext = filename.split('.')[-1]
            filename = "%s.%s" % (uuid.uuid4(), ext) # chance of collision <1e-50
            return os.path.join('uploads/cars/photos', filename)

    photo = models.ImageField(upload_to=getCarPhotoFilePath())

我已经看到了使用 @staticmethod 和 @classmethod 装饰器的建议（c.f. https://stackoverflow.com/a/9264153/790075），但我发现当我这样做时，函数永远不会执行并且文件名最终看起来像：/path/to/file/<classmethod object>，与文件路径中嵌入的方法对象，这当然不是故意的！

以下哪一个是首选模式？有没有更好的办法？

Answer 1

我建议你：

import this

对我来说，这属于 Zen of Python 的部分说明：

Simple is better than complex.
Complex is better than complicated.

我认为您的简单解决方案更好。但是，您的情结并不过分复杂。我想你可能会没事的。只是我的两分钱。

Answer 2

有一个命名约定可以防止名称污染。

• 使用_get_car_photo_file_path 将您的函数标记为内部函数（虽然没有隐藏）；
• 在课堂外使用__get_car_photo_file_pathto prevent access。

您可以在CarPhoto 类中添加这样的类方法或静态方法，这比添加可调用类更简单（后者让我想起Java 为一种方法而定义匿名类的方式）。

名称将清楚地表明_get_car_photo_file_path 是一个实现细节，而不是接口的一部分，从而防止污染类的命名空间。作为CarPhoto的方法，该函数不会污染模块的命名空间。

Answer 3

目前在我正在使用的代码中，我们有最简单的代码的变体。唯一的区别是，由于该函数是供内部使用的，所以它带有_前缀。

def _get_car_photo_file_path(instance, filename):
    [...]

class CarPhoto(models.Model):
    photo = models.ImageField(upload_to=_get_car_photo_file_path)

但是，我确实相信这会更 Pythonic（或者更确切地说是 OOP）：

class CarPhoto(models.Model):

    @staticmethod
    def _get_file_path(instance, filename):
        [...]

    photo = models.ImageField(upload_to=_get_file_path)