Simplification of code

Open Beginner friendly
#1,445 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
1/5
Estimated time
Under an hour
Newbie friendliness
72/100
Issue type
Documentation
Clarity
Clearly specified
Activity status
Stale
Tech stack
csharp
Domain
documentation

Research direction

Open xml/System.Linq/Enumerable.xml and find the Enumerable.ToLookup documentation example linked by the issue. Replace the Convert.ToChar(p.Company.Substring(0, 1)) selector with the simpler character-indexer form, then verify that the example remains valid and accurately reflects the documented API.

Written by the indexing model from the issue text.

Description

area-System.Linq dotnet-api/prod Pri2 untriaged

In the code example, .ToLookup(p => Convert.ToChar(p.Company.Substring(0, 1)) is an overkill when a simple code such as .ToLookup(p => p.Company[0] achieves same result given that String class supports char indexer to fetch the underlying value.

Document Details

⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

Dominant language
C#
Stars
950
Forks
1.7k
Avg merge
3d 5h
Merged PRs (30d)
38

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from dotnet/dotnet-api-docs

All issues in dotnet/dotnet-api-docs

Similar issues

More C# issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.