Javascript-форум (https://javascript.ru/forum/)
-   Общие вопросы Javascript (https://javascript.ru/forum/misc/)
-   -   Прошу помочь оптимизировать код (https://javascript.ru/forum/misc/49225-proshu-pomoch-optimizirovat-kod.html)

Master_Sergius 04.08.2014 12:42

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

Arramis 05.08.2014 16:08

Цитата:

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

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

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

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

Erolast 05.08.2014 16:46

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

Arramis 05.08.2014 17:21

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

Erolast 05.08.2014 19:16

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


Часовой пояс GMT +3, время: 05:40.