Почему изменилось поведение if?

Question

[Владимир Грабко]

func ErrKeyNotExists() error {
	return errors.New("key not exists")
}

if ErrKeyNotExists() == ErrKeyNotExists() {
	log.Println("ok")
} else {
	log.Println("err")
}

Почему возвращает "err"? У меня в коде много таких функций и после обновления к 1.7 всё сломалось.

UPD.
Из-за того что errors возвращает ссылку то if смотрит одинаков ли адрес, а не значение.
Проверил вот таким кодом

q := 1
w := 1
if &q == &w {
	log.Println("ok")
} else {
	log.Println("err")
}

Как заставить if сравнивать значение?

Comments

[Александр Павлюк]

Вы лучше выносите ошибки в константы, а не в функции, возвращающие их.

[Владимир Грабко]

Александр Павлюк: спасибо. Так и сделаю

Answer 1

[Алексей]

if ErrKeyNotExists().Error() == ErrKeyNotExists().Error() {

Если может возвращаться nil, то нужно доп проверки делать.

Answer 2

[Никита]

Вообще надо делать проверку на != nil.
Если нужно классифицировать ошибки, то лучше создать свои структуры ошибок и проверять на type.

Comments

[Владимир Грабко]

там у меня свитч в коде. И тупой перебор возможных ошибок. А не знаете где найти пример этой самой структуры?

[Никита]

Владимир Грабко: Конечно, всегда рад за вас погуглить или посмотреть в документацию.
https://blog.golang.org/error-handling-and-go
https://golang.org/pkg/errors/

И там и там есть примеры.

[Владимир Грабко]

Никита: вы даже не представляете... Но я думал что вы о чём-то другом говорите.

Answer 3

[Евгений Иваха]

Тут не поведение `if` поменялось. С ним всё правильно. Функция каждый раз возвращает указатель на новую, хоть и идентичную конструкцию. Само собой, при сравнении адресов, они оказываются разными. И не нужно сравнивать текст ошибок. Правильно написали – вынести типовые ошибки в константы, из функций возвращать типовую ошибку и сравнивать с ними же.