FLGO-45 Fix json fields #46
Conversation
modules/forumapi/datacontroller.go
Outdated
| Moderators: moderators, | ||
| Id: dbForum.ForumID, | ||
| Title: dbForum.Name, | ||
| Description: dbForum.Description, |
There was a problem hiding this comment.
а мы можем description по другому назвать? :) с type и description есть проблемки в свифте
modules/forumapi/datacontroller.go
Outdated
| @@ -13,7 +13,7 @@ func getForumBlocks(dbForums []dbForum, dbModerators map[uint32][]dbModerator) * | |||
| for _, dbForum := range dbForums { | |||
| if dbForum.ForumBlockID != currentForumBlockID { | |||
| forumBlock := pb.Forum_ForumBlock{ | |||
| Id: dbForum.ForumBlockID, | |||
| XXX_Id: dbForum.ForumBlockID, | |||
There was a problem hiding this comment.
типа эта айдишка не должна светиться?
There was a problem hiding this comment.
Ага. Параметр технический, раньше вырезать его из json-а было очень просто, теперь приходится костыли городить.
There was a problem hiding this comment.
просто, если он не секретный, то может хрен с ним, пускай будет?)
There was a problem hiding this comment.
Ок. Однако, надобность в аналоге json:"-" может возникнуть и в другом месте, мы просто скрываем проблему.
golang/protobuf#52 - Open
There was a problem hiding this comment.
протобаф планируется использовать только для отдачи клиентам (пока что - дальше хз что будет) и при необходимости можно создавать внутренние модели для конвертации и уже из них создавать протобуфные - например если будет сложный алгоритм преобразования дбшных моделей, иногда эффективней потратить больше памяти для промежуточных моделек. В таком случае не понадобится модификация генератора протомоделек) Но вообще не думаю что будет что-то настолько сложное что понадобится извращаться. ИМХО :)
protobuf/schema/forum_models.proto
Outdated
| POLL = 2; | ||
| unknown = 0; | ||
| topic = 1; | ||
| poll = 2; |
There was a problem hiding this comment.
https://developers.google.com/protocol-buffers/docs/style - там раздел про енумы
There was a problem hiding this comment.
Эт все здорово, но как сделать, чтоб в json их строковое представление было в нижнем регистре?
There was a problem hiding this comment.
это настолько принципиально? можно указать в маршалере параметр EnumAsInt например
There was a problem hiding this comment.
Это плохо, когда инструмент навязывает собственные решения и не дает альтернатив.
golang/protobuf#555 - Open, как обычно
There was a problem hiding this comment.
Я специально завел строковые значения, чтоб не городить массу непонятных интов для клиентов :) Возвращаться обратно к тому же - не, спасибо
There was a problem hiding this comment.
Это плохо, когда инструмент навязывает собственные решения и не дает альтернатив.
golang/protobuf#555 - Open, как обычно
в таком случае поддержку кастомных имен надо добавлять во всех языках которые поддерживают протобуф) в общем такое себе - не вижу большой проблемы в том что кейсы енума будут в апперкейсе) ЗАТО_ИХ_СРАЗУ_ЗАМЕТНО!
No description provided.