Go解压目录调用os.ReadDir返回ENOTDIR(3)错误该如何解决？

根因说明 #

你触发ENOTDIR报错的核心原因是混淆了zip归档内的虚拟文件系统和本地磁盘的物理文件系统：

• os.ReadDir是操作系统层面读取本地磁盘路径的接口，但你此时仅在内存中打开了zip压缩包，并未将包内的目录解压写入本地磁盘，你拼接的./+file.Name路径在本地根本不存在，自然会报「目标路径不是目录」的错误。
• 除此之外你的代码还存在多个逻辑问题：
  • 拆分目录名时lastFileName := split[len(split) - 2]存在数组越界风险，当遍历到zip根目录时，split切片长度为1，减2会直接触发panic
  • 判断文件后缀时取split[0]匹配pfx/cert，实际类似file011.pfx的文件名，后缀是切片最后一位，取第一位永远匹配不到目标后缀
  • defer zippedFile.Close()写在for循环内部，会导致大量文件句柄堆积到函数退出时才释放，存在资源泄漏风险
  • 定义的mapTest变量赋值后未被使用，属于冗余逻辑

修复方案 #

不需要调用本地文件系统接口，直接在遍历zip包内文件的过程中完成目录校验和文件读取即可，修复后代码如下：

func openZipFile(sc ServerConfig, path string) ([]byte, error) {
	// 预编译正则，避免每次循环重复编译，提升性能
	dirRegex, err := regexp.Compile(`^[A-Za-z]+$`)
	if err != nil {
		return nil, err
	}

	zipReader, err := zip.OpenReader(path)
	if err != nil {
		return nil, err
	}
	defer zipReader.Close()

	// 先遍历所有条目，收集符合纯字母命名要求的目录
	validDirs := make(map[string]struct{})
	for _, file := range zipReader.File {
		if file.FileInfo().IsDir() {
			split := strings.Split(strings.TrimSuffix(file.Name, "/"), "/")
			lastDirName := split[len(split)-1]
			if dirRegex.MatchString(lastDirName) {
				validDirs[file.Name] = struct{}{}
			}
		}
	}

	// 再次遍历所有条目，读取符合要求的目录下的文件
	for _, file := range zipReader.File {
		if file.FileInfo().IsDir() {
			continue
		}
		// 拆分文件路径，判断父目录是否为合法目录
		splitPath := strings.Split(file.Name, "/")
		if len(splitPath) < 2 {
			continue
		}
		parentDir := splitPath[0] + "/"
		if _, ok := validDirs[parentDir]; !ok {
			continue
		}

		// 直接读取zip内的文件内容，不需要写入本地
		zippedFile, err := file.Open()
		if err != nil {
			zippedFile.Close()
			return nil, err
		}
		content, err := io.ReadAll(zippedFile)
		// 立即关闭句柄，不要用defer
		zippedFile.Close()
		if err != nil {
			return nil, err
		}

		// 提取文件后缀
		splitName := strings.Split(splitPath[len(splitPath)-1], ".")
		if len(splitName) < 2 {
			continue
		}
		suffix := splitName[len(splitName)-1]

		// 按业务需求赋值
		sc.ClusterSetupZip[suffix] = ZipStructure{}
		switch suffix {
		case "pfx":
			sc.ClusterSetupZip[suffix] = ZipStructure{Pfx: content}
		case "cert":
			sc.ClusterSetupZip[suffix] = ZipStructure{Cert: content}
		}
	}
	return nil, nil
}

优化点说明 #

• 预编译正则表达式，避免每次循环重复编译，提升执行性能
• 不需要操作本地磁盘，直接读取zip包内的文件内容，减少不必要的IO开销
• 手动控制文件句柄的关闭时机，避免defer堆积导致的资源泄漏
• 增加了切片长度校验，避免数组越界panic

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