为何代码中null被当作字符串拼接至文件名？如何解决？

这是个很常见的空值处理坑，咱们可以从三个方向来解决这个问题，彻底避免nullFile.txt这种奇怪的文件名出现：

方案1：让checkOS返回默认路径而非null #

与其返回null让调用方踩坑，不如在未知系统时直接返回一个合法的默认路径（比如程序当前运行目录），这样不管什么系统都能拿到可用的路径。而且注意哦，你原代码里Mac的路径写的是/home/，其实Mac的用户目录是/Users/，这个小细节可以顺便修正下：

public static String checkOS() {
    String OS = System.getProperty("os.name").toLowerCase();
    if(OS.indexOf("win") >= 0){
        return "C:/";
    } else if(OS.indexOf("nix") >= 0 || OS.indexOf("nux") >= 0 || OS.indexOf("aix") > 0 ){
        return "/home/";
    } else if(OS.indexOf("mac") >= 0){
        return "/Users/"; // 修正Mac的默认用户目录路径
    } else{
        // 返回程序当前运行的目录，确保路径合法
        return System.getProperty("user.dir") + "/";
    }
}

方案2：在调用方增加null检查 #

如果没办法修改checkOS方法，那就在接收返回值的地方先做null判断，再处理拼接逻辑：

String storePath = checkOS();
String fileName = "File.txt";
String fullPath;

if (storePath != null) {
    fullPath = storePath + fileName;
} else {
    // 两种选择：要么用默认目录保存，要么提示错误
    fullPath = System.getProperty("user.dir") + "/" + fileName;
    // 或者直接抛出异常，让上层感知到不支持的系统
    // throw new UnsupportedOperationException("当前操作系统不支持自动选择存储路径");
}

// 执行文件保存操作

方案3：让checkOS抛出异常而非返回null #

如果你希望程序在遇到未知系统时明确报错，而不是默默用默认路径，那可以让checkOS直接抛出异常，强制调用方处理这种异常情况：

public static String checkOS() throws UnsupportedOperationException {
    String OS = System.getProperty("os.name").toLowerCase();
    if(OS.indexOf("win") >= 0){
        return "C:/";
    } else if(OS.indexOf("nix") >= 0 || OS.indexOf("nux") >= 0 || OS.indexOf("aix") > 0 ){
        return "/home/";
    } else if(OS.indexOf("mac") >= 0){
        return "/Users/";
    } else{
        throw new UnsupportedOperationException("不支持的操作系统：" + OS);
    }
}

调用方必须捕获或者声明这个异常，这样就不会出现null被误拼接的情况了。

选哪种方案？ #

• 如果希望程序在任何系统下都能正常运行，优先选方案1；
• 如果需要明确告知用户/上层系统不支持当前OS，选方案3；
• 如果没办法修改checkOS的代码，那就用方案2。

内容的提问来源于stack exchange，提问作者bhatnaushad