修改Java代码不良代码，可能你也这么用，快来看下吧

{

put(“a”, 1);

put(“b”, 2);

}

};

private static List list = new ArrayList() {

{

add(“a”);

add(“b”);

}

};

正例：

private static Map<String, Integer> map = new HashMap<>();

static {

map.put(“a”, 1);

map.put(“b”, 2);

};

private static List list = new ArrayList<>();

static {

list.add(“a”);

list.add(“b”);

};

建议使用 try-with-resources 语句

Java 7 中引入了 try-with-resources 语句，该语句能保证将相关资源关闭，优于原来的 try-catch-finally 语句，并且使程序代码更安全更简洁。

反例：

private void handle(String fileName) {

BufferedReader reader = null;

try {

String line;

reader = new BufferedReader(new FileReader(fileName));

while ((line = reader.readLine()) != null) {

…

}

} catch (Exception e) {

…

} finally {

if (reader != null) {

try {

reader.close();

} catch (IOException e) {

…

}

}

}

}

正例：

private void handle(String fileName) {

try (BufferedReader reader = new BufferedReader(new FileReader(fileName))) {

String line;

while ((line = reader.readLine()) != null) {

…

}

} catch (Exception e) {

…

}

}

删除未使用的私有方法和字段

删除未使用的私有方法和字段，使代码更简洁更易维护。若有需要再使用，可以从历史提交中找回。

反例：

public class DoubleDemo1 {

private int unusedField = 100;

private void unusedMethod() {

…

}

public int sum(int a, int b) {

return a + b;

}

}

正例：

public class DoubleDemo1 {

public int sum(int a, int b) {

return a + b;

}

}

删除未使用的局部变量

删除未使用的局部变量，使代码更简洁更易维护。

反例：

public int sum(int a, int b) {

int c = 100;

return a + b;

}

正例：

public int sum(int a, int b) {

return a + b;

}

删除未使用的方法参数

未使用的方法参数具有误导性，删除未使用的方法参数，使代码更简洁更易维护。但是，由于重写方法是基于父类或接口的方法定义，即便有未使用的方法参数，也是不能删除的。

反例：

public int sum(int a, int b, int c) {

return a + b;

}

正例：

public int sum(int a, int b) {

return a + b;

}

删除表达式的多余括号

对应表达式中的多余括号，有人认为有助于代码阅读，也有人认为完全没有必要。对于一个熟悉 Java 语法的人来说，表达式中的多余括号反而会让代码显得更繁琐。

反例：

return (x);

return (x + 2);

int x = (y * 3) + 1;

int m = (n * 4 + 2);

正例：

return x;

return x + 2;

int x = y * 3 + 1;

int m = n * 4 + 2;

工具类应该屏蔽构造函数

工具类是一堆静态字段和函数的集合，不应该被实例化。但是，Java 为每个没有明确定义构造函数的类添加了一个隐式公有构造函数。所以，为了避免 java "小白"使用有误，应该显式定义私有构造函数来屏蔽这个隐式公有构造函数。

反例：

public class MathUtils {

public static final double PI = 3.1415926D;

public static int sum(int a, int b) {

return a + b;

}

}

正例：

public class MathUtils {

public static final double PI = 3.1415926D;

private MathUtils() {}

public static int sum(int a, int b) {

return a + b;

}

}

删除多余的异常捕获并抛出

用 catch 语句捕获异常后，什么也不进行处理，就让异常重新抛出，这跟不捕获异常的效果一样，可以删除这块代码或添加别的处理。

反例：

private static String readFile(String fileName) throws IOException {

try (BufferedReader reader = new BufferedReader(new FileReader(fileName))) {

String line;

StringBuilder builder = new StringBuilder();

while ((line = reader.readLine()) != null) {

builder.append(line);

}

return builder.toString();

} catch (Exception e) {

throw e;

}

}

正例：

private static String readFile(String fileName) throws IOException {

try (BufferedReader reader = new BufferedReader(new FileReader(fileName))) {

String line;

StringBuilder builder = new StringBuilder();

while ((line = reader.readLine()) != null) {

builder.append(line);

}

return builder.toString();

}

}

公有静态常量应该通过类访问

虽然通过类的实例访问公有静态常量是允许的，但是容易让人它误认为每个类的实例都有一个公有静态常量。所以，公有静态常量应该直接通过类访问。

反例：

public class User {

public static final String CONST_NAME = “name”;

…

}

User user = new User();

String nameKey = user.CONST_NAME;

正例：

public class User {

public static final String CONST_NAME = “name”;

…

}

String nameKey = User.CONST_NAME;

不要用NullPointerException判断空

空指针异常应该用代码规避（比如检测不为空），而不是用捕获异常的方式处理。

反例：

public String getUserName(User user) {

try {

return user.getName();

} catch (NullPointerException e) {

return null;

}

}

正例：

public String getUserName(User user) {

if (Objects.isNull(user)) {

return null;

}

return user.getName();

}

使用String.valueOf(value)代替""+value

当要把其它对象或类型转化为字符串时，使用 String.valueOf(value) 比""+value 的效率更高。

反例：

int i = 1;

String s = “” + i;

正例：

int i = 1;

String s = String.valueOf(i);

过时代码添加 @Deprecated 注解

