BestTime_System implementation problems...

Unity Engine
,
1

I am working in a new game that is almost finished, but i cant get my script to work correctly to give me a bestTime rather than the currentTime I am saving result in PlayerPrefs and i have various if()s to verify all cases…

for example:
float _bestTime_F starts with no value, meaning 0 at the first run of the game.
I have a float _currentTime_F that is constantly updating every game loop.

  • I need that if my _currentTime_F is > than my _bestTime_F,
    meaning your best time still same because you get a higher time than you Best.
  • Also verify if my _currentTime_F is < than my _besTime_F,
    meaning that you have made a new Best Score…
  • And if my _bestTime_F is <= _currentTime_F…
    player Best Time maintain the _bestTime_F value…
    meaning player have finished the level and no new Best Time has been made.

I will appreciate any advice. ::JCD

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

public class _Timer : MonoBehaviour
{
    // VARIABLES____________________...

    public Text timer_TXT;
    private string _currentTime_S;
    private float _currentTime_F;


    public Text bestTime_TXT;
    public string bestTime_S;
    private float _bestTime_F; 
      


    // FUNCTIONS_____________________...

    void Start()
    {
        //_bestTime_F = PlayerPrefs.GetFloat("BestTime", Mathf.Infinity);
       
        _bestTime_F = PlayerPrefs.GetFloat("BestTime");

        Debug.Log(_bestTime_F);
       
    } //start.

   
    private void Update()
    {
        Timer();

    } //update.


    private void Timer()
    {
        timer_TXT.text = System.TimeSpan.
            FromSeconds((int)Time.timeSinceLevelLoad).
            ToString();

        _currentTime_S = timer_TXT.text;     
       



        _currentTime_F += Time.deltaTime;
        //Debug.Log(_currentTime_F);

        if (_bestTime_F < _currentTime_F)
        {
            _bestTime_F = _currentTime_F;
        }

      


        if (_Spikes.s_destroyed_B) /* testing when player dies but it will be called when level clear */
        {
            if (_bestTime_F < _currentTime_F)
            {
                _bestTime_F = _currentTime_F;

                PlayerPrefs.SetFloat("BestTime", _bestTime_F);
                //PlayerPrefs.Save();

                Debug.Log("Your BestTime starts being less than your CurrentTime...");
            }               

            if (_currentTime_F >= _bestTime_F)
            {
                //_bestTime_F = _currentTime_F;

                //_bestTime_F = PlayerPrefs.GetFloat("BestTime");

                PlayerPrefs.SetFloat("BestTime", _bestTime_F);
                //PlayerPrefs.Save();

                Debug.Log("Your BestTime is greater or equal than your CurrentTime...");
            }

            if (_currentTime_F < _bestTime_F)
            {
                //_bestTime_F = _currentTime_F;

                PlayerPrefs.SetFloat("BestTime", _bestTime_F);

                Debug.Log("Congratulations, you set a new BestTime!");
            }
        }           

    } //timer.

    
} // END TIMER.

5497273–563653–_Timer.cs (2.24 KB)

2

You failed to mentioned what is actually not working. Here’s the problems I noticed at a quick glance.

  • Line 55, whenever _bestTime_F is less than _currentTime_F, you are setting _bestTime_F to _currentTime_F. This seems backwards assuming lower times are better.
  • Line 57, you are setting _bestTime_F to _currentTime_F, but the rest of the code in this method depends on comparisons between these two variables. Why set them equal to each other here, but do any of the “if” statements with them later? Makes no sense
  • Line 65, this “if” statement can never be true because if it were true you would have set the two variables equal to each other back on line 57.
  • Line 91, if _currentTime_F is better than _bestTime_F, why aren’t you saving _currentTime_F as the new BestTime?
3

One pointer to making this stuff cleaner: when you want to implement something persistent like a high score saver, put it all in its own static class and then access it ONLY through that class. Here’s a SUPER-simple example:

using UnityEngine;

public static class BestScore
{
    private const string s_BestScore = "BestScore";
    public static int Score
    {
        get
        {
            return PlayerPrefs.GetInt(s_BestScore, 0);
        }
        set
        {
            PlayerPrefs.SetInt(s_BestScore, value);
        }
    }
}

Once you have that, NEVER go inside that class again.

To check if your current score is higher and set it,

if (CurrentScore > BestScore.Score)
{
  Debug.Log( "You got the best score!");
  BestScore.Score = CurrentScore;
}

Anywhere you want to use it, just use it:

Debug.Log( "Your best score is " + BestScore.Score);
1 Like
4

@Joe_Censored

Joe, I re-write the script and i come with a way to get what i was looking for. If you want to copy code and run it by yourself and see results and you have a better way to achieve this just let me know. Take a look below…

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

