zl
Имхо вполне очевидно
Ошибок нет, результат пуст
Ошибок нет, результат пуст
и дальше паника
Size: a a a
2020 April 18
zl
надо делать очевидно
zl
чтоб и дураку было понятно
zl
очевидным будет возвращать или хороший результат, который не надо потом проверять, или ошибку, которую при желании обработать можно
C
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
Конкретно в данном случае ошибка лучше смотрится
zl
Мне nil nil больше импонирует если результат массив
а почему не пустой массив?
zl
можно сразу циклы запускать не проверяя
C
а почему не пустой массив?
А ну да, по идее будет пустой массив
RS
Мне nil nil больше импонирует если результат массив
ну вот если тип результата срез, тогда
nil, nil будет означать, что сам срез найден, а так он пустRS
а почему не пустой массив?
а зачем аллокейтить массив если он и так пуст?
zl
ну вот если тип результата срез, тогда
nil, nil будет означать, что сам срез найден, а так он пусттогда в клиентском коде лишние строчки будут
C
а зачем аллокейтить массив если он и так пуст?
Чтобы исключить потенциальную панику
zl
сначала на ошибку проверка, потом на nil
C
а зачем аллокейтить массив если он и так пуст?
По хорошему, если массив был найден, он не должен быть nil
RS
По хорошему, если массив был найден, он не должен быть nil
с чего бы? в Go
nil можно считать пустым массивомVM
с чего бы? в Go
nil можно считать пустым массивом+