Golang vksdk, проблема с типом данных, как решить?

Question

[re-incarnation]

Добрый день/вечер, у меня возникла проблема с удалением сообщения через библиотеку SevereCloud/vksdk.

lp.MessageNew(func(_ context.Context, obj events.MessageNewObject) {
				var admins = 380236476
				var event = 2000000001

				if obj.Message.FromID == admins{
				if obj.Message.PeerID == event {
					b := params.NewMessagesDeleteBuilder()
					b.DeleteForAll(true)
					b.MessageIDs(obj.Message.ID)


					_, err := vk.MessagesDelete(b.Params)
						if err != nil {
							log.Fatal(err)
						}
				}
				}
	})

При выполнении кода он выдаёт ошибку по причине того, что я пытаюсь использовать тип int как тип []int

./main.go:68:18: cannot use test (type int) as type []int in argument to b.MessageIDs

Как я могу конвертировать int в []int или же как нибудь по другому решить эту проблему?

Answer 1

[Евгений Мамонов]

вместо b.MessageIDs(obj.Message.ID) напишите b.MessageIDs([]int{obj.Message.ID})

Comments

[re-incarnation]

Теперь пишет что один из параметров утерян либо не правильный)

One of the parameters specified was missing or invalid: message_ids or conversation_message_ids is undefined

[Евгений Мамонов]

re-incarnation, покажите полный пример кода

[re-incarnation]

Евгений Мамонов,

package main

import (
	"context"
	"log"

	"github.com/SevereCloud/vksdk/v2/api"
	"github.com/SevereCloud/vksdk/v2/api/params"
	"github.com/SevereCloud/vksdk/v2/longpoll-bot"
	"github.com/SevereCloud/vksdk/v2/events"
)

func main() {
	token := "token" 
	vk := api.NewVK(token)

	group, err := vk.GroupsGetByID(api.Params{})
	if err != nil {
		log.Fatal(err)
	}

	lp, err := longpoll.NewLongPoll(vk, group[0].ID)
	if err != nil {
		log.Fatal(err)
	}

	lp.MessageNew(func(_ context.Context, obj events.MessageNewObject) {
				var admins = 380236476
				var event = 2000000001

				if obj.Message.FromID == admins{
				if obj.Message.PeerID == event {
					b := params.NewMessagesDeleteBuilder()
					b.DeleteForAll(true)
					b.MessageIDs([]int{obj.Message.ID})


					_, err := vk.MessagesDelete(b.Params)
						if err != nil {
							log.Fatal(err)
						}
				}
				}
	})







	log.Println("Start longpoll")
	if err := lp.Run(); err != nil {
		log.Fatal(err)
	}
}

[Евгений Мамонов]

а obj.Message.ID не пустой случайно?
сделайте что-то вроде

fmt.Printf("obj.Message.ID: %v\n", obj.Message.ID)
b.MessageIDs([]int{obj.Message.ID})

вызов функции b.MessageIDs сейчас определённой верный, вот эта функция из исходников

// MessageIDs parameter.
func (b *MessagesDeleteBuilder) MessageIDs(v []int) *MessagesDeleteBuilder {
	b.Params["message_ids"] = v
	return b
}

Т.е. скорее всего проблема уже в самих данных, которые туда передаются

Добавьте еще строку
fmt.Printf("b.Params: %+v\n", b.Params)
перед _, err := vk.MessagesDelete(b.Params)
посмотрим что оно вообще возвращает

[re-incarnation]

Евгений Мамонов, Да, если выводить MessageID то о всегда пустой выходит, пишет в каждых сообщениях 0.
Пробовал ещё сделать так:

lp.MessageNew(func(_ context.Context, obj events.MessageNewObject) {
var admins = 380236476
var event = 2000000001

if obj.Message.FromID == admins{
if obj.Message.PeerID == event {
b := params.NewMessagesDeleteBuilder()
b.DeleteForAll(true)
b.MessageIDs([]int{obj.Message.ConversationMessageID})

_, err := vk.MessagesDelete(b.Params)
if err != nil {
log.Fatal(err)
}
}
}
})

