Побить можете? https://github.com/anboo/sms-gateway/blob/master/service/di.go Это, конечно же, не di Но вот сделал такой билдер

Question

Побить можете? https://github.com/anboo/sms-gateway/blob/master/service/di.go Это, конечно же, не di Но вот сделал такой билдер

объектов

Чтобы можно было из любого места достать уже созданный объект:
https://github.com/anboo/sms-gateway/blob/master/http/send_sms_handler.go#L23
formattedPhoneNumber, regionCodeByIp, err := service.GetObjectStorage().GetLocationManager().ParsePhoneAndFormatE164(
https://github.com/anboo/sms-gateway/blob/master/http/send_sms_handler.go#L32
p, err := service.GetObjectStorage().GetPhoneNumberManager().GetProviderForPhoneNumber(

Смотрел уберовский DI, там как то что то неудобно все, на функциях и с парсингом рефлексий и т.п.

Я не нашел как решить вопрос связи между объектами
Может я с самого начала ошибся, захотев оперировать всей этой шнягой как объектами?

#backend #go #programming #russian

0

22.11.2021

13 ответов

12 просмотров

Dmitriy Sviridov

Есть же wire. Нафига эти велосипеды?

0

22.11.2021

Danil Andreyev Автор вопроса

Dmitriy Sviridov
Есть же wire. Нафига эти велосипеды?

Гляну

0

22.11.2021

Михаил Макарычев

Сервис-локаторы в 2021 - это сильно, конечно. Что вам мешает руками граф зависимостей строить?

0

22.11.2021

Danil Andreyev Автор вопроса

Михаил Макарычев
Сервис-локаторы в 2021 - это сильно, конечно. Что ...

Я пришел из мира с автовайрингом и DI с коробки(

0

22.11.2021

Михаил Макарычев

Danil Andreyev
Я пришел из мира с автовайрингом и DI с коробки(

Я тоже. Но скорее всего в том мире, откуда вы пришли, били по рукам за service locator

0

22.11.2021

Roman Timofeev

А зачем вам тут https://github.com/anboo/sms-gateway/blob/master/service/di.go#L12 если вы его не используете?

0

22.11.2021

Danil Andreyev Автор вопроса

Roman Timofeev
А зачем вам тут https://github.com/anboo/sms-gatew...

Это чисто первый работоспособный v0.0.1-alpla вариант стоража уже собранных объектов Он там сто раз переписывался и всё еще удалится Спасибо

0

22.11.2021

Roman Timofeev

Danil Andreyev
Это чисто первый работоспособный v0.0.1-alpla вари...

Если что, у вас там гонка будет без mutex

0

22.11.2021

Roman Timofeev

https://github.com/anboo/sms-gateway/blob/master/service/location_manager.go#L25 не надо тут паниковать. https://github.com/anboo/sms-gateway/blob/master/service/location_manager.go#L33 проверка l.mixmind != nil не имеет смысла, кмк. https://github.com/anboo/sms-gateway/blob/master/service/location_manager.go#L36 тут нужен логгер.

0

22.11.2021

Roman Timofeev

https://github.com/anboo/sms-gateway/blob/master/storage/sms_storage.go#L10 почему int, а не time.Time?

0

22.11.2021

Danil Andreyev Автор вопроса

Roman Timofeev
https://github.com/anboo/sms-gateway/blob/master/s...

А почему не имеет смысла? mixmind не может быть nil? Вроде может - https://play.golang.org/p/OdmfYs46-B6 Или о другом речь?

0

22.11.2021

Roman Timofeev

Danil Andreyev
А почему не имеет смысла? mixmind не может быть ni...

А, может, если mixmindDatabasePath == "", но я бы всю эту историю с geoip2.Open вынес в отдельный метод

0

22.11.2021

Roman 🎄 · Accepted Answer

Roman 🎄

А тестировать как будете? Вот вам надо будет написать юнит-тест, который будет имитировать отправку смс, но без реальной отправки, как реализуете?

0

22.11.2021

170 похожих чатов

Побить можете? https://github.com/anboo/sms-gateway/blob/master/service/di.go Это, конечно же, не di Но вот сделал такой билдер

13 ответов

Похожие вопросы