Plusieurs « if » ou une combinaison « if/else if » avec « return » ?

Bonsoir,

Voici un morceau d’un fichier que je viens de finir d’écrire :

exec(channel) {
    if (channel.type === 'text') {
        return channel.guild.channels.get(config.defaultChannel).send(i18n.t('events:channel.create.text', { channel: channel, interpolation: { escapeValue: false }}));
    }
    if (channel.type === 'voice') {
        return channel.guild.channels.get(config.defaultChannel).send(i18n.t('events:channel.create.voice', { channel: `**${channel}**`, interpolation: { escapeValue: false }}));
    }
    if (channel.type === 'category') {
        return channel.guild.channels.get(config.defaultChannel).send(i18n.t('events:channel.create.category', { category: `**${channel.name}**`}));
    }
}

Le contenu des if n’est pas important. En gros, exec() s’exécute à la création d’un nouveau channel. En fonction de son type, un message différent est envoyé. Je me demandais si, au niveau pratique ainsi qu’au niveau performance, il valait mieux faire suivre trois if ou bien un if suivi de deux else if.

Merci d’avance .

Je viens de penser à faire une combinaison if/else if comprenant chacun une ligne. Une variable contenant la chaîne de caractère puis de faire return à la fin de la fonction. Est-ce mieux ?

@Heziode : Si tu veux le faire avec 3 œufs, va falloir revoir tes conditions, parce que là t’en fais qu’avec 2 œufs

C’est corrigé X)

C’est en effet cette bibliothèque ! J’ai longtemps hésité à sauter le pas et finalement, je trouve cela fantastique  !

C’est en effet l’objectif de mes return Dans le cas d’un code qui retourne quelque chose dans chaque bloc de la condition, on m’a conseillé de faire plusieurs bloc if ou ne pas mettre de else. Par exemple :

const age = 33;
if (estMajeur(age) {
    console.log('Santé !');
}

const estMajeur = age => {
    if (age > 18) {
        return true;
    }
    return false;
}

En fait, la problématique revient à la simplification du code où à l’optimisation de l’exécution. La correction de @Zérotisme est bien plus clair que de répéter la méthode à rallonge qui permet d’envoyer un message. Pour le coup, je ne sais pas du tout quel choix faire  .

Je vais penser au switch que je n’utilise pas assez à cause de ce qui m’a été dit lors de mon apprentissage : proscrire. Malgré le fait de savoir que c’est trop extrême, je n’arrive pas à y penser. Dans tous les cas, si je pars sur le switch, il faut que je réfléchisse à ce qui est à l’intérieur. Hormis si break jouera d’une certaine façon le rôle de return ?

Voilà le résultat final que j’obtiens :

let options;

switch (channel.type) {
    case 'text':
        options = { channel: channel, interpolation: { escapeValue: false }};
        break;
    case 'voice':
        options = { channel: `**${channel}**`, interpolation: { escapeValue: false }}
        break;
    case 'category':
        options = { category: `**${channel.name}**` }
}

return channel.guild.channels.get(config.defaultChannel).send(i18n.t(`events:channel.create.${channel.type}`, options));

Est-ce la meilleure manière que je puisse écrire cette conditions ?

Moi j’approuve.

Tu as oublié le break dans le cas du "category" même si ce n’est pas vraiment utile. Tu ne gère pas les erreurs ? Si on passe autre chose que "text", "voice" ou "category" que va-t-il ce passer ?

Connaissant i18next, il va simplement t’afficher la clé "events:channel.create.foo" par exemple.

Après, si tu veux un truc « très » optimisé, le mieux c’est de faire des mesures avec d’autres façon de faire.

J’ai omis exprès le dernier break car la suite doit être exécuté quoi qu’il arrive. Il en est de même pour l’absence de default car channel.type ne peut prendre que les trois valeurs que tu vois ci-dessus.

Par contre, bizarrement, il y a un autre cas où j’utilise une variable directement dans i18n.t() qui retourne une erreur mais le code de ma précédente réponse fonctionne à merveille.

Mon objectif est que cela soit le plus performant possible tout en étant lisible pour un humain (autre ou moi dans le futur). Je pense garder comme ça !