[💬] 오브젝트 풀 개선점 의견 받음

[익명(220.70)]

using System.Collections.Generic;
using System.IO;
using System.Linq;
using UnityEngine;

/// <summary>
/// 간단한 범용 오브젝트 풀
/// </summary>
/// <typeparam name="T">풀에서 바로 꺼내서 사용할 컴포넌트 타입</typeparam>
public class SimpleObjectPool<T> where T : MonoBehaviour
{
private readonly Transform objHolder;
private readonly T targetPrefab;
private readonly List<T> usingPool = new List<T>();
private readonly List<T> sleepingPool = new List<T>();

public SimpleObjectPool(T objToUse, Transform objHolder, int preWarmCount = 5)
{
targetPrefab = objToUse;
this.objHolder = objHolder;
PreWarm(preWarmCount);
}

/// <summary>
/// 풀의 상태가 정상적인지 체크함
/// 만약 비정상적이라면 풀을 정리하는 기능도 겸함
/// </summary>
/// <returns>true : 정상</returns>
public bool CheckPoolStatus()
{
var ret = objHolder && targetPrefab;
if (ret) return true; //정상적이지 않은 경우 리스트를 정리함
usingPool.Clear();
sleepingPool.Clear();

return false;
}

/// <summary>
/// 오브젝트를 미리 추가로 생성할 수 있는 기능을 제공하는 method
/// </summary>
/// <param name="countToGenerate">미리 추가로 생성할 오브젝트 수</param>
public void PreWarm(int countToGenerate)
{
GenerateObjects(countToGenerate);
}

/// <summary>
/// 오브젝트를 명시적으로 풀에 반환함
/// </summary>
/// <param name="obj">sleeping pool에 반환할 오브젝트</param>
public void ReturnObject(T obj)
{
obj.gameObject.SetActive(false);
if (!usingPool.Remove(obj)) //풀에 의해 제어되지 않고 있는 오브젝트를 반환 시도하는 경우 예외 발생
throw new InvalidDataException();
sleepingPool.Add(obj);
}

/// <summary>
/// 오브젝트를 미리 생성함
/// </summary>
/// <param name="count"> 생성할 오브젝트 수 </param>
private void GenerateObjects(int count = 5)
{
for (var i = 0; i < count; i++)
{
var newObj = Object.Instantiate(targetPrefab.gameObject, objHolder).GetComponent<T>();
newObj.gameObject.SetActive(false);
sleepingPool.Add(newObj);
}
}

/// <summary>
/// 미사용 오브젝트를 가져와 활성화하고 반환함
/// </summary>
/// <param name="getNonActivated"> 오브젝트를 활성화되지 않은 체로 반환 </param>
/// <returns> 활성화된 오브젝트 </returns>
public T GetObj(bool getNonActivated = false)
{
if (sleepingPool.Count == 0)
{
if (!ClearUsingPool())
GenerateObjects();
}

var retObj = sleepingPool[0];
sleepingPool.Remove(retObj);
usingPool.Add(retObj);
if (!getNonActivated)
retObj.gameObject.SetActive(true);

return retObj;
}

private readonly List<T> tempList = new List<T>();

/// <summary>
/// 풀의 무결성을 유지하기 위해 미사용 중인 오브젝트가 사용 중 풀에 있는 경우 정리함
/// </summary>
/// <returns> 청소된 오브젝트가 있는지 (없으면 false, 있으면 true) </returns>
private bool ClearUsingPool()
{
tempList.Clear();
tempList.AddRange(usingPool.Where(obj => !obj.gameObject.activeSelf));
sleepingPool.AddRange(tempList);
usingPool.RemoveAll(tempList.Contains);
return tempList.Count != 0;
}
}

범용으로 만들어두고 잘 쓰고 있긴 하거든

근데 오브젝트 풀링 특성 상 안들어가는 프로젝트가 드물다 보니 더 괜찮게 다듬어보고 싶어서

의견 물어보러 왔음

추가 기능, 성능 상 개선점 의견 받음

인붕이들은 더 잘 만들어서 쓰고 있겠지만 쓰고 싶은 사람은 가져다 써라

[익명(221.140)]

checkPoolStatus랑 clearUsingPool 이상함. objectPool 자체가 문제가 없다면 필요없는데 함수인데? checkPoolStatus는 함수 이름과 다르게 오브젝트 풀을 비우는 역할이지 않나? 리스트안에 오브젝트가 들어있으면 리스트만 비우면 되는게 아니라 오브젝트 제거도 해야할텐데?

[익명(221.140)]

그리고 오브젝트풀을 사용할껌ㄴ monobehaviour을 그대로 상속받기보다 랩핑해서 쓰는게 나을 수도 있음

[익명(220.70)]

오브젝트 제거는 내가 놓쳤던 부분이 맞는 것 같네. 저게 씬이 변경됐을 때 스크립터블 오브젝트쪽에서 쓰는 풀에서 문제가 생긴 적이 있어서 넣어 놓은 거임. check에 정리까지 몰아 놓은 건 어짜피 비정상적인 상태라면 사용하지 못하기 때문에 함께 둔 건데 명확히 하려면 사실 분리 하는 게 맞긴 하지 ㅇㅇ.

[익명(220.70)]

clearUsingPool을 만들어 놓은 이유가 오브젝트를 사용하는 주체가 풀이 아니라 외부에 있기 때문에 사용하고 돌려놓는 책임을 사용하는 쪽에 뒀기 때문임. 그러다 보니 usingPool은 오브젝트를 리스트에 갖고있긴 하지만 이게 어떻게 사용되는지는 풀에서 판단할 방법이 없음. 그래서 필요한 함수임 오브젝트가 모자랄 때 새로 만드는 것 보다는 usingPool에 혹시라도 놀고 있는 오브젝트가 있으면 스캔해서 찾아 보고 그걸 사용하는 게 더 싸게 먹히기도 하고.

[익명(220.70)]

그리고 mono는 제네릭 한정자라 상속과 무관함

[익명(221.140)]

내가 회사 여러군데 다니면서 저런 짓 한적이 한번도 없고 하는 것도 본적이 없는데

[익명(221.140)]

mono는 예를들어 KMonoBehaviour : MonoBehaviour같이 프로그래머가 관리하는 모노 객체로 만들라는 의미였음. 오브젝트 풀을 사용한다는 것 자체가 프로그래머가 계속 관리해줘야하니까

[익명(220.70)]

나만 쓰는게 아니니까 일종의 보험이라고 생각하면 될 것 같다

[정공개발자(firetruck4545)]

where 키워드를 처음봤는데 덕분에 공부한다