Блог

У меня есть функция, которая добавляет имя класса к элементу dom, выполняется, предоставленный скрипт работает,

 function rm_row(){ 
  
  // Static 
  var chk_inherent =  arcapi.dataResult().columnNameToIndex('Inherent Risk');
  var chk_residual = arcapi.dataResult().columnNameToIndex('Residual Risk')
  var chk_perf = arcapi.dataResult().columnNameToIndex(arcapi.getSetting('Column name'));
  
   d3.selectAll("#"   arcapi.chartId()   " .dataTables_scrollBody tbody tr").classed('removerow', function() {
    
    const perf = this.children[chk_perf].innerHTML
    const inherent = this.children[chk_inherent].innerHTML
    const residue = this.children[chk_residual].innerHTML
    const cf_regex = perf.replace(/[`~%]/gi, '');
    
    
    
  //  **** Check *****
    const inh_isEmpty = inherent === '' || inherent === 'Null'
    const rsk_isEmpty = residue === '' || residue === 'Null'
    const perf_isInvalid = cf_regex < 0 || cf_regex > 100
    
      /** Logic [if inherent is empty AND  Residual risk is empty AND is perf value is incorrect = True] **/
      return inh_isEmpty amp;amp; rsk_isEmpty amp;amp; perf_isInvalid === true
    });
}
  
  

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