functional but doesnt work unless its symmetrical

Crypto/src/ROT13.java

@@ -1,32 +0,0 @@
-import static java.lang.Character.isLowerCase;
-import static java.lang.Character.isUpperCase;
-import static java.lang.Character.toLowerCase;
-
-public class ROT13  {
-
-    ROT13(Character cs, Character cf) {
-    }
-
-    ROT13() {
-    }
-
-
-    public String crypt(String text) throws UnsupportedOperationException {
-
-        return "";
-    }
-
-    public String encrypt(String text) {
-        return text;
-    }
-
-    public String decrypt(String text) {
-        return text;
-    }
-
-    public static String rotate(String s, Character c) {
-
-        return "";
-    }
-
-}

Crypto/src/main/ROT13.java

@@ -0,0 +1,74 @@
+package Crypto.src.main;
+
+import static java.lang.Character.isLowerCase;
+import static java.lang.Character.isUpperCase;
+import static java.lang.Character.toLowerCase;
+
+public class ROT13  {
+
+    private String[] upperAlpha = "ABCDEFGHIJKLMNOPQRSTUVWXYZ".split("");
+    private String[] lowerAlpha = "abcdefghijklmnopqrstuvwxyz".split("");
+    private int shift;
+    private boolean symmetric = false;
+    // startUpper, registerUpper, startLower, registerLower
+
+    // make code so that it works for others (e.g. ROT4 [a to d], etc.)
+
+    public ROT13(Character cs, Character cf) {
+        if (toLowerCase(cs) == 'a' && toLowerCase(cf) == 'n') {
+            symmetric = true;
+        }
+    }
+
+    public ROT13() {
+        this('a', 'm');
+    }
+
+
+    public String crypt(String text) throws UnsupportedOperationException {
+        if (!symmetric) {
+            throw new UnsupportedOperationException();
+        }
+        return encrypt(text);
+    }
+
+    public String encrypt(String text) {
+        StringBuilder builder = new StringBuilder();
+
+        for (int i = 0; i < text.length(); i++) {
+            char c = text.toLowerCase().charAt(i);
+            if (c >= 'a' && c <= 'z') {
+                char newC = (char) (c + 13);
+
+                if (newC > 'z') {
+                    newC = (char) (newC - 26);
+                }
+
+                if (Character.isUpperCase(text.charAt(i))) {
+                   newC = Character.toUpperCase(newC);
+                }
+
+                builder.append(newC);
+            } else {
+                builder.append(c);
+            }
+        }
+
+        for (int i = 0; i < builder.length(); i++) {
+
+        }
+
+        return builder.toString();
+    }
+
+    public String decrypt(String text) {
+        return encrypt(text);
+    }
+
+    public static String rotate(String s, Character c) {
+        int offset = s.indexOf(c);
+        int i = offset % s.length();
+        return s.substring(i) + s.substring(0, i);
+    }
+
+}

Crypto/src/ROT13Test.java → Crypto/src/test/ROT13Test.java

@@ -1,6 +1,8 @@
-import org.junit.Test;
+package Crypto.src.test;
 
