Спецам Javascript. Что тут можно оптимизировать?

Question

[Urukhayy]

Написал простую игру - "Пятнашки", основанную на блоках HTML.
Использовал: HTML, CSS3, Javascript (jQuery).
Вот ссылка - testims.orisale.ru
JS думаю сами знаете как глянуть.
Если нужен исходник, отпишитесь, выложу.
Мне нужны советы по оптимизации. Что можно улучшить, что исправить, что переделать...

Answer 1

[1001001 111]

Держи jsfiddle.net/Tiro/zG3hV/3

Answer 2

[Alexander Sydorenko]

А Вы знали что костяшки нельзя рандомно разбрасывать по полю пятнашек? ;) Для того чтобы они всегда складывались необходим специальный алгоритм размешивания суть которого должна сводиться к (перестановка двух случайних косточек) * 2n?

Источник

Поэтому сначала оптимизируйте генерацию случайной доски.

У Имя Фамилия в алгоритме, кстати, та же ошибка..

Comments

[Urukhayy]

2n - ?

[Alexander Sydorenko]

@Urukhayy: Парное количество раз имелось ввиду, не знал как схематически лучше изобразить.

Answer 3

[aen]

Для начала не засерайте глобальную область видимости.

Comments

[Urukhayy]

То есть писать функции в объектах?

[aen]

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

[Urukhayy]

@aen: Вариант, что ответом ниже самый оптимизированный?

Answer 4

[Илья Агафонов]

Советую почитать про паттерны программирования на JS. Например про модульную систему

Answer 5

[UbuRus]

$(".square").click(function()
неэффективно

Нужно так:

$("body").on("click", ".square", function()

upd. лол, раз тут набежали объясню

Заглянув в исходники jQuery можно увидеть, что разные $(selector).click это алиасы на $(selector).on
https://github.com/jquery/jquery/blob/master/src/e...

А вот тут видно, что во втором случае у нас будет только один листенер, в первом случае их будет 15. Сильно на производительность не повлияет, но если он захочет сделать 2048нашки то будет заметна разница.
https://github.com/jquery/jquery/blob/master/src/e...

Comments

[Urukhayy]

В чем разница то?

[UbuRus]

@MaxKorz: Идиот ты(SO Driven Developer) и ник идиотский:
https://github.com/jquery/jquery/blob/master/src/e...