с
потенциальный
Size: a a a
2020 April 18
zl
потенциальный
а это весомый аргумент отказаться от библиотеки или так себе?
zl
с полгода библиотеку применяю, ничего такого не заметил
жалко выбрасывать только из-за такой мелочи
жалко выбрасывать только из-за такой мелочи
IS
а это весомый аргумент отказаться от библиотеки или так себе?
Это вопрос из разряда "если известно, что у машины иногда не работает педаль тормоза, это повод от нее отказаться?"
IS
Ответ в том числе может быть "не ездите на ней, используйте как склад" :)
IS
Или "почините"
IS
А кто-то продолжает ездить, и для него норм
zl
Это вопрос из разряда "если известно, что у машины иногда не работает педаль тормоза, это повод от нее отказаться?"
грязные оптимизаторы могут проклятие наложить за такие высказывания
zl
в том же fasthttp сплошь и рядом применяются такие подходы
https://github.com/valyala/fasthttp/blob/master/bytesconv.go
https://github.com/valyala/fasthttp/blob/master/bytesconv.go
с
сплошь и рядом это один файл
zl
т.е. fasthttp пройдет race-детектор, потому что один раз?
с
проходит в CI
zl
дискриминация какая-то
валялкину можно в unsafe, а китайцам нет
валялкину можно в unsafe, а китайцам нет
RS
A: возвращать
https://play.golang.org/p/VCxC2jf1gcs
B: возвращать
https://play.golang.org/p/Ul-ZZPf_YB6
nil, nil в случае если объект не найденhttps://play.golang.org/p/VCxC2jf1gcs
B: возвращать
nil, ErrNotFound в случае если объект не найденhttps://play.golang.org/p/Ul-ZZPf_YB6
RS
Какой из выше перечисленных вариантов вам более по душе?
Анонимный опрос
Проголосовало: 22zl
nil, ErrNotFound - это более очевидно
zl
я на днях с этим столкнулся - читал доки через строчку, а там надо еще и проверять на nil оказалось