High Score is not Saving Unity

Unity Engine
1 
using UnityEngine;
using UnityEngine.UI;

public class ScoreManager : MonoBehaviour
{

///<summary> The transform ofour player, for determining position. </summary>
[Header("Please Drag Objects Here")]
public Transform player;
///<summary> The UI element to display our current score. </summary>
public Text currentScoreText;
///<summary> The UI element to display our high score. </summary>
public Text highScoreText;

// Good idea to keep strings like this in a field, to avoid typos later.
private const string highScoreKey = "HighScore";

// Hold our score values.
[Header("Do not change values")]
[SerializeField] private int highScore = 0;
[SerializeField] private int currentScore = 0;

///<summary>
/// Get our current high score if we have it and dipslay it in the UI.
///</summary>
private void start()
{
// TODO: The high score could be retrieved in Awake()

// Get the current high score from player prefs, 0 if id doesn't exist.
highScore = PlayerPrefs.GetInt(highScoreKey, 0);

// Set the UI text to the current high score.
highScoreText.text = highScore.ToString();
}

///<summary>
/// Calculate our current score based on z position. Display current score to UI.
/// Check if current score is greater than high score. If so, update appropriately.
///</summary>
private void Update()
{
// Calculate the current score based on player positon, round to int.
currentScore = Mathf.RoundToInt(player.position.z);
// Display current score in the UI.
currentScoreText.text = currentScore.ToString();

// Check if our new score is greater than our previous high score.
if (currentScore > highScore)
{
// Change the high score to the new current value.
highScore = currentScore;
// Update the high score UI text.
highScoreText.text = highScore.ToString();

// TODO: Delete me, unless for some odd reason I am needed... But I should not be :smile:
PlayerPrefs.SetInt(highScoreKey, highScore);
PlayerPrefs.Save();
}
}

///<summary>
/// When this gameobject is disabled, save our players highscore.
///</summary?
private void OnDisable()
{
/*
* OnDisable() is called anytime this object is disabled. This
* includes changing scenes and quitting the application.
*/

// Set our high score.
PlayerPrefs.SetInt(highScoreKey, highScore);
// Save our data.
PlayerPrefs.Save();

// TODO: Delete me, for debugging only!
Debug.Log("I'm being Disabled! The high score is currently: " + highScore);
}

private bool SaveHighScore(int newScore)
{
int highScore = PlayerPrefs.GetInt("HighScore", 0);
bool gotNewHighScore = newScore > highScore;

if (gotNewHighScore)
{
PlayerPrefs.SetInt("HighScore", newScore);
PlayerPrefs.Save();
}

return gotNewHighScore;
}
}

Btw I did not make this code because I tried for 4 days and couldn’t do a high score so I copied someone else’s so idk what it means by TODO:

2

First things first, you really need to describe your problem in more detail. Posting code with a title “it doesn’t work” helps no one.
Second thing, a comment with TODO usually literally means “to do”, stuff that still needs to be changed or added to the code. You also have a comment like FIXME, which as the name suggests, means that the piece of code is broken/bugged.