51 lines
1.0 KiB
Markdown
51 lines
1.0 KiB
Markdown
### for 循环中禁止使用defer
|
||
- 直接在 for 循环中使用 defer 很可能使得 defer 不能执行，导致内存泄露或者其他资源问题，所以应该将 defer 放到外层。
|
||
- 若确实需要使用 defer，可以将逻辑封装为一个独立函数或者使用闭包
|
||
- 错误
|
||
```go
|
||
func readFiles(files []string) {
|
||
for i:=0;i<len(files);i++{
|
||
f,err:=os.Open(files[i])
|
||
if err!=nil{
|
||
println(err.Error())
|
||
continue
|
||
}
|
||
|
||
// bug here
|
||
// 在循环中的 defer 只有在循环结束后才会执行
|
||
// 若 files 很多，会导致大量文件句柄未及时释放
|
||
defer f.Close()
|
||
|
||
bf,err:=io.ReadAll(f)
|
||
if err!=nil{
|
||
println(err.Error())
|
||
}else{
|
||
println(string(bf))
|
||
}
|
||
}
|
||
}
|
||
```
|
||
- 正确
|
||
```go
|
||
func readFiles(files []string) {
|
||
for i:=0;i<len(files);i++{
|
||
readFile(files[i])
|
||
}
|
||
}
|
||
|
||
func readFile(name string){
|
||
f,err:=os.Open(name)
|
||
if err!=nil{
|
||
println(err.Error())
|
||
return
|
||
}
|
||
defer f.Close()
|
||
bf,err:=io.ReadAll(f)
|
||
if err!=nil{
|
||
println(err.Error())
|
||
}else{
|
||
println(string(bf))
|
||
}
|
||
}
|
||
|
||
``` |