refactoring: index.js

[nikolai-shabalin]

Изменения для #43 - привести код index.js к единому стилю

[pepelsbey]

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

[nikolai-shabalin]

К единому стилю привести не проблема, но по хорошему, как и с #45, там бы всё переписать. Там очень плохо всё. Человек из #25, обещал всё переписать. Поэтому я и не трогал inner.js.

В целом ничто мне не мешает порефакторить inner.js.

[nikolai-shabalin]

Все jquery объекты теперь именуются с $, чтобы сразу было видно, что там что-то про Jquery.

Удалил

• неиспользуемый код
• неспользуемые переменные

jquery-селекторы вынес в шапку. Повторяющиеся jquery селекторы вынес, теперь в DOM стучимся реже.

Конструкции

var blabla, 
blabla2;

переделал в

var blabla;
var blabla2;

так нагляднее и вероятнее, что не попадёт в window.

Длинные строки ограничил 120 символами по ширине.

Некоторые переменные назвал по-человечески, хотя автору более понятно. Я не всё осилил =(

Дублирующиеся куски вынес в функцию.

Двойные кавычки, сменил на одинарные там где это возможно.

Добавил точек с запятой в конец строк.

Всё что можно было зачейнить зачейнил. Всё что не смог не зачейнил.

В целом для рефактора, мне кажется достаточно. По правильному это надо переписать. Опять же ждём #25

[pepelsbey]

Спасибо! По такому коду точно будет проще переписывать :) Ну и не стыдно за исходники сейчас.