当一段代码过时，但为了兼容又无法直接删除，不希望以后有人再使用它时，可以添加 @Deprecated 注解进行标记。在文档注释中添加 @deprecated 来进行解释，并提供可替代方案。

正例：

/**

• 保存

• @deprecated 此方法效率较低，请使用{@link newSave()}方法替换它

*/

@Deprecated

public void save(){

// do something

}

让代码远离 bug

禁止使用构造方法 BigDecimal(double)

BigDecimal(double) 存在精度损失风险，在精确计算或值比较的场景中可能会导致业务逻辑异常。

反例：

BigDecimal value = new BigDecimal(0.1D); // 0.100000000000000005551115…

正例：

BigDecimal value = BigDecimal.valueOf(0.1D);; // 0.1

返回空数组和空集合而不是 null

返回 null ，需要调用方强制检测 null ，否则就会抛出空指针异常。返回空数组或空集合，有效地避免了调用方因为未检测 null 而抛出空指针异常，还可以删除调用方检测 null 的语句使代码更简洁。

反例：

public static Result[] getResults() {

return null;

}

public static List getResultList() {

return null;

}

public static Map<String, Result> getResultMap() {

return null;

}

public static void main(String[] args) {

Result[] results = getResults();

if (results != null) {

for (Result result : results) {

…

}

}

List resultList = getResultList();

if (resultList != null) {

for (Result result : resultList) {

…

}

}

Map<String, Result> resultMap = getResultMap();

if (resultMap != null) {

for (Map.Entry<String, Result> resultEntry : resultMap) {

…

}

}

}

正例：

public static Result[] getResults() {

return new Result[0];

}

public static List getResultList() {

return Collections.emptyList();

}

public static Map<String, Result> getResultMap() {

return Collections.emptyMap();

}

public static void main(String[] args) {

Result[] results = getResults();

for (Result result : results) {

…

}

List resultList = getResultList();

for (Result result : resultList) {

…

}

Map<String, Result> resultMap = getResultMap();

for (Map.Entry<String, Result> resultEntry : resultMap) {

…

}

}

优先使用常量或确定值来调用 equals 方法

对象的 equals 方法容易抛空指针异常，应使用常量或确定有值的对象来调用 equals 方法。当然，使用 java.util.Objects.equals() 方法是最佳实践。

反例：

public void isFinished(OrderStatus status) {

return status.equals(OrderStatus.FINISHED); // 可能抛空指针异常

}

正例：

public void isFinished(OrderStatus status) {

return OrderStatus.FINISHED.equals(status);

}

public void isFinished(OrderStatus status) {
自我介绍一下，小编13年上海交大毕业，曾经在小公司待过，也去过华为、OPPO等大厂，18年进入阿里一直到现在。

深知大多数Java工程师，想要提升技能，往往是自己摸索成长或者是报班学习，但对于培训机构动则几千的学费，着实压力不小。自己不成体系的自学效果低效又漫长，而且极易碰到天花板技术停滞不前！

因此收集整理了一份《2024年Java开发全套学习资料》，初衷也很简单，就是希望能够帮助到想自学提升又不知道该从何学起的朋友，同时减轻大家的负担。

既有适合小白学习的零基础资料，也有适合3年以上经验的小伙伴深入学习提升的进阶课程，基本涵盖了95%以上Java开发知识点，真正体系化！

由于文件比较大，这里只是将部分目录截图出来，每个节点里面都包含大厂面经、学习笔记、源码讲义、实战项目、讲解视频，并且会持续更新！

如果你觉得这些内容对你有帮助，可以扫码获取！！（备注Java获取）

本次面试答案，以及收集到的大厂必问面试题分享：

《一线大厂Java面试题解析+核心总结学习笔记+最新讲解视频+实战项目源码》，点击传送门即可获取！
OrderStatus.FINISHED.equals(status);

}

public void isFinished(OrderStatus status) {
自我介绍一下，小编13年上海交大毕业，曾经在小公司待过，也去过华为、OPPO等大厂，18年进入阿里一直到现在。

深知大多数Java工程师，想要提升技能，往往是自己摸索成长或者是报班学习，但对于培训机构动则几千的学费，着实压力不小。自己不成体系的自学效果低效又漫长，而且极易碰到天花板技术停滞不前！

因此收集整理了一份《2024年Java开发全套学习资料》，初衷也很简单，就是希望能够帮助到想自学提升又不知道该从何学起的朋友，同时减轻大家的负担。[外链图片转存中…(img-WrpX9xvv-1711886630604)]

[外链图片转存中…(img-6QNpkqNC-1711886630605)]

[外链图片转存中…(img-zDjpvGFg-1711886630605)]

既有适合小白学习的零基础资料，也有适合3年以上经验的小伙伴深入学习提升的进阶课程，基本涵盖了95%以上Java开发知识点，真正体系化！

由于文件比较大，这里只是将部分目录截图出来，每个节点里面都包含大厂面经、学习笔记、源码讲义、实战项目、讲解视频，并且会持续更新！

如果你觉得这些内容对你有帮助，可以扫码获取！！（备注Java获取）

本次面试答案，以及收集到的大厂必问面试题分享：

[外链图片转存中…(img-sokOEXqm-1711886630606)]

《一线大厂Java面试题解析+核心总结学习笔记+最新讲解视频+实战项目源码》，点击传送门即可获取！