Как исправить ошибку при проверке наличия роли у юзера?

Question

[sinaee]

const ComUserID = interaction.user.id;
let rUser = interaction.guild.members.cache.get(ComUserID);
const userRoles = rUser.roles.cache.filter((roles) => roles.id !== rUser.guild.id).map((role) => role.id)
const HaveOrNotAdmRole = rUser.roles.cache.get('1136984503886303253');
console.log(userRoles)
if (userRoles === '',`${HaveOrNotAdmRole.id}`){    //ошибку выдает здесь

Никак не могу понять, как это пофиксить т.к. только начал изучать discord.js
Если у того кто прописывает команду есть нужная роль, то всё работает как нужно, но если её нет, то бот крашит с ошибкой:
TypeError: Cannot read properties of undefined (reading 'id')
Хотел сделать так, чтобы если прописавший команду юзер имел нужную роль, то команда воспроизводилась, а если юзер не имеет нужную роль, то выдавало бы embed с ошибкой. В итоге застрял на том что есть. Пожалуйста помогите.

Comments

[Александр]

userRoles === '',`${HaveOrNotAdmRole.id}`

а что это вообще за конструкция? что вы ожидали получить от этого выражения?
в правой части вы воспользовались оператором "запятая", который выполняет все выражения, но возвращает результат только последнего.

это значит, что из выражение1, выражение2, выражение3, выражениеN, вернётся только последнее - выражениеN

поэтому в вашем случае пустая строка '' игнорируется, и возвращается только `${HaveOrNotAdmRole.id}`

после этого сравнение выглядит так: userRoles === `${HaveOrNotAdmRole.id}`

userRoles—массив, `${HaveOrNotAdmRole.id}` — строка.
при строгом сравнении === не происходит преобразование типов к одному, такое сравнение как у вас ВСЕГДА вернёт false, сам if НИКОГДА не сработает, вы сравниваете яблоко с табуреткой

даже если исправить ошибку, толку от такой конструкции 0.

Answer 1

[sinaee]

//const userRoles = rUser.roles.cache.filter((roles) => roles.id !== rUser.guild.id).map((role) => role.id) бесполезная строчка
const HaveOrNotAdmRole = interaction.guild.roles.cache.get('1136984503886303253') // чуть изменена
if (rUser.roles.cache.find(role => role.id == HaveOrNotAdmRole){   }

посидел, подумал, поискал и вот полностью рабочий код. Причем работает имеено так как задумывалось. Если у пользователя есть роль, то комманда работает, если ее нет, то выводит ошибку через отдельный embed

Answer 2

[Владимир Коротенко]

if (userRoles === '' && HaveOrNotAdmRole,`${HaveOrNotAdmRole.id}`){ }

В общем то вам написали что у undefined нет свойства id

Comments

[sinaee]

в таком случае команда работает даже если нет, нужной роли. :(

[Владимир Коротенко]

sinaee, с чего бы это?
В переводе на русский
если userRoles равно пустой строке и HaveOrNotAdmRole определено то делаем что то

[sinaee]

Владимир Коротенко, Суть была в том что у одного юзера есть несколько ролей, одна из которых дает доступ к боту, а
userRoles в свою очередь хранила все id ролей юзера, который ввел команду. В то же время
HaveOrNotAdmRole выводило undefined в том случае если у юзера нет роли открывающей доступ к боту.
И сама суть была в том что HaveOrNotAdmRole.id пыталось получить id того, чего нет и именно из-за этого бот крашил.

[Владимир Коротенко]

sinaee, нда я и забыл какой Js гибкий. Нормальный компилятор не пропустил бы id с запятой.

Answer 3

[Synbulat Biishev]

Не следует называть участников как user, а пользователей как member. Это все же разные объекты.

Участника из интеракции можно достать через interaction.member. Для проверки роли можно использовать member.roles.cache.has('id')
В итоге код должен выглядеть так

if (interaction.member.roles.cache.has('1136984503886303253')) {}