When debugging, don’t start with the end result. Start at the source, verify your assumptions.
Check that your conditions for collision are valid. Both objects need colliders, at least one needs a rigidbody (your player?).
Add a “Debug.Log(“string”)” at the top of the function to make sure the function is actually being called. Also put one inside the if statement to make sure that condition is true.
If all that happens, then you know where the code reaches, and the problem is somewhere between your PlayerScore class and your UI element. Do you have your PlayerScore coded so that it will automatically update the UI when “PlayerScoreValue” is changed?
Start with that, and please use Code Tags when pasting code into the forums.