Прошу помочь оптимизировать код

[Erolast]

6. this.next = function (num=0) Это не работает. Правильно делать так.

Нет, это как раз правильно и это работает, но только в лисе. Значения параметров по умолчанию - часть еще неутвержденного ES6.

[WorM32]

Нет, это как раз правильно и это работает, но только в лисе. Значения параметров по умолчанию - часть еще неутвержденного ES6.

Это понятно, но вряд ли ТС писал данный код только под лису)

[Master_Sergius]

Ого, сколько всего. Ясно, большое спасибо. Буду работать над ошибками. Надо бы алгоритмизацию и паттерны подтянуть. А куча разных ЯП сказывается, последнее время - питоню.
Ещё раз, всем большое спасибо, но если ещё чего есть добавить, буду рад.

[Arramis]

но если ещё чего есть добавить, буду рад.

Помимо всего того что вам уже сказали, ваш код не обеспечивает чистоту для глобального объекта (window).
У вас имена функций не конфликтно - безопастные, в таком случае весь код надо сувать в единственный кастомный глобальный объек с уникальным именем, вот так:

var __MyUniqueName = __MyUniqueName || {

	Questions: function() {
		this.question_count = 5;	 
		this.process_answer = function (answer, num) { /* ... */ }
	},

	clear: function() { /* ... */ }
	...
	
};

[Erolast]

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

[Arramis]

Erolast,
можно и в анонимную, но городить деревья это ты переборщил, ничего плохого не будет если сувать в объект (даже на быстродействие не повлияет). Как минимум порядок в коде и делать это надо уже если строк кода больше 200 - 300. Так что смысл имеет

[Erolast]

Ну надо же меру знать в порядке.