Quel est le moyen le plus lisible d'écrire des fonctions imbriquées en javascript?

Question

Exemple de ce code, j'ai écrit ne semble pas aussi lisible que ce pourrait être:

function getShortMessages(messages) { 
    return messages.filter((messages) => { 
     return messages.message.length < 50 
    }).map((object) => { 
     return object.message 
    }) 
} 

Answer 1

function getShortMessages(messages) { 
 
    return messages.filter(message => message.message.length < 50) 
 
    .map(message => message.message) 
 
}

Answer 2

semble bien pour moi pour être honnête. Ce que vous pouvez faire est de remplacer le "50" par une variable dans votre fichier js.

var MESSAGE_MAX_LENGTH= 50; 

Et recadrez un peu la façon dont vous adresse la fonction

function getShortMessages(messages) { 
    return messages 
     .filter((messageObject) => { 
      return messageObject.message.length < MESSAGE_MAX_LENGTH 
     }) 
     .map((messageObject) => { 
      return messageObject.message 
     }); 
} 

Aussi je trouve que lorsque aborder un éventail de messages et en cours d'exécution par la fonction filter, il est préférable de ne pas appeler l'objet messages mais item ou messageObject

par ailleurs, object dans la fonction map est un peu sinistre, appelez messageObject encore une fois, par exemple pour que vous sachiez ce que vous utilisez spécifiquement

Answer 3

dans ES 6 vous pouvez utiliser des raccourcis, comme ceci:

function getShortMessages(messages) { 
    return messages.filter(messages => messages.message.length < 50).map(object => object.message) 
} 

Son dépend de vous que l'on est lisible. En une ligne, vous n'avez pas besoin d'utiliser {} ni return et si vous utilisez la fonction de tableau, avec 1 paramètre vous n'avez pas besoin non plus (messages) =>, vous pouvez utiliser seulement