Octobot/src/Extensions/UserExtensions.cs at 133cc8886590417c3f49c479c247eee271f56d41 - TeamInklings/Octobot - Forge Basement

TeamInklings/Octobot

mirror of https://github.com/TeamOctolings/Octobot.git synced 2025-05-05 05:26:28 +03:00

Octol1ttle e6f53b13f0

Split extension methods into separate classes (#161 )

This PR splits the extension methods contained in `Extensions.cs` into
separate classes in the `Octobot.Extensions` namespace. This was done
for multiple reasons:
1) The `Extensions.cs` violates SRP (Single Responsibility Principle) -
it takes upon itself every extension method for many types
2) Having a separate class for each extended type is a standard practice
- take a look at
[Remora.Discord](https://github.com/Remora/Remora.Discord/tree/main/Backend/Remora.Discord.Rest/Extensions)
or [osu!](https://github.com/ppy/osu/tree/master/osu.Game/Extensions)
3) Having all extension methods in one file makes it hard to find the
method you want

2023-10-12 15:37:25 +00:00

11 lines

294 B

C#

Raw Blame History

 using Remora.Discord.API.Abstractions.Objects;
 namespace Octobot.Extensions;
 public static class UserExtensions
 {
     public static string GetTag(this IUser user)
     {
         return user.Discriminator is 0000 ? $"@{user.Username}" : $"{user.Username}#{user.Discriminator:0000}";
     }
 }