Engine Crashing on if Statements??

Development Programming & Scripting C++
, , ,
1

I don’t know why this is happening cause the code works for a while but after the functions get called a few times the engine crashes. Both functions are input functions.
Crashes on these line:

if (LockedOnWidget->GetWidgetClass() != nullptr)

LockedOnWidget is a UWidgetComponent*

if (LockedOnActor != nullptr)

LockedOnActor is an AActor*

Any suggestions or reasons for this would be appreciated.:slight_smile:
Thanks in advance.

2

Did you try

if (LockedOnActor)
3

Yes.

1 Like
4

Strange…

5

Any suggestions?

6

Maybe a silly question but are you checking them in the right order?

if (LockedOnActor != nullptr){
   if (LockedOnWidget->GetWidgetClass() != nullptr){
       // extra code here
   }
}

You need to be sure LockedOnActor is ok before accessing it’s GetWidgetClass method.

1 Like
7

Oh sorry forgot to mention they’re in different cpp files. LockedOnActor* check(see if its been set) is in the character class and is used to access a function in the pawn class that uses LockedOnWidget->GetWidgetClass() check(see if its been set).

8

UPROPERTY()
UObject *LockedOnWidget;

(garbage collector)

9

wdym.

10

Did you declared your UObject pointer using UPROPERTY()?

11

do it

12

I did from the start.

13

The most likely thing is the object containing your comparison is dodgy. What is the this ptr for the object as it crashes? What is the flow to the function that is crashing? as perhaps the object has been GC’d but still being trigger by a timer or somthing.

14

so… is not the garbaje collector then…

¿Your widget is a blueprint derived from a C++ class?

Maybe it’s not instantiated.

Try to get the class like this:

TSubclassOf< UMyClass > WidgetClass;

And set it using the blueprint

15


Here’s an example of what’s happening.

16

You are using LockedOnWidget in two diferent classes… is LockedOnWidget a global variable or…?

17

Those two classes are different universes… nothing you do in one will affect the other.

18

Try declaring LockedOnWidget in the GameInstace… then you can access to the same variable in diferent clasess (it will be global)

19

Its a component on a pawn.

20

Look into what casting is and then you’ll understand what I’m doing.