变量、参数、返回值优先声明为collection或list等接口类型,而非arraylist等具体实现类,以提升可替换性与抽象性;避免原始类型、硬编码容量、重复遍历及忽略泛型。

用 Collection 和 List 接口声明变量,别用 ArrayList
接口抽象的核心动作就这一个:变量、参数、返回值都优先写成接口类型。比如写 List<string> names = new ArrayList()</string>,而不是 ArrayList<string> names = new ArrayList()</string>。
这样做不是为了“看起来更规范”,而是让后续替换实现类时完全不改调用方代码。某天发现数据量大了要换 LinkedList 做频繁插入,或换 CopyOnWriteArrayList 做读多写少的并发场景,只需改构造那一行。
常见错误现象:ArrayList 直接暴露在 public 方法签名里,导致下游被迫依赖具体实现;或者用 Arrays.asList() 返回的 List,却当成可修改集合去 add(),结果抛 UnsupportedOperationException。
- 所有方法入参尽量用
Collection或List,除非你明确需要ArrayList的随机访问性能或扩容策略 - 返回值如果是内部构造的新集合,也优先返回
List接口,别返回ArrayList -
Arrays.asList()返回的是固定大小的List,要可变请包一层new ArrayList(...)
避免在业务逻辑里硬编码 HashMap 的初始化容量和负载因子
很多人一看到“性能优化”就立刻给 HashMap 写上 new HashMap(16, 0.75f),但实际多数业务场景根本不需要——JDK 8+ 的 HashMap 在首次 put 时才真正分配数组,初始容量只是个提示。
立即学习“Java免费学习笔记(深入)”;
真正影响可维护性的是把容量逻辑散落在各处:比如某个服务里写死 new HashMap(128),另一个地方又写 new HashMap(64),而没人知道这个数字是怎么来的。一旦数据规模变化,这些 magic number 就变成坏味道。
使用场景:只有当你能预估 key 数量且对内存敏感(如缓存层、高频批处理),才考虑显式设置初始容量。
- 优先用无参构造,让 JVM 自适应;90% 的业务 Map 不值得手动调参
- 如果真要设容量,用
(int) Math.ceil(expectedSize / 0.75)算,别拍脑袋写 16/32/64 - 负载因子别乱改,0.75 是空间与时间的平衡点,改成 0.9 容易引发链表过长,改成 0.5 又浪费内存
别在集合上做重复过滤或嵌套循环,先转成 Set 或用 Stream.distinct()
典型坏代码:for (User u1 : users) { for (User u2 : users) { if (u1.getId().equals(u2.getId())) ... }},或者反复调用 list.contains(x) 判断是否存在。
这种写法不仅慢(O(n²) 或 O(n) 每次),更关键的是它把“去重”“查存在”这些语义藏在了过程式逻辑里,别人读代码时得推演半天才知道你在干啥。换成接口抽象后,意图立刻清晰。
性能影响明显:1000 个元素的 ArrayList.contains() 平均要遍历 500 次;而 HashSet.contains() 是常数时间。
- 需要频繁查存在的,构造时就转成
Set:Set<long> userIds = new HashSet(userList.stream().map(User::getId).toList())</long> - 去重优先用
Stream.distinct(),比手写if (!seen.add(x)) continue更直白 - 注意
distinct()依赖equals/hashCode,实体类没重写这两个方法的话,去重会失效
泛型擦除不是借口——所有集合必须带类型参数,禁用原始类型
写 List list = new ArrayList() 或 Map map = new HashMap() 是在给自己埋雷。编译器不再帮你检查类型安全,运行时 ClassCastException 往往出现在离出问题的地方很远的位置。
这不是“多写几个字母”的事。泛型是 Java 集合框架最基础的抽象契约,放弃它等于放弃编译期校验能力。尤其在团队协作中,原始类型会让 IDE 提示失灵、重构失败、甚至掩盖 Integer 和 int 拆箱空指针这类低级但难定位的问题。
容易踩的坑:Object 强转时以为是 String,结果是 BigDecimal;或者用 get(0) 取值后直接调 .length(),IDE 不报错,运行时报 NullPointerException。
- 哪怕类型嵌套深,也要写全:
Map<string list object>>></string>,别嫌烦 - 用
var时注意:局部变量可以靠推导,但字段、方法参数、返回值仍需显式泛型 - 第三方库返回原始类型(如老版 MyBatis)?立刻包装成带泛型的集合,别让它污染你的代码边界
真正难的不是语法,是坚持把类型信息从数据源头开始带进来。一旦中间某层妥协用了原始类型,后面整条链路就失去抽象保护。











