I already tried that | Official Unity Discord | Page 1

umbral yarrow Jun 20, 2022, 7:23 PM

#

Can you show me all your code related to this? I need to see where you're adding the listener and where Amount() belongs.

rapid estuary Jun 20, 2022, 7:23 PM

#

alrigt

#

the whole file?

umbral yarrow Jun 20, 2022, 7:23 PM

#

yes.

rapid estuary Jun 20, 2022, 7:24 PM

#

https://pastebin.com/QiKZCtrA

Pastebin

using UnityEngine;using UnityEngine.UI;using System;using System.Co...

Pastebin.com is the number one paste tool since 2002. Pastebin is a website where you can store text online for a set period of time.

#

Everything worked literally yesterday

umbral yarrow Jun 20, 2022, 7:25 PM

#

confirm.onClick.RemoveListener(() => { Amount(); });

rapid estuary Jun 20, 2022, 7:25 PM

#

Mhm

#

What about it?

umbral yarrow Jun 20, 2022, 7:26 PM

#

So if you run this code right now, "test 2" never gets printed to the console?

rapid estuary Jun 20, 2022, 7:27 PM

#

umbral yarrow So if you run this code right now, "test 2" never gets printed to the console?

nope

umbral yarrow Jun 20, 2022, 7:28 PM

#

Does React get called?

rapid estuary Jun 20, 2022, 7:28 PM

#

Obviously

umbral yarrow Jun 20, 2022, 7:30 PM

#

hmmm

#

i am really not sure...

#

Why can't you just

#

in the inspector, assign the button's onclick event to Amount()?

rapid estuary Jun 20, 2022, 7:32 PM

#

umbral yarrow in the inspector, assign the button's onclick event to Amount()?

hard to explain

#

one of the reasons is that the gameobjects that call it are clones

#

it's just that I can't

#

It's the only solution and I have no clue why it doesn't work and I'm so fucking pissd kill me

umbral yarrow Jun 20, 2022, 7:37 PM

#

okay one more thing to try

rapid estuary Jun 20, 2022, 7:37 PM

#

wait I will do some analysis since I got the old version of the file as well

umbral yarrow Jun 20, 2022, 7:38 PM

#

void Test(string someText) { Debug.Log(someText); }
...
button.AddListener(delegate { Test("Hello World"); });

#

does this work?

rapid estuary Jun 20, 2022, 7:41 PM

#

