[💬] 뭐가 더 좋은 접근법일까

[ints*(neinnew)]

public static bool IsComponentMissing<T>(this MonoBehaviour monoBehaviour, T component, out T getComponent) where T : Component
{
if (component == null)
{
Debug.LogWarning("Component " + typeof(T) + " is missing. Adding it now.");
getComponent = monoBehaviour.GetComponent<T>();
return true;
}
else
{
getComponent = null;
return false;
}
}

public static void EnsureExist<T>(T component, Action ensuring) where T : Component
{
if (component != null) return;

Debug.LogWarning("Component " + typeof(T) + " is missing. Adding it now.");
ensuring.Invoke();
}

public static T EnsureExist<T>(this MonoBehaviour monoBehaviour, T component) where T : Component
{
if (component != null) return component;

Debug.LogWarning("Component " + nameof(T) + " is missing. Adding it now.");
return monoBehaviour.GetComponent<T>();
}

public static void EnsureExist<T>(this MonoBehaviour monoBehaviour, ref T component) where T : Component
{
if (component != null) return;

Debug.LogWarning("Component " + nameof(T) + " is missing. Adding it now.");
component = monoBehaviour.GetComponent<T>();
}

void Awake()
{
// 1
if (this.IsComponentMissing(_rigidbody, out var getComponent)) _rigidbody = getComponent;

// 2
Utils.EnsureExist(_rigidbody, () => _rigidbody = GetComponent<Rigidbody>());

// 3
_rigidbody = this.EnsureExist(_rigidbody);

// 4
this.EnsureExist(ref _rigidbody);
}

1. If문이 함수 밖으로 나와서 길어지거나 중괄호가 필요함 

2. 파라미터를 일일히 작성해주긴 뭔가 그럼

3. 이미 있는 경우에도 대입하는게 거슬림

4. ref를 쓰는게 그냥 아니꼬움

평가좀 ㄱㄱ

[ㅇㅇ 1(211.197)]

3번 아님 ref 말고 out 쓰는 건 어떰? - dc App

[ints*(neinnew)]

4번에는 out을 못씀

[ㅇㅇ 1(211.197)]

사실 폰이라 코드가 제대로 안보임 ㅋㅋ - dc App

[ㅇㅇ 1(211.197)]

아 확장메서드네 그럼 어쩔 수 없지. 3, 4번 두개 선택할게. - dc App

[ㅇㅇ 2(59.19)]

찐따

[ㅇㅇ 3(112.146)]

5. _rigidbdy ??= GetComponent();

[ints*(neinnew)]

이게 깔끔하긴 한데 유니티엔진 오브젝트엔 ??= fake null땜에 못씀... 안에 다른거 못넣는것도 있기도 하고

[ㅇㅇ 3(112.146)]

GetComponent(typeof(Rigidbody));