C# High Score not saving

Questions & Answers
1

I’m building a app for android and i can’t get the high score to save.
using UnityEngine;
using System.Collections;

[RequireComponent(typeof(CircleCollider2D))]
public class DamageEntity : MonoBehaviour {

public float damage = 15;
public bool damageOnEnter = true;
public bool damageOnExit = false;
public bool damageOnStay = true;
static int score = 0;
static int highScore = 0;

static public void AddPoint(){
	

	
	if(score > highScore) {
		highScore = score;
	}
}
// Use this for initialization
void Start() {

}

void OnTriggerEnter2D(Collider2D otherObj) {
    if (damageOnEnter) {
					if (otherObj.GetComponent<BaseEntity> () != null) {
							otherObj.GetComponent<BaseEntity> ().TakeDamage (damage);
			score ++;

					}
			}
		if (damageOnEnter) {
			if (otherObj.GetComponent<Spawner>() != null) {
				otherObj.GetComponent<Spawner>().TakeDamage(damage);
			score ++;
				
			}
		}
			if (damageOnEnter) {
				if (otherObj.GetComponent<Spawner2>() != null) {
					otherObj.GetComponent<Spawner2>().TakeDamage(damage);
			score ++;
				}
		}
				if (damageOnEnter) {
					if (otherObj.GetComponent<Spawner3>() != null) {
						otherObj.GetComponent<Spawner3>().TakeDamage(damage);
			score ++;
					}
    }
}



void OnTriggerStay2D(Collider2D otherObj) {
    if (damageOnStay) {
					if (otherObj.GetComponent<BaseEntity> () != null) {
							otherObj.GetComponent<BaseEntity> ().TakeDamage (damage * Time.deltaTime);
					}
			}
			}
					
void OnGUI(){
	GUI.Label (new Rect (10, 10, 100, 20), "Score" + score);
	GUI.Label (new Rect (20, 20, 100, 20), "HighScore" + highScore);

}
void Update () {
	highScore = score;
	score = 0;
	highScore = PlayerPrefs.GetInt("highScore", 0);
	PlayerPrefs.SetInt("highScore", highScore);
	if(score > highScore) {
		highScore = score;
}

}
According to most people this should be fine for saving score but it’s not.

2

Your logic is wrong here, but I don’t know how to tell you to fix it. You are executing this code every frame, which you should not have to do. Line 68:

 score = 0;

…makes no sense to me. Why do you want to reset the to 0 every frame? You probably want to do the following at specific points like when the player loses/wins or at specific checkpoints in the game. You might also want to call. PlayerPrefs.Save() at breaks in the game.

void UpdateHighScore() {
     int currHighScore = PlayerPrefs.GetInt("highScore", 0);
     if (score > currHighScore) {
          highScore = score;
          PlayerPrefs.SetInt("highScore", highScore;
     }
}