ChatGPT幫我看下這段代碼有什么問(wèn)題

更新時(shí)間：2023年02月16日 08:37:27 作者：程序猿阿朗

今天一個(gè)很簡(jiǎn)單的功能，觸發(fā)了一個(gè) BUG，處理后我想起了最近爆火的 ChatGPT，于是我嘗試測(cè)試 ChatGPT 能否發(fā)現(xiàn)這個(gè) BUG，這篇文章會(huì)先介紹功能代碼，然后手動(dòng)分析 BUG 原因,需要的朋友可以參考下

今天一個(gè)很簡(jiǎn)單的功能，觸發(fā)了一個(gè) BUG，處理后我想起了最近爆火的 ChatGPT，于是我嘗試測(cè)試 ChatGPT 能否發(fā)現(xiàn)這個(gè) BUG。這篇文章會(huì)先介紹功能代碼，然后手動(dòng)分析 BUG 原因；接著測(cè)試 ChatGPT 能否發(fā)現(xiàn)這個(gè) BUG。

先說(shuō)下結(jié)論，測(cè)試結(jié)束，ChatGPT 今后可能是我編程路上的好幫手。

前言

今天在測(cè)試一個(gè)準(zhǔn)備上線的新功能時(shí)，里面有一段簡(jiǎn)單的函數(shù)，出現(xiàn)了一個(gè) BUG。我先說(shuō)一下這個(gè)函數(shù)的功能，看下你會(huì)怎么寫。

功能描述

輸入：一個(gè)文件夾路徑。

輸出：這個(gè)文件夾中的文件夾數(shù)量。

這是一個(gè)實(shí)現(xiàn)起來(lái)比較簡(jiǎn)單的函數(shù)，我順手寫了下面的代碼：

String pathString = "/Users/darcy/";
File fileDir = new File(pathString);
if (!fileDir.exists() || !fileDir.isDirectory()) {
    throw new RuntimeException("xxx....");
}
String[] fileList = fileDir.list();
int count = 0;
for (String filePath : fileList) {
    if (new File(pathString + filePath).isDirectory()) {
        count++;
    }
}
System.out.println(count);

功能是實(shí)現(xiàn)了，但是很明顯，代碼比較繁瑣，于是我又順手優(yōu)化了一下。

Path path = Paths.get("/Users/darcy/");
if (!Files.exists(path) || !Files.isDirectory(path)) {
    throw new RuntimeException("xxx....");
}
long dirCount = Files.list(path).filter(Files::isDirectory).count();
System.out.println(dirCount);

效果是一樣的，因?yàn)槭褂昧?JDK 7 引入的 Files 類，讓代碼簡(jiǎn)單了不少。

不過(guò)正是這段代碼，觸發(fā)了 BUG，我先賣個(gè)關(guān)子，你先看下 BUG 在什么地方。

問(wèn)題分析

你看出問(wèn)題了嗎？改造后的代碼運(yùn)行少量次數(shù)的情況下，都是可以正常輸出的。但是它有一個(gè)問(wèn)題，就是 Files.list 返回的 Stream 流沒(méi)有進(jìn)行關(guān)閉，這就導(dǎo)致如果這段代碼被頻繁的調(diào)用，一段時(shí)間后將會(huì)因?yàn)榇蜷_(kāi)的文件過(guò)多而報(bào)錯(cuò)，可以測(cè)試出這種異常情況。

問(wèn)題復(fù)現(xiàn)

在循環(huán)中不斷運(yùn)行這段代碼：

 while (true){
     Path path = Paths.get("/Users/darcy/");
     if (!Files.exists(path) || !Files.isDirectory(path)) {
         throw new RuntimeException("xxx....");
     }
     long dirCount = Files.list(path).filter(Files::isDirectory).count();
     System.out.println(dirCount);
 }

一段時(shí)間后控制臺(tái)收到報(bào)錯(cuò)：Too many open files

問(wèn)題原因

報(bào)錯(cuò)的原因是因?yàn)槊總€(gè)進(jìn)程可以打開(kāi)的文件數(shù)量是有限制的，如果一直打開(kāi)不進(jìn)行關(guān)閉，在達(dá)到限制時(shí)會(huì)報(bào)出這個(gè)錯(cuò)誤。

不妨讓代碼運(yùn)行速度降低，然后監(jiān)控下隨著程序的運(yùn)行，運(yùn)行進(jìn)程持有的文件數(shù)是否不斷增加。

