Merge pull request #40 from carlos3g/master

REFACTOR

Author: carlos3g

index.js

@@ -1,11 +1,10 @@
 require('dotenv').config();
 const { Alice } = require('./src/Alice');
 const {
-  Bot,
+  About,
   Coin,
   Commands,
   Cron,
-  Doc,
   Links,
   Lyric,
   Report,
@@ -15,11 +14,10 @@ const {
 } = require('./src/commands');
 
 const alice = new Alice([
-  new Bot(),
+  new About(),
   new Coin(),
   new Commands(),
   new Cron(),
-  new Doc(),
   new Links(),
   new Lyric(),
   new Report(),

src/commands/about.js

@@ -0,0 +1,38 @@
+const STRINGS = {
+  help: `
+Mostra informações sobre o bot.
+
+*uso:* \`\`\`!about [--args]\`\`\`
+
+*args válidos:* 
+  \`\`\`--help\`\`\` -> _mostra esta mensagem._
+  `.trim(),
+
+  about: `
+Alice foi criada utilizando a biblioteca \`\`\`whatsapp-web.js\`\`\` como base sob licença Apache 2.0. Saiba mais em _https://github.com/pedroslopez/whatsapp-web.js_
+
+Não sabe como usar um comando? Use a flag help, ex: \`\`\`!lyric --help\`\`\`. Se não conhece os comandos, use \`\`\`!commands\`\`\`
+
+_Quer contribuir? Então dá uma olhada em https://github.com/Coding-in-community/alice_
+  `.trim(),
+};
+
+class About {
+  constructor() {
+    this.name = 'about';
+    this.strings = STRINGS;
+  }
+
+  execute(data, message) {
+    const { args } = data;
+
+    if (args.includes('help')) {
+      message.reply(this.strings.help);
+      return;
+    }
+
+    message.reply(this.strings.about);
+  }
+}
+
+module.exports = About;

src/commands/bot.js

@@ -1,6 +1,18 @@
 const axios = require('axios');
 const cheerio = require('cheerio');
 
+const STRINGS = {
+  help: `
+Retorna dados de uma moeda disponível no CoinMarketCap (https://coinmarketcap.com).
+
+*uso:* \`\`\`!coin [--args] coin_name\`\`\`
+
+*args válidos:* 
+  \`\`\`--all\`\`\` -> _retorna todos os dados disponíveis._
+  \`\`\`--help\`\`\` -> _mostra essa mensagem._
+  `.trim(),
+};
+
 async function loadCheerio(url) {
   try {
     const { data } = await axios.get(url);
@@ -13,8 +25,8 @@ async function loadCheerio(url) {
 async function getCoinStats(url) {
   const $ = await loadCheerio(url);
 
-  if (!(typeof $ === 'function')) {
-    return null;
+  if (typeof $ !== 'function') {
+    throw new Error('Moeda não encontrada.');
   }
 
   const priceStatistics = $('.sc-16r8icm-0.nds9rn-0.dAxhCK')
@@ -49,34 +61,38 @@ 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_  
-
-a flag _all_ pode retornar dados em excesso, sua utilização repetida será considera spam
-    `.trim();
+    this.strings = STRINGS;
   }
 
   async execute(data, message) {
     const { args, text } = data;
 
-    if (!text) {
-      message.reply(this.defaultMessage);
+    if (args.includes('help')) {
+      message.reply(this.strings.help, undefined, { linkPreview: false });
       return;
     }
 
     const url = this.getUrl(text);
     let coinStats = await getCoinStats(url);
 
-    if (!coinStats) {
-      message.reply('Moeda não encontrada.');
-      return;
-    }
-
     if (!args.includes('all')) {
       coinStats = coinStats.slice(0, 3);
     }
 
+    const output = Coin.formatStats(coinStats);
+    message.reply(output);
+  }
+
+  getUrl(text) {
+    if (!text) {
+      throw new Error('Nenhuma moeda foi passada.');
+    }
+
+    const path = text.replace(/\s/g, '-').toLowerCase();
+    return this.BASE_URL + path;
+  }
+
+  static formatStats(coinStats) {
     let output = '';
 
     coinStats.forEach((s) => {
@@ -85,12 +101,7 @@ a flag _all_ pode retornar dados em excesso, sua utilização repetida será con
       output += string;
     });
 
-    message.reply(output.trim());
-  }
-
-  getUrl(text) {
-    const path = text.replace(/\s/g, '-').toLowerCase();
-    return this.BASE_URL + path;
+    return output.trim();
   }
 }
 

src/commands/coin.js

@@ -1,7 +1,14 @@
-class Commands {
-  constructor() {
-    this.name = 'commands';
-    this.defaultMessage = `
+const STRINGS = {
+  help: `
+Lista todos os comandos disponíveis.
+
+*uso:* \`\`\`!commands [--args]\`\`\`
+
+*args válidos:* 
+  \`\`\`--help\`\`\` -> _mostra esta mensagem._
+  `.trim(),
+
+  availableCommands: `
 Os seguintes comandos estão disponiveis:
 - !bot
 - !coin
@@ -14,11 +21,24 @@ Os seguintes comandos estão disponiveis:
 - !search
 - !suggest
 - !wiki
-    `.trim();
+  `.trim(),
+};
+
+class Commands {
+  constructor() {
+    this.name = 'commands';
+    this.strings = STRINGS;
   }
 
-  execute(_, message) {
-    message.reply(this.defaultMessage);
+  execute(data, message) {
+    const { args } = data;
+
+    if (args.includes('help')) {
+      message.reply(this.strings.help);
+      return;
+    }
+
+    message.reply(this.strings.availableCommands);
   }
 }
 

src/commands/commands.js

@@ -1,6 +1,27 @@
 const events = require('events');
 const { chattools, Time } = require('../utils');
 
+const STRINGS = {
+  help: `
+Repete uma mensagem em um determinado período de tempo. Cada mensagem é representada por uma thread.
+
+*uso:* \`\`\`!cron --args [--kwargs=<type>] ...\`\`\`
+
+*args válidos:* 
+  \`\`\`--create\`\`\` -> _cria uma nova thread._
+  \`\`\`--destroy\`\`\` -> _para e apaga uma thread._
+  \`\`\`--stop\`\`\` -> _para uma thread._
+  \`\`\`--start\`\`\` -> _inicia uma thread._
+  \`\`\`--log\`\`\` -> _mostra as threads existentes._
+  \`\`\`--help\`\`\` -> _mostra esta mensagem._
+
+*kwargs válidos:*
+  \`\`\`--s=<int>\`\`\` -> _define um periodo em segundos._
+  \`\`\`--m=<int>\`\`\` -> _define um periodo em minutos._
+  \`\`\`--h=<int>\`\`\` -> _define um periodo em horas._
+  \`\`\`--d=<int>\`\`\` -> _define um periodo em dias._
+  `.trim(),
+};
 const emitter = new events.EventEmitter();
 
 function Thread(id, text, message, timer) {
@@ -22,35 +43,18 @@ class Cron {
     this.name = 'cron';
     this.threads = [];
     this.counter = 0;
-    this.defaultMessage = `
-Repete uma mensagem em um determinado período de tempo. Cada mensagem é representada por uma thread.
-
-*uso:* \`\`\`!command [--args] [--kwargs=<type>] ...\`\`\`
-
-*args válidos:* 
-  \`\`\`--create\`\`\` -> _cria uma nova thread._
-  \`\`\`--destroy\`\`\` -> _para e apaga uma thread._
-  \`\`\`--stop\`\`\` -> _para uma thread._
-  \`\`\`--start\`\`\` -> _inicia uma thread._
-  \`\`\`--log\`\`\` -> _mostra as threads existentes._
-
-*kwargs válidos:*
-  \`\`\`--s=<int>\`\`\` -> _define um periodo em segundos._
-  \`\`\`--m=<int>\`\`\` -> _define um periodo em minutos._
-  \`\`\`--h=<int>\`\`\` -> _define um periodo em horas._
-  \`\`\`--d=<int>\`\`\` -> _define um periodo em dias._
-`.trim();
+    this.strings = STRINGS;
   }
 
   async execute(data, message) {
     const isAdm = await chattools.isAdm(message);
 
-    if (isAdm) {
-      message.reply(this.runs(data, message));
+    if (!isAdm) {
+      message.reply('staff only.');
       return;
     }
 
-    message.reply('staff only.');
+    message.reply(this.runs(data, message));
   }
 
   create(data, message) {
@@ -127,13 +131,14 @@ Repete uma mensagem em um determinado período de tempo. Cada mensagem é repres
       destroy: () => this.destroy(data.text),
       start: () => this.start(data.text),
       stop: () => this.stop(data.text),
+      help: () => this.strings.help,
     };
 
     if (methods[data.args[0]]) {
       return methods[data.args[0]]();
     }
 
-    return this.defaultMessage;
+    throw new Error('Nenhum arg válido foi passado.');
   }
 
   isIdValid(id) {

src/commands/cron.js

@@ -1,11 +1,10 @@
 /* eslint-disable global-require */
 
 module.exports = {
-  Bot: require('./bot'),
+  About: require('./about'),
   Coin: require('./coin'),
   Commands: require('./commands'),
   Cron: require('./cron'),
-  Doc: require('./doc'),
   Links: require('./links'),
   Lyric: require('./lyric'),
   Report: require('./report'),