Add highscore to playerprefs setup

Hello
I have a nice working score and Lives counter working between my scenes using playerprefs.
My issue is, I want to implement a highs core as well, no matter how I try the highscore text either reads zero or the same value as the current score.

I know there are other work arounds but i really want to use it with this setup…
First script is my GameManager second one is the updater script which is on my text component.

public class GameStatus : MonoBehaviour
{
    public GameObject[] Hearts;



    public int numLives = 3;
    public int score = 0;
    public int highScore = 0;

   



    // Start is called before the first frame update
    void Start()
    {

        score = PlayerPrefs.GetInt("score", 0);
        numLives = PlayerPrefs.GetInt("lives", 3);
        highScore = PlayerPrefs.GetInt("highscore", 0);



    }

  
    void Update()
    {
      
        // TESTING HIGHSCORE
       if(score > PlayerPrefs.GetInt("highscore", 0))
        {
            PlayerPrefs.SetInt("highscore", highScore);
            Debug.Log("NOPE");

        }

Second scipt:

public class scorePanelUpdaterHS : MonoBehaviour
{

    GameStatus gs;

    // Update is called once per frame
    void Update()
    {
        //Bad way of grabbing data
        GameObject go = GameObject.Find("GameStatus");
        if(go == null)
        {

            Debug.LogError("Failed to find that object");
            this.enabled = false;
            return;
        }

        //actual script gamestatus

        GameStatus gs = go.GetComponent<GameStatus>();

        GetComponent<Text>().text = "highscore: " + gs.highScore;
       





    }
}

I would recommend against splattering PlayerPrefs calls all over your code. Centralize it.

Here’s my basic approach:

Tracking simple high score / low time single-entry leaderboard:

For the highest score: Unity High score tracking - Pastebin.com

Usage:

TheBest.RecordScoreIfHigher( lastGamePlayScore);

For the lowest time: Unity best time tracking - Pastebin.com

Usage:

TheBest.RecordTimeIfLower( lastGamePlayTime);

To retrieve the best score or time, use one of these:

int bestScore = TheBest.BestScore;
float bestTime = TheBest.BestTime;

Full usage example:

Thank you Kurt for the code and some explanation.

unfortunately as I am still learning and followed a tutorial the end I would like a solution more tailored to the setup I already have.

I am SO CLOSE, I can feel it…

Well first thing is to find out where things are going wrong in the above code.

To help gain more insight into your problem, I recommend liberally sprinkling Debug.Log() statements through your code to display information in realtime.

Doing this should help you answer these types of questions:

  • is this code even running? which parts are running? how often does it run? what order does it run in?
  • what are the values of the variables involved? Are they initialized? Are the values reasonable?

Knowing this information will help you reason about the behavior you are seeing.

You can also put in Debug.Break() to pause the Editor when certain interesting pieces of code run, and then study the scene

You could also just display various important quantities in UI Text elements to watch them change as you play the game.

If you are running a mobile device you can also view the console output. Google for how on your particular mobile target.

Here’s an example of putting in a laser-focused Debug.Log() and how that can save you a TON of time wallowing around speculating what might be going wrong:

For instance, this logic seems very suspect but because playerprefs is so visually spammy it’s hard to see immediately:

You’re saying (in the above code):

  • if your score is better than the highscore playerpref:
  • set the highscore playerpref equal to the highscore and print NOPE

I would imagine you would want to set the highscore playerpref equal to the score and print YEP

Again, all that spam just makes it hard to reason about.

Here’s another way to tidy it up:

Here’s an example of simple persistent loading/saving values using PlayerPrefs:

https://gist.github.com/kurtdekker/01da815d2dfd336a925ae38019c3a163

Useful for a relatively small number of simple values.