Показать сообщение отдельно
Старый 01.02.2016, 21:33
Фенёк вне форума Посмотреть профиль Отправить личное сообщение для Фенёк Найти все сообщения от Фенёк
  № 12  
Ответить с цитированием
Фенёк

Регистрация: May 2011
Сообщений: 221
GBee
2 дня

Оооо, ребят, я тут получил фидбек, пощу тут

Цитата:
Ваш код написан не оптимально

отсутствует попытка использования пулов

алгоритмически не эффективная сортировка

отсутствие кэширований и отсечений

не оптимальная работа с тайлами - для подобных объектов, эффективнее использовать один класс рендерер, без создания объекта наследующего Sprite на каждый тайл

Изометрическая сортировка реализована некорректно. Прямоугольные объекты были не просто так - с ними нельзя использовать элементарную сортировку по глубине, т.к. возникнут артефакты в определенных случаях. Решениями являются к примеру использование топологической сортировки, попиксельный z buffer или нарезание объектов на вертикальные полосы сортирующиеся индивидуально

Странные архитектурные решения вроде менеджера ресурсов под каждый отдельный атлас

Нелогичная работа с изометрическими размерами, координатами, конвертированием между screen и iso space

Полностью отсутствует ощущение, что проект можно расширять: вбитые названия и отсутствие возможности разнообразить карту.
То есть похоже что люди ДАЖЕ НЕ ОТКРЫВАЛИ КОД. Ну или пробежались по нему очень нехотя. Короче, слов мне не хватает, чтобы выразить все эмоции, написал в ответ им гневное письмо.

Вобщем, господа, в компании ВОЛКА очевидно работают олигофрены, не только ленивые, но и не желающие разбираться в математике. Вы уж простите, за эмоциональный всплеск.