Akkariin/Grasscutter
1
0
Fork 0
mirror of https://github.com/Grasscutters/Grasscutter.git synced 2025-01-07 00:42:57 +08:00
Code Issues Projects Releases Wiki Activity

refactor: simplify map operations

Browse Source 
we can use computeIfAbsent here
This commit is contained in:
Breno A. 2024-06-09 09:13:18 -03:00
parent 3119e0fffc
commit c7119aae68
2 changed files with 2 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/main/java/emu/grasscutter/game/managers/SotSManager.java
View File

@ -218,11 +218,7 @@ public class SotSManager extends BasePlayerManager {
public CityInfoData getCityInfo(int cityId) { public CityInfoData getCityInfo(int cityId) {
if (player.getCityInfoData() == null) player.setCityInfoData(new HashMap<>()); if (player.getCityInfoData() == null) player.setCityInfoData(new HashMap<>());
var cityInfo = player.getCityInfoData().get(cityId); var cityInfo = player.getCityInfoData().computeIfAbsent(cityId, CityInfoData::new);
if (cityInfo == null) {
cityInfo = new CityInfoData(cityId);
player.getCityInfoData().put(cityId, cityInfo);
}
return cityInfo; return cityInfo;
} }

4
src/main/java/emu/grasscutter/game/player/Player.java
View File

@ -599,9 +599,7 @@ public class Player implements PlayerHook, FieldFetch {
GameData.getSceneTagDataMap().values().stream() GameData.getSceneTagDataMap().values().stream()
.filter(SceneTagData::isDefaultValid) .filter(SceneTagData::isDefaultValid)
.forEach(sceneTag -> { .forEach(sceneTag -> {
if (this.getSceneTags().get(sceneTag.getSceneId()) == null) { this.getSceneTags().computeIfAbsent(sceneTag.getSceneId(), k -> new HashSet<>());
this.getSceneTags().put(sceneTag.getSceneId(), new HashSet<>());
}
this.getSceneTags().get(sceneTag.getSceneId()).add(sceneTag.getId()); this.getSceneTags().get(sceneTag.getSceneId()).add(sceneTag.getId());
}); });
} }