Он запускался, но при появлении сообщении которое подходит под критерии в if, просто говорит что сообщение не найдено.

2021/05/23 01:57:47 api: Access denied: message can not be found

[Евгений Мамонов]

В общем проблема на стороне Go решилась, надо разбираться что не так с данными.

Как вариант можно поставить https://github.com/davecgh/go-spew
и сделать spew.Dump(obj.Message) чтобы посмотреть что там вообще в этом obj.Message

[re-incarnation]

Евгений Мамонов, спасибо что показали эту библиотеку, я когда с php на go переходил думал здесь так же дампать можно будет через встроенную функцию)
Вот что вернуло

(object.MessagesMessage) {
AdminAuthorID: (int) 0,
Action: (object.MessagesMessageAction) {
ConversationMessageID: (int) 0,
Email: (string) "",
MemberID: (int) 0,
Message: (string) "",
Photo: (object.MessagesMessageActionPhoto) {
Photo100: (string) "",
Photo200: (string) "",
Photo50: (string) ""
},
Text: (string) "",
Type: (string) ""
},
Attachments: ([]object.MessagesMessageAttachment) {
},
ConversationMessageID: (int) 15116,
Date: (int) 1621701349,
FromID: (int) 259189762,
FwdMessages: ([]object.MessagesMessage) {
},
ReplyMessage: (*object.MessagesMessage)(),
Geo: (object.BaseMessageGeo) {
Coordinates: (object.BaseGeoCoordinates) {
Latitude: (float64) 0,
Longitude: (float64) 0
},
Place: (object.BasePlace) {
Address: (string) "",
Checkins: (int) 0,
City: (interface {}) ,
Country: (interface {}) ,
Created: (int) 0,
ID: (int) 0,
Icon: (string) "",
Latitude: (float64) 0,
Longitude: (float64) 0,
Title: (string) "",
Type: (string) "",
IsDeleted: (object.BaseBoolInt) false,
TotalCheckins: (int) 0,
Updated: (int) 0,
CategoryObject: (object.BaseCategoryObject) {
ID: (int) 0,
Title: (string) "",
Icons: ([]object.BaseImage)
}
},
Showmap: (int) 0,
Type: (string) ""
},
PinnedAt: (int) 0,
ID: (int) 0,
Deleted: (object.BaseBoolInt) false,
Important: (object.BaseBoolInt) false,
IsHidden: (object.BaseBoolInt) false,
IsCropped: (object.BaseBoolInt) false,
Out: (object.BaseBoolInt) false,
WasListened: (object.BaseBoolInt) false,
Keyboard: (object.MessagesKeyboard) {
AuthorID: (int) 0,
Buttons: ([][]object.MessagesKeyboardButton) ,
OneTime: (object.BaseBoolInt) false,
Inline: (object.BaseBoolInt) false
},
Template: (object.MessagesTemplate) {
Type: (string) "",
Elements: ([]object.MessagesTemplateElement)
},
Payload: (string) "",
PeerID: (int) 2000000001,
RandomID: (int) 0,
Ref: (string) "",
RefSource: (string) "",
Text: (string) (len=26) "test",
UpdateTime: (int) 0,
MembersCount: (int) 0,
ExpireTTL: (int) 0
}

[Евгений Мамонов]

Всё верно, ID пустой.
Есть соображения почему он пустой?

[re-incarnation]

Евгений Мамонов, ну я посмотрел похожие вопросы, и как я понял ( мб правильно, мб нет ) что бы получать все параметры из Message, нужно пройти какую-то модерацию или что-то типо того. Я задал вопрос в поддержку по этому поводу, как отпишут сообщу сюда.

К слову я здесь должен указывать Message.ID или же ConversationMessageID ?

