Thread: Encrypt Decrypt

The method Encrypt should for example
input:
ab
cd

output:acbd

the method decrypt should take same input and output:abcd

what could be wrong:

	public static String encrypt(String plainText) {
 
 
	String s = plainText.toLowerCase();// Taking out upper cases
 
	s = s.replaceAll("\\s","");//Taking out blank spaces
 
	float squareRoot = (float) Math.sqrt(s.length());
 
	int dimension = (int) Math.ceil(squareRoot);
 
	int x = dimension;
 
	int y = dimension;
 
	char[][] box = new char [x][y];// This is the box: a 2 dimensional array
 
	int counter = 0;
 
	for(int i = 0; i<x; i++){
 
		for(int j = 0; j<y; j++){
 
		if(counter<s.length()){	
 
           box[i][j]= s.charAt(counter);
 
          counter++;         
		}		
	}
		}
	java.util.Arrays.deepToString(box);
 
	 StringBuilder sb = new StringBuilder();
 
	for(int j =0; j<y; j++){
 
			for(int i = 0; i<x; i++){
 
				sb.append(box[i][j]);
			}			
}
 
return (sb.toString());
 
	}
 
 
 
	   public static String decrypt(String cypherText) {
 
 
	String s = cypherText.toLowerCase();// Taking out upper cases
 
	s = s.replaceAll("\\s","");//Taking out blank spaces
 
	float squareRoot = (float) Math.sqrt(s.length());
 
	int dimension = (int) Math.ceil(squareRoot);
 
	int x = dimension;
 
	int y = dimension;
 
	char[][] box = new char [x][y];// This is the box: a 2 dimensional array
 
	int counter = 0;
 
	for(int i = 0; i<x; i++){
 
		for(int j = 0; j<y; j++){
 
		if(counter<s.length()){	
 
           box[i][j]= s.charAt(counter);
 
          counter++;         
		}		
	}
		}
	java.util.Arrays.deepToString(box);
 
	 StringBuilder sb = new StringBuilder();
 
	for(int i = 0; i<x; i++){
 
			for(int j = 0; j<y; j++){
 
				sb.append(box[i][j]);
			}			
}
 
return (sb.toString());
 
	}

Sorry, the input is abcd

But I have to put it into a caesar box before encrypt/decrypt

decrypt should put it into a caesar box like

a c
b d

Fixed:

	public static String encrypt(String plainText) {
 
 
	String s = plainText.toLowerCase();// Taking out upper cases
 
	s = s.replaceAll("\\s","");//Taking out blank spaces
 
	float squareRoot = (float) Math.sqrt(s.length());
 
	int dimension = (int) Math.ceil(squareRoot);
 
	int x = dimension;
 
	int y = dimension;
 
	char[][] box = new char [x][y];// This is the box: a 2 dimensional array
 
	int counter = 0;
 
	for(int i = 0; i<x; i++){
 
		for(int j = 0; j<y; j++){
 
		if(counter<s.length()){	
 
           box[i][j]= s.charAt(counter);
 
          counter++;         
		}		
	}
		}
	java.util.Arrays.deepToString(box);
 
	 StringBuilder sb = new StringBuilder();
 
	for(int j =0; j<y; j++){
 
			for(int i = 0; i<x; i++){
 
				sb.append(box[i][j]);
			}			
}
 
return (sb.toString());
 
	}
 
 
 
	   public static String decrypt(String cypherText) {
 
 
	String s = cypherText.toLowerCase();// Taking out upper cases
 
	s = s.replaceAll("\\s","");//Taking out blank spaces
 
	float squareRoot = (float) Math.sqrt(s.length());
 
	int dimension = (int) Math.ceil(squareRoot);
 
	int x = dimension;
 
	int y = dimension;
 
	char[][] box = new char [x][y];// This is the box: a 2 dimensional array
 
	int counter = 0;
 
	for(int j = 0; j<y; j++){
 
		for(int i = 0; i<x; i++){
 
		if(counter<s.length()){	
 
           box[i][j]= s.charAt(counter);
 
          counter++;         
		}		
	}
		}
	java.util.Arrays.deepToString(box);
 
	 StringBuilder sb = new StringBuilder();
 
	for(int i = 0; i<x; i++){
 
			for(int j = 0; j<y; j++){
 
				sb.append(box[i][j]);
			}			
}
 
return (sb.toString());
 
	}

