Что бы вы сказали если вам такой код дали в поддержку?

Question

[v- smerti]

Сейчас пишу морду для одной игрушки. Потом я думаю будет ещё как минимум пару человек поддерживать мою поделку.

Конечно когда будет игра готова, то будет и модулей с расширениями побольше. Написал пару документов к ядру дабы вам было проще понять как работает поделка. Мне к примеру всё понятно и выглядит просто. Возможно потому что я это писал не так давно.

Интересует объективный взгляд со стороны.

https://github.com/vzglad-smerti/game

Answer 1

[Mikhail Osher]

Развидеть.

Comments

[v- smerti]

простите что?

[Олег Цилюрик]

v- smerti:
это он плохо закусывал ... к вечеру. ;-)

Answer 2

[Александр Вульф]

1. ошибки доставляют. Что такое dock? Корабельный док?
2. Версия jQuery адски старая. Обновить.
3. Доки в odt? Серьезно? Даже скачивать не стал.
4. За отступы в табах я, лично, убиваю. Только 2 пробела. Только хардкор.
5. Весь css в одном/двух файлах. Серьезно? И без препроцессора? Это ад, ей-богу.
6. Адовое | количество | ашибок
7. Названия методов То с Большой буквы, то С Маленькой. жуть в Общем.

За сим я остановил просмотр кода. Ужс, лично для меня. Я бы с этим работать не стал. Извините.

Comments

[v- smerti]

2. потдержка ie 7 и 8
3.исправлю
4.а как по мне то зря
5. то каша а не ксс случайно попало в репу. Я его разобью по модулям потом. https://github.com/vzglad-smerti/game/tree/master/...
6. всё исправил. Но а что делать с https://github.com/vzglad-smerti/game/blob/master/... не пойму

7. исправлено

[Александр Вульф]

v- smerti: logic - это логика. logik - непонятно что)

Answer 3

[moondogy]

1. jquery в 2015 - это не серьезно
2. либы в репозитории - это не серьезно.
Дальше не смотрел

Comments

[v- smerti]

я весь проэкт залил.

[Stalker_RED]

А можно аргументированно, что не так с jQuery в 2015?

[v- smerti]

Stalker_RED: я думаю он о версии которую я использую (1.11.3)

[moondogy]

Stalker_RED: не предназначен он для сложных фронтов, игр и т.д. Раньше он был чуть ли не единственным вариантом, сейчас появились более осмысленные фреймворки, заточенные больше на сложную логику, а не на манипуляции с DOMом. Конкретно что не так с jQuery - нету di, templating, структуры

[v- smerti]

moondogy: ну шаблонизатор уже написал. осталось понять что за структуры и di

[v- smerti]

moondogy: подождите. Структуры это же обычные обьекты. Я даже подобие бд написал https://github.com/vzglad-smerti/game/tree/master/...

[moondogy]

Сергей Сергеев: а это типичный ответ многостаночника, который шаблончики на cms-ки натягивает и ничего крупного и сложного сам не писал, и в командной разработке участия не принимал.
З.Ы. Ларавел и кохана - PHP фреймворки, а nodjs к фронтенду не имеет никакого отношения, интересно, к чему ты это написал?

Answer 4

[sim3x]

относительно стилей
https://youtu.be/UvyYsRMdsW8?t=1h20m31s

Поищи что такое gulp-grunt - ето удобно собирать кучу файлов с понятными именами в один еще и оптимизировать все ето налету

Посмотри как организован бутстрап - их ведь не зря любят

Ошибки в текстах коммитов доставляют

Comments

[v- smerti]

ошибки исправлю как только напишу исправлялку ошибок для терминала )))

[sim3x]

v- smerti: пиши на английском - так хоть отмазка будет

Answer 5

[Котик Антон]

1. Придерживание какого-либо codestyle отсутствует, смесь пробелов и табуляции.
   Начать можно с этого: events.yandex.ru/lib/talks/566, чтобы понять, почему это важно. Зачем learn.javascript.ru/coding-style. Там же есть ссылки на наиболее популярные стили. Выбираете любой и начинаете его придерживаться.
   
2. Не хватает комментариев в коде.
   Сейчас это может ещё не так критично, но когда кода станет на порядок больше, без комментариев станет совсем тоскливо. Главное не пуститесь в другую крайность - не начните комментировать совсем очевидные вещи или даже каждую строчку (бывают случаи).
   
3. В HTML5 есть множество тегов, которые следовало бы использовать: main, header, footer, ... (вместо <div id="header">, <div id="footer">), для кодировки <meta charset="utf-8">, для скриптов атрибут type="text/javascript" можно не писать.
   

Comments

[v- smerti]

1. где про это почитать?
2. сделаю

Answer 6

[abcyu]

Бывает и хуже. Терпимо. Надеюсь за поддержку хорошо платят. За маленькие денежки не хотелось бы ковырять.