Java中if-else语句、方法调用与变量管理的常见陷阱及修正指南

import java.util.Scanner; // Needed to make Scanner available

public static int age()
{
    int age;
    Scanner scanner = new Scanner(System.in);
    System.out.println("How old are you? ");
    age = Integer.parseInt(scanner.nextLine());
    return age;
}

age(); // 问题一：孤立的方法调用

public static void yesornodisability()
{
    String disabled;
    Scanner scanner = new Scanner(System.in);
    System.out.println("Are you registered disabled(Yes / No)? ");
    disabled = scanner.nextLine();
    return; // 问题五：方法设计与返回值类型
}

yesornodisability(); // 问题一：孤立的方法调用

public static int total()
{
    int total;
    total = 10;
    return total;
}

total(); // 问题一：孤立的方法调用

public static void swimmingprice()
{
    if (age()<=18);{ // 问题二：条件语句的语法错误； 问题三：重复的方法调用
    total() = total()/2}; // 问题四：方法返回值与变量赋值的混淆

    else if (age()>=65);{ // 问题二：条件语句的语法错误； 问题三：重复的方法调用
        total()=total()-3}; // 问题四：方法返回值与变量赋值的混淆

    else if (yesornodisability().equals("Yes");{ // 问题二：条件语句的语法错误； 问题五：方法设计与返回值类型
        total() = total()-4}; // 问题四：方法返回值与变量赋值的混淆

    System.out.println("The swimming price for you is "+total()+" pounds.");
}

import java.util.Scanner; // 导入Scanner类

public class SwimmingPriceCalculator { // 类名应更具描述性

    // 1. 获取年龄的方法，返回整数年龄
    public static int getAge() {
        Scanner scanner = new Scanner(System.in);
        System.out.println("How old are you? ");
        int age = Integer.parseInt(scanner.nextLine());
        // 注意：在实际应用中，Scanner应该被关闭，但如果频繁创建关闭会影响性能。
        // 对于简单的命令行程序，可以考虑在main方法中创建一次Scanner并传递。
        // 为了示例简洁，此处保持原样，但需了解其潜在问题。
        // scanner.close(); // 实际项目中应关闭Scanner
        return age;
    }

    // 2. 获取残疾状态的方法，返回布尔值更清晰
    public static boolean getDisabilityStatus() {
        Scanner scanner = new Scanner(System.in);
        System.out.println("Are you registered disabled (Yes / No)? ");
        String disabledInput = scanner.nextLine().trim().toLowerCase(); // 转换为小写并去除空格，增强健壮性
        // scanner.close(); // 实际项目中应关闭Scanner
        return disabledInput.equals("yes"); // 如果输入是"yes"则返回true
    }

    // 3. 计算游泳价格的方法
    public static void calculateSwimmingPrice() {
        // 3.1 变量声明与初始化
        int userAge = getAge(); // 调用一次getAge()并存储结果
        boolean isDisabilityRegistered = getDisabilityStatus(); // 调用一次getDisabilityStatus()并存储结果

        // 初始价格
        double totalPrice = 10.0; // 使用double类型以支持小数折扣

        // 3.2 条件判断与价格计算
        // 注意：if-else if 语句的顺序很重要。
        // 如果一个条件满足，其对应的代码块执行后，整个if-else if链就会结束。
        // 考虑折扣优先级，例如，如果小于18岁且残疾，是先打折还是先减免？
        // 当前逻辑是：先判断年龄是否小于等于18，再判断是否大于等于65，最后判断是否残疾。
        // 这种顺序意味着：如果小于18岁，即使残疾也只享受50%折扣，不会再减4磅。
        // 如果需要同时享受多个折扣，则不能使用if-else if，而应该使用独立的if语句。

        if (userAge <= 18) { // 正确的if语句语法，没有分号
            totalPrice = totalPrice / 2; // 50% 折扣
            System.out.println("您符合儿童折扣（50%）。");
        } else if (userAge >= 65) { // 正确的else if语句语法
            totalPrice = totalPrice - 3; // 减3磅
            System.out.println("您符合老年折扣（-3磅）。");
        } 

        // 独立判断残疾折扣，允许与其他折扣叠加（如果上面不是else if）
        // 如果需要残疾折扣可以与其他年龄折扣叠加，则这里不应是else if，而应是独立的if
        // 如下面所示，如果前面是if-else if，这里就不能再用else if了，除非是互斥条件
        // 考虑到原题意，可能希望是互斥的，即只应用一个最高折扣。
        // 但如果想叠加，可以这样写：
        // if (isDisabilityRegistered) {
        //     totalPrice = totalPrice - 4;
        // }
        // 假设原意是互斥的，且残疾折扣是最后考虑的（优先级最低或独立于年龄）
        // 如果要实现“如果上述年龄折扣都不满足，才考虑残疾折扣”，则保持else if
        else if (isDisabilityRegistered) { // 正确的else if语句语法
            totalPrice = totalPrice - 4; // 减4磅
            System.out.println("您符合残疾折扣（-4磅）。");
        }

        System.out.println("您的游泳价格是 " + totalPrice + " 磅。");
    }

    public static void main(String[] args) {
        // 在main方法中调用业务逻辑方法
        calculateSwimmingPrice();
    }
}