External/mage
1
0
Fork 1
mirror of https://github.com/magefree/mage.git synced 2025-12-29 23:12:10 -08:00
Code Issues Projects Releases Packages Wiki Activity Actions

AI: improved performance and server stability in games with "choose name" effects (related to #11285)

Browse source
This commit is contained in:
Oleg Agafonov 2025-05-07 17:21:29 +04:00
parent 00dc6f4742
commit 99ca1e6029
1 changed files with 43 additions and 16 deletions
Download patch file Download diff file Expand all files Collapse all files

59
Mage/src/main/java/mage/cards/repository/CardRepository.java
View file

@ -43,6 +43,9 @@ public enum CardRepository {
private Dao<CardInfo, Object> cardsDao;
// store names lists like all cards, lands, etc (it's static data and can be calculated one time only)
private static final Map<String, Set<String>> namesQueryCache = new HashMap<>();
// sets with exclusively snow basics
public static final Set<String> snowLandSetCodes = new HashSet<>(Arrays.asList(
"CSP",
@ -156,8 +159,11 @@ public enum CardRepository {
return snowLandSetCodes.contains(setCode);
}
public Set<String> getNames() {
Set<String> names = new TreeSet<>();
public synchronized Set<String> getNames() {
Set<String> names = namesQueryCache.computeIfAbsent("getNames", x -> new TreeSet<>());
if (!names.isEmpty()) {
return names;
}
try {
QueryBuilder<CardInfo, Object> qb = cardsDao.queryBuilder();
qb.distinct().selectColumns("name", "modalDoubleFacedSecondSideName", "secondSideName", "flipCardName", "spellOptionCardName");
@ -172,8 +178,11 @@ public enum CardRepository {
return names;
}
public Set<String> getNonLandNames() {
Set<String> names = new TreeSet<>();
public synchronized Set<String> getNonLandNames() {
Set<String> names = namesQueryCache.computeIfAbsent("getNonLandNames", x -> new TreeSet<>());
if (!names.isEmpty()) {
return names;
}
try {
QueryBuilder<CardInfo, Object> qb = cardsDao.queryBuilder();
qb.distinct().selectColumns("name", "modalDoubleFacedSecondSideName", "secondSideName", "flipCardName", "spellOptionCardName");
@ -189,8 +198,11 @@ public enum CardRepository {
return names;
}
public Set<String> getNonbasicLandNames() {
Set<String> names = new TreeSet<>();
public synchronized Set<String> getNonbasicLandNames() {
Set<String> names = namesQueryCache.computeIfAbsent("getNonbasicLandNames", x -> new TreeSet<>());
if (!names.isEmpty()) {
return names;
}
try {
QueryBuilder<CardInfo, Object> qb = cardsDao.queryBuilder();
qb.distinct().selectColumns("name", "modalDoubleFacedSecondSideName", "secondSideName", "flipCardName", "spellOptionCardName");
@ -210,8 +222,11 @@ public enum CardRepository {
return names;
}
public Set<String> getNotBasicLandNames() {
Set<String> names = new TreeSet<>();
public synchronized Set<String> getNotBasicLandNames() {
Set<String> names = namesQueryCache.computeIfAbsent("getNotBasicLandNames", x -> new TreeSet<>());
if (!names.isEmpty()) {
return names;
}
try {
QueryBuilder<CardInfo, Object> qb = cardsDao.queryBuilder();
qb.distinct().selectColumns("name", "modalDoubleFacedSecondSideName", "secondSideName", "flipCardName", "spellOptionCardName");
@ -227,8 +242,11 @@ public enum CardRepository {
return names;
}
public Set<String> getCreatureNames() {
Set<String> names = new TreeSet<>();
public synchronized Set<String> getCreatureNames() {
Set<String> names = namesQueryCache.computeIfAbsent("getCreatureNames", x -> new TreeSet<>());
if (!names.isEmpty()) {
return names;
}
try {
QueryBuilder<CardInfo, Object> qb = cardsDao.queryBuilder();
qb.distinct().selectColumns("name", "modalDoubleFacedSecondSideName", "secondSideName", "flipCardName", "spellOptionCardName");
@ -244,8 +262,11 @@ public enum CardRepository {
return names;
}
public Set<String> getArtifactNames() {
Set<String> names = new TreeSet<>();
public synchronized Set<String> getArtifactNames() {
Set<String> names = namesQueryCache.computeIfAbsent("getArtifactNames", x -> new TreeSet<>());
if (!names.isEmpty()) {
return names;
}
try {
QueryBuilder<CardInfo, Object> qb = cardsDao.queryBuilder();
qb.distinct().selectColumns("name", "modalDoubleFacedSecondSideName", "secondSideName", "flipCardName", "spellOptionCardName");
@ -261,8 +282,11 @@ public enum CardRepository {
return names;
}
public Set<String> getNonLandAndNonCreatureNames() {
Set<String> names = new TreeSet<>();
public synchronized Set<String> getNonLandAndNonCreatureNames() {
Set<String> names = namesQueryCache.computeIfAbsent("getNonLandAndNonCreatureNames", x -> new TreeSet<>());
if (!names.isEmpty()) {
return names;
}
try {
QueryBuilder<CardInfo, Object> qb = cardsDao.queryBuilder();
qb.distinct().selectColumns("name", "modalDoubleFacedSecondSideName", "secondSideName", "flipCardName", "spellOptionCardName");
@ -282,8 +306,11 @@ public enum CardRepository {
return names;
}
public Set<String> getNonArtifactAndNonLandNames() {
Set<String> names = new TreeSet<>();
public synchronized Set<String> getNonArtifactAndNonLandNames() {
Set<String> names = namesQueryCache.computeIfAbsent("getNonArtifactAndNonLandNames", x -> new TreeSet<>());
if (!names.isEmpty()) {
return names;
}
try {
QueryBuilder<CardInfo, Object> qb = cardsDao.queryBuilder();
qb.distinct().selectColumns("name", "modalDoubleFacedSecondSideName", "secondSideName", "flipCardName", "spellOptionCardName");