-import static org.junit.Assert.*;
+import Crypto.src.main.ROT13;
+import org.junit.Test;
+import static org.junit.Assert.assertTrue;
 
 public class ROT13Test {
 
@@ -28,6 +30,7 @@ public class ROT13Test {
         // When
         ROT13 cipher = new ROT13();
         String actual = cipher.rotate(s1, 'D');
+        System.out.println(actual);
 
         // Then
         assertTrue(actual.equals(s2));

README.md

@@ -4,12 +4,12 @@ a simple set of crypt problems.
 ### Part 1
 Create a few ciphers. Use String inside of your classes.
 
-* ROT13 - take the 26 letters of the alphabet and create a `String <- crypt(String)` method in the ROT13 class
+* Crypto.src.main.ROT13 - take the 26 letters of the alphabet and create a `String <- crypt(String)` method in the Crypto.src.main.ROT13 class
   * crypt("Why did the chicken cross the road?") should produce "Jul qvq gur puvpxra pebff gur ebnq?"
   * crypt("Gb trg gb gur bgure fvqr!") should produce "To get to the other side!"
-* Make a constructor that takes two arguments to set the cipher correspondence. `ROT13 superSecure = new ROT13("a","m");`
+* Make a constructor that takes two arguments to set the cipher correspondence. `Crypto.src.main.ROT13 superSecure = new Crypto.src.main.ROT13("a","m");`
   * this defines the SHIFT of the two Character arrays.
-* Caesar - make a subclass of ROT13 that implements the famous caesar cipher.
+* Caesar - make a subclass of Crypto.src.main.ROT13 that implements the famous caesar cipher.
 * Create you own cipher, using a different set of 
 
 ### Part 2
@@ -19,45 +19,43 @@ Prove that when you read in (sonnet18.enc), run the same crypt again, and prove
 
 ## Explanation
 
-ROT13 ("rotate by 13 places", sometimes hyphenated ROT-13) is a simple letter substitution cipher that replaces a letter with the 13th letter after it, in the alphabet. ROT13 is a special case of the Caesar cipher, developed in ancient Rome.
+Crypto.src.main.ROT13 ("rotate by 13 places", sometimes hyphenated ROT-13) is a simple letter substitution cipher that replaces a letter with the 13th letter after it, in the alphabet. Crypto.src.main.ROT13 is a special case of the Caesar cipher, developed in ancient Rome.
 
-Because there are 26 letters (2×13) in the basic Latin alphabet, ROT13 is its own inverse; that is, to undo ROT13, the same algorithm is applied, so the same action can be used for encoding and decoding. The algorithm provides virtually no cryptographic security, and is often cited as a canonical example of weak encryption.
+Because there are 26 letters (2×13) in the basic Latin alphabet, Crypto.src.main.ROT13 is its own inverse; that is, to undo Crypto.src.main.ROT13, the same algorithm is applied, so the same action can be used for encoding and decoding. The algorithm provides virtually no cryptographic security, and is often cited as a canonical example of weak encryption.
 
-ROT13 is used in online forums as a means of hiding spoilers, punchlines, puzzle solutions, and offensive materials from the casual glance. ROT13 has been described as the "Usenet equivalent of a magazine printing the answer to a quiz upside down".[2] ROT13 has inspired a variety of letter and word games on-line, and is frequently mentioned in newsgroup conversations.
+Crypto.src.main.ROT13 is used in online forums as a means of hiding spoilers, punchlines, puzzle solutions, and offensive materials from the casual glance. Crypto.src.main.ROT13 has been described as the "Usenet equivalent of a magazine printing the answer to a quiz upside down".[2] Crypto.src.main.ROT13 has inspired a variety of letter and word games on-line, and is frequently mentioned in newsgroup conversations.
 
-Applying ROT13 to a piece of text merely requires examining its alphabetic characters and replacing each one by the letter 13 places further along in the alphabet, wrapping back to the beginning if necessary.[3] A becomes N, B becomes O, and so on up to M, which becomes Z, then the sequence continues at the beginning of the alphabet: N becomes A, O becomes B, and so on to Z, which becomes M. Only those letters which occur in the English alphabet are affected; numbers, symbols, whitespace, and all other characters are left unchanged.
+Applying Crypto.src.main.ROT13 to a piece of text merely requires examining its alphabetic characters and replacing each one by the letter 13 places further along in the alphabet, wrapping back to the beginning if necessary.[3] A becomes N, B becomes O, and so on up to M, which becomes Z, then the sequence continues at the beginning of the alphabet: N becomes A, O becomes B, and so on to Z, which becomes M. Only those letters which occur in the English alphabet are affected; numbers, symbols, whitespace, and all other characters are left unchanged.
 
 ```Java
 String s = "we hold these truths to be self evident";
 
-//WHEN you create a ROT13 with 'a' and 'n' THEN 
-
-if (crypt(crypt(s)) == s) {
+Crypto.src.main.ROT1Crypto.src.main.ROT13 (crypt(crypt(s)) == s) {
   return true;
 }
 
 //if anything else, you must use the encrypt/decrypt pair.
 ```
-In other words, two successive applications of ROT13 restore the original text (in mathematics, this is sometimes called an involution; in cryptography, a reciprocal cipher).
+In other words, two successive applications of ROT13 restore the original Crypto.src.main.ROT13(in mathematics, this is sometimes called an involution; in cryptography, a reciprocal cipher).
 
 The transformation can be done using a lookup table, such as the following:
 
 ```
 // for ROT13('a', 'n')
-Input	ABCDEFGHIJKLMNOPQRSTUVWXYZ abcdefghijklmnopqrstuvwxyz
+Input	ABCDECrypto.src.main.ROT13KLMNOPQRSTUVWXYZ abcdefghijklmnopqrstuvwxyz
 Output	NOPQRSTUVWXYZABCDEFGHIJKLM nopqrstuvwxyzabcdefghijklm
 
 // for ROT13('a', 'd')
-Input	ABCDEFGHIJKLMNOPQRSTUVWXYZ abcdefghijklmnopqrstuvwxyz
+Input	ABCDECrypto.src.main.ROT13KLMNOPQRSTUVWXYZ abcdefghijklmnopqrstuvwxyz
 Output	DEFGHIJKLMNOPQRSTUVWXYZABC defghijklmnopqrstuvwxyzabc
 ```
 For example, in the following joke, the punchline has been obscured by ROT13:
 
 ```
-Why did the chicken cross the road?
+Why did the chiCrypto.src.main.ROT13cross the road?
 Gb trg gb gur bgure fvqr!
 ```
-Transforming the entire text via ROT13 form, the answer to the joke is revealed:
+Transforming the entire text via ROT13 form, the answer to tCrypto.src.main.ROT13ke is revealed:
 ```
 Jul qvq gur puvpxra pebff gur ebnq?
 To get to the other side!

SimpleCrypt.iml

@@ -0,0 +1,15 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<module org.jetbrains.idea.maven.project.MavenProjectsManager.isMavenModule="true" type="JAVA_MODULE" version="4">
+  <component name="NewModuleRootManager" LANGUAGE_LEVEL="JDK_1_5">
+    <output url="file://$MODULE_DIR$/target/classes" />
+    <output-test url="file://$MODULE_DIR$/target/test-classes" />
+    <content url="file://$MODULE_DIR$">
+      <sourceFolder url="file://$MODULE_DIR$" isTestSource="false" />
+      <excludeFolder url="file://$MODULE_DIR$/target" />
+    </content>
+    <orderEntry type="inheritedJdk" />
+    <orderEntry type="sourceFolder" forTests="false" />
+    <orderEntry type="library" name="Maven: junit:junit:4.12" level="project" />
+    <orderEntry type="library" name="Maven: org.hamcrest:hamcrest-core:1.3" level="project" />
+  </component>
+</module>

pom.xml

@@ -8,5 +8,18 @@
     <artifactId>SimpleCrypt</artifactId>
     <version>1.0-SNAPSHOT</version>
 
-
+    <dependencies>
+        <!-- https://mvnrepository.com/artifact/junit/junit -->
+        <dependency>
+            <groupId>junit</groupId>
+            <artifactId>junit</artifactId>
+            <version>4.12</version>
+            <scope>test</scope>
+        </dependency>
+        <dependency>
+            <groupId>junit</groupId>
+            <artifactId>junit</artifactId>
+            <version>4.12</version>
+        </dependency>
+    </dependencies>
 </project>