sdl2-demo icon indicating copy to clipboard operation
sdl2-demo copied to clipboard

Предложение некотрых изменений

Open besworland opened this issue 4 years ago • 1 comments

Добрый день,

решил быть несколько более конструктивным после нашей беседы на Хабре и предложить вам несколько изменений. По большому счёту основные изменения лежат в области переработки кода класса Animation, который, по моему мнению, не должен наследовать от Sprite. Вместо этого в структуре player отдельно создаются необходимые объекты типа Sprite, а затем часть информации из них используется для инициализации Animation. Логика этого изменения предельно проста: Анимация не является Sprite, но Sprite может быть анимирован. Изменения не критические и не несут большое значение, но с точки зрения архитектуры имеют смысл.

Буду рад услышать ваше мнение :) С уважением Сергей

besworland avatar Sep 22 '21 12:09 besworland

Добрый день, так, это интересно, спасибо. Часть ваших изменений полностью соответствует тому, что я уже закоммитил в свой локальный репозиторий :)

Другую часть изменений я должен посмотреть внимательнее, прямо сейчас времени нет. Идея отделить анимации от спрайтов разумная, надо обдумать.

Спасибо!

ssloy avatar Sep 22 '21 12:09 ssloy