Pig-Latin Exercise Updates (#1249)

* Create version File for Pig-Latin Challenge

* Update PigLatinTranslator.java

Updated tests.

Could someone please double-check that this test class works as intended?

* Update PigLatin Tests

Build failed last time.

* Update PigLatinTranslator.java

Fixed reference solution so words with "y" after a consonant cluster are taken care of properly.

* Update PigLatinTranslatorTest.java

Fix test to match correct reference solution.

* Update PigLatinTranslator.java

Made method more efficient and fixed up formatting

* Update PigLatinTranslator.java

* Fix Character and String Comparison

exercises/pig-latin/.meta/src/reference/java/PigLatinTranslator.java

@@ -26,6 +26,11 @@ class PigLatinTranslator {
             return word + AY;
         }
 
+        if (getLocationOfYAfterConsonantCluster(word) != -1) {
+            int yPos = getLocationOfYAfterConsonantCluster(word);
+            return word.substring(yPos) + word.substring(0, yPos) + AY;
+        }
+
         if (wordStartsWithPrefixes(word, THR, SCH)) {
             return word.substring(3) + word.substring(0, 3) + AY;
         }
@@ -41,6 +46,17 @@ class PigLatinTranslator {
         return word.substring(1) + word.toCharArray()[0] + AY;
     }
 
+    private static int getLocationOfYAfterConsonantCluster(String word) {
+        for (int i = 0; i < word.length() - 1; i++) {
+            if ((Character.toString(word.charAt(i)).matches(VOWELS_REGEX))) {
+                return -1;
+            } else if ((i != 0) && word.charAt(i) == 'y') {
+                return i;
+            }
+        }
+      return -1;
+    }
+
     private boolean wordStartsWithVowelLike(String word) {
         return word.startsWith(YT) || word.startsWith(XR) || word.substring(0, 1).matches(VOWELS_REGEX);
     }

exercises/pig-latin/.meta/version

@@ -0,0 +1 @@
+1.1.0

exercises/pig-latin/src/test/java/PigLatinTranslatorTest.java

@@ -23,12 +23,17 @@ public class PigLatinTranslatorTest {
                 {"object", "objectay"},
                 {"under", "underay"},
 
+                // Word beginning with vowel follwed by a 'qu'
+                {"equal", "equalay"},
+
                 // First letter and ay are moved to the end of words that start with consonants
                 {"pig", "igpay"},
                 {"koala", "oalakay"},
-                {"yellow", "ellowyay"},
                 {"xenon", "enonxay"},
 
+                // Word beginning with 'q' without a following 'u'
+                {"qat", "atqay"},
+
                 // Ch is treated like a single consonant
                 {"chair", "airchay"},
 
@@ -53,6 +58,15 @@ public class PigLatinTranslatorTest {
                 // Xr is treated like a single vowel
                 {"xray", "xrayay"},
 
+                // 'Y' is treated like a consonant at the beginning of a word
+                {"yellow", "ellowyay"},
+
+                // 'Y' is treated like a vowel at the end of a consonant cluster
+                {"rhythm", "ythmrhay"},
+
+                // 'Y' as second letter in a two letter word
+                {"my", "ymay"},
+
                 // Phrases are translated
                 {"quick fast run", "ickquay astfay unray"}
         });