Как пользоваться check/handle в Golang?

Question

[Vladimir]

Я пробовал писать handler'ы для оператора check по статье "Обработка ошибок в Go 2", но вот, например, этот код, взятый из статьи, не компилируется:

func printSum(a, b string) error {
    handle err { return err }
    x := check strconv.Atoi(a)
    y := check strconv.Atoi(b)
    fmt.Println("result:", x + y)
    return nil
}

Ошибка:

check-handle/main.go:13:12: syntax error: unexpected err at end of statement
check-handle/main.go:14:5: syntax error: non-declaration statement outside function body

Что бы я не делал, у меня не получается написать handler ошибок. Пожалуйста, помогите. Если не сложно, посоветуйте как лучше обрабатывать ошибки, так как у меня в коде копипаст из обработчика ошибки, вот например:

rawSubStart, err := parseTime(rawSub.Start)
if err != nil {
	return err
}
rawSubEnd, err := parseTime(rawSub.End)
if err != nil {
	return err
}

Answer 1

[Владислав]

А у вас го версии 2?

Подсказка: версии 2 не существует :), а в статье описаны идеи, которые, возможно, когда нибудь будут добавлены в го в будущем, которое пока не наступило.

в статье ведь написано:

Черновики дизайна это даже не предложения (proposals), с которых начинается любое изменение в библиотеке, тулинге или языке Go. Это начальная точка для обсуждения дизайна, предложенная командой Go после нескольких лет работы над данными вопросами. Всё, что описано в черновиках с большой долей вероятности будет изменено, и, при наилучших раскладах, воплотится в реальность только через несколько релизов (я даю ~2 года).

Если не сложно, посоветуйте как лучше обрабатывать ошибки, так как у меня в коде копипаст из обработчика ошибки, вот например:

Да, так и обрабатывать, это го, тут так принято... Можно вынести это в отдельную функцию, которая будет возвращать `start, end, err`.

Comments

[Vladimir]

Как тогда лучше обрабатывать ошибки? Я видел где-то пример с анонимной функцией, в которой написан обработчик ошибки. Как Вы считаете, это хорошая практика?

[Vladimir]

Мне кажется, что копипастить обработчик как то не очень хорошо и это засоряет код.

[Владислав]

Vladimir, я обновил комментарий. А можете привести пример? Опускать ошибки я бы не советовал, всегда их обрабатывайте.

[Vladimir]

res, err := http.Get(episode.VideoUrl)
if err != nil {
	logger.Println(err)
	report := Report{"newEpisodes", err.Error()}
	bot.reportToChannel(report)
	continue
}
if res.StatusCode != 200 {
	logger.Println(err)
	report := Report{"newEpisodes", err.Error()}
	bot.reportToChannel(report)
	continue
}
p, listType, err := m3u8.DecodeFrom(res.Body, false)
if err != nil {
	logger.Println(err)
	report := Report{"newEpisodes", err.Error()}
	bot.reportToChannel(report)
	continue
}
if listType != m3u8.MASTER {
	logger.Println("Playlist is not Master", episode.VideoUrl)
	report := Report{"Playlist is not Master", episode.VideoUrl}
	bot.reportToChannel(report)
	continue
}
videoUrl, ok := findX480Url(p.(*m3u8.MasterPlaylist).Variants)
if ok == false {
	logger.Println("No 480p", episode.VideoUrl)
	report := Report{"No 480p", episode.VideoUrl}
	bot.reportToChannel(report)
	continue
}

[Владислав]

Vladimir, это достаточно обычное явление в го, по другому никак.
вот пример из достаточно популярного кода на го https://github.com/gohugoio/hugo/blob/master/parse...
но вообще можете открыть любой код, везде будет

if err != nil {
  return err
}

так как это "основа" языка

[Vladimir]

Владислав Фурсов, спасибо! Я, кстати, использую hugo для сайта.

[Владислав]

Vladimir, в вашем примере происходит обработка ошибок на месте и куча дублирующего кода, который намного сложнее, чем "return err".

Ваш пример, написанный опытным го программистом, будет выглядеть как-то так:

func main() {
	err := doJob()
	if err != nil {
		logger.Println(err)
		report := Report{"newEpisodes", err.Error()}
		bot.reportToChannel(report)
	}
}

func doJob() error {
	res, err := http.Get(episode.VideoUrl)
	if err != nil {
		return err
	}
	if res.StatusCode != 200 {
		return errors.New("bad status")
	}

	p, listType, err := m3u8.DecodeFrom(res.Body, false)
	if err != nil {
		return fmt.Errorf("decode m3u8: %w", err) 
	}

	if listType != m3u8.MASTER {
		return fmt.Errorf("playlist is not Master: %w", err) 
	}

	videoUrl, ok := findX480Url(p.(*m3u8.MasterPlaylist).Variants)
	if ok == false {
		return fmt.Errorf("no 480p: %w", err) 
	}

	return nil
}

намного чище, не правда ли?

[Vladimir]

Владислав Фурсов, Да, Спасибо!