perf(Role): linear speed position getter

[kyranet]

Please describe the changes this PR makes and why it should be merged:

Alternative to #9490, fixes #3925.

Iterations	Old	New	Difference
10_000	631.24ms	174.44ms	27.63%
100_000	8.01s	1.89s	23.59%

Basically skips the sorted call, which is an algorithm with an O(n * log(n)) cost, as well as the indexOf, which is a O(n) call. With this, not only it's not longer allocating a Collection nor an Array (making the only allocation be the iterator used in reduce), but the performance also goes down to a single O(n).

🤖 Generated by Copilot at 8b28bc4

position getter
improved with reduce function
snowflakes in winter

Status and versioning classification:

• Code changes have been tested against the Discord API, or there are no code changes
• I know how to update typings and have done so, or typings don't need updating

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
discord-js	❌ Failed (Inspect)			May 5, 2023 0:45am
discord-js-guide	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 5, 2023 0:45am

[jaw0r3k]

Can you make the same for channels?

[kyranet]

Yes, in a follow-up PR, I wanted to push this one first to get reviews in case the code can be improved any further.

[jaw0r3k]

Yes, in a follow-up PR, I wanted to push this one first to get reviews in case the code can be improved any further.

Idk if using for loop wont be a little bit faster, as it will be not calling the function everytime
Except that it looks good

[iCrawl]

🐎

[github-actions]

⚡️ Lighthouse report for the changes in this PR:

Category	Score
🔴 Performance	39
🟢 Accessibility	98
🟢 Best practices	100
🟢 SEO	94
🟠 PWA	70

Lighthouse ran on https://discord-js-guide-git-fork-kyranet-perf-rolefas-cb6b68-discordjs.vercel.app/guide/home/introduction