Правильно ли написан код?

Question

[nurpais95]

https://github.com/nurpais/FullPageScroll можно ли использовать этот код на реальном проекте

Comments

[edward_freedom]

сделай класс из него. На JQ, удобно было бы использовать, типо так $("$main").FullPageWhile();

Answer 1

[Дмитрий Лузанов]

- Глобальные переменные
- Жестко забитые классы

Если вы собираетесь использовать в качестве плагина - то нет. Ели на вашем проекте работает и вас устраивает - ок.

Answer 2

[Jecob Holus]

все правильно

Answer 3

[SagePtr]

лучше заверните хотя бы в замыкание, чтобы ваш код не засорял глобальный объект window.