Hilfe benötigt

[Frey]

Hallo,

ich hab Fragen zu dem untenstehenden Programm...

1. Gibt es Verbesserungsvorschläge?
2. Wie bring ich in println nen Zeilenabstand rein(Weil is ja alles direkt untereinander gemacht)?

Danke für eure Unterstützung

Frey

/*Das ist der Programmstart.Damit tituliere ich den Start und den Namen*/
/*Funktion importiert. New Line       \t = Tabulator in println*/
public class Substitution {
  public static void main(String[] args) {
  System.out.println("Dieses Java-Programm verschlüsselt Ihre Eingabe.");
    String Text = "";
    String Crypt = "";
    for (int i = 0; i <= Text.length() -1; i++){
      char c = Text.charAt(i);
      switch (c){
        case 'A': c = 'X'; break;
        case 'B': c = 'O'; break;
        case 'C': c = 'I'; break;
        case 'D': c = 'T'; break;
        case 'E': c = 'U'; break;
        case 'F': c = 'R'; break;
        case 'G': c = 'A'; break;
        case 'H': c = 'M'; break;
        case 'I': c = 'P'; break;
        case 'J': c = 'W'; break;
        case 'K': c = 'C'; break;
        case 'L': c = 'Q'; break;
        case 'M': c = 'H'; break;
        case 'N': c = 'Z'; break;
        case 'O': c = 'F'; break;
        case 'P': c = 'K'; break;
        case 'Q': c = 'D'; break;
        case 'R': c = 'Y'; break;
        case 'S': c = 'V'; break;
        case 'T': c = 'E'; break;
        case 'U': c = 'B'; break;
        case 'V': c = 'G'; break;
        case 'W': c = 'J'; break;
        case 'X': c = 'S'; break;
        case 'Y': c = 'L'; break;
        case 'Z': c = 'N'; break;
        case 'a': c = 'x'; break;
        case 'b': c = 'o'; break;
        case 'c': c = 'i'; break;
        case 'd': c = 't'; break;
        case 'e': c = 'u'; break;
        case 'f': c = 'r'; break;
        case 'g': c = 'a'; break;
        case 'h': c = 'm'; break;
        case 'i': c = 'p'; break;
        case 'j': c = 'w'; break;
        case 'k': c = 'c'; break;
        case 'l': c = 'q'; break;
        case 'm': c = 'h'; break;
        case 'n': c = 'z'; break;
        case 'o': c = 'f'; break;
        case 'p': c = 'k'; break;
        case 'q': c = 'd'; break;
        case 'r': c = 'y'; break;
        case 's': c = 'v'; break;
        case 't': c = 'e'; break;
        case 'u': c = 'b'; break;
        case 'v': c = 'g'; break;
        case 'w': c = 'j'; break;
        case 'x': c = 's'; break;
        case 'y': c = 'l'; break;
        case 'z': c = 'n'; break;
        case ' ': c = '*'; break;
        case '!': c = '='; break;
        case '?': c = '+'; break;
        case '.': c = '¢'; break;
        default : ;
      }
      Crypt=Crypt+c;
    }
    System.out.println("Verschlüsselter Text:    "+Crypt);
    String Decrypt = "";
    for (int i = 0; i<= Crypt.length() -1; i++){
      char c = Crypt.charAt(i);
      switch (c){
        case 'X': c = 'A'; break;
        case 'O': c = 'B'; break;
        case 'I': c = 'C'; break;
        case 'T': c = 'D'; break;
        case 'U': c = 'E'; break;
        case 'R': c = 'F'; break;
        case 'A': c = 'G'; break;
        case 'M': c = 'H'; break;
        case 'P': c = 'I'; break;
        case 'W': c = 'J'; break;
        case 'C': c = 'K'; break;
        case 'Q': c = 'L'; break;
        case 'H': c = 'M'; break;
        case 'Z': c = 'N'; break;
        case 'F': c = 'O'; break;
        case 'K': c = 'P'; break;
        case 'D': c = 'Q'; break;
        case 'Y': c = 'R'; break;
        case 'V': c = 'S'; break;
        case 'E': c = 'T'; break;
        case 'B': c = 'U'; break;
        case 'G': c = 'V'; break;
        case 'J': c = 'W'; break;
        case 'S': c = 'X'; break;
        case 'L': c = 'Y'; break;
        case 'N': c = 'Z'; break;
        case 'x': c = 'a'; break;
        case 'o': c = 'b'; break;
        case 'i': c = 'c'; break;
        case 't': c = 'd'; break;
        case 'u': c = 'e'; break;
        case 'r': c = 'f'; break;
        case 'a': c = 'g'; break;
        case 'm': c = 'h'; break;
        case 'p': c = 'i'; break;
        case 'w': c = 'j'; break;
        case 'c': c = 'k'; break;
        case 'q': c = 'l'; break;
        case 'h': c = 'm'; break;
        case 'z': c = 'n'; break;
        case 'f': c = 'o'; break;
        case 'k': c = 'p'; break;
        case 'd': c = 'q'; break;
        case 'y': c = 'r'; break;
        case 'v': c = 's'; break;
        case 'e': c = 't'; break;
        case 'b': c = 'u'; break;
        case 'g': c = 'v'; break;
        case 'j': c = 'w'; break;
        case 's': c = 'x'; break;
        case 'l': c = 'y'; break;
        case 'n': c = 'z'; break;
        case '*': c = ' '; break;
        case '=': c = '!'; break;
        case '+': c = '?'; break;
        case '¢': c = '.'; break;
        default : ;
      }
      Decrypt = Decrypt+c;
    }
    System.out.println("Entschlüsselter Text:    "+Decrypt);
    /*System.out.println("(C) by Raphael Frey");*/
  }
}

 

[Jango]

2. Wie bring ich in println nen Zeilenabstand rein(Weil is ja alles direkt untereinander gemacht)?

\n

 

[Schandro]

Du kannst einfach ein array Alphabet machen
hab ich schonmal gemacht, deswegen kopier ichs einfach mal rein um dir zeit zu sparen.
String[] Alphabet = {"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"};
damit kannst du einfacher auf die Buchstaben zugreifen, auch auf die Großbuchstaben per
.toUpperCase();

 

[Frey]

Und die Sonderzeichen? kann ich die auch in den String werfen?

 

[Ocean-Driver]

Klar, warum nicht?

 

[Frey]

OK, dass mit den Sonderzeichen schnall ich...

Aber kann mir wer ein Bsp. zu meinem machen mit dem array Alphabet, weil dass schnall ich nicht...

THX

 

[Schandro]

achso tschuldigung hab net genau hingesehen. Hab gedacht die Zeichen würden nur z.b. um 10 Stellen verschoben werden wie bei der cäsarischen Verschlüsselung. Deine Verschlüsselung hat ja anscheinend kein solches System.

Könntest das natürlich trotzdem mit 2 Arrays machen, also im ersten stehen alle Zeichen die zu ersetzen sind und im zweiten die die dafür eingesetzt werden sollen.

bsp.

String[] DeCrypt = {"a","b","c"};
String[] Crypt   = {"u","-","1"};

for(int i=0;i<DeCrypt.length;++i){
       Text = Text.replace(DeCrypt[i],Crypt[i]);}

// a wird zu u, b zu - und c zu 1

Damit könntste 1. etliche Zeilen sparen und hättest viel weniger Arbeit bei der Dechiffrierung, dafür bräuchtest du nämlich das replace in der schleife einfach nur umzudrehen.

 

[Frey]

ach, jetzt is klar...

Danke an alle