0. GORM конечно ультра оверхед, он скорее усложняет код здесь, не только производительность уменьшает. А AutoMigrate его это вообще кошмарный кошмар как будто не production ready изначально

1. Вместо Create с проверкой на ErrNotExists, можно FirstOrCreate with attrs gorm

2. Лучше не инкрементировать счетчик в репозитории в ссылке на модель Vote

3. Если 2 параллельных запроса придет c одним и тем же номером, то никакой защиты нет. Нужна блокировка на уровне строк и уникальный constraint если голос уже есть

4. Обновление счетчика и создание голоса в бд лучше делать в одной транзакции априори, а то голос создадим, а инкрементировать не сможем - неконсистентность. И по сути это один метод репозитория - создания голоса, а не два разных.

5. Зависимости с большой буквы не надо называть, если это не необходимо экспортировать реально

6. Интерфейсы в месте реализации, а не использования, как то слишком не туда не сюда выглядят на примере, в чем гибкость и удобство в Вашем случае?

7. Снова про интерфейсы. Если используется чистая архитектура и для DI куда удобнее класть интерфейсы в места использования, а у вас вообще как то непонятно вышло) Ваш код даже невозможно юнит тестами покрыть по сути, если даже опустить, что Handler зависит от репы бд напрямую...

8. Никогда в жизни не стоит внешние вызовы через дефолтную конфигурацию http.Client делать без таймаутов и прочих ограничений по соединениям и т.п.

9. Нету общепринятого gracefull shutdown, если сервис будет рестартиться во время каких то операций и открытых соединений то капут, потеря голосов, ошибки у клиентов, неконстистентность, вопросы, негодования, неудобства, смерть

10. Если так важно отправлять смс, то стоит предпринимать делать несколько попыток. А если не получилось, то откатывать транзакцию к бд. Иначе полная ахинея выходит.

Все эти пункты учесть можно за 10 минут без усложнения кода и реализации ровно ни на сколько, но зато работало бы как надо и выглядело более менее

У вас самая нечеловечная капча во всем интернете, которая вылезает без причины на некоторых сервисах яндекса по нескольку раз, из-за чего авто ру я для себя уже послал куда подальше, например =)
К слову, с первой попытки, даже при полном умиротворении и старании, редко удается её решить

Тем не менее, на рынке авторешения капч, ваша решается автоматизированными средствами, а не ручными, судя по минимальнейшему кол-ву попыток и скорости.
Т.е. вы максимально неудобны для людей, но максимально удобны для роботов 😂

Суммируете данные по УИКам по конкретной территории, сравниваете со статистикой голосования по этой территории в ДЭГ, voila.

Что конкретно сравнивать?) Кол-во голосов?)

Кстати что то в яндексе последнее время намного ниже рынка зп :(

P.S:
Дареному коню в зубы не смотрят конечно, но чисто из интереса посмотреть, не продешевил ли я со своим подарком, не удержался глянуть ценники на эти штуки… И охренел...

Через три недели, проведенных в СИЗО, предпринимателя, похоже, пытали и убили. Официальное следствие утверждает о самоубийстве