2012-09-19 49 views
0

這是一個乾淨代碼類型的問題。覆蓋其他值的參數名稱

應用程序發送適當地址的電子郵件。但是當設置變量X時,它將覆蓋所有電子郵件地址。所有郵件都發送到X.變量X指定的地址正在從配置文件在部署時間閱讀和IoC容器設置 某事像:

public class EmailSender { 

    @Value("config.option.X") 
    private String X; 

    public void sendEmail() { 
     ... 
     if (X != null) message.setEmailAddress(X); 
     internalEmailSender.send(message); 
    } 
} 

,我不知道如何正確地命名X。 replacerEmailAddress? emailAddressOverrider?任何更好的想法?

回答

0

什麼是變量X被設置的上下文?我會嘗試命名它以某種方式引用該條件。

+0

它在被部署在讀取配置文件中設置。我會在我的問題中寫更多的細節 – piotrek

+0

我想我更多地指的是設置覆蓋電子郵件地址的商業原因是什麼?如果有一個簡潔的理由來設置它,試着以某種方式使用該原因命名該變量。 – chinabuffet

+0

哦。這是爲了測試。我們的應用程序發送電子郵件給用戶,但在開發和UAT我們希望它發送郵件在一個特定的地址,而不是垃圾郵件用戶 – piotrek

0

對我來說,它看起來像默認選項。

所以我將其命名爲這樣的:

公共類EmailSender {

@Value("config.option.defaultEmailValue") 
private String defaultEmailValue; 

public void sendEmail() { 
    ... 
    if (defaultEmailValue != null){ 
     message.setEmailAddress(defaultEmailValue); 
    } 
    internalEmailSender.send(message); 
} 

}