ekolis / ekolis/FrEee

Convert simple classes that mainly just store data to records

Open
#315 1 comment 0 reactions 0 assignees View on GitHub
performance technical debt
Dominant language
C#
Stars
86
Forks
14
PR merge metrics
No merged PRs in 30d

Description

**What needs to be cleaned up?**

There are a lot of simple classes that mainly just store some data and don't have much in the way of references to other classes that need to maintain referential integrity. Convert them to data classes. Umm, I mean, records, this is not Kotlin!

**How will this benefit us?**

Records are supposed to have better performance than classes I think? And the syntax is simpler since properties can be converted to constructor parameters and equals/hashcode/etc are auto-implemented. This means that you don't need to copy them to reuse them because equality "just works".

**What potential drawbacks are there to making this change?**

Just the time consumed doing it and any potential bugs introduced. Also records are immutable so that could be a problem.

**Candidate classes**
* GridColumnConfig

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.