perf: Compute 'missing from minimum' stats only when necessary

This commit is contained in:
yairm210 2024-08-16 16:01:45 +03:00
parent 8206101f34
commit 84af8fa3e0

View File

@ -118,11 +118,14 @@ class TileStatFunctions(val tile: Tile) {
if (road != null) listOfStats.add(road.name to roadStats)
if (improvement != null) listOfStats.add(improvement.name to improvementStats)
val statsFromMinimum = missingFromMinimum(listOfStats.toStats(), minimumStats)
listOfStats.add("Minimum" to statsFromMinimum)
if (minimumStats != Stats.ZERO) {
val statsFromMinimum = missingFromMinimum(listOfStats.toStats(), minimumStats)
listOfStats.add("Minimum" to statsFromMinimum)
}
if (observingCiv != null &&
listOfStats.toStats().gold != 0f && observingCiv.goldenAges.isGoldenAge())
if (observingCiv != null && observingCiv.goldenAges.isGoldenAge()
&& listOfStats.toStats().gold != 0f
)
listOfStats.add("Golden Age" to Stats(gold = 1f))
// To ensure that the original stats (in uniques, terrains, etc) are not modified in getTileStats, we clone them all
@ -133,7 +136,7 @@ class TileStatFunctions(val tile: Tile) {
private fun missingFromMinimum(current: Stats, minimumStats: Stats): Stats {
// Note: Not `for ((stat, value) in other)` - that would skip zero values
val missingStats = Stats()
for (stat in Stat.values()) {
for (stat in Stat.entries) {
if (current[stat] < minimumStats[stat])
missingStats[stat] = minimumStats[stat] - current[stat]
}
@ -203,7 +206,7 @@ class TileStatFunctions(val tile: Tile) {
val cachedAllStatPercentFromObjectCityUniques = uniqueCache.forCityGetMatchingUniques(
city, UniqueType.AllStatsPercentFromObject, stateForConditionals)
for (unique in cachedAllStatPercentFromObjectCityUniques) {
for (stat in Stat.values())
for (stat in Stat.entries)
addStats(unique.params[1], stat, unique.params[0].toFloat())
}
@ -217,7 +220,7 @@ class TileStatFunctions(val tile: Tile) {
val cachedAllStatPercentFromObjectCivUniques = uniqueCache.forCivGetMatchingUniques(
observingCiv, UniqueType.AllStatsPercentFromObject, stateForConditionals)
for (unique in cachedAllStatPercentFromObjectCivUniques) {
for (stat in Stat.values())
for (stat in Stat.entries)
addStats(unique.params[1], stat, unique.params[0].toFloat())
}
}