lp.MessageNew(func(_ context.Context, obj events.MessageNewObject) {
var admins = 380236475
var event = 2000000001

if obj.Message.FromID != admins{
if obj.Message.PeerID == event {
b := params.NewMessagesDeleteBuilder()
b.DeleteForAll(true)
b.MessageIDs([]int{obj.Message.ID})

_, err := vk.MessagesDelete(b.Params)
if err != nil {
log.Fatal(err)
}
}
}
})

Т.к. когда я пишу Message.ID мне выводит ошибку и не запускается в принципе.

2021/05/23 14:03:38 api: One of the parameters specified was missing or invalid: message_ids or conversation_message_ids is undefined

Ну а когда ConversationMessageID, то он запускается, но при появлении сообщении с параматрами, выдаёт ошибку

2021/05/23 14:05:58 api: Access denied: message can not be found

[Евгений Мамонов]

Судя по названию функции и по сообщению об ошибке - передавать таки надо Message.ID.
Прояснилось почему Message.ID нулевой?

[re-incarnation]

Евгений Мамонов, так, извиняюсь что пропал, жесткий полетел, вот что ответили в помощи вк:

Здравствуйте, Данил!

У таких сообщений нет глобальных id. Для бота нужно использовать поле conversation_message_id и метод https://vk.com/dev/messages.getByConversationMessageId, чтобы получить такое сообщение. В https://vk.com/dev/messages.delete соответственно тоже есть поле conversation_message_ids, нужно использовать именно его.

С уважением,
Команда ВКонтакте

Буду пробовать работать с КонверсMessageID

[Евгений Мамонов]

Теперь стало понятно что делать.
Я пока не нашел в их мануале как правильно создать Builder для удаления по conversation_message_id.
Предлагаю попробовать небольшой "хак", чтобы убедиться, что оно реально работает.
А как заработает - надо будет найти правильное решение, сначала давайте сделаем чтобы заработало.
Судя по вашему примеру - вам не нужны вычитывать эти сообщения чтобы удалить, верно?
Попробуйте вот такой вариант

// исходя из логики в их мануале и того, что там есть 
// NewMessagesGetByConversationMessageIDBuilder и NewMessagesDeleteBuilder
// по идее должен быть или NewMessagesDeleteByConversationMessageIDBuilder
// или MessagesDeleteBuilder должен быть метод ConversationMessageIDs
// Но нет ни того, ни другого.
// Возможно еще не сделали :)
// Будем работать с тем, что есть.
b := params.NewMessagesDeleteBuilder()
b.DeleteForAll(true)

// судя по их мануалу https://vk.com/dev/messages.delete
// они понимают параметр conversation_message_ids
// пропишем этот параметр в params сами, раз у MessagesDeleteBuilder этого параметра нет.
params = b.Params
params["conversation_message_ids"] = []int{obj.Message.ConversationMessageID}

_, err := vk.MessagesDelete(params)

Такой вариант должен сработать.
Дайте знать как попробуете ))

[re-incarnation]

Евгений Мамонов, к сожалению такой вариант не сработал))

./main.go:56:10: use of package params without selector
./main.go:57:9: use of package params without selector
./main.go:59:30: use of package params without selector

По поводу что мне нужно:"Ассистент в беседе который будет попросту удалять все сообщения кроме сообщений администратора в бесед, т.к. бот будет лишь для одной беседы, можно не париться над адаптивностью под все беседы, и использовать простой статичный if messageFromId = такой-то id"

По поводу кода, мб попробовать в этом коде сделать функцию исходя из мануала автора?

https://pkg.go.dev/github.com/SevereCloud/vksdk/v2...

Примерно с такой структурой кода:
1 - реагирование на новое сообщение
2 - функция MessagesGetByConversationMessageID
3 - уже сама функция удаления

P.S. может быть в конце я бред написал, прошу строго не судить за это)

[Евгений Мамонов]

Ой, я забыл, что есть еще пакет params, давайте заменим params на p, вот так:

p = b.Params
p["conversation_message_ids"] = []int{obj.Message.ConversationMessageID}
_, err := vk.MessagesDelete(p)

