ReSharper
 
Get ReSharper
You are viewing the documentation for an earlier version of ReSharper.

Code Inspection: Safe cast expression always succeeds, consider direct cast instead

Last modified: 21 July 2022

To safely cast a reference variable from a derived type to a base type or vice versa, you can use as operator. Safely means that the code will not throw an exception, but rather a variable to which you assign a value will be assigned null if the cast cannot be performed.

If the types are compatible, a safe cast using as would always succeed, so in this case the explicit cast would be enough as redundant casts might decrease performance.

A cast from base to derived type might be redundant if the variable of a base type was already checked for compatibility with the derived type. In the example below the assignment of string str is only possible when obj is of the type string and it is not null. So we can either use a direct cast or rewrite this code to separate the cast from null-checking.

When a safe cast is used for a conversion from a base to derived type, then it is not necessary at all (the compiler will implicitly perform the cast anyway) and can be safely removed.