無法讓我的密碼實驗室工作

Question

我需要我的chiper實驗室的幫助。我的指令是：

Write a program that accepts any number of strings as command-line arguments and displays those strings encrypted with the Atbash cipher. You program should be as modular as possible and use good object oriented programming techniques. Your program must be thoroughly documented using javadoc comments.

我有String alphabet = "ABCDEFGHIJKLMNOPQRSTUVWXYZ";應該編碼字符串，這樣一個將返回Z和B將返回Y和等。我在Eclipse中完成了我的密碼實驗，並且沒有運行。我不確定我做錯了什麼。

public class CaesarCipher { 
    public static void main(String[] args) { 
    CaesarCipher cc = new CaesarCipher(); 
    } 

    public static final int ALPHASIZE = 26; 
    public static final char [] alpha = {'A','B','C','D','E','F','G','H','I','J','K','L','M','N','O','P','Q','R','S','T','U','V','W','X','Y','Z'}; 
    protected char[] encrypt = new char[ALPHASIZE]; 
    protected char[] decrypt = new char[ALPHASIZE]; 

    public CaesarCipher() { 
    for (int i=0; i<ALPHASIZE; i++) 
     encrypt[i] = alpha[(i + 3) % ALPHASIZE]; 
    for (int i=0; i<ALPHASIZE; i++) 
     decrypt[encrypt[i] - 'A'] = alpha[i]; 
    } 

    /** Encryption Method */ 
    public String encrypt(String secret) { 
    char[] mess = secret.toCharArray(); 
    for (int i=0; i<mess.length; i++) 
     if (Character.isUpperCase(mess[i])) 
     mess[i] = encrypt[mess[i] - 'A']; 
    return new String(mess); 
    } 

    /** Decryption Method */ 
    public String decrypt(String secret) { 
    char[] mess = secret.toCharArray(); 
    for (int i=0; i<mess.length; i++) 
     if (Character.isUpperCase(mess[i])) 
     mess[i] = decrypt[mess[i] - 'A']; 
    return new String(mess); 
    } 
} 

Answer 1

試試這個：

public class CaesarCipher { 

    private static final int A = (int)'A'; 
    private static final int Z = (int)'Z'; 

    public static void main(String[] args) { 
     for (String s : args) 
      System.out.println(new Encoder(s)); 
    } 

    private static class Encoder { 
     private String encoded; 
     public Encoder(String s) { 
      s = s.toUpperCase(); 
      StringBuilder sb = new StringBuilder(); 
      for (char c : s.toCharArray()) 
       sb.append((char)(A + Z - (int)c)); 
      encoded = sb.toString(); 
     } 

     @Override 
     public String toString() { return encoded; } 
    } 

} 

Answer 2

所有main方法不被調用類，你的主要方法是在構造函數中

這是非常令人困惑的代碼，並且現在不在。非常好的形式。

你可能想要做的是在你的主要方法中包含大部分代碼。您可以使用「使用良好的面向對象的編程技術」，將代碼從那裏組織到其他類中。

我會做的是一樣的東西

public class CaesarCipher 
{ 
    public static void main(String[] args) 
    { 
     for(int i=0; i<args.length; i++) 
     { 
      Cypher cypher = new Cypher(args[i]); 
      System.Out.Println(cypher.Print()); 
     } 
    } 
} 

，然後在另一個文件（或者同一個文件也沒關係）

public class Cypher 
{ 
    // fields to represent your cypher 

    public Cypher(String s) 
    { 
     //load the input string into your cypher here. 
    } 

    public String Print() 
    { 
     //print the encrypted string 
    } 
} 

你可以選擇無論是在構造函數中進行加密時，打印方法或其他方式。