dotnet-ad / dotnet-ad/Mvvmicro

Optimization: Avoid the encapsulation of stored factories in the container

Open
#7 0 comments 0 reactions 0 assignees View on GitHub
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.