[Евгений Мамонов]

Если идти по пути MessagesGetByConversationMessageID - это будут избыточные вызовы, но тоже можно.
Получаете сообщение по ConversationMessageID, достаёте из него MessageID и потому удаляете.

[re-incarnation]

Евгений Мамонов,

./main.go:59:31: undefined: p

[Евгений Мамонов]

Надо двоеточие добавить, без компилятора не очень удобно :)
p := b.Params

[re-incarnation]

Евгений Мамонов, я пробовал ещё тогда, он говорил что один из параметров не указан)
А именно :

api: One of the parameters specified was missing or invalid: message_ids or conversation_message_ids is undefined

[Евгений Мамонов]

Вот так должно сработать, этой строкой мы как раз и указываем один из этих обязательных параметров
p["conversation_message_ids"] = []int{obj.Message.ConversationMessageID}

Когда мы раньше использовали MessageID - то там данные были не правильные, т.к. нам 0 всегда был.
В общем такой вариант должен сработать.

[Евгений Мамонов]

Не просто писать код, когда нет возможности его проверить :)

[re-incarnation]

Евгений Мамонов, к сожалению вылазит всё та же ошибка)

2021/05/28 17:30:32 api: One of the parameters specified was missing or invalid: conversation_message_ids requests peer_id parameter

[Евгений Мамонов]

давайте убедимся, что данные правильные, добавьте эту строку перед удалением
fmt.Printf("%+v\n", p)

[re-incarnation]

Евгений Мамонов,

2021/05/28 17:59:02 map[conversation_message_ids:[15382] delete_for_all:true]
2021/05/28 17:59:02 api: One of the parameters specified was missing or invalid: conversation_message_ids requests peer_id parameter

[Евгений Мамонов]

Жесть, параметры таки заполнены правильно.
Получается либо MessageConversationID (15382) некорректный, либо запрос на API идёт не правильный.
Есть как минимум два варианта, которые есть смысл пробовать.
1) сначала получить сообщения по их MessageConversationID, потом их удалить по ID.
2) сделать вызов API вручную с правильно заполненными параметрами и посмотреть ответ.

[re-incarnation]

Евгений Мамонов,

lp.MessageNew(func(_ context.Context, obj events.MessageNewObject) {
    b := params.NewMessagesGetByConversationMessageIDBuilder()
    if obj.Message.FromID != 380236476 {
    b.PeerID(2000000001)
    b.ConversationMessageIDs([]int{obj.Message.ConversationMessageID})

    _, err := vk.MessagesDelete(b.Params)
  //  log.Printf("%s, %s" , b.Params)
    if err != nil {
    log.Fatal(err)
}
}
})

2021/05/28 22:07:50 api: Access denied: message can not be deleted (3)
exit status 1

Правильно? Хотя если было бы правильно, то работало наверное)
Вот что возвращает b.Params в этом случае

map[conversation_message_ids:[%!s(int=15510)] peer_id:%!s(int=2000000001)], %!s(MISSING)

[Евгений Мамонов]

Скорее как то так

getParamsBuilder := params.NewMessagesGetByConversationMessageIDBuilder()
getParamsBuilder.ConversationMessageIDs([]int{obj.Message.ConversationMessageID})

response, err := vk.MessagesGetByConversationMessageID(getParamsBuilder.Params)
// if err != nil ...

ids := []int{}
for _, item := range response.Items {
     fmt.Printf("%+v\n\n", item) // посмотрите что вообще приходит и что там за данные
     ids = append(ids, item.ID)
}

delParamsBuilder := params.NewMessagesDeleteBuilder()
delParamsBuilder.DeleteForAll(true)
delParamsBuilder.MessageIDs(ids)
_, err := vk.MessagesDelete(delParamsBuilder.Params)

[Евгений Мамонов]

ну и сразу посмотрите что приходит вообще в response
spew.Dump(response)

[re-incarnation]

Евгений Мамонов,

./funcs_vk.go:101:12: no new variables on left side of :=

