Extract some player attributes to data files

CMake/Assets.cmake

@@ -142,6 +142,12 @@ set(devilutionx_assets
   nlevels/l5data/cornerstone.dun
   nlevels/l5data/uberroom.dun
   txtdata/Experience.tsv
+  txtdata/classes/barbarian/attributes.tsv
+  txtdata/classes/bard/attributes.tsv
+  txtdata/classes/monk/attributes.tsv
+  txtdata/classes/rogue/attributes.tsv
+  txtdata/classes/sorcerer/attributes.tsv
+  txtdata/classes/warrior/attributes.tsv
   ui_art/diablo.pal
   ui_art/hellfire.pal
   ui_art/creditsw.clx

Packaging/resources/assets/txtdata/classes/barbarian/attributes.tsv

@@ -0,0 +1,18 @@
+Attribute	Value
+baseStr	40
+baseMag	0
+baseDex	20
+baseVit	25
+maxStr	255
+maxMag	0
+maxDex	55
+maxVit	150
+blockBonus	30
+adjLife	1152
+adjMana	0
+lvlLife	128
+lvlMana	0
+chrLife	128
+chrMana	64
+itmLife	160
+itmMana	64

Packaging/resources/assets/txtdata/classes/bard/attributes.tsv

@@ -0,0 +1,18 @@
+Attribute	Value
+baseStr	20
+baseMag	20
+baseDex	25
+baseVit	20
+maxStr	120
+maxMag	120
+maxDex	120
+maxVit	100
+blockBonus	25
+adjLife	1472
+adjMana	192
+lvlLife	128
+lvlMana	128
+chrLife	64
+chrMana	96
+itmLife	96
+itmMana	112

Packaging/resources/assets/txtdata/classes/monk/attributes.tsv

@@ -0,0 +1,18 @@
+Attribute	Value
+baseStr	25
+baseMag	15
+baseDex	25
+baseVit	20
+maxStr	150
+maxMag	80
+maxDex	150
+maxVit	80
+blockBonus	25
+adjLife	1472
+adjMana	352
+lvlLife	128
+lvlMana	128
+chrLife	64
+chrMana	64
+itmLife	96
+itmMana	96

Packaging/resources/assets/txtdata/classes/rogue/attributes.tsv

@@ -0,0 +1,18 @@
+Attribute	Value
+baseStr	20
+baseMag	15
+baseDex	30
+baseVit	20
+maxStr	55
+maxMag	70
+maxDex	250
+maxVit	80
+blockBonus	20
+adjLife	1472
+adjMana	352
+lvlLife	128
+lvlMana	128
+chrLife	64
+chrMana	64
+itmLife	96
+itmMana	96

Packaging/resources/assets/txtdata/classes/sorcerer/attributes.tsv

@@ -0,0 +1,18 @@
+Attribute	Value
+baseStr	15
+baseMag	35
+baseDex	15
+baseVit	20
+maxStr	45
+maxMag	250
+maxDex	85
+maxVit	80
+blockBonus	10
+adjLife	576
+adjMana	-128
+lvlLife	64
+lvlMana	128
+chrLife	64
+chrMana	128
+itmLife	64
+itmMana	128

Packaging/resources/assets/txtdata/classes/warrior/attributes.tsv

@@ -0,0 +1,18 @@
+Attribute	Value
+baseStr	30
+baseMag	10
+baseDex	20
+baseVit	25
+maxStr	250
+maxMag	50
+maxDex	60
+maxVit	100
+blockBonus	30
+adjLife	1152
+adjMana	-64
+lvlLife	128
+lvlMana	64
+chrLife	128
+chrMana	64
+itmLife	128
+itmMana	64

Source/items.cpp

@@ -2747,10 +2747,10 @@ void CalcPlrItemVals(Player &player, bool loadgfx)
 	player._pFireResist = std::clamp(fr, 0, MaxResistance);
 	player._pLghtResist = std::clamp(lr, 0, MaxResistance);
 
-	vadd = (vadd * PlayersData[static_cast<size_t>(player._pClass)].itmLife) >> 6;
+	vadd = (vadd * GetClassAttributes(player._pClass).itmLife) >> 6;
 	ihp += (vadd << 6); // BUGFIX: blood boil can cause negative shifts here (see line 757)
 
-	madd = (madd * PlayersData[static_cast<size_t>(player._pClass)].itmMana) >> 6;
+	madd = (madd * GetClassAttributes(player._pClass).itmMana) >> 6;
 	imana += (madd << 6);
 
 	player._pMaxHP = ihp + player._pMaxHPBase;

Source/loadsave.cpp

@@ -414,7 +414,7 @@ void LoadPlayer(LoadHelper &file, Player &player)
 	player._pDamageMod = file.NextLE<int32_t>();
 	player._pBaseToBlk = file.NextLE<int32_t>();
 	if (player._pBaseToBlk == 0)
-		player._pBaseToBlk = PlayersData[static_cast<std::size_t>(player._pClass)].blockBonus;
+		player._pBaseToBlk = GetClassAttributes(player._pClass).blockBonus;
 	player._pHPBase = file.NextLE<int32_t>();
 	player._pMaxHPBase = file.NextLE<int32_t>();
 	player._pHitPoints = file.NextLE<int32_t>();

Source/pack.cpp

@@ -453,7 +453,7 @@ void UnPackPlayer(const PlayerPack &packed, Player &player)
 
 	player._pExperience = SDL_SwapLE32(packed.pExperience);
 	player._pGold = SDL_SwapLE32(packed.pGold);
-	player._pBaseToBlk = PlayersData[static_cast<std::size_t>(player._pClass)].blockBonus;
+	player._pBaseToBlk = GetClassAttributes(player._pClass).blockBonus;
 	if ((int)(player._pHPBase & 0xFFFFFFC0) < 64)
 		player._pHPBase = 64;
 
@@ -542,7 +542,7 @@ bool UnPackNetPlayer(const PlayerNetPack &packed, Player &player)
 	player._pStatPts = packed.pStatPts;
 
 	player._pExperience = SDL_SwapLE32(packed.pExperience);
-	player._pBaseToBlk = PlayersData[static_cast<std::size_t>(player._pClass)].blockBonus;
+	player._pBaseToBlk = GetClassAttributes(player._pClass).blockBonus;
 	player._pMaxManaBase = baseManaMax;
 	player._pManaBase = baseMana;
 	player._pMemSpells = SDL_SwapLE64(packed.pMemSpells);

Source/pfile.cpp

@@ -678,10 +678,11 @@ bool pfile_ui_set_hero_infos(bool (*uiAddHeroInfo)(_uiheroinfo *))
 
 void pfile_ui_set_class_stats(unsigned int playerClass, _uidefaultstats *classStats)
 {
-	classStats->strength = PlayersData[playerClass].baseStr;
-	classStats->magic = PlayersData[playerClass].baseMag;
-	classStats->dexterity = PlayersData[playerClass].baseDex;
-	classStats->vitality = PlayersData[playerClass].baseVit;
+	const ClassAttributes &classAttributes = GetClassAttributes(static_cast<HeroClass>(playerClass));
+	classStats->strength = classAttributes.baseStr;
+	classStats->magic = classAttributes.baseMag;
+	classStats->dexterity = classAttributes.baseDex;
+	classStats->vitality = classAttributes.baseVit;
 }
 
 uint32_t pfile_ui_get_first_unused_save_num()

Source/player.cpp

@@ -1703,16 +1703,16 @@ int Player::GetCurrentAttributeValue(CharacterAttribute attribute) const
 
 int Player::GetMaximumAttributeValue(CharacterAttribute attribute) const
 {
-	PlayerData plrData = PlayersData[static_cast<std::size_t>(_pClass)];
+	const ClassAttributes &attr = GetClassAttributes(_pClass);
 	switch (attribute) {
 	case CharacterAttribute::Strength:
-		return plrData.maxStr;
+		return attr.maxStr;
 	case CharacterAttribute::Magic:
-		return plrData.maxMag;
+		return attr.maxMag;
 	case CharacterAttribute::Dexterity:
-		return plrData.maxDex;
+		return attr.maxDex;
 	case CharacterAttribute::Vitality:
-		return plrData.maxVit;
+		return attr.maxVit;
 	}
 	app_fatal("Unsupported attribute");
 }
@@ -2062,14 +2062,14 @@ uint32_t Player::getNextExperienceThreshold() const
 
 int32_t Player::calculateBaseLife() const
 {
-	const PlayerData &playerData = PlayersData[static_cast<size_t>(_pClass)];
-	return playerData.adjLife + (playerData.lvlLife * getCharacterLevel()) + (playerData.chrLife * _pBaseVit);
+	const ClassAttributes &attr = GetClassAttributes(_pClass);
+	return attr.adjLife + (attr.lvlLife * getCharacterLevel()) + (attr.chrLife * _pBaseVit);
 }
 
 int32_t Player::calculateBaseMana() const
 {
-	const PlayerData &playerData = PlayersData[static_cast<size_t>(_pClass)];
-	return playerData.adjMana + (playerData.lvlMana * getCharacterLevel()) + (playerData.chrMana * _pBaseMag);
+	const ClassAttributes &attr = GetClassAttributes(_pClass);
+	return attr.adjMana + (attr.lvlMana * getCharacterLevel()) + (attr.chrMana * _pBaseMag);
 }
 
 Player *PlayerAtPosition(Point position)
@@ -2279,24 +2279,24 @@ void CreatePlayer(Player &player, HeroClass c)
 	player = {};
 	SetRndSeed(SDL_GetTicks());
 
-	const PlayerData &playerData = PlayersData[static_cast<size_t>(c)];
+	const ClassAttributes &attr = GetClassAttributes(c);
 
 	player.setCharacterLevel(1);
 	player._pClass = c;
 
-	player._pBaseStr = playerData.baseStr;
+	player._pBaseStr = attr.baseStr;
 	player._pStrength = player._pBaseStr;
 
-	player._pBaseMag = playerData.baseMag;
+	player._pBaseMag = attr.baseMag;
 	player._pMagic = player._pBaseMag;
 
-	player._pBaseDex = playerData.baseDex;
+	player._pBaseDex = attr.baseDex;
 	player._pDexterity = player._pBaseDex;
 
-	player._pBaseVit = playerData.baseVit;
+	player._pBaseVit = attr.baseVit;
 	player._pVitality = player._pBaseVit;
 
-	player._pBaseToBlk = playerData.blockBonus;
+	player._pBaseToBlk = attr.blockBonus;
 
 	player._pHitPoints = player.calculateBaseLife();
 	player._pMaxHP = player._pHitPoints;
@@ -2314,7 +2314,7 @@ void CreatePlayer(Player &player, HeroClass c)
 	player._pInfraFlag = false;
 
 	player._pRSplType = SpellType::Skill;
-	SpellID s = playerData.skill;
+	SpellID s = PlayersData[static_cast<size_t>(c)].skill;
 	player._pAblSpells = GetSpellBitmask(s);
 	player._pRSpell = s;
 
@@ -2397,7 +2397,7 @@ void NextPlrLevel(Player &player)
 	} else {
 		player._pStatPts += 5;
 	}
