Ceasar Verschlüsselung

[Veraenderer]

Hi,
ich habe ein Problem:
Ich habe ein Program programmiert welches einen eingegeben Text verschlüsseln soll. Allerdings kommt es sobald ich den Text mit der Caesar Methode verschlüssle zu folgenden Fehler:
Die Buchstaben werden zwar verschlüsselt (und es kommt zu keiner Fehlermeldung), aber die Buchstaben werden falsch verschlüsselt.
Also eigentlich soll aus einem a ein f werden aber es wird ein d. Das ganze wird noch mysteriöser dadurch das alle Buchstaben entweder zu a,b,c,d oder e verschlüsselt werden. Also werden z.B. alle a, g, j,1 usw. zu d's.

Da der Fehler schwierig zu erklären ist habe ich das Program angehängt.

Ich vermute der Fehler liegt in der Schleife mit den vielen if's.

Ich würde mich auch darüber freuen, wenn ihr mir Tipps geben könntet wie ich Teile die funktionieren eleganter lösen könnte.

Spoiler: relevanter Code:

public class Verschluessel
{

    String input;
    public Verschluessel(String input1)
    {
        input=input1;
        
    }
    //Dreht den Text um
    public void reverse () {
        input=new StringBuffer(input).reverse().toString();
    }
    //Caesar Verfahren
    public void caesar () {
        int x=5;
        String alphabetl="abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ1234567890";
        char [] alphabet=new char[alphabetl.length()];
        alphabet=alphabetl.toCharArray();
        char [] theText = new char [input.length ()];
        theText=input.toCharArray();
        char [] newalphabet=new char[alphabetl.length()];
        System.out.println (alphabetl.length ());
        for (int i=0; i<alphabetl.length(); i++) {
            if ((i+x)<alphabetl.length()) {
            newalphabet [i]=alphabet[i+x];
        }
        else {
            newalphabet [i]=alphabet[i+x-alphabetl.length()];
        }
    
    } 
    System.out.println (theText);
        for (int i=0; i<input.length();i++) {
            if (theText[i]=='a') {
            theText[i]=newalphabet[0];
                        System.out.println (newalphabet[0]);
            }
            if (theText[i]=='b') {
            theText[i]=newalphabet[1];
                        System.out.println (newalphabet[1]);
            }
            
            if (theText[i]=='c') {
            theText[i]=newalphabet[2];
                        System.out.println (newalphabet[2]);
            }
            if (theText[i]=='d') {
            theText[i]=newalphabet[3];
                        System.out.println (newalphabet[3]);
            }
            if (theText[i]=='e') {
            theText[i]=newalphabet[4];
                        System.out.println (newalphabet[4]);
            }
            if (theText[i]=='f') {
            theText[i]=newalphabet[5];
                        System.out.println (newalphabet[5]);
            }
             
            if (theText[i]=='g') {
            theText[i]=newalphabet[6];
                        System.out.println (newalphabet[6]);
            }
            if (theText[i]=='h') {
            theText[i]=newalphabet[7];
            }
            if (theText[i]=='i') {
            theText[i]=newalphabet[8];
            }
            if (theText[i]=='j') {
            theText[i]=newalphabet[9];
            }
            if (theText[i]=='k') {
            theText[i]=newalphabet[10];
            }
            if (theText[i]=='m') {
            theText[i]=newalphabet[11];
            }
            if (theText[i]=='n') {
            theText[i]=newalphabet[12];
            }
            if (theText[i]=='l') {
            theText[i]=newalphabet[13];
            }
            if (theText[i]=='o') {
            theText[i]=newalphabet[14];
            }
            if (theText[i]=='p') {
            theText[i]=newalphabet[15];
            }
            if (theText[i]=='q') {
            theText[i]=newalphabet[16];
            }
            if (theText[i]=='r') {
            theText[i]=newalphabet[17];
            }
            if (theText[i]=='s') {
            theText[i]=newalphabet[18];
            }
            if (theText[i]=='t') {
            theText[i]=newalphabet[19];
            }
            if (theText[i]=='u') {
            theText[i]=newalphabet[20];
            }
            if (theText[i]=='v') {
            theText[i]=newalphabet[21];
            }
            if (theText[i]=='w') {
            theText[i]=newalphabet[22];
            }
            if (theText[i]=='x') {
            theText[i]=newalphabet[23];
            }
            if (theText[i]=='y') {
            theText[i]=newalphabet[24];
            }
            if (theText[i]=='z') {
            theText[i]=newalphabet[25];
            }
            if (theText[i]=='A') {
            theText[i]=newalphabet[26];
            }if (theText[i]=='B') {
            theText[i]=newalphabet[27];
            }if (theText[i]=='C') {
            theText[i]=newalphabet[28];
            }if (theText[i]=='D') {
            theText[i]=newalphabet[29];
            }if (theText[i]=='E') {
            theText[i]=newalphabet[30];
            }if (theText[i]=='F') {
            theText[i]=newalphabet[31];
            }if (theText[i]=='G') {
            theText[i]=newalphabet[32];
            }if (theText[i]=='H') {
            theText[i]=newalphabet[33];
            }if (theText[i]=='I') {
            theText[i]=newalphabet[34];
            }if (theText[i]=='J') {
            theText[i]=newalphabet[35];
            }if (theText[i]=='K') {
            theText[i]=newalphabet[36];
            }if (theText[i]=='L') {
            theText[i]=newalphabet[37];
            }if (theText[i]=='M') {
            theText[i]=newalphabet[38];
            }if (theText[i]=='N') {
            theText[i]=newalphabet[39];
            }if (theText[i]=='O') {
            theText[i]=newalphabet[40];
            }if (theText[i]=='P') {
            theText[i]=newalphabet[41];
            }if (theText[i]=='Q') {
            theText[i]=newalphabet[42];
            }if (theText[i]=='R') {
            theText[i]=newalphabet[43];
            }if (theText[i]=='S') {
            theText[i]=newalphabet[44];
            }if (theText[i]=='T') {
            theText[i]=newalphabet[45];
            }if (theText[i]=='U') {
            theText[i]=newalphabet[46];
            }if (theText[i]=='V') {
            theText[i]=newalphabet[47];
            }if (theText[i]=='W') {
            theText[i]=newalphabet[48];
            }if (theText[i]=='X') {
            theText[i]=newalphabet[49];
            }if (theText[i]=='Y') {
            theText[i]=newalphabet[50];
            }if (theText[i]=='Z') {
            theText[i]=newalphabet[51];
            }if (theText[i]=='1') {
            theText[i]=newalphabet[52];
            }if (theText[i]=='2') {
            theText[i]=newalphabet[53];
            }if (theText[i]=='3') {
            theText[i]=newalphabet[54];
            }if (theText[i]=='4') {
            theText[i]=newalphabet[55];
            }if (theText[i]=='5') {
            theText[i]=newalphabet[56];
            }if (theText[i]=='6') {
            theText[i]=newalphabet[57];
            }if (theText[i]=='7') {
            theText[i]=newalphabet[58];
            }if (theText[i]=='8') {
            theText[i]=newalphabet[59];
            }if (theText[i]=='9') {
            theText[i]=newalphabet[60];
            }if (theText[i]=='0') {
            theText[i]=newalphabet[61];
            }

        }
        System.out.println (theText);
        input="".valueOf (theText);
    }
    public String output () {
        return input;
    }
}

