h1or4dme8/osu-lazer
1
0
Fork 0
mirror of https://github.com/ppy/osu.git synced 2025-02-08 12:23:21 +08:00
Code Issues Releases Wiki Activity

Combine countless update methods all called together into a single method

Browse Source
This commit is contained in:
Dean Herbert 2025-01-24 18:00:05 +09:00
parent 1bfb4d4c2b
commit 17b1739ae4
No known key found for this signature in database
1 changed files with 19 additions and 33 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

52
osu.Game/Screens/OnlinePlay/Match/RoomSubScreen.cs
View File

@ -355,11 +355,11 @@ namespace osu.Game.Screens.OnlinePlay.Match
public override void OnResuming(ScreenTransitionEvent e)
{
base.OnResuming(e);
updateBeatmap();
updateSpecifics();
beginHandlingTrack();
Scheduler.AddOnce(updateMods);
Scheduler.AddOnce(updateRuleset);
Scheduler.AddOnce(updateUserStyle);
}
protected bool ExitConfirmed { get; private set; }
@ -448,9 +448,7 @@ namespace osu.Game.Screens.OnlinePlay.Match
updateUserMods();
updateBeatmap();
updateMods();
updateRuleset();
updateUserStyle();
updateSpecifics();
if (!item.AllowedMods.Any())
{
@ -501,43 +499,31 @@ namespace osu.Game.Screens.OnlinePlay.Match
UserModsSelectOverlay.Beatmap.Value = Beatmap.Value;
}
private void updateMods()
private void updateSpecifics()
{
if (!this.IsCurrentScreen() || SelectedItem.Value is not PlaylistItem)
return;
var rulesetInstance = GetGameplayRuleset().CreateInstance();
Mods.Value = GetGameplayMods().Select(m => m.ToMod(rulesetInstance)).ToArray();
}
private void updateRuleset()
{
if (!this.IsCurrentScreen() || SelectedItem.Value is not PlaylistItem)
return;
Ruleset.Value = GetGameplayRuleset();
}
private void updateUserStyle()
{
if (!this.IsCurrentScreen() || SelectedItem.Value is not PlaylistItem)
return;
if (UserStyleDisplayContainer == null)
return;
PlaylistItem gameplayItem = SelectedItem.Value.With(ruleset: GetGameplayRuleset().OnlineID, beatmap: new Optional<IBeatmapInfo>(GetGameplayBeatmap()));
PlaylistItem? currentItem = UserStyleDisplayContainer.SingleOrDefault()?.Item;
if (gameplayItem.Equals(currentItem))
return;
UserStyleDisplayContainer.Child = new DrawableRoomPlaylistItem(gameplayItem)
if (UserStyleDisplayContainer != null)
{
AllowReordering = false,
AllowEditing = true,
RequestEdit = _ => OpenStyleSelection()
};
PlaylistItem gameplayItem = SelectedItem.Value.With(ruleset: GetGameplayRuleset().OnlineID, beatmap: new Optional<IBeatmapInfo>(GetGameplayBeatmap()));
PlaylistItem? currentItem = UserStyleDisplayContainer.SingleOrDefault()?.Item;
if (gameplayItem.Equals(currentItem))
return;
UserStyleDisplayContainer.Child = new DrawableRoomPlaylistItem(gameplayItem)
{
AllowReordering = false,
AllowEditing = true,
RequestEdit = _ => OpenStyleSelection()
};
}
}
protected virtual APIMod[] GetGameplayMods()