umbral yarrow ```csharp void Test(string someText) { Debug.Log(someText); } ... button.AddList...

no

umbral yarrow Jun 20, 2022, 7:45 PM

#

yeah i'm not sure then... you could try making a small dummy script that just forwards calls to Amount() and set the OnClick event in the inspector to call the dummy script.

rapid estuary Jun 20, 2022, 7:46 PM

#

im having a meltdown rn

#

why

#

you think it might be an unity bug?

umbral yarrow Jun 20, 2022, 7:53 PM

#

Wait you're calling React() everyframe?

#

That means you're adding a listener, every. single. frame?

rapid estuary Jun 20, 2022, 7:53 PM

#

no

#

fucking

#

this bot sucks

#

if this is met ^

umbral yarrow Jun 20, 2022, 7:54 PM

#

Are you trying to make it so the player can only click the button if a condition is met?

rapid estuary Jun 20, 2022, 7:57 PM

#

yes

#

and till yesterday it worked

#

like holy shit the only thing I changed is that I fixed some UI bugs nothing else

umbral yarrow Jun 20, 2022, 7:57 PM

#

Why not just set button.interactable to false when they shouldn't be able to click it?

#

OR

#

Always let them click it, but check the condition in Amount() ?

rapid estuary Jun 20, 2022, 7:58 PM

#

but it worked before

#

I don't know what made it not work

#

and I'm furious

umbral yarrow Jun 20, 2022, 7:59 PM

#

are you here to solve your problem or continue to be mad and solve nothing?

rapid estuary Jun 20, 2022, 7:59 PM

#

hold on let me restart unity it looks like a unity problem tbh

#

nope

umbral yarrow Jun 20, 2022, 8:02 PM

#

If you're not willing to take a different approach then I'm not sure how to help you any further

rapid estuary Jun 20, 2022, 8:02 PM

#

umbral yarrow If you're not willing to take a different approach then I'm not sure how to help...

look I cant just do it the unity way using OnClick()

#

it's hard to explain

#

I require this

#

fucking hell

#

@umbral yarrow would you mind if I sent you the entire file

umbral yarrow Jun 20, 2022, 8:09 PM

#

Haven't you already in that pastebin?

rapid estuary Jun 20, 2022, 8:09 PM

#

Not file

#

Project

#

The entire unity project

#

I have no fucking clue whats wrong

umbral yarrow Jun 20, 2022, 8:11 PM

#

uh no. I'm not interested in downloading your unity project, sorry.

rapid estuary Jun 20, 2022, 8:36 PM

#

ok why isn't it working I have no clue

umbral yarrow Jun 20, 2022, 8:43 PM

#

@onyx ferry there's a thread for his issue already here.

#

@iron cloak you too.

#

they've linked their code above

iron cloak Jun 20, 2022, 8:49 PM

#

Amount() is quite a big function that could die in a couple of ways. If there's an error in there, it might be silently dying without finishing the function. Try this. Add this debug log right before you call the AddClick:

Debug.Log($"Adding a listener to {confirm.gameObject.name}");

and then inside of Amount(), put in this log and comment everything else out. Let's make sure it can call the function first.

Debug.Log("Amount has been executed");

rapid estuary Jun 20, 2022, 8:49 PM

#

iron cloak `Amount()` is quite a big function that could die in a couple of ways. If there'...

it doesn't even return test2 at the top

onyx ferry Jun 20, 2022, 8:50 PM

#

Update, Instance IDs seem to be different between Inspector and code, so betting on a bad reference

iron cloak Jun 20, 2022, 8:50 PM

#

rapid estuary it doesn't even return test2 at the top

If it's running in another thread and it dies then it will never report back to the main thread that it needs to log something

#

I don't know how this is running, but I'm making a wild guess

rapid estuary Jun 20, 2022, 8:51 PM

#

onyx ferry Update, Instance IDs seem to be different between Inspector and code, so betting...

how is this even possible is beyond me

onyx ferry Jun 20, 2022, 8:51 PM

#

How are you getting a reference to the button?

rapid estuary Jun 20, 2022, 8:52 PM

#

using public Button confirm;

#

I attached the button to the script, yes

onyx ferry Jun 20, 2022, 8:53 PM

#

Okay, so create another field of the same type, drag-drop the button (make sure it's the correct one), and in the code, log if confirm == subject where subject is the field you just created.

#

Prints out false? You have a bad ref

rapid estuary Jun 20, 2022, 8:53 PM

#

alright

#

wait

#

okay so the ID is the same when it comes to button

#

button component ID: 20138

#

it's the same

#

my bad

#

it doesn't explain why it doesn't work though

onyx ferry Jun 20, 2022, 8:55 PM

#

Does some of the code run on another thread? If you're not dealing with threads new Thread, async/await, it's a no

#

(coroutines are not threaded)

rapid estuary Jun 20, 2022, 8:56 PM

#

nope, I'm not

onyx ferry Jun 20, 2022, 8:56 PM

#

So, right before you add the listener, throw new Exception("Test"), same thing in the Amount method

#

If you don't get any exception, then the code that adds the listener isn't running

#

(disregard any "Unreachable code" warnings this might create, after adding the throw instructions)

rapid estuary Jun 20, 2022, 8:59 PM

#

it does throw the first exception (before the listener)

iron cloak Jun 20, 2022, 8:59 PM

#

onyx ferry So, right before you add the listener, `throw new Exception("Test")`, same thing...

Ah, debugging by destruction. The last most powerful tool of a confused developer

iron cloak Jun 20, 2022, 9:00 PM

#

rapid estuary it does throw the first exception (before the listener)

Okay, so it reaches that code and is definitely adding the listener. Now remove it and try to get the error to come from the first line of Amount()

rapid estuary Jun 20, 2022, 9:02 PM

#

when I click the button no exception

#

nothing

iron cloak Jun 20, 2022, 9:02 PM

#

Okay, so then the code is definitely quietly dying

#

Comment out everything that isn't the throw

#

/* after the throw
*/ right before the end of the function

onyx ferry Jun 20, 2022, 9:04 PM

#

After that verification (what digi is saying), you might need to explore if it actually adds the listener. Maybe some silenced error makes it fail?
You'll need to log confirm.onClick.GetPersistentEventCount() both before and after you add the listener. It should be printing 0, 1.

rapid estuary Jun 20, 2022, 9:06 PM

#

still no exception

onyx ferry Jun 20, 2022, 9:06 PM

#

The code snippet in https://docs.unity3d.com/ScriptReference/Events.UnityEventBase.GetPersistentListenerState.html will print out info about each registered listener, use wisely

iron cloak Jun 20, 2022, 9:07 PM

#

rapid estuary still no exception

Even when the only thing in Amount() is the throw?

rapid estuary Jun 20, 2022, 9:07 PM

#

yes

iron cloak Jun 20, 2022, 9:08 PM

#

How about changing the add listener line to confirm.onClick.AddListener(() => throw new Exception("Test"));

#

Actually hang on

confirm.onClick.AddListener(() => { Amount(); });

I think this should be

confirm.onClick.AddListener(() => Amount());

#

Has that been tried yet?

onyx ferry Jun 20, 2022, 9:08 PM

#

I have another destructive debug method: making the button reference null after you add the listener, so it catches any modifications made to the button

umbral yarrow Jun 20, 2022, 9:08 PM

#

oh god was this a anonymous scoping issue the whole time

onyx ferry Jun 20, 2022, 9:09 PM

#

That shouldn't be an issue, both should compile to the same thing

rapid estuary Jun 20, 2022, 9:09 PM

#

iron cloak Actually hang on ```cs confirm.onClick.AddListener(() => { Amount(); }); ``` I _...

yes

umbral yarrow Jun 20, 2022, 9:10 PM

#

Does this work: #988524198362239106 message ?

rapid estuary Jun 20, 2022, 9:10 PM

#

also, the exception was thrown

#

confirm.onClick.AddListener(() => throw new Exception("Test 3"));

rapid estuary Jun 20, 2022, 9:11 PM

#

umbral yarrow Does this work: https://discord.com/channels/489222168727519232/9885241983622391...

yes

umbral yarrow Jun 20, 2022, 9:11 PM

#

yeah i figured it would

#

hmmm

onyx ferry Jun 20, 2022, 9:11 PM

#

Three possible syntaxes, have you tried them all?
AddListener(() => { Amount(); });
AddListener(() => Amount());
AddListener(Amount);

rapid estuary Jun 20, 2022, 9:11 PM

#

let's try again

umbral yarrow Jun 20, 2022, 9:12 PM

#

If those 3 don't work, I think we try just writing a simple script that has nothing in it other than the code we're focused on fixing

rapid estuary Jun 20, 2022, 9:13 PM

#

no

#

no

#

no

#

the audacity of this bitch

onyx ferry Jun 20, 2022, 9:14 PM

#

Proceed with <#988524198362239106 message>
And <#988524198362239106 message>

rapid estuary Jun 20, 2022, 9:15 PM

#

I will tommorow if you don't mind. I already spend 8 hours working and got school tommorow.

#

and it's already very late

umbral yarrow Jun 20, 2022, 9:15 PM

#

damn, i was hoping to have a conclusion to this one 😦

#

i was stumped too

rapid estuary Jun 20, 2022, 9:16 PM

#

we'll try again tommorow

#

good night

onyx ferry Jun 20, 2022, 9:18 PM

#

Things to try for tomorrow
Print out listener info
Isolate the issue by writing a new, simple script with a button and a listener to see if the issue lies in Amount or outside of it
Check if the listener doesn't get cleared in between the add and the event raise

rapid estuary Jun 21, 2022, 2:28 PM

#

morning

rapid estuary Jun 21, 2022, 2:29 PM

#

onyx ferry --- Things to try for tomorrow Print out listener info Isolate the issue by writ...

when it comes to printing out listener info, do I make a new UnityEvent or what

#

or something along the lines of:

#

https://pastebin.com/UsxP2j4E

Pastebin

for (int i = 0; i < confirm.GetPersistentEventCount(); + - Pastebin...

Pastebin.com is the number one paste tool since 2002. Pastebin is a website where you can store text online for a set period of time.

#

and I wrote this small script which works

#

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;

public class Test : MonoBehaviour
{

    public Button confirm;

    void Test2(string someText) { Debug.Log(someText); }
    void Start()
    {
        confirm.onClick.AddListener(delegate { Test2("Hello world"); });
    }
}

#

when it comes to clearing the listener i dont think so

onyx ferry Jun 21, 2022, 4:05 PM

#

onyx ferry The code snippet in <https://docs.unity3d.com/ScriptReference/Events.UnityEventB...

To print listener info you take inspiration of the code in that page

#

That's what you have in the pastebin

#

So run that loop after you add the listener and see what it says

rapid estuary Jun 21, 2022, 4:54 PM

#

onyx ferry To print listener info you take inspiration of the code in that page

yeah but confirm is a button

onyx ferry Jun 21, 2022, 4:54 PM

#

confirm.onClick

#

Is the event

rapid estuary Jun 21, 2022, 4:56 PM

#

tried that

#

doesn't work

#

UnityEngine.Component.GetComponent[T] () (at <0ee480759f3d481d82ada245dc74f9fd>:0)
InteractionRadius.React (UnityEngine.Vector3 center, System.Single radius) (at Assets/Scripts/InteractionRadius.cs:361)
InteractionRadius.Update () (at Assets/Scripts/InteractionRadius.cs:431)```

