Как перезаписать переменную?

Question

[gotohell]

Привет!
Есть функции они обрабатывает значения инпута сравнивают их с таблицей и если значения находятся в пределах допустимых значений (Больше или ровно) добавляет их в переменную а переменная отражает все эти значения на графике.

И вторая функция находит точку на графике X Y.

Все это работает! Но....

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

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

Помогите разобраться!

https://codepen.io/webmasterotboga/pen/PKYYeY

Comments

[Kovalsky]

Это не относится к ошибке, но вы в функции checkEntry при каждом выполнении добавляете обработчик click для .go

[gotohell]

Никита Полевой: Я понимаю что это не ошибка, просто не могу понять как перезаписать переменную при повторном обращении к функции.

[Kovalsky]

gotohell: какую именно переменную?

[gotohell]

Переменные coord, coordinatYX

[Kovalsky]

gotohell: для того чтобы их переписать, надо им присваивать другое значение. Вы пишите "Задумывалась так, когда вводишь новые значения в инпуте он стирает старые данные", но у вас в коде этого нет - по клику на .go вы добавляете новые значения в coord:

$('.table-data').not('.hide').each(function() {
    coord[iter] = {
      y: $(this).data('pascalmax'),
      x: $(this).data('rashodmin')
    }
    coord[iter + 1] = {
      y: $(this).data('pascalmin'),
      x: $(this).data('rashodmax')
    }
    iter += 2;
  });

И в coordinatYX тоже добавляете новые значения, и кстати неправильно добавляете, ведь у вас iter увеличивается сразу на 2:

$('.table-data').each(function() {
    coordinatYX[iter] = {
      y: $('.pascal').val(),
      x: $('.rashod').val()
    }
  })

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

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

[gotohell]

Никита Полевой: Прошу помогите разобраться! Хотя бы урок на то как нужно перезаписывать массивы?!

Answer 1

[Павел Корнилов]

По ссылке на исходники вижу:
error 404

***
Но только когда вводишь новые значения инпута он не стирает старые значения которые записались в переменную от предыдущих действии.
***
Это решается примерно так:

var input= ...;
var X;
input.onchange= function() {
X= this.value;
}

Comments

[gotohell]

Спасибо! Проблема уже решена!

[Павел Корнилов]

На здоровье! Может, кому-то еще пригодится решение.