什么时候适合在 Java 中为实例变量赋值？

Question

这是一个关于最佳实践的问题。在采用面向对象的方法时，我想出了三种不同的方法来做同样的事情。在我未经训练的眼中，它们似乎都没有“错误”，但我知道每种语言和风格都有其最佳实践，我想知道这三种方式中的任何一种是否违反了一些我尚未学习的“最佳实践”。

方式一：（先声明，然后在构造函数中赋值）

public class CCipher {

    private String alphabet;
    private String shiftedAlphabet;
    private int mainKey;

    public CCipher(int key){

        mainKey = key;
        alphabet = "abcdefghijklmnopqrstuvwxyz";
        shiftedAlphabet = alphabet.substring(mainKey) 
                        + alphabet.substring(0, mainKey);
    }

方式2：（同时声明和赋值）

public class CCipher {

    private String alphabet = "abcdefghijklmnopqrstuvwxyz";
    private String shiftedAlphabet;
    private int mainKey;

    public CCipher(int key){

        mainKey = key;
        shiftedAlphabet = alphabet.substring(mainKey) 
                        + alphabet.substring(0, mainKey);
    }

方式3：（有些东西在非get/set方法中初始化）

public class CCipher {

    private String alphabet;
    private String shiftedAlphabet;
    private int mainKey;

    public CCipher(int key){

        mainKey = key;
        alphabet = "abcdefghijklmnopqrstuvwxyz";

    }

    public String encrypt(String input){
        shiftedAlphabet = alphabet.substring(mainKey) 
                        + alphabet.substring(0, mainKey);
        // ... code to encrypt input ...
    }

    public String decrypt(String input){
        shiftedAlphabet = alphabet.substring(26 - mainKey) 
                + alphabet.substring(0, 26 - mainKey);
        // ... code to decrypt input
    }
}

就我个人而言，对于这个特定的家庭作业，我真的最喜欢第三种方式，因为它符合我要解决的问题的逻辑。但是错了，好吧错了……

Answer 1

第二个版本似乎还可以。但是常量确实是静态的最终字符串。

public class CCipher {

    private static final String ALPHABET = "abcdefghijklmnopqrstuvwxyz";
    private final String shiftedAlphabet;
    private final int mainKey;

    public CCipher(int key) {
        mainKey = key;
        shiftedAlphabet = ALPHABET.substring(mainKey) 
                        + ALPHABET.substring(0, mainKey);
    }

---

Christopher Schneider 指出，在加密和解密中使用了不同的移位字母。作为现实中的 CCipher 对象，可能会加密或解密，使其成为局部变量。

需要两个不同的非最终延迟初始化字段，这很麻烦。

public class CCipher {

    private static final String ALPHABET = "abcdefghijklmnopqrstuvwxyz";
    //private final String shiftedEncryptAlphabet;
    //private final String shiftedDecryptAlphabet;
    private final int mainKey;

    public CCipher(int key) {
        mainKey = key;
        shiftedAlphabet = ALPHABET.substring(mainKey) 
                        + ALPHABET.substring(0, mainKey);
    }

    public String encrypt(String input){
        String shiftedAlphabet = alphabet.substring(mainKey) 
                    + alphabet.substring(0, mainKey);
        // ... code to encrypt input ...
    }

    public String decrypt(String input){
        String shiftedAlphabet = alphabet.substring(26 - mainKey) 
            + alphabet.substring(0, 26 - mainKey);
        // ... code to decrypt input
    }

Answer 2

关于alphabet - 如果它永远不会改变，第二个 sn-p 可能最接近最佳实践，但它仍然不完全存在 - 它应该是 private static final "constant"：

public class CCipher {   
    private static final String alphabet = "abcdefghijklmnopqrstuvwxyz";
    // ...
}

关于 shiftedAlphabet - 第三个 sn-p 肯定不太有利（尽管在技术上不是“错误”） - 在每次调用 encrypt 或 decrypt 时，您重新计算不受以下影响的 shiftedAlphabet input 以任何方式。这可能没有错，但很浪费（换句话说——作为一名老师，我肯定会为此扣分，即使代码确实有效）。

总结一下 - 第二个 sn-p 可能是三个中最好的，但我会将 alphabet 的修饰符修复为 private static final。