#

OH WAIT

#

huh

#

ok so error solved but it doesn't debug anything

#

like it doesn't log anything

#

https://pastebin.com/18yfegvD

Pastebin

confirm.onClick.AddListener(Amount); - Past...

Pastebin.com is the number one paste tool since 2002. Pastebin is a website where you can store text online for a set period of time.

onyx ferry Jun 21, 2022, 5:01 PM

#

Log confirm.onClick.GetPersistentEventCount() just before the loop

rapid estuary Jun 21, 2022, 5:03 PM

#

0

onyx ferry Jun 21, 2022, 5:04 PM

#

Add a dummy listener and retry

#

Not sure it'll even work, those methods get persistent listeners, the ones added at runtime might not be persistent

rapid estuary Jun 21, 2022, 5:08 PM

#

still no

onyx ferry Jun 21, 2022, 5:09 PM

#

So these aren't persitent

#

Onto the next step

#

Remove the dummy listener, and right after you add the listener that doesn't work, set confirm to null

#

Then play the game. As soon as you get a NullRef, stop the game and inspect the line it got thrown on to see if you're messing with the listeners at that point.
If you're not, then put the confirm = null after the point the exception was thrown, and repeat the steps

rapid estuary Jun 21, 2022, 5:11 PM

#

so like

#

