Вы пропускаете '#' на ваш призыв JQuery:
window.outputBefore = function(x) {
$("#output-before").text(x); // You were missing the # here
};
Кроме того, некоторые другие вещи: Изменить
window.ACTIVE_NUMBER;
window.LAST_NUMBER;
To:
var ACTIVE_NUMBER;
var LAST_NUMBER;
Кроме того, вам необходимо изменить LAST_NUMBER перед обновлением ACTIVE_NUMBER, таким образом, он запоминает, что ACTIVE_NUMBER было до его текущего значения.
window.checker = function() {
LAST_NUMBER = ACTIVE_NUMBER; // Added this line
var foundnumber = parseInt($(".tool.active").index() + 1);
ACTIVE_NUMBER = foundnumber;
};
И тогда в вашей функции мыши, необходимо для вывода LAST_NUMBER:
$(".tool").click(function() {
if ($(this).hasClass("active")) {
console.log("It is already active.");
} else {
$(".tool").removeClass("active");
$(this).addClass("active");
checker();
outputBefore(ACTIVE_NUMBER); // Add this line
output(LAST_NUMBER);
}
});
И поставил точку с запятой после всех ваших функций.
Полный код:
$(document).ready(function() {
// global variables
var ACTIVE_NUMBER;
var LAST_NUMBER;
// global functions
window.output = function(x) {
$("#output").text(x);
};
window.outputBefore = function(x) {
$("#output-before").text(x);
};
window.checker = function() {
LAST_NUMBER = ACTIVE_NUMBER;
var foundnumber = parseInt($(".tool.active").index() + 1);
ACTIVE_NUMBER = foundnumber;
};
// the starting value
// if(LAST_NUMBER == undefined){
// LAST_NUMBER = 1;
// }
checker();
output(ACTIVE_NUMBER);
outputBefore(LAST_NUMBER);
// Changes applied onclick event
$(".tool").click(function() {
if ($(this).hasClass("active")) {
console.log("It is already active.");
} else {
$(".tool").removeClass("active");
$(this).addClass("active");
checker();
outputBefore(ACTIVE_NUMBER);
output(LAST_NUMBER);
}
});
});
Отличное объяснение. Большое спасибо! – Dawid