negezor / vk-io

Modern VK API SDK for Node.js
https://npm.im/vk-io
MIT License
548 stars 85 forks source link

Updates.isGroup #108

Closed alienvspredator closed 6 years ago

alienvspredator commented 6 years ago

В вашем коде вы выполняете подключение к Bots Long Poll API или к User Long Poll API отталкиваясь от параметров, установленных при инициализации.

const {
  pollingGroupId
} = this.vk.options;
const isGroup = pollingGroupId !== null;

Но можно сделать иначе. Метод groups.getById? без параметров, с ключом доступа сообщества, возвращает объект этого же сообщества, тогда как этот же метод с ключом доступа пользователя возвращает ошибку о пропущенном параметре group_ids.

negezor commented 6 years ago

А что мешает вам так поступить?)

alienvspredator commented 6 years ago

Ничего, это лишь предложение по улучшению.

negezor commented 6 years ago

Дело в том что это больше "магическое" API которое может позже перестать работать.

alienvspredator commented 6 years ago

Понял.

alienvspredator commented 6 years ago

Это не магическое API, об этом указано в документации к методу: https://vk.com/dev/groups.getById Описание метода следующее: "Возвращает информацию о заданном сообществе или о нескольких сообществах." Так же в описании параметров к методу ни один из параметров не является обязательным. Это задокументированная возможность данного метода, просто описана немного не чётко.

negezor commented 6 years ago

Но это и называется "магическим" API.