MaibornWolff / metric-gardener

BSD 3-Clause "New" or "Revised" License
5 stars 0 forks source link

Change coupling metric #360

Open ChristianHuehn opened 1 month ago

ChristianHuehn commented 1 month ago

File coupling metric is currently not further described on how and what it will count. This should be fixed according to this list:

// kinda equivalent to const a: A = new A(); const b: B = a.buildB(); const c: C = b.buildC(); const d: D = c.buildD(); d.doStuff();


This means that you need knowledge over the implementation of the other classes and therefor coupling them together.
- [ ] We count statics too

### Background information ###
It can be helpful to also check the visual studio implementation of it: https://learn.microsoft.com/en-us/visualstudio/code-quality/code-metrics-class-coupling?view=vs-2022
BridgeAR commented 1 month ago

We should ideally have different type of coupling metrics, not only a single one.

ChristianHuehn commented 1 month ago

Exactly, that is our plan 👍 . But first we want a basic version, that works and is tested. Before we move on adding new coupling metrics.