From 7b6be39dbd8cf9632750bd959548fb700a9a702c Mon Sep 17 00:00:00 2001 From: Arad Alvand Date: Wed, 13 Sep 2023 07:10:29 +0330 Subject: [PATCH] Use `.ToCharArray()` instead of `.ToArray()` on `string` The former is preferable performance-wise. See https://stackoverflow.com/a/37139124 --- src/Sqids/SqidsEncoder.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/Sqids/SqidsEncoder.cs b/src/Sqids/SqidsEncoder.cs index 9b541c5..eff3787 100644 --- a/src/Sqids/SqidsEncoder.cs +++ b/src/Sqids/SqidsEncoder.cs @@ -112,7 +112,7 @@ public SqidsEncoder(SqidsOptions options) ); _blockList = options.BlockList.ToArray(); // NOTE: Arrays are faster to iterate than HashSets, so we construct an array here. - _alphabet = options.Alphabet.ToArray(); + _alphabet = options.Alphabet.ToCharArray(); ConsistentShuffle(_alphabet); }