discord.js角色检查优化

Question

我目前正在建立一个不和谐验证机器人，这个机器人的一个功能，如果检查你是否有一个角色从每一个需要的类别，然后在最后产生你的角色的总结。目前，我有一个工作解决方案，它的功能完全符合我的要求，但它使用了一个很大的if堆栈，我对js很陌生，但被告知尽可能不要使用大的if/if else if堆栈。我已经研究了switch的案例，但不知道如何应用这些案例，所以我想知道是否有一种更优化的方法来实现我所需要的。我现在的代码如下：

var hasrole = false;
var role = "";
var subrole = "";
var istype1 = false;
var istype2 = false;
var istype3 = false;
var istype4 = false;
var counter = 0;

if (message.member.roles.cache.find(r => r.id === "736032207248031765")) {
    istype1 = true;
    role = "Type 1";
    hasrole = true;
    counter ++;
}
if (message.member.roles.cache.find(r => r.id === "736032822053175306")) {
    istype2 = true;
    role = "Type 2";
    hasrole = true;
    counter ++;
}
if (message.member.roles.cache.find(r => r.id === "736032670013980702")) {
    role = "Type 3";
    hasrole = true;
    counter++
    if (message.member.roles.cache.find(r => r.id === "736032716411371581")) {
        istype1 = true;
        subrole = "Type 1";
    } else if (message.member.roles.cache.find(r => r.id === "736032780122849340") {
        istype2 = true;
        subrole = "Type 2";
    } else {
        istype3 = true;
    }
}
if (message.member.roles.cache.find(r => r.id === "736032870703038534")) {
    istype4 = true;
    role = "Type 4";
    hasrole = true;
    counter ++;
}
if (message.member.roles.cache.find(r => r.id === "736033471474172025")) {
    istype2 = true;
    role = "Type 5";
    hasrole = true;
    counter ++;
}
if (counter >= 1) {
    message.channel.send("More than 1 main role, please have only 1");
    return;
} else if (hasrole == false) {
    message.channel.send("No main role please get one")
    return;
}

Answer 1

好的，我假设您的代码运行良好，只是需要一点优化。

我的第一个建议是使用数组并迭代您的类型。

我的第二个建议是不要硬编码您的角色ID。

我的第三条建议是，也许可以通读您的代码，并尝试理解它，因为它看起来非常混乱。

Type3角色检查中的代码很混乱，而Type5角色检查验证了Role2值。

这是我的代码版本

const roles = ['736032207248031765', '736032822053175306', '736032670013980702', '736032870703038534', '736033471474172025'];
let userRoles = [];

roles.forEach(r => {
    if(message.member.roles.cache.has(r)){
        userRoles.push(r);
    }
});

if (userRoles.length >= 1) {
    message.channel.send("More than 1 main role, please have only 1");
    return;
} else if (userRoles.length == 0) {
    message.channel.send("No main role please get one")
    return;
}

正如您所看到的，这比一整条IFS链要方便得多。