Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New exercise for Asynchronous programming in C# #1894

Open
mdrakiburrahman opened this issue Mar 6, 2022 · 3 comments
Open

New exercise for Asynchronous programming in C# #1894

mdrakiburrahman opened this issue Mar 6, 2022 · 3 comments

Comments

@mdrakiburrahman
Copy link
Contributor

It would be great to have an exercise on Async programming as it is used in many real world dotnet applications.

The Golang track had one called "Prallel Letter Frequency"that introduce Goroutines, which I found quite useful.

@ErikSchierboom
Copy link
Member

We have the same exercise (https://github.com/exercism/csharp/tree/main/exercises/practice/parallel-letter-frequency), but it looks like the implementation is actually wrong and uses synchronous code. Would you be interested in converting the exercise to allow for asynchronous (rather: parallel) programming?

@mdrakiburrahman
Copy link
Contributor Author

@ErikSchierboom - sure! I've never contributed a scenario to Exercism - but if you can point me the necessary reqs (Unit testing etc) and assign the issue to me, I'm happy to contribute.

@ErikSchierboom
Copy link
Member

@mdrakiburrahman You don't even have to create a new scenario, just to update the existing one to accommodate for people using parallel execution. So what I think needs to be done is to change the Calculate method to return a Task<Dictionary<char, int>> instead of Dictionary<char, int>. This means updating three files:

Finally, you should also add yourself to the exercise's contributors list in https://github.com/exercism/csharp/blob/main/exercises/practice/parallel-letter-frequency/.meta/config.json#L5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants