Check for null pointer, good pratice ?

[The_E]

@BiggestSmile;131482

Well, it’s always better to check pointer before using, but

if (GEngine)
   GEngine->AddOnScreenDebugMessage(-1, 5.f, FColor::Yellow, TEXT("BeginPlay"));

But this piece of code is executed in actor’s BeginPlay function, as i see, there is no way GEngine could be null by this time, i believe

Belief is not knowledge, however. If the API contract states that a given value can be null, then it can be null at any time, and you have to check for that. Even if you think you know that it can’t be null at a given point in time, because you cannot be certain that that behaviour will stay that way in subsequent revisions.