while (true){
    Path path = Paths.get("/Users/darcy/");
    if (!Files.exists(path) || !Files.isDirectory(path)) {
        throw new RuntimeException("xxx....");
    }
    long dirCount = Files.list(path).filter(Files::isDirectory).count();
    System.out.println(dirCount);
    Thread.sleep(1000);
}

下面是用到的命令。

## 查找進(jìn)程 ID
?  ~ jps -mlVv | grep testFileList
## 查看指定進(jìn)程引用的文件數(shù)量
?  ~ lsof -p 62248 | wc -l

從測(cè)試結(jié)果中可以看到隨著程序的運(yùn)行，進(jìn)程持有的文件數(shù)量不斷上升。

問(wèn)題解決

原因已經(jīng)知道了，既然 Files.list 返回的 Stream 流沒(méi)有進(jìn)行關(guān)閉，那么進(jìn)行關(guān)閉就可以解決問(wèn)題。

while (true){
    Path path = Paths.get("/Users/darcy/");
    if (!Files.exists(path) || !Files.isDirectory(path)) {
        throw new RuntimeException("xxx....");
    }
    // 方式1：手動(dòng)關(guān)閉流
    // Stream<Path> pathStream = Files.list(path);
    // long dirCount = pathStream.filter(Files::isDirectory).count();
    // pathStream.close();
    // 方式2：使用 try resource 自動(dòng)關(guān)閉流
    try (Stream<Path> pathStream = Files.list(path);){
        long dirCount = pathStream.filter(Files::isDirectory).count();
        System.out.println(dirCount);
    }
    Thread.sleep(1000);
}

再次運(yùn)行后，引用的文件數(shù)量不再上漲，BUG 解決。

?  ~ lsof -p 62879 | wc -l
     321
?  ~ lsof -p 62879 | wc -l
     321
?  ~ lsof -p 62879 | wc -l
     321
?  ~ lsof -p 62879 | wc -l
     321
?  ~ lsof -p 62879 | wc -l
     321
?  ~ lsof -p 62879 | wc -l
     321

ChatGPT 分析

如果這段代碼使用 ChatGPT 分析，它能發(fā)現(xiàn)問(wèn)題嗎？我先告訴你答案，它能發(fā)現(xiàn)，而且描述的很好，并且給出了修復(fù)建議。

ChatGPT 分析代碼

我問(wèn)：

下面這段 Java 代碼有什么問(wèn)題嗎？

Path path = Paths.get("/Users/darcy/");
if (!Files.exists(path) || !Files.isDirectory(path)) {
    throw new RuntimeException("fileDir.isDirectory() error");
}
long dirCount = Files.list(path).filter(Files::isDirectory).count();
System.out.println(dirCount);

ChatGPT：

這段代碼首先會(huì)檢查 /Users/darcy/ 目錄是否存在并且是一個(gè)目錄，如果不是，則會(huì)拋出 RuntimeException。接著，它會(huì)計(jì)算 /Users/darcy/ 目錄下的子目錄數(shù)量并將其打印到控制臺(tái)。

這段代碼看起來(lái)沒(méi)有語(yǔ)法錯(cuò)誤，但是在運(yùn)行時(shí)可能會(huì)拋出 IOException，因?yàn)閷?duì)文件系統(tǒng)進(jìn)行操作時(shí)可能會(huì)出現(xiàn)各種異常情況。特別是，如果指定的路徑不存在，則會(huì)拋出 NoSuchFileException。因此，在代碼中最好捕獲可能發(fā)生的異常并進(jìn)行處理，或者在方法簽名中聲明拋出異常以通知調(diào)用方。

此外，還需要注意的是，Files.list(path) 方法返回的是一個(gè) Stream<Path> 對(duì)象，需要注意其資源釋放問(wèn)題，以避免潛在的資源泄漏問(wèn)題。通常情況下，可以使用 try-with-resources 語(yǔ)句來(lái)自動(dòng)關(guān)閉流，例如：

try (Stream<Path> stream = Files.list(path)) {
    long dirCount = stream.filter(Files::isDirectory).count();
    System.out.println(dirCount);
} catch (IOException e) {
    // 處理異常
}

下面是對(duì)話截圖：

ChatGPT 實(shí)現(xiàn)功能

問(wèn)：

幫我用 Java 寫一串代碼。功能描述。

輸入：一個(gè)文件夾路徑。

輸出：這個(gè)文件夾中的文件夾數(shù)量。

ChatGPT 回答：

以下是 Java 代碼示例，用于計(jì)算指定文件夾路徑下的文件夾數(shù)量：

