Hey @henon @ScarletKuro what's your opinion on this idea?

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.57%. Comparing base (28bc599) to head (5b1b285).
Report is 385 commits behind head on dev.

@@            Coverage Diff             @@
##              dev    #9493      +/-   ##
==========================================
+ Coverage   89.82%   90.57%   +0.74%     
==========================================
  Files         412      407       -5     
  Lines       11878    12715     +837     
  Branches     2364     2463      +99     
==========================================
+ Hits        10670    11516     +846     
+ Misses        681      638      -43     
- Partials      527      561      +34     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Wouldn't something like this be better if we chase performance / allocation?

private const string Chars = "abcdefghijklmnopqrstuvwxyz0123456789";
private const int RandomStringLength = 8;

internal static string Create(ReadOnlySpan<char> prefix)
{
    Span<char> identifierSpan = stackalloc char[prefix.Length + RandomStringLength];
    prefix.CopyTo(identifierSpan);

    Span<byte> randomBytes = stackalloc byte[RandomStringLength];
    RandomNumberGenerator.Fill(randomBytes);

    for (int i = 0; i < RandomStringLength; i++)
    {
        identifierSpan[prefix.Length + i] = Chars[randomBytes[i] % Chars.Length];
    }

    return identifierSpan.ToString();
}

I feel like GUID is too heavy just to get 8 chars from a 32-char GUID. I don't think we need super uniqueness, and RandomNumberGenerator is a true cryptographic random generator.

Sure, would suffice. I do something similar in my apps actually.

You are right. I didn't think about that, great idea. However, the allocations on the heap are unchanged, but it's faster. Now I have the following code, but "only" the Random instance:

private const string Chars = "abcdefghijklmnopqrstuvwxyz0123456789";
private const int CharsLength = 35;
private const int RandomStringLength = 8;

internal static string Create(ReadOnlySpan<char> prefix)
{
    Span<char> identifierSpan = stackalloc char[prefix.Length + RandomStringLength];
    prefix.CopyTo(identifierSpan);

    for (var i = 0; i < RandomStringLength; i++)
    {
        var index = Random.Shared.Next(CharsLength);
        identifierSpan[prefix.Length + i] = Chars[index];
    }

    return identifierSpan.ToString();
}

with these results:

I would update all references where the identifier can be used 👍

I have three notes:

• I'm not sure about the prefix in MudComponentBase. Currently, it's mudinput but maybe mud or the default prefix would fit better.
• Should the new identifier class be public? Then others could use it as well.
• Are you ok with the name of the class and the method?

• I'm not sure about the prefix in MudComponentBase. Currently, it's mudinput but maybe mud or the default prefix would fit better.

As long as it doesn't break any unit tests, etc., then I'm okay with the rename. The shorter, the better.

• Should the new identifier class be public? Then others could use it as well.

No.

• Are you ok with the name of the class and the method?

Yes.

As long as it doesn't break any unit tests

Okay, they do break. I'll leave it as it is. PR is ready 👍.

Thanks!