iotexproject / iotexproject/iotex-core

Candidate State Manager and Candidate State Reader

Open
#4,301 0 comments 0 reactions 0 assignees View on GitHub
refactor
Dominant language
Go
Stars
1.6k
Forks
382
Avg merge
4d 22h
Merged PRs (30d)
17

Description

1. CandidateStateManager extends CandidateStateReader
2. Clean up the two interfaces
3. Use appropriate one, e.g., use StateReader in the readonly functions
### Description of the code refactor

### Why is this needed
- [] better/less memory usage
- [] reduce running time
- [] clean (easy-to-understand) data structure or algorithm
- [] remove dead code
- [] better code design or structure (like reduce complexity)
- [] reduce/decouple dependencies between code modules
- [] increase test coverage
- [] better code readability

Contributor guide

Open the contributing guide

Research direction

Start by locating the CandidateStateManager and CandidateStateReader definitions and their usages, especially the readonly functions. Determine how the two interfaces should be cleaned up and which functions should use StateReader; the work is done when the intended interface separation is implemented and relevant tests pass.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
blockchain
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.