использовать списочный контекст. тогда все автоматом встанет на свои места и все вопросы про конкретное значение переменной пропадут

Интересуешься темой разработки и поддержки высоконагруженных приложений, связанных с финансовыми инструментами? Онлайн-митап ВТБ – специально для тебя!
17 декабря пройдет онлайн-митап “Как обеспечить надежность мобильного приложения при увеличении нагрузки в 10 раз”. Участие бесплатное.

Предварительная регистрация поможет не забыть о митапе и получить материалы после - https://clck.ru/SMuCA

Вернуть условную ошибку ErrNoUser

ммм, где мне про это почитать?

да просто не конкретное значение возвращать, а список значений. если ничего не найдено, список будет пустой

ну вот это как раз плохой вариант. ошибка, которая не ошибка.

если может быть “не нашлось”, а может и ошибка - надо  (user, bool, error) возвращать

То есть, предлагаете просто отказаться от возвращений пользовательских типов данных  в принципе?

Я гуглил в этом направлении:


https://stackoverflow.com/questions/28447297/how-to-check-for-an-empty-struct

Если функция возвращает одного клиента, можно nil вернуть. А если возвращает список, то список юзеров (пустой если не найдено)

смотрите. это просто два разных подхода к работе с данными. либо использовать прослойку, которая возвращает вам список значений (ну, например, дай мне всех пользователей с таким-то списком айдишников, или такими-то именами). То, что ничего не найдено - это не ошибка. Это результат поиска. Ошибка это или нет - должен решать тот, кто принимал решение о поиске.
Второй варианта - это когда модель сама себя ищет. Например, дай мне юзера с таким-то айдишником. В этом случае уместно возвращать 2 значения. Сам результат и ошибку.

Вариант с тремя переменными выглядит неплохо. Во всяком случае его можно масштабировать и понятно как использовать везде.

Другое дело, является ли это неким общепринятым вариантом или нет.

с двумя - точно общепринятый

с тремя - реже.

обычно первой тогда возвращают указатель, и nil означает, что ничего не найдено

если вы можете возвращать указатель - можно ак и делать

Просто, возможно(!), мне кто-нибудь скажет: чувак, никто не возвращает пользовательские типы данных потому что тебе надо этот структ выносить глобально, он не может быть nil, непонятно как нормально его сравнивать в каждом случае. Поэтому просто не возвращай - и нет проблем.

Я просто предполагаю и такой вариант

mongo.ErrNoDocuments

даже io.EOF есть.

но это не делает эти варианты хорошими

+. Например в монге эта ошибка возвращается только в FindOne, а если ничего не найдено в Find это типа норм, пустой слайс и без ошибки

sql.ErrNoRows, но это дело вкуса