Related to the IItemLoader interface

[ntl92bk]

I think the IItemLoader interface should be:

public interface IItemLoader<T> {
        T SpawnItem();
        void DeSpawnItem(T instance);
}

You should not pass a prefab into the spawn item method, it leaks the logic how to create an item (from a prefab)

[QuocHieuNguyen]

I think the IItemLoader interface should be:

public interface IItemLoader<T> {
        T SpawnItem();
        void DeSpawnItem(T instance);
}

You should not pass a prefab into the spawn item method, it leaks the logic how to create an item (from a prefab)

It's just the design preference, but I get your point, I will consider it.