python - 让被调用方法处理调用者类本可以处理的业务是一个不好的做法吗？

Question

我在这里写了一些代码，并面临着这个好的/坏的实践困境:

        message_id = self.get_message_id(msg)
        Notification.set_notification_type(self, message_id)

如您所见，我从 self.get_message_id(msg) 获得了一些返回，并将其作为参数发送给 Notification.set_notification_type()。

我的问题是:我应该这样做:

        Notification.set_notification_type(self, msg)

然后，在 Notification.py 文件上:

        def set_notification_type(obj, msg):
            message_id = obj.get_message_id(msg)

也就是说，我应该在方法中接收原始内容并使用 obj 参数来处理我必须处理的内容，还是应该发送 obj 已处理的所有内容？

谢谢!

Answer 1

这将更具可读性，并且可以清楚地定义您的 set_notification_type 方法将提供的内容以及它期望的参数。

def get_notification_type(messageId):
        //Do stuff to populate notificationType
        return notificationType;

message_id = self.get_message_id(msg)
notification_type = Notification.get_notification_type(message_id)

我是c#背景的。所以语法可能有错误。尽量保持方法定义尽可能清晰，以便于重用。