10个Java开发者必须改掉的过时编程习惯
10个Java开发者必须改掉的过时编程习惯
"如果你一直拖着过去不放，就无法向前进步。"这正是我在代码审查时对团队说的话。然而，每次我打开一个拉取请求时，我仍然看到过时的Java特性、坏习惯和应该在几年前就被淘汰的编码习惯。所以，如果你今天在使用Java，无论你是初级、中级还是高级开发者，这篇文章都适合你 🫵 。其中一些观点可能会让你感到不舒服。有些可能与你所学的相违背。但这正是你应该读到最后的原因。
1. 错误使用 Optional.get()
Optional是一个很棒的特性，但我看到许多开发者误用了它：
Optional<String> value = getValue();
String result = value.get(); // 可能抛出 NoSuchElementException!
这是错误的！
如果你在使用**Optional**，就要拥抱它的API：
String result = value.orElse("default");
// 或者
value.ifPresent(val -> System.out.println(val));
// 或者
String result = value.orElseThrow(() -> new IllegalArgumentException("Value missing"));
2. 硬编码值而不是使用常量
这是我在代码中看到的最大罪过之一：
if (status == 3) {
// do something
}
一个新的初级开发者加入团队，他不知道 3 是什么意思。然后他错误地修改了逻辑。砰。生产环境出现了bug。
正确的方式：
// 使用 static final
public static final int STATUS_COMPLETED = 3;
if (status == STATUS_COMPLETED) {
// do something
}
// 更好的方式 - 使用枚举：
if (status == Status.COMPLETED) {
// do something
}
为什么？ 因为这样你的代码变得自文档化且安全。
3. 有缺陷的单例模式的双重检查锁定
经典错误：
public class MyClass {
private static MyClass instance;
private MyClass() {
// 私有构造函数来强制单例模式
}
public static MyClass getInstance() {
if (instance == null) {
synchronized(MyClass.class) {
if (instance == null) {
instance = new MyClass();
}
}
}
return instance;
}
}
除非使用volatile完美地完成，否则这在Java中是有缺陷的。在并发环境中可能出现微妙的错误。
这里是修复方法：
public class MyClass {
private static volatile MyClass instance; // <<< 修复：添加 'volatile'
private MyClass() {
// 私有构造函数来强制单例模式
}
public static MyClass getInstance() {
if (instance == null) {
synchronized(MyClass.class) {
if (instance == null) {
instance = new MyClass();
}
}
}
return instance;
}
}
更好的方式：
// 使用枚举单例：
public enum Singleton {
INSTANCE;
public void doSomething() {
// 实现
}
}
// 或者使用 static final：
public class MySingleton {
public static final MySingleton INSTANCE = new MySingleton();
}
这些方式安全、简单，并且避免了并发错误。
4. Vector 和 Hashtable
现在是2025年。然而，有时我仍然看到这样的代码：
Vector<String> vector = new Vector<>();
Hashtable<String, String> table = new Hashtable<>();
Vector和Hashtable都是Java早期的遗留类。它们是同步的，因此比现代替代方案慢得多。
应该使用什么替代：
// 对于List：
List<String> list = new ArrayList<>();
// 对于Map：
Map<String, String> map = new HashMap<>();
如果你需要线程安全，使用：
List<String> list = Collections.synchronizedList(new ArrayList<>());
Map<String, String> map = Collections.synchronizedMap(new HashMap<>());
// 或者更好的：
ConcurrentHashMap<String, String> map = new ConcurrentHashMap<>();
为什么要停止使用Vector和Hashtable？
- 在现代应用中性能差
- 存在更好的替代方案，对同步有更精细的控制
5. 原始类型
我仍然审查到这样的PR：
List list = new ArrayList();
list.add("Hello");
list.add(123); // 什么？在字符串列表中放整数？
这是危险的💀。
原始类型移除了类型安全性，导致在运行时爆发的微妙错误。
正确的方式：
List<String> list = new ArrayList<>();
list.add("Hello");
// list.add(123); // 编译错误 - 很好！
提示： 总是使用泛型。你的IDE会帮你早期发现错误。
6. StringBuffer（当你不需要同步时）
许多开发者使用这个：
StringBuffer sb = new StringBuffer();
sb.append("Hello ");
sb.append("World");
但除非你在做多线程工作，否则StringBuffer是不必要的慢。
应该使用什么替代：
StringBuilder sb = new StringBuilder();
sb.append("Hello ");
sb.append("World");
StringBuilder更快，对于单线程操作来说已经足够了，这是你大部分代码的情况。
7. 直接使用 SimpleDateFormat
这是连高级开发者都会犯的经典错误：
SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-dd");
String date = sdf.format(new Date());
问题在哪里？🤔 SimpleDateFormat是不线程安全的。如果这段代码在Web应用或多线程环境中运行，它可能抛出奇怪的日期格式化错误。
应该使用什么替代：
DateTimeFormatter formatter = DateTimeFormatter.ofPattern("yyyy-MM-dd");
String date = LocalDate.now().format(formatter);
新的java.time包（从Java 8开始）是线程安全的，并且远远优越。
8. 使用 System.out.println 进行日志记录
我实际上见过企业应用程序中有数百个**System.out.println()**语句。
当你调试生产问题时，你会后悔的。
正确的方式：
使用日志框架，如SLF4J with Logback或Log4j2：
private static final Logger logger = LoggerFactory.getLogger(MyClass.class);
logger.info("Processing order: {}", orderId);
logger.error("Error processing payment", ex);
- 你可以获得日志级别（info、debug、error）
- 你可以将日志重定向到文件
- 你可以控制日志格式和保留期
🙅♂️永远不要在生产代码中部署System.out.println
9. 过度使用同步而不是使用现代并发
许多开发者仍然这样做：
synchronized(this) {
// do something
}
但Java已经发展了🪴。java.util.concurrent提供了强大、更安全、性能更好的工具：
- 使用ReentrantLock进行显式锁定
- 使用ConcurrentHashMap而不是同步映射
- 使用AtomicInteger、AtomicBoolean等
- 使用ExecutorService而不是手动管理线程
**除非绝对必要，否则避免使用synchronized。**并发很难——让现代库来帮助你。
10. 过时的集合API方法
我有时仍然看到这样的代码：
Enumeration<String> e = myVector.elements();
while (e.hasMoreElements()) {
System.out.println(e.nextElement());
}
❌ 当你可以使用现代for-each循环时，停止使用Enumeration和Iterator：
for (String item : myList) {
System.out.println(item);
}
// 或者更好的 - 使用流：
myList.forEach(System.out::println);
这些更清洁、现代、更可读。