RepoMirrors/osu
1
0
Fork 0
mirror of https://github.com/ppy/osu synced 2025-01-22 05:43:14 +00:00
Code Issues Packages Projects Releases Wiki Activity

Make constructors design more pleasent to eyes

Browse Source
This commit is contained in:
Salman Ahmed 2021-01-03 18:34:58 +03:00
parent 152e9ecccf
commit c8423d1c46
1 changed files with 6 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
osu.Game/Online/Rooms/BeatmapAvailability.cs
View File

@ -20,35 +20,17 @@ namespace osu.Game.Online.Rooms
/// </summary> /// </summary>
public readonly double? DownloadProgress; public readonly double? DownloadProgress;
/// <summary> private BeatmapAvailability(DownloadState state, double? downloadProgress = null)
/// Constructs a new non-<see cref="DownloadState.Downloading"/> beatmap availability state.
/// </summary>
/// <param name="state">The beatmap availability state.</param>
/// <exception cref="ArgumentException">Throws if <see cref="DownloadState.Downloading"/> was specified in this constructor, as it has its own constructor (see <see cref="BeatmapAvailability(DownloadState, double)"/>.</exception>
public BeatmapAvailability(DownloadState state)
{ {
if (state == DownloadState.Downloading)
throw new ArgumentException($"{nameof(DownloadState.Downloading)} state has its own constructor, use it instead.");
State = state; State = state;
DownloadProgress = null;
}
/// <summary>
/// Constructs a new <see cref="DownloadState.Downloading"/>-specific beatmap availability state.
/// </summary>
/// <param name="state">The beatmap availability state (always <see cref="DownloadState.Downloading"/>).</param>
/// <param name="downloadProgress">The beatmap's downloading current progress.</param>
/// <exception cref="ArgumentException">Throws if non-<see cref="DownloadState.Downloading"/> was specified in this constructor, as they have their own constructor (see <see cref="BeatmapAvailability(DownloadState)"/>.</exception>
public BeatmapAvailability(DownloadState state, double downloadProgress)
{
if (state != DownloadState.Downloading)
throw new ArgumentException($"This is a constructor specific for {DownloadState.Downloading} state, use the regular one instead.");
State = DownloadState.Downloading;
DownloadProgress = downloadProgress; DownloadProgress = downloadProgress;
} }
public static BeatmapAvailability NotDownload() => new BeatmapAvailability(DownloadState.NotDownloaded);
public static BeatmapAvailability Downloading(double progress) => new BeatmapAvailability(DownloadState.Downloading, progress);
public static BeatmapAvailability Downloaded() => new BeatmapAvailability(DownloadState.Downloaded);
public static BeatmapAvailability LocallyAvailable() => new BeatmapAvailability(DownloadState.LocallyAvailable);
public bool Equals(BeatmapAvailability other) => other != null && State == other.State && DownloadProgress == other.DownloadProgress; public bool Equals(BeatmapAvailability other) => other != null && State == other.State && DownloadProgress == other.DownloadProgress;
public override string ToString() => $"{string.Join(", ", State, $"{DownloadProgress:0.00%}")}"; public override string ToString() => $"{string.Join(", ", State, $"{DownloadProgress:0.00%}")}";