处理多个错误或是相应处理它们的一般方式

IT行业相对于一般传统行业，发展更新速度更快，一旦停止了学习，很快就会被行业所淘汰。所以我们需要踏踏实实的不断学习，精进自己的技术，尤其是初学者。今天golang学习网给大家整理了《处理多个错误或是相应处理它们的一般方式》，聊聊，我们一起来看看吧！

我有这段代码，我不喜欢这种感觉，更不用说 golint 不喜欢它与 error 应该是返回多个项目时的最后一个类型（golint）。解释一下这段代码：

• 我想让用户决定是否关心返回的任何错误
• 特别是在此代码中，有时不需要或不需要音频文件，可以忽略它
• 无论用户在做什么，都可能始终需要视频和输出文件

我愿意以任何方式重构它（无论是分解它、移动东西等等）。go 中是否有更惯用的方式来完成这样的事情？

// download will download the audio and video files to a particular path
func (r *redditvideo) download() (outputvideofilename string, outputaudiofilename string, errvideo error, erraudio error) {
    outputvideofile, errvideo := downloadtemporaryfile(
        r.videourl,
        temporaryvideofileprefix,
    )
    if errvideo == nil {
        outputvideofilename = outputvideofile.name()
    }

    outputaudiofile, erraudio := downloadtemporaryfile(
        r.audiourl,
        temporaryaudiofileprefix,
    )
    if erraudio == nil {
        outputaudiofilename = outputaudiofile.name()
    }

    return outputvideofilename, outputaudiofilename, errvideo, erraudio
}

同样，我发现自己稍后在代码中再次使用了相同的模式：

// SetFiles sets up all of the input files and the output file
func (l *localVideo) SetFiles(inputVideoFilePath string, inputAudioFilePath string, outputVideoFilePath string) (errVideo error, errAudio error, errOutputVideo error) {
	// Set input video file
	l.inputVideoFile, errVideo = os.Open(inputVideoFilePath)
	if errVideo != nil {
		return
	}
	if errVideo = l.inputVideoFile.Close(); errVideo != nil {
		return
	}

	// Set output video file
	l.outputVideoFile, errOutputVideo = os.Create(outputVideoFilePath)
	if errOutputVideo != nil {
		return
	}
	if errOutputVideo = l.outputVideoFile.Close(); errOutputVideo != nil {
		return
	}

	// IMPORTANT! Set input audio file LAST incase it is nil
	l.inputAudioFile, errAudio = os.Open(inputAudioFilePath)
	if errAudio != nil {
		return
	}
	if errAudio = l.inputAudioFile.Close(); errVideo != nil {
		return
	}

	return
}

这次在这段代码中再次出现了一些与上面相同的情况：

• 我们关心视频和输出的设置，可能关心也可能不关心音频
• 有多个错误需要由用户来处理

解决方案

您在标准库中可以看到很多特定函数，它们包装了更通用的非导出函数。请参阅下面的注释代码。

// download is a rather generic function
// and probably you can refactor downloadTemporaryFile
// so that even this function is not needed any more.
func (r *RedditVideo) download(prefix, url string) (filename string, error error) {
    outputFile, err := downloadTemporaryFile(
        r.VideoURL,
        prefix,
    )

    if err == nil {
        return "", fmt.Errorf("Error while download: %s", err)
    }

    return outputFile.Name(), nil
}

// DownloadVideo wraps download, handing over the values specific
// to the video download
func (r *RedditVideo) DownloadVideo() (filename string, averror error) {
    return r.download(TemporaryVideoFilePrefix, r.VideoURL)
}

// DownloadAudio wraps download, handing over the values specific
// to the audio download
func (r *RedditVideo) DownloadAudio() (filename string, averror error) {
    return r.download(TemporaryAudioFilePrefix, r.AudioURL)
}

func main() {

    r := RedditVideo{
        VideoURL: os.Args[1],
        AudioURL: os.Args[2],
    }

    var videoerror, audioerror error
    var videofileName, audiofileName string

    if videofileName, videoerror = r.DownloadVideo(); videoerror != nil {
        fmt.Println("Got an error downloading the video")
    }

    if audiofileName, audioerror = r.DownloadAudio(); audioerror != nil {
        fmt.Println("Got an error downloading the audio")
    }

    fmt.Printf("Video:\t%s\nAudio:\t%s", videofileName, audiofileName)
}

这样，返回的错误来自哪个下载就很明显了。

今天关于《处理多个错误或是相应处理它们的一般方式》的内容就介绍到这里了，是不是学起来一目了然！想要了解更多关于的内容请关注golang学习网公众号！