Окей.
1. Manager в названии класса BuffManager ни к месту. Это не Manager. Назови лучше просто Buff. А ActiveBuff переименуй в BuffState.
2. Метод BuffManager#getBuff переименуй в BuffManager#of. BuffManager.of(5) выглядит красивее, чем BuffManager.getBuff(5).
3. В методах onActive, isReady ты пробрасываешь через параметры tier, duration. Пробрасывай лучше весь ActiveBuff.
4. Почему BuffManager#applyBuffEffect и BuffManager#removeBuffEffect статические - мне непонятно.
5. BuffManager#getDurationForDisplay тоже почему-то статический. По идее, если нужен бесконечный баф, то можно было бы переопределить этот метод, чтобы выводить --:--.
6. То что майн и ты завязали текущий стейт на простом ActiveBuff - не очень удобно. А что если понадобится еще один динамический параметр? Решение - создавать ActiveBuff через BuffManager, т.е. немного изменить класс
public class BuffManager<T extends ActiveBuff> {
...
public T newState(int tier, int duration) {
return new ActiveBuff(id, tier, duration);
}
...
}
И естественно везде ActiveBuff заменить на T. Таким образом можно будет создавать кастомные стейты. Но тогда придется изменить readBuffFromNBT. Нужно будет сначала читать id, создавать ActiveBuff при помощи BuffManager, а затем вызывать ActiveBuff#readFromNBT.
7. private Map<Integer, ActiveBuff> activeBuffs сделай final.
8. В BuffsLivingBase #haveActiveBuffs нет смысла в проверке на null.
9. BuffsLivingBase#activeBuffsIdSet должен возвращать Set<Integer>.
10. BuffsLivingBase#activeBuffsCollection должен возвращать Set<ActiveBuff>.
11. Замени везде итераторы на foreach циклы.
for (ActiveBuff buff : activeBuffsCollection()) {
...
}
12. new BuffsOverlay(). Создай объект один раз, а потов вызывай какой-нибудь draw.
13. То что ты никогда не синхронишь кулдаун с сервера на клиент немного странно. Что если на сервере лаги? Тогда эффект на клиенте пропадет значительно раньше. Лаги на клиенте - аналогично. По факту эффект пропадет, но игрок будет думать, что баф наложен.
Это можно было бы оформить, как либу (или просто как пачку классов для прямого копипаста), сделать немного по-красивее и больше функционала. У меня есть к чему придраться. Если интересно, могу высказать.