v0.8.9 (Bugfixes + translations) released to Google Play!

Useful links
Source code of the game - Contribution guide - ATCS Editor - Translate the game on Weblate - Example walkthrough - Andor's Trail Directory - Join the Discord
Get the game (v0.8.9) from Google, F-Droid, our server, or itch.io

Game stats: iqhan master shows Up twice

A place to submit bugs to the Andor's Trail Development Team.
Fagris
Posts: 16
Joined: Sun Oct 25, 2020 7:11 pm
android_version: 9.0 - Pie

Game stats: iqhan master shows Up twice

Post by Fagris »

While trying to grind for the ChaR I noticed the game stats counts the iqhan master twice thus wasting a precious space of the monster count top 5. :!:
I assume this could be due to their robes in red and purple?

While we're at it, could this top5 be expanded to top7 or even Top10? this would be an enhancement of course, no bug ;)
You do not have the required permissions to view the files attached to this post.
Riccious, lvl 71
User avatar
rijackson741
Posts: 4451
Joined: Tue Aug 20, 2013 2:04 am
android_version: 10 - Android 10
Location: Somewhere in Dhayavar
Contact:

Re: Game stats: iqhan master shows Up twice

Post by rijackson741 »

Hmm. Yes, there are two NPCs with the name "Iqhan chaos master", and different internal IDs. The name is just a label you see in the game, and is ignored for most purposes. Seeing them show up twice is a little annoying, but fixing it would be a lot of work.

Yes, we could extend the list. Every kill is recorded in the savegame.
Level:71, XP:6493739, PV:608, FQ:84
HP:210, AC:212, AD:58-77, AP:4, ECC:16%, CM:1.5, BC:188, DR:3
Gold: 237559 | RoLS:1, RoL:1, GoW:1, VSH:1, RoFLS:1, WoB:1
HH:1, WA:1, CS:2, Cl:1, IF:4, Ev:3, Re:2, WP:DA:1, WP:1S:1, WP:B:1, AP:L:1, FS:DW:2, S:DW:1
Fagris
Posts: 16
Joined: Sun Oct 25, 2020 7:11 pm
android_version: 9.0 - Pie

Re: Game stats: iqhan master shows Up twice

Post by Fagris »

thanks for the quick reply.
Just wanted add, Same issue with Iqhan chaos evoker as the second "entity"of this monster made it to the top 5 as well :-D
understood there will be no change here given the effort and as it's a kind of 'corner case'
Riccious, lvl 71
User avatar
Antison
Posts: 5254
Joined: Mon Mar 28, 2011 11:33 pm
android_version: 14 - Android 14
Location: A home without a beagle is just a house

Re: Game stats: iqhan master shows Up twice

Post by Antison »

It's not a "corner case", because it's not really a bug. Think of it this way: You got two baseball teams playing a game. Each one is playing in a different uniform, but both sets of players are considered "players". Just because they wear different clothes doesn't make them different things.
"A home without a beagle is just a house"
Fagris
Posts: 16
Joined: Sun Oct 25, 2020 7:11 pm
android_version: 9.0 - Pie

Re: Game stats: iqhan master shows Up twice

Post by Fagris »

Sorry but I don't get your logic.
where is the point of having same titled monsters 2 times in the game stats. 4 of 5 slots are occupied with 2 types of monsters, Chaos master and evoker respectively. there is no benefit / sense for me as player in this behaviour.
Riccious, lvl 71
reizy
VIP
Posts: 38
Joined: Thu Feb 21, 2019 4:46 pm
android_version: 9.0 - Pie

Re: Game stats: iqhan master shows Up twice

Post by reizy »

rijackson741 wrote: Thu Mar 25, 2021 10:22 pmbut fixing it would be a lot of work.
Is changing the label realy a big deal?
User avatar
rijackson741
Posts: 4451
Joined: Tue Aug 20, 2013 2:04 am
android_version: 10 - Android 10
Location: Somewhere in Dhayavar
Contact:

Re: Game stats: iqhan master shows Up twice

Post by rijackson741 »

The name is the same so that not every monster with that name is quite the same. It's deliberate, to give variety (and there's a lot of such NPCs), so changing the name is not an option. Changing the ID to be the same is also not an option, because that's impossible unless the multiple NPCs become only one. So the only option would be to change the code to look at not only ID, but also name. In which cases would we do that? So sorry, but it is not so simple to fix this.
Level:71, XP:6493739, PV:608, FQ:84
HP:210, AC:212, AD:58-77, AP:4, ECC:16%, CM:1.5, BC:188, DR:3
Gold: 237559 | RoLS:1, RoL:1, GoW:1, VSH:1, RoFLS:1, WoB:1
HH:1, WA:1, CS:2, Cl:1, IF:4, Ev:3, Re:2, WP:DA:1, WP:1S:1, WP:B:1, AP:L:1, FS:DW:2, S:DW:1
Einsame Hirte
Posts: 27
Joined: Sun Jan 24, 2021 8:22 pm
android_version: 13 - Android 13

Re: Game stats: iqhan master shows Up twice

Post by Einsame Hirte »

I haven't looked at the code, but how about generating the kill-count display based on the cumulative kills for monsters with the same labels? I have no idea how the data is stored, but I have to think it should be fairly straightforward to iterate through the kill-count array/list/database/whatever and sum up counts for each unique label, then generate the list based that.
User avatar
rijackson741
Posts: 4451
Joined: Tue Aug 20, 2013 2:04 am
android_version: 10 - Android 10
Location: Somewhere in Dhayavar
Contact:

Re: Game stats: iqhan master shows Up twice

Post by rijackson741 »

The only thing that's tracked is the internal ID, which is always unique. There are many cases where the name is the same, but there is more than one internal ID (for example, there are nine versions of Burhczyd). The only real option would be to change the name so that the names of killable monsters are always unique.
Level:71, XP:6493739, PV:608, FQ:84
HP:210, AC:212, AD:58-77, AP:4, ECC:16%, CM:1.5, BC:188, DR:3
Gold: 237559 | RoLS:1, RoL:1, GoW:1, VSH:1, RoFLS:1, WoB:1
HH:1, WA:1, CS:2, Cl:1, IF:4, Ev:3, Re:2, WP:DA:1, WP:1S:1, WP:B:1, AP:L:1, FS:DW:2, S:DW:1
Einsame Hirte
Posts: 27
Joined: Sun Jan 24, 2021 8:22 pm
android_version: 13 - Android 13

Re: Game stats: iqhan master shows Up twice

Post by Einsame Hirte »

Right, but the player doesn't know or care that there are multiple internal versions of the same monster. They just want to know the total number of times they killed the same name. So, just create list that tallies the kills for each name (in php or perl you'd use an associative array with the monster name as index; I guess in Java you'd use a HashMap), then sort that list by descending kills and take the top 5.

I'm no good at java, but here's the idea in pseudocode...

Code: Select all

for each monsterId in killedMonstersByID
{
	killsByName{monsterId.name}+=monsterId.kills
}

int i=0
for each monsterName in sortByDescendingValues(%killsByName)	
{
	if (i++ >= 5) break;
	addToOutputString(monsterName, killsByName{monsterName})
}
Looking at getTop5MostCommonlyKilledMonsters() in model/GameStatistics.java, I think it would be just an extra three or four lines. I've been meaning to get my feet wet in Java and Android development, and this would be a good exercise. Anyone mind if I try to contribute a fix like this?
Post Reply