Can someone help me with my highscore script?

There are few problems with those scripts.

Problems in the HighScore script

• At the Start Event Function you're using GetFloat to get the highscore but you never assign it?

• You shouldn't get the "highScoreText" Component each time you set new high score, just initialize it in using the Awake Event Function.

• Why do you reassign, the currentScore and the highScore in the FixedUpdate Event Function? you already assigned the value on update, even though, you should only assign it once at awake/start.

Problems in the ScoreCounter script

• You're rounding the same value twice "Mathf.Round(timeStart)" just round it once and save it in a variable to use it again.

• You shouldn't communicate between 2 scripts using the PlayerPrefs, just get a reference of the script.

This is a fix of the code:

 public class HighScore : MonoBehaviour
 {
     public ScoreCounter ScoreCounter;
 
     public Text highScoreText;
     public float highScore;
 
     string highScoreKey = "highScore";
 
     public void Awake()
     {
         highScoreText = GetComponent<Text>();
         highScore = PlayerPrefs.GetFloat(highScoreKey);
     }
 
     public void Update()
     {
         if (ScoreCounter.currentTime > highScore)
         {
             highScore = ScoreCounter.currentTime;
             highScoreText.text = highScore.ToString();
             PlayerPrefs.SetFloat(highScoreKey, highScore);
         }
     }
 }
 
 public class ScoreCounter : MonoBehaviour
 {
     public float timeStart = 0;
     public Text textBox;
     public float currentTime;
 
     void Start()
     {
         textBox.text = timeStart.ToString();
     }
 
     void Update()
     {
         timeStart += Time.deltaTime;
         float roundedTime = Mathf.Round(timeStart);
         currentTime = roundedTime;
         textBox.text = roundedTime.ToString();
     }
 }

In the class HighScore in the update function you use PlayerPrefs.SetFloat("highScore", highScore) Shouldn't this be PlayerPrefs.SetFloat("highScore, currentScore)? Because it looks like you're now updating the highscore to what it already was