tazerman2 Posted February 8, 2017 Posted February 8, 2017 (edited) nice code but where is the reward ? and from where get id from reward ? i dont see get some Id from like adena Edited February 8, 2017 by tazerman2
pirama Posted March 1, 2017 Posted March 1, 2017 Code will be improved? sure ... but author is banned ...
Tryskell Posted March 1, 2017 Posted March 1, 2017 (edited) There isn't that much to improve, replace ConcurrentHashMap occurences by regular HashMap (no need of concurrency here, slow the code for nothing) and as a basic core, if you got no bugs, it's almost good to go. Edited March 1, 2017 by Tryskell
xxdem Posted March 1, 2017 Posted March 1, 2017 There isn't that much to improve, replace ConcurrentHashMap occurences by regular HashMap (no need of concurrency here, slow the code for nothing) and as a basic core, if you got no bugs, it's almost good to go. it can be coded way better. Theres no need of external data structures at all to make this work. This is a shitcode
Tryskell Posted March 1, 2017 Posted March 1, 2017 it can be coded way better. Theres no need of external data structures at all to make this work. This is a shitcode You don't need external data structure for anything, if you think like this. Almost everything can be hardcoded on enum. 40k spawns on int array, you will like it.
xxdem Posted March 1, 2017 Posted March 1, 2017 You don't need external data structure for anything, if you think like this. Almost everything can be hardcoded on enum. 40k spawns on int array, you will like it. bs. I am talking about this particular case, you don't need hashmaps to store players on their factions at all
Tryskell Posted March 1, 2017 Posted March 1, 2017 (edited) The hashmap is used either for Faction type storing, or as a temporary container. There is no additional storing HashMap otherwise, so I don't really see what you mean. getFactionPlayers( isn't even used and could be deleted (or replaced by a stream, which still produce garbage). And as I said higher anyway, factions could benefit from enum as there isn't a lot of data and it won't be edited. http://www.maxcheaters.com/topic/211856-acis-faction-system-unlimited-factions/?p=2614452 Edited March 1, 2017 by Tryskell
Recommended Posts