Herman
Herman
func (ph *PostHandlers) MmakeNewPost описка
Herman
ну или нет
Herman
func (ph *PostHandlers) Ffindbyid
Herman
в любом случае так не надо
Anonymous
а это знаешь для чего
Anonymous
у меня он не видел приватное поле в структуре
Anonymous
пришлось с заглавной писать
Herman
еще я бы рекомендовал писать сначала экспортируемые функции, а потмо неэкспортируемые
Anonymous
че то я тупанул
Herman
у тебя в каком-то пакете идет прямо наоборот
Anonymous
в каком?
Herman
в смысле
Anonymous
???
Herman
а что такое экспортируемая функция?
Anonymous
а нет вспомнил
Anonymous
с заглавной буквы короче
Anonymous
все я понял про что ты
Anonymous
исправлю
Anonymous
ну да рил кринжово выглядит
Anonymous
у меня сначала идут экспортируемые потом не экспортируемая а потом опять экспортируемая
Herman
когда начинаешь читать пакет, больше интересуешься экспортируемыми
Herman
как и больше всего, имхо, интересуешься конструктором
Herman
поэтому я за то, чтобы он был первый
Ivan
func (r *FinanceRepo) MakeTransaction почему-то создает нового юзера. не очевидно
Emil
func (r *FinanceRepo) MakeTransaction почему-то создает нового юзера. не очевидно
Изначально база пустая, решил, пусть создается пользователь по транзакции, вроде удобно тестить. Лучше сразу базу с пользователями некоторыми создавать?
Herman
мне кажется от русского надо избавляться
Herman
flag.BoolVar(&local, "local", false, "хост")
Ivan
лучше проверить пользователя до вызова этой функции, либо уж вернуть ошибку и потом создать
Emil
лучше проверить пользователя до вызова этой функции, либо уж вернуть ошибку и потом создать
И по запросу транзакции будет не 200 код, не будет самой транзакции, но будет сообщение и факт создания пользователя?
Herman
Herman
ну как я понял они по разу используются
Herman
так что да
Emil
Резонно
(мне просто нравятся блоки констант)
Herman
ну start = start это точно не стоит)
Herman
Herman
типа приходит в рублях
Herman
а, все
Herman
то есть если в рублях, то отдаем так
Herman
а если не в рублях, то ходим в сервис и?
Herman
сложно пока)
Herman
а по логике должно всегда возвращаться в рублях?
Emil
Нет, по запросу, если не рубли, то там дальше в сервис идём и получаем валюту и возвращаем
Herman
сложно
Herman
то есть в базе баланс лежит в рублях?
Emil
Да
Herman
я бы облегчил хенлдер и перенес условие с походом в cur service в функцию баланса
Herman
типа Balance(curr string, id uuid )
Herman
очень тонкий сервис и толстый хенлдер
Ivan
func NewCurService(apiKey string) ICurrency {
return &curService{ApiKey: apiKey}
} , лучше возвращать конкретный тип. а интерфейс обьявлять в месте использования
Herman
Herman
ну и if state
Herman
без ==
Emil
И правда
Herman
аа, там else, а в else if
Herman
а почему не else if {} ?
Herman
да и вообще странный метод, я пока не понял
Herman
можно и без else походу
Emil
Намудрил что-то видимо)
Дебажил его вечность)
Herman
if state {
h.newResponse(c, http.StatusConflict, duplicate, nil)
return true }
Herman
а потом
Herman
if err := h.cache.CacheKey(c, key); err != nil {
h.newResponse(c, http.StatusInternalServerError, cacheFail, err)
return true
}
Herman
тогда return false действительно можно достичь
Emil
Помняю 👌👌
Emil
func NewCurService(apiKey string) ICurrency {
return &curService{ApiKey: apiKey}
} , лучше возвращать конкретный тип. а интерфейс обьявлять в месте использования
А в месте использования это где? Мне так нравилось, как везде возвращается интерфейс
Herman
Herman
везде выше есть
Emil
А остальное приходит через query параметры
Herman
но оно же все равно использует это в анмаршале
Herman
делая "" ?
Herman
можно написать json:"-"