
Stanislav
17.05.2018
08:47:46

Dmitry
17.05.2018
08:48:23
а потом не инициализировали cfg
ну а дальше в методе не использовали и в вызове метода

Google

Dmitry
17.05.2018
08:51:07
если вернуться к моей проблеме, есть у кого какие мысли?)

Sergey
17.05.2018
08:55:58
или скорее всего что-то неправильно парсится на 33

Stanislav
17.05.2018
08:57:10
а попробуй не cPointer.TelegramToken, а *cPointer.TelegramToken

Sergey
17.05.2018
08:57:30

Artem
17.05.2018
08:57:32
имхо проблема в 25 строке, а именно в кривости файла

Dmitry
17.05.2018
08:57:43
эти варианты все пробовал
уже от безысходности
щас гляну 33 и 25

Artem
17.05.2018
08:58:01
потому что (MISSING) просто так никто не пишет

Sergey
17.05.2018
08:58:26

Stanislav
17.05.2018
08:58:55
так, падажжи, yaml.Unmarshal тебе тоже не ругается, что ты ему кидаешь структуру, а не указатель на нее?

Sergey
17.05.2018
08:59:19

Google

Sergey
17.05.2018
08:59:29
func (c *Config)

Stanislav
17.05.2018
09:01:11
а это меня ввело в заблуждение то, что вверху var c Config. Надеюсь за этот код деньги не платят?)

Sergey
17.05.2018
09:02:19
но используется наименьший скоуп, так что ничего страшного

Stanislav
17.05.2018
09:02:58
а вообще я бы порекомендовал это дело переписать и сделать:
1. либо var cfg Config, либо func (cfg *Config)
2. вообще сделать var c *Config (или var cfg *Config)
Как только это сделаешь - у тебя или заработает, или ты сразу поймешь, в чем проблема
если, конечно, не стоит задача запутать чтеца по максимуму)

Dmitry
17.05.2018
09:04:55

Stanislav
17.05.2018
09:05:34

Dmitry
17.05.2018
09:05:54
вон видно здвиг в yaml

Stanislav
17.05.2018
09:06:47
ээ, что-то или у меня со зрением, или раньше там было
if err == nil {
log.Printf("Unmarshal: %v", c)
}
???

Dmitry
17.05.2018
09:07:51
было, ничего не пушил

Sergey
17.05.2018
09:09:02

Stanislav
17.05.2018
09:10:15
значит надо в отпуск :D

Alexandr
17.05.2018
09:13:02
пытаюсь запустить google go на vs code, а он что-то требует пакет go get github.com/derekparker/delve/cmd/dlv
я его стараюсь вытянуть, но просто ничего не происходит
он не загружается =)

Димка
17.05.2018
09:15:25
404 же пишет)

Dmitry
17.05.2018
09:16:22

Stanislav
17.05.2018
09:21:21
404 же пишет)
потому что ссылка выше - это неправильная ссылка на директорию)

Google

Stanislav
17.05.2018
09:21:50
то есть через браузер по ссылке выше не попадешь куда надо)

Dmitry
17.05.2018
09:22:21

Stanislav
17.05.2018
09:23:19

Dmitry
17.05.2018
09:24:25
я не об этом случае

Alexandr
17.05.2018
09:25:36
а кто кодит на VSCode? Можете дать tasks.json файл?
для Go

Dmitry
17.05.2018
09:25:47

Alexandr
17.05.2018
09:25:48
и launch.json

Dmitry
17.05.2018
09:26:17

Stanislav
17.05.2018
09:26:28
внимательнее на ваш второй пример
так что не так?) ну можно еще прям принудительно создавать структуру, читабельность кода даже повысится: https://play.golang.com/p/TvRy37rqUV7

Dmitry
17.05.2018
09:27:04
ну в моем случае указатель не нужен, как я и говорил
распарсенные токены менятся не собираются
у вас отличие только в указателе

Alexandr
17.05.2018
09:27:56

Stanislav
17.05.2018
09:29:13
причем нормально разделив имена

Dmitry
17.05.2018
09:30:37
ну если вы о памяти, то можно и указатель

Диёр
17.05.2018
09:31:47

Google

Dmitry
17.05.2018
09:31:49
у меня там ничего нет для го, все из коробки работает хорошо

Диёр
17.05.2018
09:32:39

Alexandr
17.05.2018
09:33:07
у меня почему-то программа только с дебаггером запускается

Dmitry
17.05.2018
09:35:44

Sergey
17.05.2018
09:38:32
я немного выпал из контекста,
на самом деле кто-то уже давал рекомендацию, что этому методу не нужен ptr receiver (или как жто называется)

Admin
ERROR: S client not available

Sergey
17.05.2018
09:40:05
(потому что он не должен менять состояние c *Config )
а так больше хз что там не работает, возможно что-то с ямлом
возможно в ямле нужно писать a: "b", вместо a: b, я уже не помню

Dmitry
17.05.2018
09:42:54
в том то и прикол, что я это и говорил)

Sergey
17.05.2018
09:47:20
но не pointer receiver
например func (cfg Config)

Dmitry
17.05.2018
09:47:46
да
в моем случае

Google

Dmitry
17.05.2018
09:48:26
@pztrn а вы что думаете по этому поводу? только ради памяти?

Stanislav
17.05.2018
09:52:55
@pztrn а вы что думаете по этому поводу? только ради памяти?
я думаю, что код надо переписать, чтобы он стал читабельным :) а то, что по ссылке - допущение на допущении. Отсюда могут быть и проблемы. У меня с ямлом никогда не было проблем, потому что я везде использую указатель на конфигурацию. Да, у меня тоже ничего не меняется, но единообразие и четкое разделение "переменная и ресивер разные" (например, var c *Config и func (cfg *Config)) еще и помогает избежать разночтений и вот таких вот брейнштормов.
Но это мой подход, к которому я привык и который, кстати говоря, нравится всем моим коллегам.

Dmitry
17.05.2018
09:53:42
@kezvision по вашей рекомендации https://github.com/krypton-code/waves-bot/blob/117912b4fcd49fe91df1f2b121d2ce425b626c22/pkg/api/config.go#L35

Sergey
17.05.2018
09:54:17

Dmitry
17.05.2018
09:54:22
extra
значения нет

Sergey
17.05.2018
09:54:47
а в yaml.unmarshal не надо указатель?

Stanislav
17.05.2018
09:54:53

Dmitry
17.05.2018
09:54:59
ща доку гляну

Sergey
17.05.2018
09:55:04

Stanislav
17.05.2018
09:55:22
но оно все равно пустое)

Dmitry
17.05.2018
09:55:52
fmt.Printf("Println token: %s", c.TelegramToken)
даже если с глаголом
пустое
у yaml пакета c другим проектом разные версии
2.2.1 против 2.2.0
но не думаю что это проблема
даже не минорная
https://github.com/go-yaml/yaml
err := yaml.Unmarshal([]byte(data), &t)