What could be wrong now?

--- Update ---

No, sorry

It should put the string character by character in a ceaser box from top to bottom left to right, so if the input is abcd the caesar box should be:

a c
b d

And then it should decrypt the caesar box and return the string: acbd (reading from left to right, top to bottom)

Yes, I am. What you said would be encrypt. This examples are given in the assignment:

Sample Input/Output
Case A
Input:
java CaesarBox -encrypt abcd
Logic:
a b
c d
Output:
acbd
Case B
Input:
java CaesarBox -encrypt abcdefghij
Logic:
a b c d
e f g h
i j

Output:
aeibfjcgdh
Case C
Input:
java CaesarBox -decrypt acbd
a b
c d
Output:
abcd

I thought I had answered it. Sorry I wasn't clear. Maybe I didn't get it as well.
Here are the instructions:

Write a Java program to encrypt a string of text using a Caesar Square: the string should be written along the rows of a square that is as small as possible, and then the encrypted string is read down the columns like so:

"A long time ago, in a galaxy far, far away" is first processed to remove case and space, to get alongtimeago,inagalaxyfar,faraway (33 characters long) and is then laid out on a square grid:

a l o n g t
i m e a g o
, i n a g a
l a x y f a
r , f a r a
w a y
The grid is 6 x 6 as 36 is the closest square number that is no less than the number of characters in the message.

Reading form top-to-bottom we get ai,lrwlmia,aoenxfynaayagggfrtoaaa - which is of course garbage ☺
You also have to decrypt such strings which may contain white space that needs to be removed ...

Learning Outcomes
Simple string manipulation
Creating/Populating/traversing an array
FAQ
Will I have to put spaces and capitalisations back in? No, you have to do neither of those things. It would be a much harder Task and beyond the scope of this course to do that.
How will my program be tested? We run the programs as though you would on the command-line, just like this:java CaesarBox -encrypt abcd or java CaesarBox ALongTimeAgo
How can I put spaces in the command-line input? If you want a String argument (which would be args[1] if you use the standard name for the arguments to main) then you can do this:
java CaesarBox -encrypt "One Flew Over A Cuckoo's Nest"
with the double quotes, and then the whole phrase "One Flew Over A Cuckoo's Nest" is made into a single String.
Do I have to leave the skeleton code as it is? Well you have to add to the skeleton code: the main method is given for you and you don't need to alter it, but you do need to fill in the other two methods. If you don't know what a method is yet then don't worry, because we'll cover those in Week 5. As a rough guide if you want to get started anyway, a method is like a tool in a program that you use by passing it information like a String, and then using the result that is returned.
Is encrypting the same as decrypting? No, it isn't, but they are very similar. You will have to write different methods for each function, as indicated in the skeleton code.
I saw in the examples that some of the input strings don't fit neatly into a square: there are spaces left over. What should I do? Do I need to include those spaces in the output? No, you shouldn't include spaces in your output.
What size square should I use? You have to work out the size of square that is as small as possible but which fits the entire string.
When decrypting, where do I put in the gaps? The output should have no gaps but during the decription process you might need to put gaps in your grid as "placeholders".
Sample Input/Output
Case A
Input:
java CaesarBox -encrypt abcd
Logic:
a b
c d
Output:
acbd
Case B
Input:
java CaesarBox -encrypt abcdefghij
Logic:
a b c d
e f g h
i j

Output:
aeibfjcgdh
Case C
Input:
java CaesarBox -decrypt acbd
a b
c d
Output:
abcd

1 - take out upper cases and spaces from string input
2 - string should be written along the rows of a caesar square
3 - case decryp it should be written top to bottom left to right
4-case decrypt should read a string from the square caeser left to right top to bottom

The first two questions have no answers because they are exactly the assignment. I suppose I have to use loops.

The code does that for some inputs but not for others.

I tried to print box[i][j] but it says i and j are undefined.