Откуда оно берёт новый элемент массива и куда оно девает значения?

Question

[gohellp]

Решил сделать задание друга, на удобном для себя языке и чтоб лишний раз не вызывать опрос с консоли решился делать это через дискорд.
Задание было таким:

Ввести n целых чисел(>0 или <0). Найти разницу чисел. Результат вывести на экран.

Я написал код для этого:

const Discord = require('discord.js');
const bot = new Discord.Client();
const {token1} = require('./conf.json');
var mas =[]
var prom =[]
let a
let i=1

bot.on("message", msg => {
    msg.content = msg.content.toLowerCase().split(" ")
    switch (msg.content[0]) {
        case"!add":
            if (!msg.author.bot){
                if (!msg.content[2]) {
                    msg.reply("Введите, хотя бы 2 значения!")
                } else {
                    mas=[]
                    for (let i = 1; msg.content[i]; i++) {
                        mas.push(Number(msg.content[i]))
                    }
                    msg.reply("Готово. Можете вводить !math")
                    a = mas.length + 1
                }
            }
        break;
        case"!math":
            try{
                prom = []
                mas.push(0)
                while(i<mas.length){
                    mas[a] = mas[i-1] - mas[i];
                    prom.push(mas[a])
                    i++
                }
                console.log(mas)
                console.log(prom)
                msg.reply("Результат: " + mas[a-1] + "\nПромежуточные результаты: " + prom)
                mas = []
            }catch (e) {
                msg.reply("Сначала введите значения!!!")
            }
        break;
    }
})

bot.login(token1);

Но вместо нормальной работы оно решает проявить себя:



1)Оно не записало промежуточный результат в prom
2)Оно откуда-то взяло ещё один элемент массива и отказалось считать
3)Оно просто отказалось считать

Я хочу понять по какому принципу возникает этот дополнительный элемент и вообще насколько чисто выполнена эта задача.
Заранее благодарен!

Comments

[Александр]

если вы хотите реализовать функционал на скрине, вам нужно использовать коллекторы сообщений.

[gohellp]

Александр, я ввод уже, в принципе, реализовал. У меня ошибка именно во второй части, тк я пол часа тыкал console.log в разные места первой половины, где идёт запись, и там все хорошо было. Где он в самих вычислениях берёт что берёт для меня полная загадка. Во вторую тоже потыкал, но это не особо дало результата

[gohellp]

molekulyarniy, да, напортачил я:D
Мало изучал непосредственно сам js и ооочень малый опыт в программировании на нём. Благодарю вас, изменю код к лучшему :D
Мне нужно было от первого до конечного элемента всё добавить, а тк есть и нулевой я не особо решился использовать .forEach

[WbICHA]

Мало изучал непосредственно сам js и ооочень малый опыт в программировании на нём.

Решил сделать задание друга, на удобном для себя языке

gohellp, ???

[gohellp]

WbICHA, я работал в основном с простой логикой на подобии пинг-понга в чатах. При более сложных задачах шёл через дебри, но оно всегда работало. Как-то так у меня сложились карты:D

[gohellp]

molekulyarniy, благодарю, днём займусь переработкой своего кода и обязательно отпишу о результатах:D

[gohellp]

molekulyarniy, да, именно оно, но, всё же не совсем так было. Генерация случайных чисел, отправка файлов. Этим я в последний раз и занимался

[WbICHA]

Дам, говнокод лютый, конечно.

a = mas.length + 1 // зачем +1?

mas.push(0) // это вообще для чего тут?

// это вообще шедевр
// то есть каждый раз мы перезаписываем разницу в элемент массива под индексом "а",
// а после берём и выводим значение под индексом "а-1", равное ранее запушенному
// нулю и спрашиваем "что не так?". Гениально.
// Про то, почему бы для этого не использовать обычную переменную, я вообще молчу.
while(i<mas.length){
  mas[a] = mas[i-1] - mas[i];
  prom.push(mas[a])
  i++
}
msg.reply("Результат: " + mas[a-1] + "\nПромежуточные результаты: " + prom)

[WbICHA]

я работал в основном с простой логикой

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

[gohellp]

WbICHA, Да, идёт ходьба по кругу там, но, учитывая, что a равно длине массива с добавленным нулем, те а-1 - последнее введённое значение оно выводит именно ноль, хоть и запись должна была уже быть произведена. Меня именно это волнует. Да и хотелось бы выяснить насколько тут примитивные методы я использовал

[WbICHA]

gohellp, да уж, я тебя уже носом ткнул в ошибку, а ты всё по прежнему. Тут уже не навыки программировать нужны, а просто логически думать.

Answer 1

[StiflerProger]

bot.on("message", msg => {

  let [type, ...data] = msg.content.toLowerCase().split(" ");

  if (msg.author.bot) return; // игнорируем бота

  switch (type) {
    case "!add":
      if (data.length < 2) return msg.reply("Введите, хотя бы 2 значения!");

      if (data.find(e => isNaN(e) || Number(e) === 0 || Math.floor(e) !== Number(e))) {
        return msg.reply("Числа должны быть целыми в диапазоне (-∞, 0), (0, ∞)");
      }

      msg.reply("Результат: " + data.slice(1).reduce((acc, cur, index) => acc - cur, data[0]));
      break;

    default:
      msg.reply("Чтобы вычислить разницу чисел. Вызовите команду !add <числа через пробел>");
      break;
  }

})

Для поставленной задачи код у тебя слишком запутанный. Не нужно сохранять нигде промежуточные результаты, и не нужно просить юзера вводить !math

Конечно какой функционал писать дело твоё) но я решил не усложнять

Comments

[gohellp]

Благодарю вас. Промежуточные значения я просто хотел сохранять чисто для себя. Про !math я и сам догадывался, но это уже дело вкуса:D
Честно говоря, код писался после долгого дня и без сильных знаний, что я уже в комментариях отмечал, от сюда и запутаность:D