public class Timer : MonoBehaviour
{
// VARIABLES____________________…

private float _currentTime_F = 0f;
private float _bestTime_F;

// FUNCIONES____________________…

private void Awake()
{
//unncomment line below, run the game and exit…
// if you need to clear all saved values to the PlayerPrefs.
//PlayerPrefs.DeleteAll();

} //awake.

private void Start()
{
_bestTime_F = PlayerPrefs.GetFloat(“BestTime”); //second time you run the game again…
// in console the value of PlayerPrefs get int “BestTime”
// will be printing the value it holds as you BestTime.

Debug.Log("Recent Best Time: " + _bestTime_F); //i explicitly need this value to be 0 the
// first run after the game being installed…

} //start.

private void Update()
{
runningTimer();

} //update.

private void runningTimer()
{
_currentTime_F += Time.deltaTime;

Debug.Log("Current Time: " + _currentTime_F);

} //timer.

private void OnApplicationQuit()
{
PlayerPrefs.SetFloat(“CurrentTime”, _currentTime_F);

//FIRST TIME PLAYER PLAY THE GAME!
if (_bestTime_F == 0) //the game will start _bestTime_F with a value of 0…
//this will only be true ONLY the very first time you run the game…
// meaning this is the very first time you have play the game!
{

//in this line i assign the _currentTime_F value to the var _BestTime_F,
// meaning at this moment i need that var _bestTime_F holds the same value
// to be saved into PlayerPrefs becuase you have made your very
// first BestTime;
_bestTime_F = _currentTime_F;

//saving…
PlayerPrefs.SetFloat(“BestTime”, _bestTime_F);

//Debugs.
Debug.Log("BestTime_F_var: " + _bestTime_F); //i explicitly need this value to be 0 the
// first run after the game being installed…

Debug.Log("CurrentTime_F_var: " + _currentTime_F); //current time or simply time obtained this run.

} //can NEVER be true again, meaning never will be 0…

//NOT THE FIRST… BUT ALL SUBSECUENT PLAYS!
if (_currentTime_F < _bestTime_F) //at this point _bestTime_F will never be 0 because you already made a new BestTime…
// now if the time you finished the level is LESS than your Time stored in _bestTime_F var,
// your _currentTime_F var will pass it value as your new BestTime.
{
PlayerPrefs.SetFloat(“BestTime”, _currentTime_F); //assigning the new lower value (_currentTime_F) to PlayerPrefs as your
// NEW BestTime.

Debug.Log("Congratulation!, you have made a new Best Time: " + PlayerPrefs.GetFloat(“BestTime”));
}

//ONLY IF THE CURRENT TIME IS EQUAL OR HIGHER THAN THE BEST SCORE YOU HAVE ALREADY MADE,
// MEANING BEST TIME WILL REMAIN THE SAME.
if (_currentTime_F >= _bestTime_F)
{
PlayerPrefs.SetFloat(“BestTime”, _bestTime_F);

Debug.Log("Your BestTime stays the same becouse you have not make a new BestTime than… Your BestTime: " +
_bestTime_F);
}

} //onapplicationquit.

} // END TIMER.

5509738–565279–Timer.cs (3.85 KB)

5

Joe, i re-write my script only for console debug and i come with a way to achieve what i was looking to get.
Please take a look and if you have a better or more efficient way to do it, just let me know.

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

public class Timer : MonoBehaviour
{
    // VARIABLES____________________...

    private float _currentTime_F = 0f;
    private float _bestTime_F;

    // FUNCIONES____________________...


    private void Awake()
    {
        //unncomment line below, run the game and exit...
        // if you need to clear all saved values to the PlayerPrefs.
        //PlayerPrefs.DeleteAll();

    } //awake.

    private void Start()
    {
        _bestTime_F = PlayerPrefs.GetFloat("BestTime"); //second time you run the game again...
                                                        // in console the value of PlayerPrefs get int "BestTime"
                                                        //  will be printing the value it holds as you BestTime.


        Debug.Log("Recent Best Time: " + _bestTime_F); //i explicitly need this value to be 0 the
                                                          // first run after the game being installed...

    } //start.

   
    private void Update()
    {
        runningTimer();

    } //update.


    private void runningTimer()
    {
        _currentTime_F += Time.deltaTime;

        Debug.Log("Current Time: " + _currentTime_F);

    } //timer.


