Strings und StringBuilder

[Jay1980]

Servus,

mir leuchtet nicht ein, wieso es mir da meinen Buchstaben nicht vertauscht?

Ziel ist dass die Buchstaben meines Eingabestrings getauscht werden.

So ist mein Versuch, aber es vertauscht nichts. Es scheint was mit der Unveränderlichkeit von Strings zu tun zu haben, wobei ich meine Vorhaben schon mit StringBuilder getestet habe. Der Block mit dem a wird für jedes a-Zeichen des Strings ausgeführt, aber getauscht wird es nicht. Einer eine Idee, was da schief läuft.

int stringLength = this.inputString.length();
		this.encString = this.inputString;
		
		for ( int i = 0; i < stringLength; i++)
		{
			
			this.encString = new StringBuilder(this.inputString); // Check ob das dann auch nur callbyvalue ist 
			
			char currentLetter = this.encString.charAt(i);
			
			System.out.println("Aktueller Durchlauf zum Encrypten mit currentLetter " + currentLetter);
			
			if (currentLetter == 'a') 
			{ 
				System.out.println( "Block zum Buchstaben a wird ausgeführt." );
				currentLetter = 'd';
				this.encString.setCharAt(i, currentLetter);
				System.out.println("Aktueller String nach dem a-Durchlauf: " + this.encString);
			}
			
			if (currentLetter == 'b') { currentLetter = 'e'; }
			if (currentLetter == 'c') { currentLetter = 'f'; }
			if (currentLetter == 'd') { currentLetter = 'g'; }
			if (currentLetter == 'e') { currentLetter = 'h'; }

 

[0x7F800000]

mir leuchtet nicht ein, wieso es mir da meinen Buchstaben nicht vertauscht?

Ziel ist dass die Buchstaben meines Eingabestrings getauscht werden.

So ist mein Versuch, aber es vertauscht nichts. Es scheint was mit der Unveränderlichkeit von Strings zu tun zu haben

nnein, das liegt wohl viel mehr an spaghetticode

int stringLength = this.inputString.length();

was soll eine derartige zeile überhaupt jemals in irgendeinem code zu suchen haben? ???:L

		this.encString = this.inputString;

ich hab kA was du hier vorhattest, aber das ist copy by reference, macht so im freien Gelände nur in seltensten fällen sinn.

			this.encString = new StringBuilder(this.inputString);

so macht die zeile davor erst recht keinen sinn, weil du alles wieder überschreibst...
Von welchem typ ist dieses encString? CharacterSequence oder irgendsoein murks oder wie?

			this.encString.setCharAt(i, currentLetter);

das kann nichts bringen, da du diese "encString" variable eh in jedem durchlauf durch ein neuen StringBuilder ersetzst, der den inhalt des ursprünglichen inputStrings enthält.

			if (currentLetter == 'b') { currentLetter = 'e'; }
			if (currentLetter == 'c') { currentLetter = 'f'; }
			if (currentLetter == 'd') { currentLetter = 'g'; }
			if (currentLetter == 'e') { currentLetter = 'h'; }

das ist alles andere als dry, bzw das ist einfach unschön. Ersetze das durch eine schleife, musst doch den kram nicht eine milliarde mal dahinschreiben...:noe:

 

[Jay1980]

Danke erstmal,

ich bin noch einmal durchgegangen, der Hinweis mit dem call by reference war gut, das wollte ich ohnehin noch klären und ich will natürlich kein call by reference. Das mit der Schleife mach ich mal, wenn das Programm richtig funktioniert - das ist jetzt erstmal das Hauptanliegen, wenn ich dafür 5min länger Copy&Paste machen muss, ist mir das aktuell Wurst.

Hier mal der leicht überarbeitete Code , einige Buchstaben werden schon ausgetauscht, aber nicht die passenden nach den Verzweigungsblöcken, dann erhalte ich bei der Anforderung sogar 'null' für den angezeigten encrypted String.

Kurz zum Verhalten:
Es wird erst ein Formular angeboten, darin wird der String eingegeben, dann wird die Klasse mit dem CaesarChiffre aufgerufen, dann wird je nach gedrücktem JButton entweder codiert oder decodiert. Aktuell lafft das bei mir nicht und irgendwo muss da ein Riesenschnitzer drin sein ....

Danke vorab für einen Hinweis.

public class JobaCaesarChiffre 
{
	private String inputString;
	private StringBuilder encString;
	private StringBuilder decString;
	
	/** Konstruktor */
	JobaCaesarChiffre( String flag, String instr )
	{
		this.inputString = instr;
		
		if ( flag == "encode")
		{
			this.computeEncryptedString();
		}
		
		if ( flag == "decode")
		{
			this.computeDecryptedString();
		}
		
	}
	
	/** Chiffre aus Klartext bilden */
	private void computeEncryptedString()
	{
		// Ein Buchstabe muss im Alphabet drei Plätze nach vorn verschoben werden
		// also Klartextzeichen | Chiffrezeichen
		this.encString = new StringBuilder(this.inputString);
		
		for ( int i = 0; i < this.inputString.length(); i++)
		{ 	
			char currentLetter = this.inputString.charAt(i);
			
			if (currentLetter == 'a') { currentLetter = 'd'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'b') { currentLetter = 'e'; this.encString.setCharAt(i, currentLetter); }		
			if (currentLetter == 'c') { currentLetter = 'f'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'd') { currentLetter = 'g'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'e') { currentLetter = 'h'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'f') { currentLetter = 'i'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'g') { currentLetter = 'j'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'h') { currentLetter = 'k'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'i') { currentLetter = 'l'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'j') { currentLetter = 'm'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'k') { currentLetter = 'n'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'l') { currentLetter = 'o'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'm') { currentLetter = 'p'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'n') { currentLetter = 'q'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'o') { currentLetter = 'r'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'p') { currentLetter = 's'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'q') { currentLetter = 't'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'r') { currentLetter = 'u'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 's') { currentLetter = 'v'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 't') { currentLetter = 'w'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'u') { currentLetter = 'x'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'v') { currentLetter = 'y'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'w') { currentLetter = 'z'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'x') { currentLetter = 'a'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'y') { currentLetter = 'b'; this.encString.setCharAt(i, currentLetter); }
			if (currentLetter == 'z') { currentLetter = 'c'; this.encString.setCharAt(i, currentLetter); }	
		}
	}
	
	/** Klartext aus Chiffre bilden */
	private void computeDecryptedString()
	{
		this.decString = new StringBuilder(this.inputString);
		
		for ( int i = 0; i < this.inputString.length(); i++)
		{
			char currentLetter = this.inputString.charAt(i);
			
			if (currentLetter == 'a') { currentLetter = 'x'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'b') { currentLetter = 'y'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'c') { currentLetter = 'z'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'd') { currentLetter = 'a'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'e') { currentLetter = 'b'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'f') { currentLetter = 'c'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'g') { currentLetter = 'd'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'h') { currentLetter = 'e'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'i') { currentLetter = 'f'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'j') { currentLetter = 'g'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'k') { currentLetter = 'h'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'l') { currentLetter = 'i'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'm') { currentLetter = 'j'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'n') { currentLetter = 'k'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'o') { currentLetter = 'l'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'p') { currentLetter = 'm'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'q') { currentLetter = 'n'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'r') { currentLetter = 'o'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 's') { currentLetter = 'p'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 't') { currentLetter = 'q'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'u') { currentLetter = 'r'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'v') { currentLetter = 's'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'w') { currentLetter = 't'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'x') { currentLetter = 'u'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'y') { currentLetter = 'v'; this.decString.setCharAt(i, currentLetter); }
			if (currentLetter == 'z') { currentLetter = 'w'; this.decString.setCharAt(i, currentLetter); }	
		}
	}
	
	/** Verschlüsselten String ausliefern */
	public StringBuilder getEncryptedString()
	{
		return this.encString;
	}
	
	/** Klartext String aus Chiffre liefern */
	public StringBuilder getDecryptedString()
	{
		return this.decString;
	}
}

Dann noch die actionPerformed-Methoden-Ausschnitt aus Gui-Klasse

else if ( e.getActionCommand() == "Cäsar-Chiffre")
		{
			// replace statusPanel-Content
			JTextArea jta = new JTextArea();
			jta.setBorder(null);
			jta.setEditable(false);
			jta.setLineWrap(true);
			jta.setWrapStyleWord(true);
			Dimension dim = new Dimension(150, 590);
			jta.setPreferredSize(dim);
			jta.setText(
					"Status: Ort - Kryptographie | Algorithmen | Cäsar"
					);
			
			this.statusPanel.removeAll();
			this.statusPanel.add(jta);
			this.statusPanel.revalidate();
			this.statusPanel.repaint();
			
			// replace centerPanel Content
			this.centerPanel.removeAll();
			JPanel caesarInputPanel = new JPanel();
			JLabel caesarInputHeaderPanelHeader = new JLabel( "Cäsar Chiffre" );
			JTextField caesarInputString = new JTextField( "hier steht eine Zeichenkette");
			
			// die JTextFields ein die Transfercontainer uebergeben
			this.dollarPostStringOne = caesarInputString;
			
			JButton caesarChiffreButtonOne = new JButton("Cäsar-Chiffre entschlüsseln");
			JButton caesarChiffreButtonTwo = new JButton("Cäsar-Chiffre verschlüsseln");
			caesarChiffreButtonOne.addActionListener( this );
			caesarChiffreButtonTwo.addActionListener( this );
			
			
			caesarInputPanel.add(caesarInputHeaderPanelHeader);
			caesarInputPanel.add(caesarInputString);
			caesarInputPanel.add(caesarChiffreButtonOne);
			caesarInputPanel.add(caesarChiffreButtonTwo);
			
			this.centerPanel.add( caesarInputPanel );
			this.centerPanel.revalidate();
			this.centerPanel.repaint();
			
		}
		else if ( e.getActionCommand() == "Cäsar-Chiffre entschlüsseln")
		{
			// replace centerPanel Content
			this.centerPanel.removeAll();
			
			JobaCaesarChiffre jcc = new JobaCaesarChiffre( "decode", this.dollarPostStringOne.getText() );
			
			StringBuilder dencText = jcc.getEncryptedString();
			String eingabeText = this.dollarPostStringOne.getText();
			
			JLabel jldenc = new JLabel();
			jldenc.setText(
					"Der entschlüsselte Text lautet: " + dencText
			);
			
			JLabel jleing = new JLabel();
			jleing.setText("Der Eingabe-String lautet: " + eingabeText );
			
			this.centerPanel.add(jldenc);
			this.centerPanel.add(jleing);
			
			this.centerPanel.revalidate();
			this.centerPanel.repaint();
		}
		else if ( e.getActionCommand() == "Cäsar-Chiffre verschlüsseln")
		{
			// replace centerPanel Content
			this.centerPanel.removeAll();
			
			JobaCaesarChiffre jcc = new JobaCaesarChiffre( "encode", this.dollarPostStringOne.getText() );
			
			StringBuilder encText = jcc.getEncryptedString();
			String eingabeText = this.dollarPostStringOne.getText();
			
			JLabel jlenc = new JLabel();
			jlenc.setText(
					"Der verschlüsselte Text lautet: " + encText
			);
			
			JLabel jleing = new JLabel();
			jleing.setText("Der Eingabe-String lautet: " + eingabeText );
			
			this.centerPanel.add(jlenc);
			this.centerPanel.add(jleing);
			
			this.centerPanel.revalidate();
			this.centerPanel.repaint();
		}

 

[Ark]

Strings vergleicht man in 99,9999% aller Fälle mit equals().

Kümmere dich aber nicht nur um das Programmieren, wenn du nicht gerade beim dummen Abtipper bleiben möchtest. Programmierer und Informatiker sind (für meine Begriffe) zwei unterschiedliche Dinge. Darum: finde einen mathematischen und endlich beschreibbaren Zusammenhang zwischen einer Zeichenkette und dem Schlüssel (natürlich sowohl für die Verschlüsselung als auch für die Entschlüsselung).

Ark

 

[nO-0n3]

Ich würde da gar nicht mit StringBuilder herumspielen macht das ganze nur zu kompliziert.
Du kannst ja auch einfach einen String nehmen (siehe folgendes Beispiel).
Das mit .equals wurde ja schon angesprochen.
Desweitern würde ich noch eine variable key dazunehmen. Also die Anzahl an Stellen um die ein Buchstabe verschoben werden sollte,
dann kannst du das auch jederzeit ändern.
Und wenn du dann noch eine Schleife nimmst für die Erstellung des enc bzw. dec Strings
dann könnte das z.B so aussehen:

private void computeEncryptedString() {
  encString = new String("");
  boolean upperCase = false;
  
  for(int i = 0;i<inputString.length();i++) {   
  
     char enc = inputString.charAt(i); //Zeichen holen
       
     if(Character.isUpperCase(enc)) upperCase = true; //Großbuchstabe
     else upperCase= false;
       
     enc+=key; // Buchstabe um key verschieben
       
     if(upperCase) { //War Großbuchstabe
        if(enc > 90) //Zu weit => kein Buchstabe mehr
       	   enc-=26;   //da zeichen + key - 26 = wieder von vorne beginnen
     }
     else { //Kleinbuchstabe
        if(enc > 122) 
           enc-=26;
     }
     encString += enc; // Zeichen hinzufügen
  }
}

Funktionieren tut das ganze dadurch das jedes Zeichen einer Zahl entspricht (siehe ASCII Tabelle ->google) somit kannst du sie ganz einfach verschieben.
Das einzige worauf du aufpassen musst ist bei Groß und Kleinschreibung da die Zahlen nicht beisammen liegen.
Analog dazu geht dann auch das decodieren. Probiers mal mit dem Code vielleicht funktioniert dein Programm ja dann.

 

[0x7F800000]

Das mit der Schleife mach ich mal, wenn das Programm richtig funktioniert - das ist jetzt erstmal das Hauptanliegen, wenn ich dafür 5min länger Copy&Paste machen muss, ist mir das aktuell Wurst.

sicher, dass nicht vielleicht eine Metzgerlehre besser wäre? -.-


Okay, Spaß beiseite:

			if (currentLetter == 'a') { currentLetter = 'd'; this.encString.setCharAt(i, currentLetter); }
/* 500 km Wurst mit Informationsgehalt=0 */
			if (currentLetter == 'z') { currentLetter = 'c'; this.encString.setCharAt(i, currentLetter); }

/*... 10 zeilen später ...*/

			if (currentLetter == 'a') { currentLetter = 'x'; this.decString.setCharAt(i, currentLetter); }
/*... nochmal die selbe wurst ...*/
			if (currentLetter == 'z') { currentLetter = 'w'; this.decString.setCharAt(i, currentLetter); }	
		}
	}

