[REFACTOR] Makes some changes to commands uses classes correctly

Author: carlos3g

src/commands/coin.js

@@ -10,7 +10,7 @@ async function loadCheerio(url) {
   }
 }
 
-async function getData(url) {
+async function getCoinStats(url) {
   const $ = await loadCheerio(url);
 
   if (!(typeof $ === 'function')) {
@@ -23,8 +23,8 @@ async function getData(url) {
     .find('tr');
   const statsArray = [];
 
-  priceStatistics.each(function () {
-    const tr = $(this);
+  priceStatistics.each((_, pS) => {
+    const tr = $(pS);
     const key = tr.find('th').text();
     let value = tr.find('td');
 
@@ -36,7 +36,7 @@ async function getData(url) {
       value = value.text();
     }
 
-    if (value !== 'No Data' || value !== 'Sem Dados') {
+    if (value !== 'No Data') {
       const object = Object.fromEntries([[key, value]]);
       statsArray.push(object);
     }
@@ -45,20 +45,10 @@ async function getData(url) {
   return statsArray;
 }
 
-function getUrl(args, text) {
-  let baseURL = 'https://coinmarketcap.com/currencies/';
-  const path = text.replace(/\s/g, '-').toLowerCase();
-
-  if (args.includes('brl')) {
-    baseURL = 'https://coinmarketcap.com/pt-br/currencies/';
-  }
-
-  return baseURL + path;
-}
-
 class Coin {
   constructor() {
     this.name = 'coin';
+    this.BASE_URL = 'https://coinmarketcap.com/currencies/';
     this.defaultMessage = `
 uso: *!coin* [--flag] name
 _--all -> mostra todas as informações disponiveis_  
@@ -71,17 +61,18 @@ a flag _all_ pode retornar dados em excesso, sua utilização repetida será con
     const { args, text } = data;
 
     if (!text) {
-      message.reply(this.defaultMessage.trim());
+      message.reply(this.defaultMessage);
       return;
     }
 
-    const url = getUrl(args, text);
-    let coinStats = await getData(url);
+    const url = this.getUrl(text);
+    let coinStats = await getCoinStats(url);
 
     if (!coinStats) {
-      message.reply('moeda não encontrada');
+      message.reply('Moeda não encontrada.');
       return;
     }
+
     if (!args.includes('all')) {
       coinStats = coinStats.slice(0, 3);
     }
@@ -96,6 +87,11 @@ a flag _all_ pode retornar dados em excesso, sua utilização repetida será con
 
     message.reply(output.trim());
   }
+
+  getUrl(text) {
+    const path = text.replace(/\s/g, '-').toLowerCase();
+    return this.BASE_URL + path;
+  }
 }
 
 module.exports = Coin;

src/commands/lyric.js

@@ -34,10 +34,12 @@ class Lyric {
     const { link } = results[0];
     const soup = await makeSoup(link);
     const title = soup.find('div', { class: 'cnt-head_title' }).find('h1');
-    const lyricsDiv = soup.find('div', { class: 'cnt-letra' });
-    const pArray = lyricsDiv.findAll('p');
-    const lyric = pArray.map((p) => removeBr(p).text).join('');
-    const output = `*${title.text}*\n\n${lyric}\n\n_${link}_`;
+    const lyrics = soup
+      .find('div', { class: 'cnt-letra' })
+      .findAll('p')
+      .map((p) => removeBr(p).text)
+      .join('');
+    const output = `*${title.text}*\n\n${lyrics}\n\n_${link}_`;
 
     message.reply(output);
   }

src/commands/report.js

@@ -1,6 +1,5 @@
 const { chattools } = require('../utils');
 
-const myID = chattools.userID(process.env.REPORT_NUMBER);
 const strings = {
   defaultMessage: `
 uso: _!report [--flag] [descrição]_
@@ -17,12 +16,12 @@ argumentos:
 class Report {
   constructor() {
     this.name = 'report';
+    this.reportID = chattools.userID(process.env.REPORT_NUMBER);
     this.strings = strings;
   }
 
   execute(data, message, client) {
     const { args, text } = data;
-
     const reportMsg = `⚠️ *${args[0]} report* ⚠️\n\n${text}`;
 
     if (args.length === 0 && text) {
@@ -33,7 +32,7 @@ class Report {
     }
 
     if (args.includes('bug') || args.includes('user')) {
-      client.sendMessage(myID, reportMsg);
+      client.sendMessage(this.reportID, reportMsg);
       message.reply(this.strings[args[0]]);
       return;
     }

src/commands/search.js

@@ -26,8 +26,8 @@ class Search {
     message.reply(`Nenhum resultado foi encontrado para: _${text}_`);
   }
 
-  static formatResult(object) {
-    const { title, link, snippet } = object;
+  static formatResult(result) {
+    const { title, link, snippet } = result;
     return `*${title}*\n\n${snippet}\n\n_${link}_`;
   }
 }

src/commands/suggest.js

@@ -1,10 +1,9 @@
 const { chattools } = require('../utils');
 
-const myID = chattools.userID(process.env.REPORT_NUMBER);
-
 class Suggest {
   constructor() {
     this.name = 'suggest';
+    this.reportID = chattools.userID(process.env.REPORT_NUMBER);
     this.defaultMessage = `
 uso: _!suggest [--flag] [sugestão]_
 
@@ -19,7 +18,6 @@ argumentos:
 
   execute(data, message, client) {
     const { args, text } = data;
-
     const reportMsg = `⚠️ *${args[0]} suggestion* ⚠️\n\n${text}`;
 
     if (args.length === 0 && text) {
@@ -34,7 +32,7 @@ argumentos:
       args.includes('remove') ||
       args.includes('change')
     ) {
-      client.sendMessage(myID, reportMsg);
+      client.sendMessage(this.reportID, reportMsg);
       message.reply('Obrigado pela colaboração!');
       return;
     }