@gogolang

Страница 1073 из 1630
Stanislav
17.05.2018
08:47:46
вы что-то путаете
да, действительно, прошу прощения

Dmitry
17.05.2018
08:48:23
https://play.golang.com/p/0y5BwrHDOZh
в 7 строке вы объявили странное

а потом не инициализировали cfg

ну а дальше в методе не использовали и в вызове метода

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

Sergey
17.05.2018
08:55:58
если вернуться к моей проблеме, есть у кого какие мысли?)
попробуй сигнатуру без указателя, возможно в ParseFile ты перетираешь что-то

или скорее всего что-то неправильно парсится на 33

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

Sergey
17.05.2018
08:57:30
или скорее всего что-то неправильно парсится на 33
всё равно в конце делаешь return c, зачем по указателю

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
щас гляну 33 и 25
после yaml.Unmarshal принтани поля

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

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
имхо проблема в 25 строке, а именно в кривости файла
точно нет, парсится в консоль все нормально, если файл кривой, выкидывает ошибку

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
было, ничего не пушил

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 же пишет)

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

Google
Stanislav
17.05.2018
09:21:50


то есть через браузер по ссылке выше не попадешь куда надо)

Dmitry
17.05.2018
09:24:25
https://play.golang.com/p/ISYtFkD9KvL с чего бы это?
внимательнее на ваш второй пример

я не об этом случае

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

для Go

Alexandr
17.05.2018
09:25:48
и launch.json

Dmitry
17.05.2018
09:26:17
а кто кодит на VSCode? Можете дать tasks.json файл?
я, а в чем именно проблема, у меня там 1 настройка

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
я, а в чем именно проблема, у меня там 1 настройка
дело в том что я не знаю что писать в этом файле

Stanislav
17.05.2018
09:29:13
распарсенные токены менятся не собираются
и только это мешает сделать указателями?

причем нормально разделив имена

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

Диёр
17.05.2018
09:31:47
дело в том что я не знаю что писать в этом файле
https://code.visualstudio.com/docs/editor/tasks#_custom-tasks

Google
Dmitry
17.05.2018
09:31:49
дело в том что я не знаю что писать в этом файле
я думал вы о глобальной настройке settings.json для го

у меня там ничего нет для го, все из коробки работает хорошо

Диёр
17.05.2018
09:32:39
у меня там ничего нет для го, все из коробки работает хорошо
gb не обнаружил, самому пришлось допиливать

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

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

у него указатель же, не?
уже да, сейчас я попробую пройтись по вашим рекомендациям

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, я уже не помню

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

Dmitry
17.05.2018
09:54:22


extra

значения нет

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

Stanislav
17.05.2018
09:54:53
extra
потому что Printf

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)



Страница 1073 из 1630