Fehler in Caesar-Chiffre

[Tete]

Moin Leute,
ich denke mein Fehler ist wahrscheinlich für Euch zum Lachen. Für meine Facharbeit soll ich zwei historische Chiffren schreiben. Da ich zu einfachen Erklärung als erste Chiffre die Caesar-Chiffre gewählt habe versuche ich diese nun zu programmieren. Bei meinem bisherigen Ansatz wird nach Eingabe des Schlüssels erneut gefragt welche Chiffre ich benutzen soll. Ehrlich gesagt bin ich etwas ratlos wieso ich nicht den chiffrierten Text sehe sondern das. Vielleicht übersehe ich auch etwas unübersehbares, aber vielleicht könnt Ihr mir helfen. (Ja, ich weiß es ist nicht kompakt und schön programmiert, aber meine Lehrerin meint, man soll erkennen, dass ich es geskriptet habe und es nicht nur "Copy-Pasted" ist.

[CODE lang="java" title="Main"]import java.util.*;
public class Main {

public static Scanner sc = new Scanner(System.in);
public static int eingabe;
public static boolean errEingabe = true;

public static void main(String[] args) {
System.out.println("Facharbeit Kryptographie von ");
System.out.println();
System.out.println("Wählen Sie folgend wie Sie Ihren Klartext mittels historischer Chiffre chiffrieren wollen:");


while(errEingabe = true) {
System.out.println("(1) Caesar-Chiffre (2) Chiffre");
eingabe = sc.nextInt();
if (eingabe == 1) {
errEingabe = false;
encryptCaes.enterKlartext();
}
else if (eingabe == 2) {
errEingabe = false;
}
else {
System.out.println("Ihre Eingabe ist ungültig. Wählen Sie bitte:");
}
}
}
}
[/CODE]
[CODE lang="java" title="encryptCaes"]import java.util.*;

public class encryptCaes {

public static Scanner ScCaes = new Scanner(System.in);
public static String wordBE;
public static int wordLength;
public static int key;
public static char[] WE;
public static char[] encryptedWE;

public encryptCaes() {

}
public static void encrypt(int pKey) {
for(int j=0;j<WE.length;j++) {
if(WE[j]<'A') {

}
else if(WE[j]>'@'&& WE[j]<'a') {
encryptedWE[j] = (char) (WE[j] + pKey);
if (encryptedWE[j]>'z') {
encryptedWE[j] = (char) (encryptedWE[j]-'a');
}
}
else if(WE[j]>'_') {

}

}
System.out.println();
for(int i =0; i<encryptedWE.length;i++) {
System.out.print(encryptedWE);
}
}
public static void enterKlartext() {
System.out.println("Caeser Verschlüsselung / Chiffrierung");
System.out.println("");
System.out.println("Bitte geben Sie folgend einen kurzen Satz ohne Umlaute und Sonderzeichen ein");
wordBE = ScCaes.next();
wordLength = wordBE.length();

char charArray[] = wordBE.toCharArray();
WE = new char [charArray.length];
encryptedWE = new char [charArray.length];

for(int i=0;i<charArray.length;i++) {
WE = charArray;
}
System.out.println("Geben Sie folgend den Schlüssel zur Verschlüsselung ein: ");
key = ScCaes.nextInt();
encryptCaes.encrypt(key);



}

}
[/CODE]

Vielen Dank !

 

[temi]

while(errEingabe = true)

Das ist kein Vergleich, sondern eine Zuweisung. Gleichheit prüft man mit ==

Die while-Schleife erwartet in der Klammer einen boolean. Da allerdings die Variable errEingabe (blöde Bezeichnung!) bereits vom Typ boolean ist, kannst du auch einfach schreiben

while(errEingabe)

 

[Tete]

Vielen Dank schonmal! Das hat den Fehler mit dem wiederholen der Frage behoben, jetzt bekomme ich allerdings nach Eingabe eines Schlüssels und eines Klartexts keine Ausgabe

 

[temi]

Vielen Dank schonmal! Das hat den Fehler mit dem wiederholen der Frage behoben, jetzt bekomme ich allerdings nach Eingabe eines Schlüssels und eines Klartexts keine Ausgabe

Dann gib es halt aus! In dem obigen Code sehe ich keine Ausgabe.

EDIT: Ah, ich sehe, es gibt eine Ausgabe in der Methode encrypt(). Wenn da nichts ausgegeben wird, ist das Array u. U. leer.

Insgesamt solltest du das Design noch mal etwas überdenken und vor allem Ein- und Ausgabe von der Verschlüsselung trennen, z. B.

public class Caesar {
   
    public String encrypt(int key, String text) {
       
    }
   
    public String decrypt(int key, String text) {
       
    }
}

D.h. in die Methode wird der Schüssel und der Text hineingegeben und das Ergebnis wird von der Methode zurückgegeben.

Außerdem bitte noch die Java-Konventionen beachten: Klassennamen beginnen mit einem Großbuchstaben.

Generell sollte das einzige "static" vor der main() Methode stehen und sonst nirgends, aber ich kenne deinen Kenntnisstand nicht.

 

[Tete]

Bezüglich des Statics wurde ich von Eclipse aufgefordert auch alles in der nebenklasse mit diesem zu versehen... Fand ich auch komisch, da ich das sonst eigentlich auch nie mache. Ich setze mich nochmal dran und melde mich dann nochmal, aber so oder so schonmal vielen Dank

 

[Tete]

Kurze Frage: Wiese meint meint Eclipse ich soll auch in der Nebenklasse ein Static einfügen?
[CODE lang="java" title="Main"]package crypt;
import java.util.*;
public class Main {

public static Scanner sc = new Scanner(System.in);
public static int eingabe,eingabeCaesar;
public static boolean errorEingabe = true;

public static void main(String[] args) {
System.out.println("Facharbeit Kryptographie von ");
System.out.println();
System.out.println("Wählen Sie folgend wie Sie Ihren Klartext mittels historischer Chiffre chiffrieren wollen:");

while(errorEingabe) {
System.out.println("(1) Caesar-Chiffre (2) Chiffre");
eingabe = sc.nextInt();

if (eingabe == 1) {
errorEingabe = false;
System.out.println("Caesar-Chiffre");
System.out.println("Möchten Sie eine Chiffrierung (1) oder eine Dechiffrierung (2) durchführen?");
eingabeCaesar = sc.nextInt();
if(eingabeCaesar == 1) {
Caesar.encrypt(sc.next(),sc.nextInt());
}
else if (eingabeCaesar == 2) {
Caesar.decrypt(sc.next(),sc.nextInt());
}
else {
System.out.println("Ihre Eingabe ist ungültig.");
errorEingabe = true;
}
}
else if (eingabe == 2) {
errorEingabe = false;
}
else {
System.out.println("Ihre Eingabe ist ungültig. Wählen Sie bitte:");
}
}
}
}
[/CODE]
[CODE lang="java" title="Caesar"]package crypt;

public class Caesar {

public char[] preEncryption, preDecryption;
public String finishedEncryption,finishedDecryption;

public String encrypt(String pText, int pKey) {
char charArray[] = pText.toCharArray();
for (int i=0;i<charArray.length; i++) {
preEncryption = charArray;
}
for (int i=0;i<charArray.length; i++) {
preEncryption = (char) (preEncryption+pKey);
}
finishedEncryption = new String(preEncryption);
return finishedEncryption;
}
public String decrypt(String pText, int pKey) {
char charArray[] = pText.toCharArray();
for (int i=0;i<charArray.length; i++) {
preDecryption = charArray;
}
for (int i=0;i<charArray.length; i++) {
preDecryption = (char) (preDecryption-pKey);
}
finishedDecryption = new String(preDecryption);
return finishedDecryption;
}
}[/CODE]

 

[temi]

Nicht statische Methoden benötigen eine Instanz der Klasse, auf der man sie aufruft.

Caesar caesar = new Caesar(); // erzeugt eine Instanz der Klasse

caesar.encrypt(...);

Das ist wie beim Scanner. Da erzeugst du auch zuerst eine Instanz der Klasse und auf dieser rufst du die Methoden auf.

 

[Tete]

Dankeschön