From 26064f146c8cfad233c5efb7ad8879e75fb810c0 Mon Sep 17 00:00:00 2001 From: Looly Date: Wed, 16 Jun 2021 02:32:12 +0800 Subject: [PATCH] fix method --- .../cn/hutool/core/collection/CollUtil.java | 21 +------ .../cn/hutool/core/collection/ListUtil.java | 54 ----------------- .../main/java/cn/hutool/core/map/MapUtil.java | 17 ++++-- .../java/cn/hutool/core/util/ArrayUtil.java | 58 +++++-------------- .../hutool/core/collection/ListUtilTest.java | 2 +- .../cn/hutool/core/util/ArrayUtilTest.java | 12 ++-- 6 files changed, 36 insertions(+), 128 deletions(-) diff --git a/hutool-core/src/main/java/cn/hutool/core/collection/CollUtil.java b/hutool-core/src/main/java/cn/hutool/core/collection/CollUtil.java index 6b5764c2b..46d2e3406 100644 --- a/hutool-core/src/main/java/cn/hutool/core/collection/CollUtil.java +++ b/hutool-core/src/main/java/cn/hutool/core/collection/CollUtil.java @@ -1141,6 +1141,9 @@ public class CollUtil { } Collection collection2 = ObjectUtil.clone(collection); + if (isEmpty(collection2)) { + return collection2; + } try { collection2.clear(); } catch (UnsupportedOperationException e) { @@ -1176,24 +1179,6 @@ public class CollUtil { return edit(collection, t -> filter.accept(t) ? t : null); } - /** - * 过滤
- * 过滤过程通过传入的Filter实现来过滤返回需要的元素内容，这个Filter实现可以实现以下功能： - * - * 
-	 * 1、过滤出需要的对象，{@link Filter#accept(Object)}方法返回true的对象将被加入结果集合中
-	 *
- * - * @param 集合元素类型 - * @param list 集合 - * @param filter 过滤器 - * @return 过滤后的数组 - * @since 4.1.8 - */ - public static List filterNew(List list, Filter filter) { - return ListUtil.editNew(list, t -> filter.accept(t) ? t : null); - } - /** * 去掉集合中的多个元素，此方法直接修改原集合 * diff --git a/hutool-core/src/main/java/cn/hutool/core/collection/ListUtil.java b/hutool-core/src/main/java/cn/hutool/core/collection/ListUtil.java index f8d5ff7ee..4de74a626 100644 --- a/hutool-core/src/main/java/cn/hutool/core/collection/ListUtil.java +++ b/hutool-core/src/main/java/cn/hutool/core/collection/ListUtil.java @@ -2,7 +2,6 @@ package cn.hutool.core.collection; import cn.hutool.core.comparator.PinyinComparator; import cn.hutool.core.comparator.PropertyComparator; -import cn.hutool.core.lang.Editor; import cn.hutool.core.lang.Matcher; import cn.hutool.core.util.ArrayUtil; import cn.hutool.core.util.ObjectUtil; @@ -423,59 +422,6 @@ public class ListUtil { return result; } - /** - * 编辑列表
- * 过滤过程通过传入的Editor实现来返回需要的元素内容，这个Editor实现可以实现以下功能： - * - * 
-	 * 1、过滤出需要的对象，如果返回null表示这个元素对象抛弃
-	 * 2、修改元素对象，返回集合中为修改后的对象
-	 *
- * 注意：此方法会修改原List！！ - * - * @param 集合元素类型 - * @param list 集合 - * @param editor 编辑器接口 - * @return 过滤后的数组 - * @since 4.1.8 - */ - public static List editNew(List list, Editor editor) { - return (List) CollUtil.edit(list, editor); - } - - /** - * 编辑列表
- * 过滤过程通过传入的Editor实现来返回需要的元素内容，这个Editor实现可以实现以下功能： - * - * 
-	 * 1、过滤出需要的对象，如果返回null表示这个元素对象抛弃
-	 * 2、修改元素对象，返回集合中为修改后的对象
-	 *
- * 注意：此方法会修改原List！！ - * - * @param 集合元素类型 - * @param list 集合 - * @param editor 编辑器接口 - * @return 过滤后的数组 - * @since 4.1.8 - */ - public static List edit(List list, Editor editor) { - if (null == list || null == editor) { - return list; - } - - final int size = list.size(); - T ele; - for (int i = 0; i < size; i++) { - ele = list.get(i); - ele = editor.edit(ele); - if(null != ele){ - list.set(i, ele); - } - } - return list; - } - /** * 获取匹配规则定义中匹配到元素的最后位置
* 此方法对于某些无序集合的位置信息，以转换为数组后的位置为准。 diff --git a/hutool-core/src/main/java/cn/hutool/core/map/MapUtil.java b/hutool-core/src/main/java/cn/hutool/core/map/MapUtil.java index fbdf31aab..e21da92d9 100644 --- a/hutool-core/src/main/java/cn/hutool/core/map/MapUtil.java +++ b/hutool-core/src/main/java/cn/hutool/core/map/MapUtil.java @@ -609,11 +609,11 @@ public class MapUtil { // ----------------------------------------------------------------------------------------------- filter /** - * 过滤
- * 过滤过程通过传入的Editor实现来返回需要的元素内容，这个Editor实现可以实现以下功能： + * 编辑Map
+ * 编辑过程通过传入的Editor实现来返回需要的元素内容，这个Editor实现可以实现以下功能： * * 
-	 * 1、过滤出需要的对象，如果返回null表示这个元素对象抛弃
+	 * 1、过滤出需要的对象，如果返回{@code null}表示这个元素对象抛弃
 	 * 2、修改元素对象，返回集合中为修改后的对象
 	 *
* @@ -621,19 +621,24 @@ public class MapUtil { * @param Value类型 * @param map Map * @param editor 编辑器接口 - * @return 过滤后的Map + * @return 编辑后的Map */ public static Map edit(Map map, Editor> editor) { if (null == map || null == editor) { return map; } - final Map map2 = ObjectUtil.clone(map); + Map map2 = ObjectUtil.clone(map); if (isEmpty(map2)) { return map2; } + try { + map2.clear(); + } catch (UnsupportedOperationException e) { + // 克隆后的对象不支持清空，说明为不可变集合对象，使用默认的ArrayList保存结果 + map2 = new HashMap<>(); + } - map2.clear(); Entry modified; for (Entry entry : map.entrySet()) { modified = editor.edit(entry); diff --git a/hutool-core/src/main/java/cn/hutool/core/util/ArrayUtil.java b/hutool-core/src/main/java/cn/hutool/core/util/ArrayUtil.java index e579768dd..f49645a80 100644 --- a/hutool-core/src/main/java/cn/hutool/core/util/ArrayUtil.java +++ b/hutool-core/src/main/java/cn/hutool/core/util/ArrayUtil.java @@ -575,21 +575,26 @@ public class ArrayUtil extends PrimitiveArrayUtil { } /** - * 过滤
- * 过滤过程通过传入的Editor实现来返回需要的元素内容，这个Editor实现可以实现以下功能： + * 编辑数组
+ * 编辑过程通过传入的Editor实现来返回需要的元素内容，这个Editor实现可以实现以下功能： * * 
-	 * 1、过滤出需要的对象，如果返回null表示这个元素对象抛弃
+	 * 1、过滤出需要的对象，如果返回{@code null}表示这个元素对象抛弃
 	 * 2、修改元素对象，返回集合中为修改后的对象
 	 *
+ *

* * @param 数组元素类型 * @param array 数组 * @param editor 编辑器接口 - * @return 过滤后的数组 + * @since 5.3.3 */ - public static T[] filter(T[] array, Editor editor) { - ArrayList list = new ArrayList<>(array.length); + public static T[] edit(T[] array, Editor editor) { + if (null == editor) { + return array; + } + + final ArrayList list = new ArrayList<>(array.length); T modified; for (T t : array) { modified = editor.edit(t); @@ -597,28 +602,8 @@ public class ArrayUtil extends PrimitiveArrayUtil { list.add(modified); } } - return list.toArray(Arrays.copyOf(array, list.size())); - } - - /** - * 编辑数组
- * 编辑过程通过传入的Editor实现来返回需要的元素内容，这个Editor实现可以实现以下功能： - * - * 

-	 * 1、修改元素对象，返回集合中为修改后的对象
-	 *
- *

- * 注意：此方法会修改原数组！ - * - * @param 数组元素类型 - * @param array 数组 - * @param editor 编辑器接口 - * @since 5.3.3 - */ - public static void edit(T[] array, Editor editor) { - for (int i = 0; i < array.length; i++) { - array[i] = editor.edit(array[i]); - } + final T[] result = newArray(array.getClass().getComponentType(), list.size()); + return list.toArray(result); } /** @@ -636,18 +621,7 @@ public class ArrayUtil extends PrimitiveArrayUtil { * @since 3.2.1 */ public static T[] filter(T[] array, Filter filter) { - if (null == filter) { - return array; - } - - final ArrayList list = new ArrayList<>(array.length); - for (T t : array) { - if (filter.accept(t)) { - list.add(t); - } - } - final T[] result = newArray(array.getClass().getComponentType(), list.size()); - return list.toArray(result); + return edit(array, t -> filter.accept(t) ? t : null); } /** @@ -659,7 +633,7 @@ public class ArrayUtil extends PrimitiveArrayUtil { * @since 3.2.2 */ public static T[] removeNull(T[] array) { - return filter(array, (Editor) t -> { + return edit(array, t -> { // 返回null便不加入集合 return t; }); @@ -697,7 +671,7 @@ public class ArrayUtil extends PrimitiveArrayUtil { * @since 3.2.1 */ public static String[] nullToEmpty(String[] array) { - return filter(array, (Editor) t -> null == t ? StrUtil.EMPTY : t); + return edit(array, t -> null == t ? StrUtil.EMPTY : t); } /** diff --git a/hutool-core/src/test/java/cn/hutool/core/collection/ListUtilTest.java b/hutool-core/src/test/java/cn/hutool/core/collection/ListUtilTest.java index cb2815cef..229486afb 100644 --- a/hutool-core/src/test/java/cn/hutool/core/collection/ListUtilTest.java +++ b/hutool-core/src/test/java/cn/hutool/core/collection/ListUtilTest.java @@ -40,7 +40,7 @@ public class ListUtilTest { @Test public void editTest(){ List a = ListUtil.toLinkedList("1", "2", "3"); - final List filter = ListUtil.edit(a, str -> "edit" + str); + final List filter = (List) CollUtil.edit(a, str -> "edit" + str); Assert.assertEquals("edit1", filter.get(0)); Assert.assertEquals("edit2", filter.get(1)); Assert.assertEquals("edit3", filter.get(2)); diff --git a/hutool-core/src/test/java/cn/hutool/core/util/ArrayUtilTest.java b/hutool-core/src/test/java/cn/hutool/core/util/ArrayUtilTest.java index 5df5ad6c6..da5dbe682 100644 --- a/hutool-core/src/test/java/cn/hutool/core/util/ArrayUtilTest.java +++ b/hutool-core/src/test/java/cn/hutool/core/util/ArrayUtilTest.java @@ -1,8 +1,6 @@ package cn.hutool.core.util; import cn.hutool.core.collection.CollUtil; -import cn.hutool.core.lang.Editor; -import cn.hutool.core.lang.Filter; import org.junit.Assert; import org.junit.Test; @@ -79,23 +77,23 @@ public class ArrayUtilTest { } @Test - public void filterTest() { + public void filterEditTest() { Integer[] a = {1, 2, 3, 4, 5, 6}; - Integer[] filter = ArrayUtil.filter(a, (Editor) t -> (t % 2 == 0) ? t : null); + Integer[] filter = ArrayUtil.edit(a, t -> (t % 2 == 0) ? t : null); Assert.assertArrayEquals(filter, new Integer[]{2, 4, 6}); } @Test public void filterTestForFilter() { Integer[] a = {1, 2, 3, 4, 5, 6}; - Integer[] filter = ArrayUtil.filter(a, (Filter) t -> t % 2 == 0); + Integer[] filter = ArrayUtil.filter(a, t -> t % 2 == 0); Assert.assertArrayEquals(filter, new Integer[]{2, 4, 6}); } @Test - public void filterTestForEditor() { + public void editTest() { Integer[] a = {1, 2, 3, 4, 5, 6}; - Integer[] filter = ArrayUtil.filter(a, (Editor) t -> (t % 2 == 0) ? t * 10 : t); + Integer[] filter = ArrayUtil.edit(a, t -> (t % 2 == 0) ? t * 10 : t); Assert.assertArrayEquals(filter, new Integer[]{1, 20, 3, 40, 5, 60}); }