ugh

#

FUCKING HELL

onyx ferry Jun 21, 2022, 5:12 PM

#

confirm.onCLick.AddListener(Amount);
confirm = null;

#

The troubleshooting stops when you don't get NullRefs anymore

rapid estuary Jun 21, 2022, 5:13 PM

#

...i'm not getting any NullRefs at the first step

onyx ferry Jun 21, 2022, 5:14 PM

#

So you're not modifying the button afterwards

#

That is the worst possible outcome

#

The listener gets removed silently at some point

#

Or, it's accessed from another script. Try: DestroyImmediate(confirm), then confirm = null;, and see if you get any errors

rapid estuary Jun 21, 2022, 5:18 PM

#

onyx ferry Or, it's accessed from another script. Try: `DestroyImmediate(confirm)`, then `c...

without adding the listener?

onyx ferry Jun 21, 2022, 5:18 PM

#

It won't make any difference, but you can add it

rapid estuary Jun 21, 2022, 5:20 PM

#

InteractionRadius.React (UnityEngine.Vector3 center, System.Single radius) (at Assets/Scripts/InteractionRadius.cs:361)
InteractionRadius.Update () (at Assets/Scripts/InteractionRadius.cs:432)```

#

Debug.Log(confirm.onClick.GetPersistentEventCount());

#

should I comment it

onyx ferry Jun 21, 2022, 5:21 PM

#

No, you move the DestroyImmediate and confirm = null below that line

rapid estuary Jun 21, 2022, 5:22 PM

#

ok

rapid estuary Jun 21, 2022, 5:23 PM

#

onyx ferry The code snippet in <https://docs.unity3d.com/ScriptReference/Events.UnityEventB...

along side this, correct?

onyx ferry Jun 21, 2022, 5:23 PM

#

After the line that threw the exception

#

The log

rapid estuary Jun 21, 2022, 5:26 PM

#

NullReferenceException: Object reference not set to an instance of an object
InteractionRadius.React (UnityEngine.Vector3 center, System.Single radius) (at Assets/Scripts/InteractionRadius.cs:362)
InteractionRadius.Update () (at Assets/Scripts/InteractionRadius.cs:432)

#

NullReferenceException: Object reference not set to an instance of an object
InteractionRadius.React (UnityEngine.Vector3 center, System.Single radius) (at Assets/Scripts/InteractionRadius.cs:359)
InteractionRadius.Update () (at Assets/Scripts/InteractionRadius.cs:432)

#

line 362: for (int i = 0; i < confirm.onClick.GetPersistentEventCount(); ++i)

#

Debug.Log(confirm.onClick.GetPersistentEventCount()); line 359

onyx ferry Jun 21, 2022, 5:27 PM

#

Move the destruction and the set to null after line 362

#

Outside of the for loop (after the loop)

rapid estuary Jun 21, 2022, 5:30 PM

#

no errors now

#

https://pastebin.com/WRRx9K2A

Pastebin

for (int i = 0; i < confirm.onClick.GetPersistentEventCount(); ++i)...

Pastebin.com is the number one paste tool since 2002. Pastebin is a website where you can store text online for a set period of time.

onyx ferry Jun 21, 2022, 5:31 PM

#

This is not after the for loop...

rapid estuary Jun 21, 2022, 5:31 PM

#

oh after ok

#

my baf

#

dd

#

still nothing

onyx ferry Jun 21, 2022, 5:32 PM

#

Alright. You said you can't start the debugger right?

rapid estuary Jun 21, 2022, 5:33 PM

#

yes

onyx ferry Jun 21, 2022, 5:34 PM

#

So, make a new object with a button on it, and add Amount as the listener like you're already doing with confirm

#

You'll just have to declare another variable to reference the temporary button

rapid estuary Jun 21, 2022, 5:45 PM

#

still fucking nothing

onyx ferry Jun 21, 2022, 5:46 PM

#

I'm afraid I have nothing else to suggest

#

Apart from deleting the entire object, re-creating it and linking the references again

rapid estuary Jun 21, 2022, 5:47 PM

#

object as in?

onyx ferry Jun 21, 2022, 5:47 PM

#

The one that has the button, and the one that has the script that adds a listener, if they're on different objects

rapid estuary Jun 21, 2022, 5:47 PM

#

I see. Thanks for your time though

rapid estuary Jun 21, 2022, 5:58 PM

#

onyx ferry The one that has the button, and the one that has the script that adds a listene...

At least I'm not the only one stumped haha

#

what the fuck

#

@onyx ferry you know what

#

I might as well pay you if you help me

onyx ferry Jun 21, 2022, 6:02 PM

#

I'm out of ideas

rapid estuary Jun 21, 2022, 6:02 PM

#

I might send you the entire project ig

#

I mean the only thing I changed

#

is some UI related stuff, aka scale etc

#

nothing

#

fucking

#

else

onyx ferry Jun 21, 2022, 6:03 PM

#

I would have suggested checking if you can inspect the event with the debugger, to see if you have access to the listener collections, but it doesn't work on your side

rapid estuary Jun 21, 2022, 6:04 PM

#

so it might be a unity bug?

#

I fucking hate my life

onyx ferry Jun 21, 2022, 6:16 PM

#

What Unity version are you on?

rapid estuary Jun 21, 2022, 6:28 PM

#

2020.3.25f1

onyx ferry Jun 21, 2022, 6:31 PM

#

I have 2020.3.3f1 installed on my side. I'll test some stuff

rapid estuary Jun 21, 2022, 6:32 PM

#

alright

#

can I send you the file including the project?

onyx ferry Jun 21, 2022, 6:36 PM

#

Yeah why not, at this point

rapid estuary Jun 21, 2022, 6:36 PM

#

ok DM since you know, private/confidential stuff

#

I mean not that it's secret

#

it's the files

#

you get the idea

onyx ferry Jun 21, 2022, 6:36 PM

#

Sure

onyx ferry Jun 21, 2022, 7:01 PM

#

Okay it's open, I'll need you to guide me through it

rapid estuary Jun 21, 2022, 7:01 PM

#

Alright

#

The issue is within the InteractionRadius.cs file

#

it is located within the Scripts folder

#

The button which the listener "operates" on is located in the following hierarchy:

#

EventSystem > Canvas > InputAmount > Button

onyx ferry Jun 21, 2022, 7:05 PM

#

Line 311 of InteractionRadius was for debugging purposes right

rapid estuary Jun 21, 2022, 7:05 PM

#

I have unity closed

#

which line is it

#

wait nvm I have it open lmao

onyx ferry Jun 21, 2022, 7:06 PM

#

confirm.onClick.RemoveListener(Amount);, it's in Amount so it shouldn't interfere

#

Unless Amount is called manually, which isn't the case

rapid estuary Jun 21, 2022, 7:06 PM

#

No it isn't for debugging purposes

#

It worked before though

onyx ferry Jun 21, 2022, 7:09 PM

#

Alright now, how do I get to that button in game?

rapid estuary Jun 21, 2022, 7:10 PM

#

here

#

onyx ferry Jun 21, 2022, 7:17 PM

#

Getting exceptions when trying to instantiate an element from the periodic table

#

Investigating...

#

ArgumentException: method arguments are incompatible

rapid estuary Jun 21, 2022, 7:18 PM

#

wait did I send you the database

rapid estuary Jun 21, 2022, 7:18 PM

#

onyx ferry `ArgumentException: method arguments are incompatible`

Weird

onyx ferry Jun 21, 2022, 7:18 PM

#

It printed out elements to the console when starting the game, so I assume that was the database

rapid estuary Jun 21, 2022, 7:18 PM

#

yes

#

I never had this error though

onyx ferry Jun 21, 2022, 7:19 PM

#

I have a lot of random ones lol

rapid estuary Jun 21, 2022, 7:19 PM

#

Oh boy

onyx ferry Jun 21, 2022, 7:19 PM

#

Editor throwing, shader graph dying etc

rapid estuary Jun 21, 2022, 7:19 PM

#

May I see them?

#

shader graph out of everything else

#

fucking wonderful

onyx ferry Jun 21, 2022, 7:20 PM

#

It's not runtime, so I assume this doesn't cause issues

rapid estuary Jun 21, 2022, 7:20 PM

#

ah

#

Wait did you open the project or did it open straight up the main "Screen"?

#

Like did it open the menu first

#

but that wouldn't change much.

onyx ferry Jun 21, 2022, 7:22 PM

#

I opened the project by double clicking the only scene file present in Assets/Scenes/

#

Okay managed to fix the instantiate button error

rapid estuary Jun 21, 2022, 7:23 PM

#

what caused it?

onyx ferry Jun 21, 2022, 7:24 PM

#

There was a handler in the On Click inspector list that was just empty lol

#

No idea what caused it, removed it and error went away

#

I have the dreaded "Confirm" button in sight now

#

It says "Hello World" when I click it

rapid estuary Jun 21, 2022, 7:25 PM

#

Yeah it was debug thingy

#

and nothing else happens?

onyx ferry Jun 21, 2022, 7:25 PM

#

Nope, the screen doesn't go away

rapid estuary Jun 21, 2022, 7:25 PM

#

let me double check if it has the proper handler just in case

onyx ferry Jun 21, 2022, 7:27 PM

#

It's added in the InteractionRadius:React method

#

Debugging to see if that executes

umbral yarrow Jun 21, 2022, 7:29 PM

#

this still didn't get solved? D:

rapid estuary Jun 21, 2022, 7:30 PM

#

umbral yarrow this still didn't get solved? D:

I told you it's cursed

#

I can feel the evil from this script ten miles away

#

it's the soviets slowing down the progress on western science

#

and yes it's the same listener

#

why the fuck it doesn't work

onyx ferry Jun 21, 2022, 7:32 PM

#

The listener gets added to the internal listeners array of the onclick event

#

But doesn't get called

#

Debugger never hits the breakpoint

rapid estuary Jun 21, 2022, 7:35 PM

#

rip

umbral yarrow Jun 21, 2022, 7:35 PM

#

have you tried just remaking the button/prefab?

rapid estuary Jun 21, 2022, 7:35 PM

#

yes

#

did absolute jack shit

onyx ferry Jun 21, 2022, 7:38 PM

#

Okay progress

#

Manually invoking the event in code runs the handler

rapid estuary Jun 21, 2022, 7:39 PM

#

yayyy

#

Okay so I gotta go guys. See you tommorow!

onyx ferry Jun 21, 2022, 8:03 PM

#

Ahh I think I got it

umbral yarrow Jun 21, 2022, 8:04 PM

#

oh please do share

onyx ferry Jun 21, 2022, 8:05 PM

#

The handler does not run if this is captured in the lambda expression, which means this doesn't exist anymore when the event is raised. And surely, the whole object is destroyed right after the UI button is shown

#

Case solved

umbral yarrow Jun 21, 2022, 8:06 PM

#

that makes sense, but how is this being captured in a statement like

button.onClick.AddListener(() => Amount()); ?

onyx ferry Jun 21, 2022, 8:07 PM

#

Amount is a method that's declared on the MonoBehaviour, so to resolve the scope it has to capture the whole class

umbral yarrow Jun 21, 2022, 8:08 PM

#

So what would be the recommended fix there?

onyx ferry Jun 21, 2022, 8:08 PM

#

And commenting out the two Destroy calls run the handler!

#

Have a separate script to run the handler on, that never gets destroyed

umbral yarrow Jun 21, 2022, 8:09 PM

#

ah, that makes complete sense

#

good detective work @onyx ferry 👍

onyx ferry Jun 21, 2022, 8:10 PM

#

let's gooo

onyx ferry Jun 21, 2022, 8:59 PM

#

So, full explanation of the story and how to fix.

I started putting breakpoints pretty much everywhere to see how the execution flows through the program.
You can place chemistry elements on a table, and fill these elements with either raw chemical elements, or compounds (like water). Then, if you combine two containers (say one flask of Na and one of H2O), it asks you how many there are to combine, so it does the reaction realistically with the stoichiometry and whatnot. The issue lies there, when you click on the "Confirm" button to combine the two containers into one, the handler does not get run.

After that I focused on the handler addition that worked half of the time, literally. The following worked
button.onClick.AddListener(() => Debug.Log("test"))
While that one didn't
button.onClick.AddListener(() => Amount())

So I moved on and tried this: `() => Debug.Log(this.gameObject.name)`, and suddenly it wasn't getting called anymore. That's pretty much where it clicked: if I included a reference to the script or the object itself, it stopped logging. In other words, if the lambda captured `this` in its scope, nothing would happen.
The only reason for that to occur is if `this` didn't exist anymore at the time the event was raised. And with Unity, it means the object got destroyed.
So I scrolled down the method until I hit the dreaded two instructions that caused it all, `Destroy(gameObject)` and `Destroy(other.gameObject)`.
The handler does not run because there is no object to run it on anymore.

I commented out the two Destroy instructions and played the game again. Added a few containers and components, combined them and the two quantity inputs and the button appeared. I hastily added random quantities and hit "Confirm".
Unity froze. "Oh no", I told myself for a bit. Then I saw it. The Visual Studio icon in the task bar was blinking orange. I switched to it, and to my delight the debugger had hit the breakpoint in the Amount method.

rapid estuary Jun 21, 2022, 9:00 PM

#

You would make a fortune as a writer

rapid estuary Jun 22, 2022, 2:02 PM

#

@onyx ferry mind sending me the file that you modified?

#

if you wouldn't mind, of course

#

if you added other stuff to the game itself I would really appreciate the whole project being sent over. And yes, I will proceed to pay you

onyx ferry Jun 22, 2022, 4:16 PM

#

To respect the "don't distribute" statement I encrypted the files with a random key then deleted them, ie. they're not recoverable.
I have the fix in mind though. You would need to make the Amount method on an object that does not get destroyed, or in a separate class completely, and pass the required info as arguments to the method.
My guess would be on the UI element where you input the quantities and confirm. That one doesn't get destroyed at all. You would just need to store the required info prior to calling it.

#

The subscription can still be done in your React method as long as the lambda does not capture this

rapid estuary Jun 22, 2022, 4:46 PM

#

fixed it, thank you so much ❤️

#

send me your email so I can send the cash via paypal

onyx ferry Jun 22, 2022, 7:24 PM

#

You're good. I don't have paypal anyway lol
I've been doing this in my free time for more than a year and never received compensation for it, that's not going to change today

#I already tried that

So, full explanation of the story and how to fix.