mfG Veraenderer

 

[Final_Striker]

Ich vermute der Fehler liegt in der Schleife mit den vielen if's.

Du veränderst einen Buchstaben nicht nur einmal sondern mehrfach.
A-> F
F-> L
...

Ich würde mich auch darüber freuen, wenn ihr mir Tipps geben könntet wie ich Teile die funktionieren eleganter lösen könnte.

Den neuen

chars

kann man direkt ausrechnen und braucht keine

newalphabet

 

[JCODA]

Das ist eine meiner zahlreichen Implementierungen davon, eher schlecht, aber "einfacher".

Spoiler: Click

public class Caesar {
	public static void main(String[] args) {		
			System.out.println(encrypt(-7,"Lpult ylpjolu Thuul klt dbykl zlpul Myhb ryhur, buk hsz zpl mblosal khzz poy Lukl olyhu rht, yplm zpl poy lpugpnlz Avljoalyslpu gb zpjo huz Ilaa buk zwyhjo \"splilz Rpuk, islpi myvtt buk nba, zv dpyk kpy kly splil Nvaa pttly ilpzalolu, buk pjo dpss cvt Opttls hbm kpjo olyhi ispjrlu, buk dpss bt kpjo zlpu.\" Khyhbm aoha zpl kpl Hbnlu gb buk clyzjoplk. Khz Thlkjolu nplun qlklu Ahn opuhbz gb klt Nyhil kly Tbaaly buk dlpual, buk ispli myvtt buk nba. Hsz kly Dpualy rht, kljral kly Zjoull lpu dlpzzlz Abljoslpu hbm khz Nyhi, buk hsz kpl Zvuul pt Mybloqhoy lz dplkly olyhinlgvnlu ohaal, uhot zpjo kly Thuu lpul huklyl Myhb."));
	}	
	static char[] alph = new char[26];		
	static{
		for(char i = 'a';i<='z';i++){
			alph[i-'a'] = i;					
		}
	}	
	public static String encrypt(int shift, String s){
		char[] arr = s.toCharArray();		
		StringBuilder sb = new StringBuilder();			
		for(int i = 0 ; i< arr.length;i++){
			char toAppend;
			if(Character.isAlphabetic(arr[i])){
				char old = Character.toLowerCase(arr[i]);				
				int index = ((old-'a'+shift)%alph.length+alph.length)%alph.length;				
				toAppend = alph[index];				
				if(!Character.isLowerCase(arr[i])){			
					toAppend = Character.toUpperCase(toAppend);
				}								
			}else{
				toAppend=arr[i];
			}
			sb.append(toAppend);
		}		
		return sb.toString();
	}	
}

 