    private void OnApplicationQuit()
    {
        PlayerPrefs.SetFloat("CurrentTime", _currentTime_F);

        //FIRST TIME PLAYER PLAY THE GAME!
        if (_bestTime_F == 0) //the game will start _bestTime_F with a value of 0...
                                //this will only be true ONLY the very first time you run the game...
                                // meaning this is the very first time you have play the game!
        {
          
           
            //in this line i assign the _currentTime_F value to the var _BestTime_F,
                // meaning at this moment i need that var _bestTime_F holds the same value
                // to be saved into PlayerPrefs becuase you have made your very
                // first BestTime;
            _bestTime_F = _currentTime_F;

            //saving...
            PlayerPrefs.SetFloat("BestTime", _bestTime_F);



            //Debugs.
            Debug.Log("BestTime_F_var: " + _bestTime_F); //i explicitly need this value to be 0 the
                                                         // first run after the game being installed...

            Debug.Log("CurrentTime_F_var: " + _currentTime_F); //current time or simply time obtained this run.

        } //can NEVER be true again, meaning never will be 0...


       
        //NOT THE FIRST... BUT ALL SUBSECUENT PLAYS!
        if (_currentTime_F < _bestTime_F) //at this point _bestTime_F will never be 0 because you already made a new BestTime...
                                          // now if the time you finished the level is LESS than your Time stored in _bestTime_F var,
                                          // your _currentTime_F var will pass it value as your new BestTime.
        {
            PlayerPrefs.SetFloat("BestTime", _currentTime_F); //assigning the new lower value (_currentTime_F) to PlayerPrefs as your
                                                              // NEW BestTime.

            Debug.Log("Congratulation!, you have made a new Best Time: " + PlayerPrefs.GetFloat("BestTime"));
        }


        //ONLY IF THE CURRENT TIME IS EQUAL OR HIGHER THAN THE BEST SCORE YOU HAVE ALREADY MADE,
        // MEANING BEST TIME WILL REMAIN THE SAME.
        if (_currentTime_F >= _bestTime_F)
        {
            PlayerPrefs.SetFloat("BestTime", _bestTime_F);

            Debug.Log("Your BestTime stays the same becouse you have not make a new BestTime than... Your BestTime: " +
                _bestTime_F);
        }


    } //onapplicationquit.


} // END TIMER.
6

Thanks @kurt , but not what is was trying to achieve.
Take a look at my re-written script and take a look what i was trying to achieve.
If you have any advice just let me know. Thanks again.

7

Here is the script re-written again for anybody who wants to contribute to write this in a efficient way.

EYE: it will not be implemented when application Quit but when Player Completes the
current Level! so this is just an example for anybody who wants to debug it over console
to see results of statements.

Thanks JCD::

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

public class Timer : MonoBehaviour
{
    // VARIABLES____________________...

    private float _currentTime_F = 0f;
    private float _bestTime_F;

    // FUNCIONES____________________...


    private void Awake()
    {
        //unncomment line below, run the game and exit...
        // if you need to clear all saved values to the PlayerPrefs.
        //PlayerPrefs.DeleteAll();

    } //awake.

    private void Start()
    {
        _bestTime_F = PlayerPrefs.GetFloat("BestTime"); //second time you run the game again...
                                                        // in console the value of PlayerPrefs get int "BestTime"
                                                        //  will be printing the value it holds as you BestTime.


        Debug.Log("Recent Best Time: " + _bestTime_F); //i explicitly need this value to be 0 the
                                                          // first run after the game being installed...

    } //start.

  
    private void Update()
    {
        runningTimer();

    } //update.


    private void runningTimer()
    {
        _currentTime_F += Time.deltaTime;

        Debug.Log("Current Time: " + _currentTime_F);

    } //timer.


    private void OnApplicationQuit()
    {
        PlayerPrefs.SetFloat("CurrentTime", _currentTime_F);

        //FIRST TIME PLAYER PLAY THE GAME!
        if (_bestTime_F == 0) //the game will start _bestTime_F with a value of 0...
                                //this will only be true ONLY the very first time you run the game...
                                // meaning this is the very first time you have play the game!
        {
         
          
            //in this line i assign the _currentTime_F value to the var _BestTime_F,
                // meaning at this moment i need that var _bestTime_F holds the same value
                // to be saved into PlayerPrefs becuase you have made your very
                // first BestTime;
            _bestTime_F = _currentTime_F;

            //saving...
            PlayerPrefs.SetFloat("BestTime", _bestTime_F);



            //Debugs.
            Debug.Log("BestTime_F_var: " + _bestTime_F); //i explicitly need this value to be 0 the
                                                         // first run after the game being installed...

            Debug.Log("CurrentTime_F_var: " + _currentTime_F); //current time or simply time obtained this run.

        } //can NEVER be true again, meaning never will be 0...


      
        //NOT THE FIRST... BUT ALL SUBSECUENT PLAYS!
        if (_currentTime_F < _bestTime_F) //at this point _bestTime_F will never be 0 because you already made a new BestTime...
                                          // now if the time you finished the level is LESS than your Time stored in _bestTime_F var,
                                          // your _currentTime_F var will pass it value as your new BestTime.
        {
            PlayerPrefs.SetFloat("BestTime", _currentTime_F); //assigning the new lower value (_currentTime_F) to PlayerPrefs as your
                                                              // NEW BestTime.


            Debug.Log("Congratulation!, you have made a new Best Time: " + PlayerPrefs.GetFloat("BestTime"));
        }


        //ONLY IF THE CURRENT TIME IS EQUAL OR HIGHER THAN THE BEST SCORE YOU HAVE ALREADY MADE,
        // MEANING BEST TIME WILL REMAIN THE SAME.
        if (_currentTime_F >= _bestTime_F)
        {
            PlayerPrefs.SetFloat("BestTime", _bestTime_F);

            Debug.Log("Your BestTime stays the same becouse you have not make a new BestTime than... Your Time was: " +
                _bestTime_F);
        }


    } //onapplicationquit.


} // END TIMER.