Ругается на уже указанную выше переменную

_, err := vk.MessagesDelete(delParamsBuilder.Params)

[re-incarnation]

Евгений Мамонов, item:

{AdminAuthorID:0 Action:{ConversationMessageID:0 Email: MemberID:0 Message: Photo:{Photo100: Photo200: Photo50:} Text: Type:} Attachments:[] ConversationMessageID:15533 Date:1622211865 FromID:380236476 FwdMessages:[] ReplyMessage: Geo:{Coordinates:{Latitude:0 Longitude:0} Place:{Address: Checkins:0 City: Country: Created:0 ID:0 Icon: Latitude:0 Longitude:0 Title: Type: IsDeleted:false TotalCheckins:0 Updated:0 CategoryObject:{ID:0 Title: Icons:[]}} Showmap:0 Type:} PinnedAt:0 ID:0 Deleted:false Important:false IsHidden:false IsCropped:false Out:false WasListened:false Keyboard:{AuthorID:0 Buttons:[] OneTime:false Inline:false} Template:{Type: Elements:[]} Payload: PeerID:2000000001 RandomID:0 Ref: RefSource: Text:test UpdateTime:0 MembersCount:0 ExpireTTL:0}

[re-incarnation]

Евгений Мамонов, дамп response

(api.MessagesGetByConversationMessageIDResponse) {
Count: (int) 1,
Items: ([]object.MessagesMessage) (len=1 cap=4) {
(object.MessagesMessage) {
AdminAuthorID: (int) 0,
Action: (object.MessagesMessageAction) {
ConversationMessageID: (int) 0,
Email: (string) "",
MemberID: (int) 0,
Message: (string) "",
Photo: (object.MessagesMessageActionPhoto) {
Photo100: (string) "",
Photo200: (string) "",
Photo50: (string) ""
},
Text: (string) "",
Type: (string) ""
},
Attachments: ([]object.MessagesMessageAttachment) {
},
ConversationMessageID: (int) 15534,
Date: (int) 1622212093,
FromID: (int) 380236476,
FwdMessages: ([]object.MessagesMessage) {
},
ReplyMessage: (*object.MessagesMessage)(),
Geo: (object.BaseMessageGeo) {
Coordinates: (object.BaseGeoCoordinates) {
Latitude: (float64) 0,
Longitude: (float64) 0
},
Place: (object.BasePlace) {
Address: (string) "",
Checkins: (int) 0,
City: (interface {}) ,
Country: (interface {}) ,
Created: (int) 0,
ID: (int) 0,
Icon: (string) "",
Latitude: (float64) 0,
Longitude: (float64) 0,
Title: (string) "",
Type: (string) "",
IsDeleted: (object.BaseBoolInt) false,
TotalCheckins: (int) 0,
Updated: (int) 0,
CategoryObject: (object.BaseCategoryObject) {
ID: (int) 0,
Title: (string) "",
Icons: ([]object.BaseImage)
}
},
Showmap: (int) 0,
Type: (string) ""
},
PinnedAt: (int) 0,
ID: (int) 0,
Deleted: (object.BaseBoolInt) false,
Important: (object.BaseBoolInt) false,
IsHidden: (object.BaseBoolInt) false,
IsCropped: (object.BaseBoolInt) false,
Out: (object.BaseBoolInt) false,
WasListened: (object.BaseBoolInt) false,
Keyboard: (object.MessagesKeyboard) {
AuthorID: (int) 0,
Buttons: ([][]object.MessagesKeyboardButton) ,
OneTime: (object.BaseBoolInt) false,
Inline: (object.BaseBoolInt) false
},
Template: (object.MessagesTemplate) {
Type: (string) "",
Elements: ([]object.MessagesTemplateElement)
},
Payload: (string) "",
PeerID: (int) 2000000001,
RandomID: (int) 0,
Ref: (string) "",
RefSource: (string) "",
Text: (string) (len=4) "test",
UpdateTime: (int) 0,
MembersCount: (int) 0,
ExpireTTL: (int) 0
}
},
ExtendedResponse: (object.ExtendedResponse) {
Profiles: ([]object.UsersUser) ,
Groups: ([]object.GroupsGroup)
}
}