...erzähl mir jetzt bitte nicht, dass du diese buchstaben per hand eingetippt hättest?!? ;(
Es geht doch nicht um die 5 Minuten copy paste.
Es geht darum, dass du 17000% mehr code gebraucht hast, als nötig :autsch:
Und darum, dass bei dir 5 Minuten völlig monotoner Arbeit keine Wut und keinen inneren Protest auslösen. Das ist schlecht. Beim Programmieren ist nämlich diese Art von "Faulheit" eine der wichtigsten Tugenden, ohne die alle nach wie vor fleißig in Assembler programmieren würden...

Das programm mit 26x-fachen funktionalität (keys unterschiedlicher länge) passt bestens in 22 Zeilen, das sind 4 zeilen weniger, als du für die bekloppten if-abfragen alleine gebraucht hast

public class CaesarChiffre {
	public static int mod(int a, int b){
		int r=a%b;
		return r<0?r+b:r;
	}
	public static String encrypt(String text, int key){
		StringBuilder builder=new StringBuilder(text);
		for(int i=0; i<builder.length(); i++){
			builder.setCharAt(i,(char)(mod(builder.charAt(i)-'a'+key,('z'-'a'+1))+'a'));
		}
		return builder.toString();
	}
	public static String decrypt(String text, int key){
		return encrypt(text,-key);
	}
	public static void main(String..._){
		String text="rhabarber";
		System.out.println(text);
		System.out.println(text=encrypt(text,3));
		System.out.println(text=decrypt(text,3));
	}
}

 

[Jay1980]

Danke für die Anmerkungen,

die ASCII Tabelle kenn ich, ich werde das beim nächsten Mal wohl auch mit dieser Tabelle lösen und dazu Schleifen verwenden, mal schauen, jetzt will ich aber den mir unerklärlichen Fehler schon ausmerzen.
Ich habe aber nun also vor, es mal so zu versuchen, die Anmerkungen zu den Strings haben dazu geführt, dass ich etwas nachgelesen habe und habe entsprechend die Sachen umgeschrieben, also den Vergleich geändert und ich dachte zu sehr ums Eck, dass ich einen veränderbaren String bauen wollte, also die StringBuilder-Klasse genommen habe, aber ich kann ja auch einen normalen String nach und nach mit einem Pluszeichen verlängern. Also nun habe ich normale Strings.

Eins ist aber weiterhin sehr komisch:
* nachdem Click auf 'Cäsar Chiffre entschlüsseln' klappt alles perfekt. Nach einem Klick auf 'Cäsar Chiffre verschlüsseln' ist aus irgendeinem Grund der Ergebnisstring viel länger, obwohl ich beim Betrachten der beiden Methoden keinen Unterschied feststellen kann.

Hier nochmal die Klassen, oben der Ausschnitt aus der actionPerformed Methode und ein Stück weiter unten die:

// ActionPerformed-Methoden-Ausschnitt
else if ( e.getActionCommand() == "Cäsar-Chiffre")
		{
			// replace statusPanel-Content
			JTextArea jta = new JTextArea();
			jta.setBorder(null);
			jta.setEditable(false);
			jta.setLineWrap(true);
			jta.setWrapStyleWord(true);
			Dimension dim = new Dimension(150, 590);
			jta.setPreferredSize(dim);
			jta.setText(
					"Status: Ort - Kryptographie | Algorithmen | Cäsar"
					);
			
			this.statusPanel.removeAll();
			this.statusPanel.add(jta);
			this.statusPanel.revalidate();
			this.statusPanel.repaint();
			
			// replace centerPanel Content
			this.centerPanel.removeAll();
			JPanel caesarInputPanel = new JPanel();
			JLabel caesarInputHeaderPanelHeader = new JLabel( "Cäsar Chiffre" );
			JTextField caesarInputString = new JTextField( "hier steht eine Zeichenkette");
			
			// die JTextFields ein die Transfercontainer uebergeben
			this.dollarPostStringOne = caesarInputString;
			
			JButton caesarChiffreButtonOne = new JButton("Cäsar-Chiffre entschlüsseln");
			JButton caesarChiffreButtonTwo = new JButton("Cäsar-Chiffre verschlüsseln");
			caesarChiffreButtonOne.addActionListener( this );
			caesarChiffreButtonTwo.addActionListener( this );
			
			
			caesarInputPanel.add(caesarInputHeaderPanelHeader);
			caesarInputPanel.add(caesarInputString);
			caesarInputPanel.add(caesarChiffreButtonOne);
			caesarInputPanel.add(caesarChiffreButtonTwo);
			
			this.centerPanel.add( caesarInputPanel );
			this.centerPanel.revalidate();
			this.centerPanel.repaint();
			
		}
		else if ( e.getActionCommand() == "Cäsar-Chiffre entschlüsseln")
		{
			// replace centerPanel Content
			this.centerPanel.removeAll();
			
			JobaCaesarChiffre jcc = new JobaCaesarChiffre( "decode", this.dollarPostStringOne.getText() );
			
			String dencText = jcc.getDecryptedString();
			String eingabeText = this.dollarPostStringOne.getText();
			
			JLabel jldenc = new JLabel();
			jldenc.setText(
					"Der entschlüsselte Text lautet: " + dencText
			);
			
			JLabel jleing = new JLabel();
			jleing.setText("Der Eingabe-String lautet: " + eingabeText );
			
			this.centerPanel.add(jldenc);
			this.centerPanel.add(jleing);
			
			this.centerPanel.revalidate();
			this.centerPanel.repaint();
		}
		else if ( e.getActionCommand() == "Cäsar-Chiffre verschlüsseln")
		{
			// replace centerPanel Content
			this.centerPanel.removeAll();
			
			JobaCaesarChiffre jcc = new JobaCaesarChiffre( "encode", this.dollarPostStringOne.getText() );
			
			String encText = jcc.getEncryptedString();
			String eingabeText = this.dollarPostStringOne.getText();
			
			JLabel jlenc = new JLabel();
			jlenc.setText(
					"Der verschlüsselte Text lautet: " + encText
			);
			
			JLabel jleing = new JLabel();
			jleing.setText("Der Eingabe-String lautet: " + eingabeText );
			
			this.centerPanel.add(jlenc);
			this.centerPanel.add(jleing);
			
			this.centerPanel.revalidate();
			this.centerPanel.repaint();
		}

--------------------------------------------------------------------------------

public class JobaCaesarChiffre 
{
	private String inputString;
	private String encString;
	private String decString;
	
	/** Konstruktor */
	JobaCaesarChiffre( String flag, String instr )
	{
		this.inputString = instr;
		
		if ( flag.equals("encode")) // Strings werden mit equals auf Gleicheit vergleichen
		{
			this.computeEncryptedString();
		}
		
		if ( flag.equals("decode"))
		{
			this.computeDecryptedString();
		}
		
	}
	
	/** Chiffre aus Klartext bilden */
	private void computeEncryptedString()
	{
		// Ein Buchstabe muss im Alphabet drei Plätze nach vorn verschoben werden
		// also Klartextzeichen | Chiffrezeichen
		
		this.encString = new String("");
		
		for ( int i = 0; i < this.inputString.length(); i++)
		{ 	
			char currentLetter = this.inputString.charAt(i);
			
			if (currentLetter == 'a') { currentLetter = 'd'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'b') { currentLetter = 'e'; this.encString = this.encString + currentLetter; }		
			if (currentLetter == 'c') { currentLetter = 'f'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'd') { currentLetter = 'g'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'e') { currentLetter = 'h'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'f') { currentLetter = 'i'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'g') { currentLetter = 'j'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'h') { currentLetter = 'k'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'i') { currentLetter = 'l'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'j') { currentLetter = 'm'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'k') { currentLetter = 'n'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'l') { currentLetter = 'o'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'm') { currentLetter = 'p'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'n') { currentLetter = 'q'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'o') { currentLetter = 'r'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'p') { currentLetter = 's'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'q') { currentLetter = 't'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'r') { currentLetter = 'u'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 's') { currentLetter = 'v'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 't') { currentLetter = 'w'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'u') { currentLetter = 'x'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'v') { currentLetter = 'y'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'w') { currentLetter = 'z'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'x') { currentLetter = 'a'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'y') { currentLetter = 'b'; this.encString = this.encString + currentLetter; }
			if (currentLetter == 'z') { currentLetter = 'c'; this.encString = this.encString + currentLetter; }	
		}
	}
	
	/** Klartext aus Chiffre bilden */
	private void computeDecryptedString()
	{
		this.decString = new String("");
		
		for ( int i = 0; i < this.inputString.length(); i++)
		{
			char currentLetter = this.inputString.charAt(i);
			
			if (currentLetter == 'a') { currentLetter = 'x'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'b') { currentLetter = 'y'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'c') { currentLetter = 'z'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'd') { currentLetter = 'a'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'e') { currentLetter = 'b'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'f') { currentLetter = 'c'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'g') { currentLetter = 'd'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'h') { currentLetter = 'e'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'i') { currentLetter = 'f'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'j') { currentLetter = 'g'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'k') { currentLetter = 'h'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'l') { currentLetter = 'i'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'm') { currentLetter = 'j'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'n') { currentLetter = 'k'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'o') { currentLetter = 'l'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'p') { currentLetter = 'm'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'q') { currentLetter = 'n'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'r') { currentLetter = 'o'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 's') { currentLetter = 'p'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 't') { currentLetter = 'q'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'u') { currentLetter = 'r'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'v') { currentLetter = 's'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'w') { currentLetter = 't'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'x') { currentLetter = 'u'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'y') { currentLetter = 'v'; this.decString = this.decString + currentLetter; }
			if (currentLetter == 'z') { currentLetter = 'w'; this.decString = this.decString + currentLetter; }
		}
	}
	
	/** Verschlüsselten String ausliefern */
	public String getEncryptedString()
	{
		return this.encString;
	}
	
	/** Klartext String aus Chiffre liefern */
	public String getDecryptedString()
	{
		return this.decString;
	}
}

 

[nO-0n3]

Hi!
Also zuerst gleich mal zu deinem Problem.
Angenommen du willst 'a' verschlüsseln:
Es kommt zu der if-Bedingung und setzt currentLetter='d', und fügt ihn dann dem String hinzu. Jetzt kommt currentLetter='d' aber
wieder in eine if-Bedingung wo du diesen auf 'g' setzt usw. Das heißt wenn du nur 'a' verschlüsselt kommt am Ende "dgjmpsvyb" raus.
Das dies beim entschlüsslen nicht passiert ist eher glücksache den es gibt durchaus Kombinationen wo das auch dort auftreten könnte (z.B bei 'a' => "xu").

Am besten verhinderst du das indem du nicht immer currentLetter neu setzt sondern einfach das entsprechende Zeichen dem String hinzufügst.
Noch besser wäre allerdings endlich von dem copy&paste wegzukommen.

Noch zwei Sachen:
Das this. vor jeder Variable innerhalb einer Funkion in deiner Klasse kannst du weglassen macht dort nicht viel Sinn.
Was anderes wäre z.B im Konstruktor wenn du dort gleichnamige Variablen übergeben bekommst.
Und statt this.string = this.string + currentLetter => this.string+=currentLetter macht das ganze nochmal kürzer.

 

[Leroy42]

aber ich kann ja auch einen normalen String nach und nach mit einem Pluszeichen verlängern.

Nein!

Du kannst auf keinerlei Art und Weise
einen String in Java verlängern.

Beispiel:

String text = "Hello " + "world";

erzeugt einen String "Hello world", verlängert aber weder
den String "Hello " noch sonst einen.

 

[Jay1980]

Danke - nun geht es, ich habe einfach ein continue in jede Zeile geschrieben - jetzt wo es läuft wird auch optimiert, bzw. werde ich die ähnlichen Algorithmen, die Zeichen ersetzen, anders implementieren. Die Anmerkungen eurerseits halfen mir sehr!

 

[Jay1980]

ROT13 ist ja ganz ähnlich zum Cäsar-Chiffre, man zählt 13 Stellen nach vorne und das habe ich nun so gemacht, nur damit die Tippgeber auch sehen können, dass es nicht immer Copy&Paste sein muss.

public class JobaRot13 
{

	String inputString;
	String encString;
	String decString;
	String flag;
	
	JobaRot13( String todoflag, String inpString )
	{
		System.out.println("Konstruktor ist da. InpStrin ist zugeteilt: " + inpString);
		inputString = inpString;
		flag = todoflag;
		
		if ( flag.equals("encode"))
		{
			computeEncodeString();
		}
		
		if ( flag.equals("decode"))
		{
			computeDecodeString();
		}
	}
	
	private void computeEncodeString()
	{
		encString = new String("");
		
		// jedes Arrayelement rannehmen
		for ( int i = 0; i < inputString.length(); i++ )
		{
			System.out.println("for-Schleife fürs Array der Buchstaben ist da mit: " + i );
			
			// Ascii Zeichen durchgehen
			// A = 65; B = 66; C = 67; ... Z = 90;
			// a = 97; b = 98; c = 99; ... z = 122;
			
			for ( int j = 97; j < 123; j++)
			{
				
				System.out.println("for-Schleife mit j ist da!");
				
				if ( inputString.charAt(i) == (char)j )
				{
					if ( j + 13 < 123 )
					{
						encString += (char)j;
						System.out.println("Block mit ASCII Zeichen ist da mit: " + j);
						break; // aus der umgebenden Schleife springen
					}
					
					if ( j + 13 > 122 )
					{
						int cj;
						cj = (j + 13) - 26;
						encString += (char)cj;
						System.out.println("Block mit zu grossem ASCII-Zeichen ist da mit: " + cj);
						break;
					}
				}
			}
		}
		
	}
	
	private void computeDecodeString()
	{
		decString = new String("");
		
		// jedes Arrayelement rannehmen
		for ( int i = 0; i < inputString.length(); i++ )
		{
			
			// Ascii Zeichen durchgehen
			// A = 65; B = 66; C = 67; ... Z = 90;
			// a = 97; b = 98; c = 99; ... z = 122;
			
			for ( int j = 97; j < 123; j++)
			{
				if ( inputString.charAt(i) == (char)j )
				{
					if ( j + 13 < 123 )
					{
						decString += (char)j;
						break; // aus der umgebenden Schleife springen
					}
					
					if ( j + 13 > 122 )
					{
						int cj;
						cj = (j + 13) - 26;
						decString += (char)cj;
						break;
					}
				}
			}
		}
	}
	
	public String getEncodeString()
	{
		return encString;
	}
	
	public String getDecodeString()
	{
		return decString;
	}
}

 

[0x7F800000]

ROT13 ist ja ganz ähnlich zum Cäsar-Chiffre, man zählt 13 Stellen nach vorne und das habe ich nun so gemacht, nur damit die Tippgeber auch sehen können, dass es nicht immer Copy&Paste sein muss.

wir sind stolz auf dich^^

 

[nO-0n3]

Auf jeden Fall schon viel besser als die copy&paste Blöcke.
Aber das es so noch nicht funktioniert weißt du hoffentlich?^^