Привести в соответствие документацию и код.

[i-pavlov]

Привести в соответствие документацию и код.

[nzinov]

Переименовал страницу "Объекты системы" в "Методы API" и убрал оттуда всякие сервисные запросы и.т.п. Теперь там что-то действительно похожее на API

[i-pavlov]

Мне кажется, что код пересчета статистики утратил свою читаемость. (500 строк все же)

Возможно, стоит сделать небольшой рефакторинг и сгруппировать все по разным классам и разным файлам?

[denspb]

Вполне.
Но, если не сложно, опиши сперва в паре десятков слов как ты видишь
результат?

30 апреля 2014 г., 22:59 пользователь Ivan Pavlov
[email protected]написал:

Мне кажется, что код пересчета статистики утратил свою читаемость. (500
строк все же)

Возможно, стоит сделать небольшой рефакторинг и сгруппировать все по
разным классам и разным файлам?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/2#issuecomment-41835592
.

[i-pavlov]

Ок
Но, над этим мне надо еще подумать.

[nzinov]

Мне не кажется, что там проблемы с читаемостью. Правда, это наверное потому, что я сам писал.) Что мне кажется проблемой: некоторые обработчики в нем не относятся к пересчету:
запуск пересчета и страница запуска. Их можно вынести. Еще, теоретически, можно вынести из main flow пересчета игры несколько методов (распознавание тестовой игры, например). Но зачем разбивать на классы, я не знаю.

[i-pavlov]

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

В дальнейшем мы будем добавлять еще статистику и найти что-то определенное будет еще сложнее.