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

Всем добрый вечер. Столкнулся с довольно странной проблемой. У меня

примерно такая система обработки команд в чат-боте: https://play.golang.org/p/lv_DQBgfm_z
Проблема в том, что для любой команды бот выполняет действие, соответствующее последней переданной в RegisterCommands. Т.е. где-то в этом методе я накосячил, но я не могу понять где. Я, должно быть, что-то не так понимаю либо про тонкости inline-функций, либо про for range, либо ещё про что-то.

У меня есть гипотеза, что функции в мапе хранятся по указателям и при итерации по срезу используется один и тот же адрес для элемента, т.е. в мапу кладётся один и тот же поинтер, чьё значение изменяется по ходу итерации и в конце становится равным указателю на action последней команды. Если это так, то как наименее криво это пофиксить?

5 ответов

4 просмотра

goland можно debug включить и по строчке смотреть что происходит у Вас. Там и какие данные приходят и какие уходят

У тебя проблема здесь for _, com := range commands { b.commandHandlers[com.Name] = func(m * Message){ com.Action() } } в каждой итерации com это указатель, ты присваеваешь хэндлеру анонимную функцию, которая забирает этот com по указателю. В итоге у тебя в мапе commandHandlers лежат функции, внутри которых com указывает на один и тот же объект

Arseny-Khoroshilov Автор вопроса
Timofey
У тебя проблема здесь for _, com := range comma...

Так и думал, ага. Попробовал сделать так: for _, com := range commands { act := com.Action b.commandHandlers[com.Name] = func(m * Message){ act() } } Работает, хоть и выглядит немного костылём.

Arseny Khoroshilov
Так и думал, ага. Попробовал сделать так: for _,...

Можно сделать вот так, чуть красивее будет for _, com := range commands { b.commandHandlers[com.Name] = func(c Command) func(m * Message){ return func(m *Message) { com.Action() } }(com) } Проверь только синтаксис, не уверен без IDE

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

Обсуждают сегодня

Всем привет, написал код ниже, но он выдает сегфолт, в чем причина? #include <stdio.h> #include <stdlib.h> #include <string.h> struct product { char *name; float price; };...
buzz базз
32
Хотел бы спросить у знающих, правильную ли я выбрал книгу для начала изучения ассемблера Юрова В.И ? Или есть более лучшие книги для начала обучения?
Botsman
25
Книга Юрова В.И пойдёт для обучения?
Botsman
24
$params = [ 'formid' => 'feedbackForm', 'formTpl' => '@CODE: <form class="form-validate" data-id="ajax_form"> <fieldset class="margin-bottom-md"> ...
Pathologic
1
Подскажите пожалуйста, а я могу вот такую штуку использовать? rpc, только реализованное в реббите https://www.rabbitmq.com/tutorials/tutorial-six-php ( или https://habr.com/ru...
Artyom
11
а мы ещё не созрели до того, чтобы создать отдельный чатик про настройку редакторов?
Cheese Syrowiecki
16
И ещё вопрос: можно ли типа как на дос как-то запариться и с помощью прерываний выводить текст, вместо функции printf ?
НѣкъиⰘижєжєиꙁъвьсєсвѣтьноѣсѣтиѥсть•
34
Всем привет! У меня почему-то по-разному отображается TListView в Debug и Release режимах (FireMonkey)! При запуске под Win приложения TListView заливается программо. в Debug ...
Александр COM
8
Ладно, ещё тупого спрошу. Код должен банально вывести значение регистра на консоль, на деле же не выводя ничего, просто оставляя нерабочую консоль (открыта, ничего не написан...
НѣкъиⰘижєжєиꙁъвьсєсвѣтьноѣсѣтиѥсть•
25
Парни, спасайте. RAD Studio 12 творит какую-то дичь. У меня есть метод (event), который вызывает другой метод, т.е. он состоит из 1-й строки вызова. Ставлю бряк на event, запу...
Eugene Krasnikov (ᴊɪɴ x)
3
Карта сайта