Skip to content

Commit 371cbba

Browse files
committed
refactoring
1 parent 799794e commit 371cbba

File tree

2 files changed

+14
-14
lines changed

2 files changed

+14
-14
lines changed

src/coverlet.core/CoverageDetails.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,14 +6,14 @@ internal class CoverageDetails
66
{
77
private double _averageModulePercent;
88

9-
public Modules Modules { get; internal set; }
9+
public int ModulesCount { get; internal set; }
1010
public double Covered { get; internal set; }
1111
public int Total { get; internal set; }
1212
public double Percent
1313
{
1414
get
1515
{
16-
if (Modules?.Count == 0) return 0;
16+
if (ModulesCount == 0) return 0;
1717
return Total == 0 ? 100D : Math.Floor((Covered / Total) * 10000) / 100;
1818
}
1919
}

src/coverlet.core/CoverageSummary.cs

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -52,12 +52,12 @@ public CoverageDetails CalculateLineCoverage(Documents documents)
5252

5353
public CoverageDetails CalculateLineCoverage(Modules modules)
5454
{
55-
var details = new CoverageDetails{Modules = modules};
56-
var accumPercent = 0.0D;
57-
58-
if (modules.Count == 0)
55+
var details = new CoverageDetails{ModulesCount = modules.Count};
56+
if (details.ModulesCount == 0)
5957
return details;
6058

59+
var accumPercent = 0.0D;
60+
6161
foreach (var module in modules)
6262
{
6363
var moduleCoverage = CalculateLineCoverage(module.Value);
@@ -155,12 +155,12 @@ public CoverageDetails CalculateBranchCoverage(Documents documents)
155155

156156
public CoverageDetails CalculateBranchCoverage(Modules modules)
157157
{
158-
var details = new CoverageDetails{ Modules = modules };
159-
var accumPercent = 0.0D;
160-
161-
if (modules.Count == 0)
158+
var details = new CoverageDetails { ModulesCount = modules.Count };
159+
if (details.ModulesCount == 0)
162160
return details;
163161

162+
var accumPercent = 0.0D;
163+
164164
foreach (var module in modules)
165165
{
166166
var moduleCoverage = CalculateBranchCoverage(module.Value);
@@ -219,12 +219,12 @@ public CoverageDetails CalculateMethodCoverage(Documents documents)
219219

220220
public CoverageDetails CalculateMethodCoverage(Modules modules)
221221
{
222-
var details = new CoverageDetails{ Modules = modules };
223-
var accumPercent = 0.0D;
224-
225-
if (modules.Count == 0)
222+
var details = new CoverageDetails { ModulesCount = modules.Count };
223+
if (details.ModulesCount == 0)
226224
return details;
227225

226+
var accumPercent = 0.0D;
227+
228228
foreach (var module in modules)
229229
{
230230
var moduleCoverage = CalculateMethodCoverage(module.Value);

0 commit comments

Comments
 (0)