Avoid using non-generic GetComponent
Prefer using the generic version of Component.GetComponent() for type safety.
How to fix
Refactor GetComponent(typeof(MyType)) calls in GetComponent<MyType>().
CQLinq source
This rule is written in CQLinq, C# LINQ over the NDepend code model. It ships with every new NDepend project: you can read it, adapt it, or use it as a starting point for your own custom rules.
// <Name>Avoid using non-generic GetComponent</Name>
// <Id>ND3201:AvoidUsingNonGenericGetComponent</Id>
warnif count > 0
from m in Application.Methods where m.IsUsing("UnityEngine.Component.GetComponent(Type)".AllowNoMatch())
select new {
m,
Debt = 8.ToMinutes().ToDebt(),
Severity = Severity.Medium
}
//<Expl>
// The *{0}* is using *UnityEngine.Component.GetComponent(Type)*.
// It should use instead the generic version of *Component.GetComponent()*
// for type safety.
//</Expl>
//<Desc>
// Prefer using the generic version of *Component.GetComponent()*
// for type safety.
//</Desc>
//<HowToFix>
// Refactor *GetComponent(typeof(MyType))* calls in *GetComponent<MyType>()*.
//</HowToFix>
Open in the interactive Rules Explorer · All default rules
API Usage / Unity: all 9 items
- Classes tagged with InitializeOnLoad should have a static constructor ND3200
- Avoid using non-generic GetComponent ND3201
- Avoid empty Unity message ND3202
- Avoid using Time.fixedDeltaTime with Update ND3203
- Use CreateInstance to create a scriptable object ND3204
- The SerializeField attribute is redundant on public fields ND3205
- InitializeOnLoadMethod should tag only static and parameterless methods ND3206
- Prefer using SetPixels32() over SetPixels() ND3207
- Don't use System.Reflection in performance critical messages ND3208