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

Apply NRT to GameplayLeaderboardScore and change GetDisplayedScore handling

Browse Source 
I don't feel too confident with the default scoring function being
assigned in the constructor to a publicly settable delegate. This just
feels a bit more elegant, and handles the (likely-never-used) case where
we need to restore the default function.

An alternative would be to provide the function as a `ctor` argument,
but I believe that wasn't done here to allow using the
`ILeaderboardScore` interface.
This commit is contained in:
Dean Herbert 2023-05-29 19:46:50 +09:00
parent fcd7a1d51a
commit 1a6d9e9ff0
2 changed files with 21 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
osu.Game/Screens/Play/HUD/GameplayLeaderboardScore.cs
View File

@ -1,10 +1,7 @@
// Copyright (c) ppy Pty Ltd <contact@ppy.sh>. Licensed under the MIT Licence.
// See the LICENCE file in the repository root for full licence text.
#nullable disable
using System;
using JetBrains.Annotations;
using osu.Framework.Allocation;
using osu.Framework.Bindables;
using osu.Framework.Extensions.Color4Extensions;
@ -50,7 +47,7 @@ namespace osu.Game.Screens.Play.HUD
public Bindable<bool> Expanded = new Bindable<bool>();
private OsuSpriteText positionText, scoreText, accuracyText, comboText, usernameText;
private OsuSpriteText positionText = null!, scoreText = null!, accuracyText = null!, comboText = null!, usernameText = null!;
public BindableLong TotalScore { get; } = new BindableLong();
public BindableDouble Accuracy { get; } = new BindableDouble(1);
@ -58,7 +55,12 @@ namespace osu.Game.Screens.Play.HUD
public BindableBool HasQuit { get; } = new BindableBool();
public Bindable<long> DisplayOrder { get; } = new Bindable<long>();
public Func<ScoringMode, long> GetDisplayScore { get; set; }
private Func<ScoringMode, long>? getDisplayScoreFunction;
public Func<ScoringMode, long> GetDisplayScore
{
set => getDisplayScoreFunction = value;
}
public Color4? BackgroundColour { get; set; }
@ -86,32 +88,31 @@ namespace osu.Game.Screens.Play.HUD
}
}
[CanBeNull]
public IUser User { get; }
public IUser? User { get; }
/// <summary>
/// Whether this score is the local user or a replay player (and should be focused / always visible).
/// </summary>
public readonly bool Tracked;
private Container mainFillContainer;
private Container mainFillContainer = null!;
private Box centralFill;
private Box centralFill = null!;
private Container backgroundPaddingAdjustContainer;
private Container backgroundPaddingAdjustContainer = null!;
private GridContainer gridContainer;
private GridContainer gridContainer = null!;
private Container scoreComponents;
private Container scoreComponents = null!;
private IBindable<ScoringMode> scoreDisplayMode;
private IBindable<ScoringMode> scoreDisplayMode = null!;
/// <summary>
/// Creates a new <see cref="GameplayLeaderboardScore"/>.
/// </summary>
/// <param name="user">The score's player.</param>
/// <param name="tracked">Whether the player is the local user or a replay player.</param>
public GameplayLeaderboardScore([CanBeNull] IUser user, bool tracked)
public GameplayLeaderboardScore(IUser? user, bool tracked)
{
User = user;
Tracked = tracked;
@ -242,7 +243,7 @@ namespace osu.Game.Screens.Play.HUD
Origin = Anchor.CentreLeft,
Colour = Color4.White,
Font = OsuFont.Torus.With(size: 14, weight: FontWeight.SemiBold),
Text = User?.Username,
Text = User?.Username ?? string.Empty,
Truncate = true,
Shadow = false,
}
@ -313,11 +314,6 @@ namespace osu.Game.Screens.Play.HUD
HasQuit.BindValueChanged(_ => updateState());
}
private void updateScore()
{
scoreText.Text = GetDisplayScore(scoreDisplayMode.Value).ToString("N0");
}
protected override void LoadComplete()
{
base.LoadComplete();
@ -328,6 +324,8 @@ namespace osu.Game.Screens.Play.HUD
FinishTransforms(true);
}
private void updateScore() => scoreText.Text = (getDisplayScoreFunction?.Invoke(scoreDisplayMode.Value) ?? TotalScore.Value).ToString("N0");
private void changeExpandedState(ValueChangedEvent<bool> expanded)
{
if (expanded.NewValue)

6
osu.Game/Screens/Play/HUD/ILeaderboardScore.cs
View File

@ -1,8 +1,6 @@
// Copyright (c) ppy Pty Ltd <contact@ppy.sh>. Licensed under the MIT Licence.
// See the LICENCE file in the repository root for full licence text.
#nullable disable
using System;
using osu.Framework.Bindables;
using osu.Game.Rulesets.Scoring;
@ -24,8 +22,10 @@ namespace osu.Game.Screens.Play.HUD
Bindable<long> DisplayOrder { get; }
/// <summary>
/// A function providing a display score. If a custom function is not provided, this defaults to using <see cref="TotalScore"/>.
/// A custom function which handles converting a score to a display score using a provide <see cref="ScoringMode"/>.
/// </summary>
/// <remarks>
/// If no function is provided, <see cref="TotalScore"/> will be used verbatim.</remarks>
Func<ScoringMode, long> GetDisplayScore { set; }
}
}