-	int hp = PlayersData[static_cast<size_t>(player._pClass)].lvlLife;
+	int hp = GetClassAttributes(player._pClass).lvlLife;
 
 	player._pMaxHP += hp;
 	player._pHitPoints = player._pMaxHP;
@@ -2408,7 +2408,7 @@ void NextPlrLevel(Player &player)
 		RedrawComponent(PanelDrawComponent::Health);
 	}
 
-	int mana = PlayersData[static_cast<size_t>(player._pClass)].lvlMana;
+	int mana = GetClassAttributes(player._pClass).lvlMana;
 
 	player._pMaxMana += mana;
 	player._pMaxManaBase += mana;
@@ -3329,7 +3329,7 @@ void ModifyPlrMag(Player &player, int l)
 	player._pBaseMag += l;
 
 	int ms = l;
-	ms *= PlayersData[static_cast<size_t>(player._pClass)].chrMana;
+	ms *= GetClassAttributes(player._pClass).chrMana;
 
 	player._pMaxManaBase += ms;
 	player._pMaxMana += ms;
@@ -3366,7 +3366,7 @@ void ModifyPlrVit(Player &player, int l)
 	player._pBaseVit += l;
 
 	int ms = l;
-	ms *= PlayersData[static_cast<size_t>(player._pClass)].chrLife;
+	ms *= GetClassAttributes(player._pClass).chrLife;
 
 	player._pHPBase += ms;
 	player._pMaxHPBase += ms;
@@ -3401,7 +3401,7 @@ void SetPlrMag(Player &player, int v)
 	player._pBaseMag = v;
 
 	int m = v;
-	m *= PlayersData[static_cast<size_t>(player._pClass)].chrMana;
+	m *= GetClassAttributes(player._pClass).chrMana;
 
 	player._pMaxManaBase = m;
 	player._pMaxMana = m;
@@ -3419,7 +3419,7 @@ void SetPlrVit(Player &player, int v)
 	player._pBaseVit = v;
 
 	int hp = v;
-	hp *= PlayersData[static_cast<size_t>(player._pClass)].chrLife;
+	hp *= GetClassAttributes(player._pClass).chrLife;
 
 	player._pHPBase = hp;
 	player._pMaxHPBase = hp;