[Евгений Мамонов]

Занятно, смотрю особо ничего не поменялось, получается делать Get особого смысла нет.
Попробуйте после получения сообщения взять его conversation_message_id и вызвать вручную метод, который описан тут https://vk.com/dev/messages.delete.
Надо разобраться как оно должно работать.

[re-incarnation]

Евгений Мамонов, наконец-то оно начало работать)
Не знаю почему эти билдеры не находили конвер айди сообщения, всё начало работать с простыми апи.

lp.MessageNew(func(_ context.Context, obj events.MessageNewObject) {

      if obj.Message.FromID != 380236476 {
      users, err := vk.MessagesDelete(api.Params{
        "conversation_message_ids": obj.Message.ConversationMessageID,
        "delete_for_all": 1,
        "peer_id": 2000000001,
      })
      if err != nil {
        log.Fatal(err)
      }
      log.Print(users)
    }
    })

и возвращать при удалении он будет:

2021/05/30 14:16:46 map[15545:1]

[Евгений Мамонов]

Отлично, наконец то :))
Возможно мы где-то во время тестов забывали вызвать b.PeerID(2000000001) и из-за этого оно не работало.
Но это не отменяет того, что билдеры не доработаны, по факту для удаления таки нельзя вызвать ConversationMessageIDs, это явная недоработка. Можете поправить их код и предложить им pull request :)

[re-incarnation]

Евгений Мамонов, что бы поправить их код, мне нужно ещё учиться и учиться)
Надеюсь здесь не запрещено задавать несколько вопросов в одной теме) Но всё же) Как я могу вывести какой либо параметр из дампа? Именно в этом случае:

lp.MessageNew(func(_ context.Context, obj events.MessageNewObject) {

      if obj.Message.Text == "test" {
        admins, err := vk.MessagesGetConversationMembers(api.Params{
          "peer_id": 2000000001,
        })
        if err != nil {
        log.Fatal(err)
        }
        spew.Dump(admins.Items)
}})

Суть кода - получить информацию о пользователях в беседе, суть которую я хочу получить - вывести из Items администраторов в беседе.
Вот ответ от сервера при дампе:

([]struct { MemberID int "json:\"member_id\""; JoinDate int "json:\"join_date\""; InvitedBy int "json:\"invited_by\""; IsOwner object.BaseBoolInt "json:\"is_owner,omitempty\""; IsAdmin object.BaseBoolInt "json:\"is_admin,omitempty\""; CanKick object.BaseBoolInt "json:\"can_kick,omitempty\"" }) (len=16 cap=19) {
(struct { MemberID int "json:\"member_id\""; JoinDate int "json:\"join_date\""; InvitedBy int "json:\"invited_by\""; IsOwner object.BaseBoolInt "json:\"is_owner,omitempty\""; IsAdmin object.BaseBoolInt "json:\"is_admin,omitempty\""; CanKick object.BaseBoolInt "json:\"can_kick,omitempty\"" }) {
MemberID: (int) 380236476,
JoinDate: (int) 1610476128,
InvitedBy: (int) 380236476,
IsOwner: (object.BaseBoolInt) true,
IsAdmin: (object.BaseBoolInt) true,
CanKick: (object.BaseBoolInt) false
},

[Евгений Мамонов]

вы можете пройтись по items в цикле и вывести уже то, что нужно, например

for _, item := range admins.Items {
    // так можно посмотреть один item
    spew.Dump(item)

    // или какой то конкретный параметр
    spew.Dump(item.SomeParam)

    // или так
    fmt.Printf("%+v\n", item) 
}

[re-incarnation]

Евгений Мамонов, Thanks

Answer 2

[Пума Тайланд]

так у вас там массив ожидается а вы просто инт передаете