javascript Ajuda com Refatoração

[Eduardo Falcão 0]

O que pode ser refatorado, apenas me mostrar o que e como pode ser refatorado nos códigos abaixo se não houver melhor ainda

rm(path.join(config.build.assetsRoot, config.build.assetsSubDirectory), err => {
  if (err) throw err
  webpack(webpackConfig, function (err, stats) {
    spinner.stop()
    if (err) throw err
    process.stdout.write(stats.toString({
      colors: true,
      modules: false,
      children: false,
      chunks: false,
      chunkModules: false
    }) + '\n\n')

module.exports = function () {
  var warnings = []
  for (var i = 0; i < versionRequirements.length; i++) {
    var mod = versionRequirements[i]
    if (!semver.satisfies(mod.currentVersion, mod.versionRequirement)) {
      warnings.push(mod.name + ': ' +
        chalk.red(mod.currentVersion) + ' should be ' +
        chalk.green(mod.versionRequirement)
      )
    }
  }
                                                 

if (warnings.length) {
    console.log('')
    console.log(chalk.yellow('To use this template, you must update following to modules:'))
    console.log()
    for (var i = 0; i < warnings.length; i++) {
      var warning = warnings[i]
      console.log('  ' + warning)
    }
    console.log()
    process.exit(1)
  }

var savedBeats = JSON.parse(localStorage.getItem('savedBeats'));
      if (savedBeats !== null && savedBeats !== undefined) {
        var resultado = [];

        function percorrer(obj) {
          for (var propriedade in obj) {
            if (obj.hasOwnProperty(propriedade)) {
              if (typeof obj[propriedade] == "object") {
                percorrer(obj[propriedade]);
              } else {
                resultado.push(obj[propriedade]);
              }
            }
          }
        }

var sincronizar = JSON.parse(localStorage.getItem('offlineBeats'));
      if (getters.online === true && sincronizar !== null && sincronizar !== undefined) {
        for (var i = 0; i < sincronizar.length; i++) {
          var obj = sincronizar[i];
          // console.log("Indice: " + i + "\n Beats: " + obj["beats"] + "\n Date: " + obj["date"] + "\n Time: " + obj["time"])
          const uid = getters.user.uid;
          const newBeat = {
            date: obj["date"],
            time: obj["time"],
            beats: obj["beats"]
          };
          const newBeatKey = firebase.database().ref().child('beats/' + uid).push().key;
          firebase.database().ref('beats/' + uid + '/' + newBeatKey).set(newBeat);
        }
        localStorage.removeItem('offlineBeats')
        commit('setOfflineBeats', null)
        commit('setSincronized', true)
      }

 

[BrunoBit 82]

Você pode trocar isso:

if (obj.hasOwnProperty(propriedade)) {
	if (typeof obj[propriedade] == "object") {
		percorrer(obj[propriedade]);
	} else {
		resultado.push(obj[propriedade]);
	}
}

Por isso:

if (!obj.hasOwnProperty(propriedade)) continue;
typeof obj[propriedade] == "object" ? percorrer(obj[propriedade]) : resultado.push(obj[propriedade]);

 

Você pode trocar isso:

var savedBeats = JSON.parse(localStorage.getItem('savedBeats'));
	if (savedBeats !== null && savedBeats !== undefined) {

por isso:

var savedBeats = JSON.parse(localStorage.getItem('savedBeats'));
	if (savedBeats) {

Mesma coisa vale pra esse trecho:

var sincronizar = JSON.parse(localStorage.getItem('offlineBeats'));
      if (getters.online === true && sincronizar !== null && sincronizar !== undefined) {

Dá pra trocar por isso:

var sincronizar = JSON.parse(localStorage.getItem('offlineBeats'));
      if (getters.online && sincronizar) {

E o warning dá pra trocar também:

Trocar esse trecho:

for (var i = 0; i < warnings.length; i++) {
	var warning = warnings[i]
	console.log('  ' + warning)
}

Por esse:

for (var i = 0; i < warnings.length; i++) {
	console.log('  ' + warnings[i])
}

 

Nesses códigos que você mostrou só consegui identificar isso, de resto eu não alteraria nada também.