[Phash]

du kannst mal eine ASCII Tabelle nehmen, und schauen, welchen Zusammenhang es da zwischen den Buchstaben und deren Wert gibt.

Daraus kannst du einen "Algorithmus" ableiten, wie man den neuen Wert berechnen kann

 

[Gast2]

statt der if's könntest du auch switch-case-anweisungen nehmen.
Aber ich würde sowieso empfehlen, den Ascii-Wert(char->int) um den Schlüssel zu erhöhen; dann brauchst du nicht die ganzen Fälle

 

[Veraenderer]

Danke für eure schnellen Antworten .

@Final_Striker
Irgendwie habe ich schon geahnt das der Fehler simpel ist ^^ .
@Dem Rest
Nochmals vielen Dank für eure Tipps und das Code Beispiel.

 

[Daassan]

noch ein paar dinge da du, entschuldige, aber recht viel dünnsinn da stehen hast...

bei der berechnung des schlüße wertes könntest % verwenden
dann kannst dir das if inner halb der schleife weglassen

for (int i=0; i<alphabetl.length(); i++) {
    if ((i+x)<alphabetl.length()) {
        newalphabet [i]=alphabet[i+x];
    } else {
        newalphabet [i]=alphabet[i+x-alphabetl.length()];
}

DIGITIERT ZUUUUUUUUU:

for (int i=0; i<alphabetl.length(); i++) {
        newalphabet[i]=alphabet[(i+x)%alphabet.length];
}

Hier:

char [] alphabet=new char[alphabetl.length()];
alphabet=alphabetl.toCharArray();
char [] theText = new char [input.length ()];
theText=input.toCharArray();
char [] newalphabet=new char[alphabetl.length()];

alphabet und theText wieso erzeugst du ein neues leeres array?
das ist unnötig und verbraucht nur ressourcen
da du eh in der nächsten zeile die Referenzen ersetzt
dann kommt mr GC und wirft frische Objekte weg
das is so als würdest du ne Packung käse kaufen
wegwerfen
und mit alten vorhanden käse ersetzen

objekte sidn wie nahrungsmittel
nich erzeugen und unbenutzt wegwerfen

joa und das rießen-if kannst einfach einfach umgehen indem du den char in nen byte umwandelst
aber das wurde ja shcon erwähnt ;D

viele grüße

PS. die nächste stufe wäre dann Vigenère cipher? ^^

 

[Veraenderer]

@Dassan
Danke.
Zu meiner Verteidigung muss ich erwähnen, dass diese Methode schon 3mal fast komplett überarbeitet hatte und dabei ist eben immer mehr Müll zusammen gekommen .

 

[Daassan]

^^ np
naja musst dir da ne gewisse disziplin angewöhnen
sonst produziert man schnell sowas ;D

naja viel spaß und erfolg noch

 

[strußi]

Einfach lösung ist
String/Char[] unverschlüsselt
char[] schlüssel //verschobenes Alphabet

private char[] schlüssel( char[] alphabet, verschiebung){
for( int i =verschriebung; i <alphabet.length; i++){
char[ i-verschiebung] schlüssel = alphabet[ i];
}
for( int i=0; i <verschiebung; i++){
char[ alphabet.lenth -verschiebung +i] schlüssel = alphabet[ i];
}
}

Char[] verschlüsselt
//variante String
for( int i =0; i < unverschlüsselt.length(); i++){
If (unverschlüsselt[ 0].equals( 'a')){
verschlüsselt[ i] =schlüssel;
else if(unverschlüsselt[ 0].equals( 'b'){
verschlüsselt[ i] =schlüssel[ 1];
... else{}
}

oder du findes eine schönere Lösung

wenn du nur auf kleinbuchstaben und ohne sonderzeichen verschlüsseln willst