Java是一种广泛使用的编程语言，但是在实际开发中，我们经常会遇到一些“烂代码”，这些代码给维护和阅读带来了困难。本文将介绍Java中的十大烂代码，并提供相应的代码示例，帮助读者避免这些问题。

1. 魔法数值

魔法数值是指在代码中直接使用的未经解释的数字。这种做法使得代码难以理解和维护。

public class Circle {
    private double radius = 5; // 半径
    
    public double calculateArea() {
        return 3.14 * radius * radius;
    }
}

上述示例中的代码使用了半径的值5，但是没有给出解释。为了提高代码的可读性和可维护性，我们应该使用常量或者注释来解释这个值。

public class Circle {
    private static final double DEFAULT_RADIUS = 5; // 默认半径

    private double radius = DEFAULT_RADIUS; 

    public double calculateArea() {
        return Math.PI * radius * radius;
    }
}

2. 长方法

长方法指的是包含过多逻辑的方法。这种方法难以理解和测试，并且对于其他开发人员来说也难以阅读和修改。

public class Calculator {
    public double calculateCircleArea(double radius) {
        double area;
        if (radius >= 0) {
            area = 3.14 * radius * radius;
        } else {
            throw new IllegalArgumentException("半径不能为负数");
        }

        return area;
    }

    public double calculateRectangleArea(double width, double height) {
        double area;
        if (width >= 0 && height >= 0) {
            area = width * height;
        } else {
            throw new IllegalArgumentException("宽度和高度不能为负数");
        }

        return area;
    }
}

上述示例中的calculateCircleArea和calculateRectangleArea方法都包含了过多的逻辑。为了提高代码的可读性和可维护性，我们应该将这些逻辑拆分成小的方法，并使用有意义的名称。

public class Calculator {
    public double calculateCircleArea(double radius) {
        validateRadius(radius);
        return Math.PI * radius * radius;
    }

    public double calculateRectangleArea(double width, double height) {
        validateWidthAndHeight(width, height);
        return width * height;
    }
    
    private void validateRadius(double radius) {
        if (radius < 0) {
            throw new IllegalArgumentException("半径不能为负数");
        }
    }

    private void validateWidthAndHeight(double width, double height) {
        if (width < 0 || height < 0) {
            throw new IllegalArgumentException("宽度和高度不能为负数");
        }
    }
}

3. 过长的参数列表

过长的参数列表使得方法难以使用和测试，并且容易引入错误。

public class Order {
    public void placeOrder(String customerName, String customerPhone, String customerAddress, String productName, int quantity) {
        // 执行下单逻辑
    }
}

上述示例中的placeOrder方法接受了过长的参数列表。为了提高代码的可读性和可维护性，我们应该封装相关的参数成为一个对象。

public class Order {
    public void placeOrder(OrderInfo orderInfo) {
        // 执行下单逻辑
    }
}

public class OrderInfo {
    private String customerName;
    private String customerPhone;
    private String customerAddress;
    private String productName;
    private int quantity;

    // Getters and setters
}

4. 过多的if-else嵌套

过多的if-else嵌套会使得代码逻辑复杂，难以维护和扩展。这种代码通常也不符合面向对象的设计原则。

public class Car {
    public void start() {
        if (isEngineRunning()) {
            if (isKeyTurnedOn()) {
                if (hasEnoughFuel()) {
                    // 启动引擎
                } else {
                    throw new IllegalStateException("燃料不足");
                }
            } else {
                throw new IllegalStateException("钥匙未打开");
            }
        } else {
            throw new IllegalStateException("发动机未运行");