java - 将对象类型作为方法参数传递是否总是设计不佳的标志？

Question

假设我有以下枚举

public enum EmailType {
    FORGET_PASSWORD, ACHIEVMENT_UNLOCK, WELCOME
}

我有一个根据类型生成电子邮件主题的函数(但它仍然需要动态数据)，例如

public String generateEmailSubject(EmailType emailType, Object obj) {
    String subject;
    switch(emailType) {
        case WELCOME:
            User user = (User) obj;
            subject = "Hello " + user.getFirstName();
        case FORGET_PASSWORD:
            User user = (User) obj;
            subject = "Forget password " + user.getEmail();
            break;
        case ACHIEVMENT_UNLOCK:
            Achievment achievment = (Achievment) obj;
            subject = "Achievment Unlock:" + achievment.getTitle();
            break;
    }

    return subject;
}

这是不好的做法吗？如果是这样，处理这个问题的好设计是什么？也许每个 EmailType 都有一个单独的方法，但这可能会导致很多方法，并且当我需要更改它们时，主题不会集中。

Answer 1

你可以为此使用多态性。

interface Subjectable {
    String getSubject();
}

class Achievement implements Subjectable {
    ...
    @Override
    public String getSubject() {
        return "Achievement unlocked: " + getTitle();
    }
}

class User implements Subjectable {
    ...
    @Override
    public String getSubject() {
        return "Forgot password: " + getEmail();
    }
}

那么您不需要显式检查对象的类型:您只需对其调用 getSubject()。