Merge pull request #65 from domaframework/lenient-snakecase

スネークケース区切りについて前バージョンとの互換機能を追加しました

Author: nakamura-to

src/main/java/org/seasar/doma/internal/util/StringUtil.java

@@ -16,6 +16,7 @@
 package org.seasar.doma.internal.util;
 
 import java.nio.CharBuffer;
+import java.util.function.Function;
 
 /**
  * {@link String} のユーティリティクラスです。
@@ -85,17 +86,45 @@ public static String fromSnakeCaseToCamelCase(String text) {
     }
 
     /**
-     * キャメルケースをアンダースコア区切りの大文字に変換します。
+     * キャメルケースをアンダースコア区切りに変換します。
+     * <p>
+     * 数字の直後に大文字が続く場合、アンダースコア区切りの対象となります。
      * 
      * @param text
      *            文字列
      * @return 変換された文字列。 ただし、{@code text} が {@code null} の場合は {@code null}、
      *         {@code text} が空文字の場合は空文字を返します。
      */
     public static String fromCamelCaseToSnakeCase(String text) {
+        return fromCamelCaseToSnakeCaseInternal(text, false);
+    }
+
+    /**
+     * キャメルケースをアンダースコア区切りに変換します。
+     * <p>
+     * 数字の直後に大文字が続く場合、アンダースコア区切りの対象となりません。
+     * 
+     * @param text
+     *            文字列
+     * @return 変換された文字列。 ただし、{@code text} が {@code null} の場合は {@code null}、
+     *         {@code text} が空文字の場合は空文字を返します。
+     */
+    public static String fromCamelCaseToSnakeCaseWithLenient(String text) {
+        return fromCamelCaseToSnakeCaseInternal(text, true);
+    }
+
+    private static String fromCamelCaseToSnakeCaseInternal(String text,
+            boolean lenient) {
         if (isNullOrEmpty(text)) {
             return text;
         }
+        Function<Character, Boolean> isNotUpperCase;
+        if (lenient) {
+            isNotUpperCase = Character::isLowerCase;
+        } else {
+            isNotUpperCase = c -> Character.isLowerCase(c)
+                    || Character.isDigit(c);
+        }
         StringBuilder result = new StringBuilder();
         CharBuffer buf = CharBuffer.wrap(text);
         while (buf.hasRemaining()) {
@@ -104,7 +133,7 @@ public static String fromCamelCaseToSnakeCase(String text) {
             buf.mark();
             if (buf.hasRemaining()) {
                 char c2 = buf.get();
-                if ((Character.isLowerCase(c) || Character.isDigit(c)) && Character.isUpperCase(c2)) {
+                if (isNotUpperCase.apply(c) && Character.isUpperCase(c2)) {
                     result.append("_");
                 }
                 buf.reset();

src/main/java/org/seasar/doma/jdbc/entity/NamingType.java

@@ -62,6 +62,10 @@ public String revert(String text) {
      * スネークケースの大文字に変換します。
      * <p>
      * たとえば、<code>aaaBbb</code> を <code>AAA_BBB</code> に変換します。
+     * <p>
+     * 2.2.0 から、入力値において数字の直後に大文字が続く箇所はアンダースコアの挿入対象になりました。
+     * 
+     * たとえば、<code>aaa3Bbb</code> を <code>AAA3_BBB</code> に変換します。
      */
     SNAKE_UPPER_CASE {
 
@@ -88,6 +92,10 @@ public String revert(String text) {
      * スネークケースの小文字に変換します。
      * <p>
      * たとえば、<code>aaaBbb</code> を <code>aaa_bbb</code> に変換します。
+     * <p>
+     * 2.2.0 から、入力値において数字の直後に大文字が続く箇所はアンダースコアの挿入対象になりました。
+     * 
+     * たとえば、<code>aaa3Bbb</code> を <code>aaa3_bbb</code> に変換します。
      */
     SNAKE_LOWER_CASE {
 
@@ -110,6 +118,66 @@ public String revert(String text) {
 
     },
 
+    /**
+     * スネークケースの大文字に変換します。ただし、アンダースコアを挿入する判断が厳格ではありません。具体的には、
+     * 入力値において数字の直後に大文字が続く箇所にアンダースコアを挿入しません。
+     * <p>
+     * この定義は、2.1.0 以前の {@link NamingType#SNAKE_UPPER_CASE} と同じ挙動をし、
+     * 過去との互換性のためだけに存在します。
+     * 
+     * @since 2.2.0
+     */
+    LENIENT_SNAKE_UPPER_CASE {
+
+        @Override
+        public String apply(String text) {
+            if (text == null) {
+                throw new DomaNullPointerException("text");
+            }
+            String s = StringUtil.fromCamelCaseToSnakeCaseWithLenient(text);
+            return s.toUpperCase();
+        }
+
+        @Override
+        public String revert(String text) {
+            if (text == null) {
+                throw new DomaNullPointerException("text");
+            }
+            return StringUtil.fromSnakeCaseToCamelCase(text);
+        }
+
+    },
+
+    /**
+     * スネークケースの小文字に変換します。ただし、アンダースコアを挿入する判断が厳格ではありません。具体的には、
+     * 入力値において数字の直後に大文字が続く箇所にアンダースコアを挿入しません。
+     * <p>
+     * この定義は、2.1.0 以前の {@link NamingType#SNAKE_LOWER_CASE} と同じ挙動をし、
+     * 過去との互換性のためだけに存在します。
+     * 
+     * @since 2.2.0
+     */
+    LENIENT_SNAKE_LOWER_CASE {
+
+        @Override
+        public String apply(String text) {
+            if (text == null) {
+                throw new DomaNullPointerException("text");
+            }
+            String s = StringUtil.fromCamelCaseToSnakeCaseWithLenient(text);
+            return s.toLowerCase();
+        }
+
+        @Override
+        public String revert(String text) {
+            if (text == null) {
+                throw new DomaNullPointerException("text");
+            }
+            return StringUtil.fromSnakeCaseToCamelCase(text);
+        }
+
+    },
+
     /**
      * 大文字に変換します。
      * <p>

src/test/java/org/seasar/doma/internal/util/StringUtilTest.java

@@ -44,7 +44,17 @@ public void testFromCamelCaseToSnakeCase() throws Exception {
         assertEquals("aaa_bbb_ccc",
                 StringUtil.fromCamelCaseToSnakeCase("aaaBbbCcc"));
         assertEquals("abc", StringUtil.fromCamelCaseToSnakeCase("abc"));
-        assertEquals("aa1_bbb_ccc", StringUtil.fromCamelCaseToSnakeCase("aa1BbbCcc"));
+        assertEquals("aa1_bbb_ccc",
+                StringUtil.fromCamelCaseToSnakeCase("aa1BbbCcc"));
+    }
+
+    public void testFromCamelCaseToSnakeCaseWithLenient() throws Exception {
+        assertEquals("aaa_bbb_ccc",
+                StringUtil.fromCamelCaseToSnakeCaseWithLenient("aaaBbbCcc"));
+        assertEquals("abc",
+                StringUtil.fromCamelCaseToSnakeCaseWithLenient("abc"));
+        assertEquals("aa1bbb_ccc",
+                StringUtil.fromCamelCaseToSnakeCaseWithLenient("aa1BbbCcc"));
     }
 
     public void testIsWhitespace() throws Exception {