dotnet-ad / dotnet-ad/Mvvmicro
Optimization: Avoid the encapsulation of stored factories in the container
Open
- Dominant language
- C#
- Stars
- 21
- Forks
- 5
- PR merge metrics
- No merged PRs in 30d
Description
The factories in the container are stored in a function `() => factory(this)`, pushing a additional function on the stack when using the factory.
The factory could be stored as is, and calls modified as `factories[type].Item2(this)` to avoid this excendentary call.
Contributor guide
No contributing guide indexed for this repository
Research direction
Locate the container implementation and the stored factories described in the issue, then trace the calls indexed by type. Compare the current wrapped-factory path with direct factory invocation and verify that factory behavior is unchanged while the extra call is removed.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp
- Domain
- mobile-dev
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100