import java.io.File;

public class FolderCount {
    public static void main(String[] args) {
        String path = "path/to/folder"; // 替換成你的文件夾路徑
        File folder = new File(path);
        int count = 0;
        if (folder.isDirectory()) {
            File[] files = folder.listFiles();
            for (File file : files) {
                if (file.isDirectory()) {
                    count++;
                }
            }
        }
        System.out.println("該文件夾中的文件夾數(shù)量為：" + count);
    }
}

該程序首先接收一個(gè)文件夾路徑作為輸入，然后創(chuàng)建一個(gè) File 對(duì)象來(lái)代表該文件夾。然后它遍歷該文件夾中的所有文件，如果找到一個(gè)文件夾，則將計(jì)數(shù)器加一。最后，它輸出計(jì)數(shù)器的值作為結(jié)果。

下面是對(duì)話截圖：

總結(jié)

在本文中，我們介紹了一個(gè)簡(jiǎn)單功能的 Java 代碼 BUG，即未關(guān)閉 Files.list() 返回的流對(duì)象，導(dǎo)致程序運(yùn)行時(shí)可能會(huì)打開(kāi)過(guò)多的文件而出現(xiàn)異常。我們通過(guò)手動(dòng)分析代碼和測(cè)試，找出了問(wèn)題所在，并提供了兩種解決方案，一種是手動(dòng)關(guān)閉流，另一種是使用 try-with-resources 自動(dòng)關(guān)閉流。最后，我們還使用 ChatGPT 測(cè)試了是否可以用 AI 自動(dòng)識(shí)別代碼中存在的問(wèn)題，結(jié)果表明 ChatGPT 可以幫助我們發(fā)現(xiàn)這類問(wèn)題，為我們的編程工作提供了有力的支持。

上面這個(gè)總結(jié)，也是 ChatGPT 幫我寫的，相對(duì)于 ChatGPT 能給出具體的代碼，我更震驚于 ChatGPT 對(duì)于語(yǔ)義的理解，不僅能理解我的輸入，還能對(duì)代碼意思進(jìn)行自然語(yǔ)言描述。今后你很可能分辨不出你所閱讀的文章是人類寫的，還是 ChatGPT 自動(dòng)生成的。

附錄：ChatGPT 寫代碼測(cè)試

以下測(cè)試結(jié)果都是 ChatGPT 的具體回答，我沒(méi)有改動(dòng)一個(gè)字符。

問(wèn)1

幫我用 Java 寫一段代碼。

輸入：一個(gè)時(shí)間字符串，這個(gè)時(shí)間信息包含年月日時(shí)分秒。

處理：提取時(shí)間字符串中的月份輸出：這個(gè)月份有多少天

ChatGPT 回答：

以下是一個(gè)示例Java代碼，它接受一個(gè)時(shí)間字符串作為輸入，并從中提取出月份并計(jì)算該月有多少天：

import java.time.*;
import java.time.format.DateTimeFormatter;

public class MonthDays {
    public static void main(String[] args) {
        String dateString = "2023-02-14 12:34:56"; // 輸入時(shí)間字符串
        DateTimeFormatter formatter = DateTimeFormatter.ofPattern("yyyy-MM-dd HH:mm:ss");
        LocalDateTime dateTime = LocalDateTime.parse(dateString, formatter);
        int daysInMonth = YearMonth.from(dateTime).lengthOfMonth(); // 提取月份并計(jì)算該月有多少天
        System.out.println(daysInMonth); // 輸出該月天數(shù)
    }
}

在這個(gè)示例中，我們使用了Java 8引入的java.time包來(lái)處理日期和時(shí)間。首先，我們將輸入時(shí)間字符串解析為一個(gè)LocalDateTime對(duì)象。然后，我們從LocalDateTime對(duì)象中提取出年月，并使用YearMonth類的lengthOfMonth()方法計(jì)算該月份有多少天。最后，我們將該天數(shù)輸出到控制臺(tái)。

下面是測(cè)試截圖：

問(wèn)2

幫我用 Java 寫一段代碼。

輸入：一個(gè)時(shí)間字符串，這個(gè)時(shí)間信息包含年月日時(shí)分秒。

輸出：把這個(gè)時(shí)間字符串進(jìn)行轉(zhuǎn)換。獲取這個(gè)時(shí)間當(dāng)天的第三小時(shí)第一秒

ChatGPT 回答：

以下是用 Java 實(shí)現(xiàn)獲取輸入時(shí)間當(dāng)天第三小時(shí)第一秒的代碼示例：

import java.time.LocalDateTime;
import java.time.LocalTime;
import java.time.format.DateTimeFormatter;

public class Main {
    public static void main(String[] args) {
        String timeString = "2023-02-14 12:34:56"; // 代表輸入的時(shí)間字符串
        DateTimeFormatter formatter = DateTimeFormatter.ofPattern("yyyy-MM-dd HH:mm:ss");
        LocalDateTime dateTime = LocalDateTime.parse(timeString, formatter); // 解析時(shí)間字符串

        // 獲取當(dāng)天的3:00:01
        LocalDateTime targetDateTime = LocalDateTime.of(dateTime.toLocalDate(), LocalTime.of(3, 0, 1));
        System.out.println(targetDateTime);
    }
}

解釋：

首先，定義一個(gè)代表輸入時(shí)間的字符串 timeString。
然后，使用 DateTimeFormatter 類創(chuàng)建一個(gè)日期時(shí)間格式化器，以便將時(shí)間字符串轉(zhuǎn)換為 LocalDateTime 對(duì)象。此處使用的格式化器模式字符串為 "yyyy-MM-dd HH:mm:ss"，其中：

yyyy 代表四位數(shù)的年份，例如 2023。
MM 代表兩位數(shù)的月份，例如 02。
dd 代表兩位數(shù)的日期，例如 14。
HH 代表兩位數(shù)的小時(shí)數(shù)（24 小時(shí)制），例如 12。
mm 代表兩位數(shù)的分鐘數(shù)，例如 34。
ss 代表兩位數(shù)的秒數(shù)，例如 56。

使用 LocalDateTime.parse() 方法將時(shí)間字符串解析為 LocalDateTime 對(duì)象。
接下來(lái)，使用 LocalDateTime.toLocalDate() 方法獲取輸入時(shí)間所在的日期，并使用 LocalTime.of() 方法創(chuàng)建一個(gè) LocalTime 對(duì)象，代表目標(biāo)時(shí)間當(dāng)天的 3:00:01。
最后，使用 LocalDateTime.of() 方法將日期和時(shí)間組合成一個(gè) LocalDateTime 對(duì)象，即獲取了輸入時(shí)間當(dāng)天的第三小時(shí)第一秒。打印該對(duì)象即可。

注意：這里的代碼示例假設(shè)輸入時(shí)間字符串的格式是固定的。如果輸入時(shí)間的格式可能會(huì)發(fā)生變化，需要根據(jù)實(shí)際情況調(diào)整格式化器模式字符串。

問(wèn)3

幫我用 Java 寫一段代碼。
輸入：一個(gè)長(zhǎng)度為10的全是小寫字母的字符串
處理1：把這個(gè)字符串按字母逆序
處理2：把這個(gè)字符串第3,6,9位字母轉(zhuǎn)換成大寫
處理3：截取字符串的前7位進(jìn)行輸出

ChatGPT 回答：

以下是用 Java 實(shí)現(xiàn)對(duì)輸入字符串進(jìn)行處理的代碼示例：

public class Main {
    public static void main(String[] args) {
        String inputStr = "abcdefghij"; // 輸入的字符串

        // 處理1：按字母逆序
        String reverseStr = new StringBuilder(inputStr).reverse().toString();

        // 處理2：第3,6,9位字母轉(zhuǎn)換成大寫
        StringBuilder resultStrBuilder = new StringBuilder(reverseStr);
        resultStrBuilder.setCharAt(2, Character.toUpperCase(reverseStr.charAt(2)));
        resultStrBuilder.setCharAt(5, Character.toUpperCase(reverseStr.charAt(5)));
        resultStrBuilder.setCharAt(8, Character.toUpperCase(reverseStr.charAt(8)));
        String resultStr = resultStrBuilder.toString();

        // 處理3：截取前7位進(jìn)行輸出
        String outputStr = resultStr.substring(0, 7);

        System.out.println(outputStr);
    }
}

解釋：

首先，定義一個(gè)代表輸入字符串的變量 inputStr。
處理1：使用 StringBuilder 類的 reverse() 方法將字符串逆序，并使用 toString() 方法轉(zhuǎn)換為普通字符串。將逆序后的字符串保存在 reverseStr 變量中。
處理2：使用 StringBuilder 類創(chuàng)建一個(gè)新的字符串構(gòu)建器，將逆序后的字符串 reverseStr 作為初始字符串。