账户类须封装余额与操作逻辑,balance为private并校验非负;存取款方法需参数校验;推荐BigDecimal防浮点误差;转账应由外部协调器原子执行;账户存储宜用ConcurrentHashMap;边界测试与日志至关重要。

账户类设计必须封装余额与操作逻辑
银行账户的核心是状态(余额)和行为(存取款),不能把 balance 设为 public,否则外部可随意修改,破坏一致性。所有变更必须经过方法校验。
-
balance字段用private double balance,初始化时校验非负 -
deposit(double amount)需判断amount > 0,否则抛IllegalArgumentException -
withdraw(double amount)必须同时检查amount > 0和balance >= amount - 建议用
BigDecimal替代double处理金额,避免浮点误差;若用double,至少在打印或日志中用String.format("%.2f", balance)格式化
转账操作必须保证原子性与账户隔离
转账涉及两个账户的余额联动更新,不是简单调两次 withdraw 和 deposit。若中途失败(如目标账户不存在、余额不足、网络中断),必须回滚,否则资金丢失或重复入账。
- 不要在
Account类里写transferTo(Account target, double amount)这种单边方法——它无法控制对方账户状态 - 应由外部协调器(如
BankService)统一处理:先锁源账户,再查目标账户有效性,再执行扣减与增加,最后释放锁 - 简易实现可用
synchronized块包裹两个账户操作,但注意锁顺序(如按accountId自然序加锁),避免死锁 - 生产环境需考虑数据库事务,Java 层仅做参数校验和业务编排
使用 ArrayList 存储账户容易引发并发与查找性能问题
很多初学者用 ArrayList 模拟银行所有账户,结果在查询、转账、统计时遍历效率低,且多线程下不安全。
- 查找账户(如通过
accountId)应改用HashMap,get()时间复杂度 O(1) - 若需按开户时间排序,可额外维护一个
TreeSet或用Stream.sorted()临时排序,而非依赖列表顺序 - 多线程访问时,
HashMap不是线程安全的,可用ConcurrentHashMap,或在外层加锁 - 避免在循环中反复调用
list.stream().filter(...).findFirst(),每次都是全量扫描
public class Bank {
private final Map accounts = new ConcurrentHashMap<>();
public void transfer(String fromId, String toId, BigDecimal amount) {
Account from = accounts.get(fromId);
Account to = accounts.get(toId);
if (from == null || to == null) throw new IllegalArgumentException("Account not found");
from.withdraw(amount); // 内部已校验
to.deposit(amount);
}
}
测试边界场景比实现功能更重要
学生项目常忽略异常路径:负金额存款、透支取款、空账户转账、重复开户。这些不是“锦上添花”,而是暴露设计缺陷的关键点。
立即学习“Java免费学习笔记(深入)”;
- 测试
new Account("A001", -100.0)应抛异常,而不是静默接受 - 对
withdraw(500.0)在余额为 499.99 的账户上调用,必须精确比较(用BigDecimal.compareTo(),不用==) - 并发测试两个线程同时从同一账户转出,验证余额最终一致性(可用
CountDownLatch控制并发) - 日志建议打在关键操作前后,如 “Before withdraw: balance=1000.00, amount=200.00”,方便定位中间态










