Est-ce une mauvaise pratique de créer setter et getters pour simplifier les fonctions en JavaScript?

Question

Exemple:

Je travaille sur une interface utilisateur pour un projet de robotique qui affiche le rapport d'étalonnage de la roue. Le ratio par défaut est de 0,89 et j'ai un bouton qui incrémente le ratio de 0,01.

function incrementRatio() { 
    if (getStatus() != "Connected" || getRatio() >= 1.00) { 
     stopEvent(); 
    } else { 
     var newRatio = getRatio() + 0.01; 

     setUIArrayValue("currentRatio", newRatio); 
     setIDText('ratiovalue', getRatio().toFixed(2)); 
    } 
} 

La fonction ci-dessus utilise accesseurs tels que:

• getStatus(), getRatio()
• setUIArrayValue (clé, valeur), setIDText (id, texte)

Certains d'entre eux contiennent une instruction mais sont fréquemment réutilisés dans le code.

Par exemple, setIDText():

function setIDText(id, text) { 
    document.getElementById(id).innerHTML = text; 
} 

Contient 1 déclaration, mais fournit une manière beaucoup plus court et lisible du texte muter. Sans ces setters et getters, des fonctions telles que incrementRatio() sembleraient plutôt encombrées.

Answer 1

nom que vous accesseurs ne sont pas vraiment ceux-ci exercent une logique réelle comme par exemple la fonction setIDText():

function setIDText(id, text) { 
    document.getElementById(id).innerHTML = text; 
} 

Ce sont plutôt des méthodes auxiliaires/logiques qui permet à la fois:

• à évitez de vous répéter.
• pour rendre votre code plus lisible et plus compréhensible en faisant abstraction des détails d'implémentation dans une fonction qui transmet un nom explicite.
  

L'utilisation de ce type de fonction est une bonne chose car il offre un meilleur code de qualité.

Answer 2

Non, ce n'est pas le cas. Cela rend le code beaucoup plus lisible, et un bon parseur le mettra en ligne pour que la différence de performance soit proche de zéro.