be a little less bad with dealing with map key checks

Author: MinaciousGrace

src/Profile.cpp

@@ -2446,11 +2446,6 @@ HighScore& Profile::GetPBHighScoreByKey(RString ck, float rate) {
 	return HighScoresByChartKey.at(ck).at(rate).at(0);
 }
 
-bool Profile::ChartkeyHasGoal(RString ck) {
-	auto it = goalmap.find(ck);
-	return it != goalmap.end();
-}
-
 // aaa too lazy to write comparators rn -mina
 ScoreGoal& Profile::GetLowestGoalForRate(RString ck, float rate) {
 	auto& sgv = goalmap[ck];
@@ -2469,7 +2464,7 @@ ScoreGoal& Profile::GetLowestGoalForRate(RString ck, float rate) {
 }
 
 void Profile::SetAnyAchievedGoals(RString ck, float rate, const HighScore& pscore) {
-	if (!ChartkeyHasGoal(ck))
+	if (!HasGoal(ck))
 		return;
 
 	auto& sgv = goalmap[ck];

src/Profile.h

@@ -280,7 +280,7 @@ class Profile
 	void CreateGoal(RString ck);
 	void DeleteGoal(RString ck, DateTime assigned);
 	map<RString, vector<ScoreGoal>> goalmap;
-	bool ChartkeyHasGoal(RString ck);
+	bool HasGoal(RString ck) { return goalmap.count(ck) == 1; }
 	ScoreGoal& GetLowestGoalForRate(RString ck, float rate);
 	void SetAnyAchievedGoals(RString ck, float rate, const HighScore& pscore);