From a9d21c4aa3a540bff6b7c85cfcc256a8e67c08dd Mon Sep 17 00:00:00 2001
From: Niko Ehrenfeuchter <nikolaus.ehrenfeuchter@unibas.ch>
Date: Tue, 13 Mar 2018 12:21:48 +0100
Subject: [PATCH] Convert field to an auto-property with a private setter.
Refers to #36
---
ATxCommon/Monitoring/Cpu.cs | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/ATxCommon/Monitoring/Cpu.cs b/ATxCommon/Monitoring/Cpu.cs
index f2e4e7f..2071129 100644
--- a/ATxCommon/Monitoring/Cpu.cs
+++ b/ATxCommon/Monitoring/Cpu.cs
@@ -43,7 +43,6 @@ namespace ATxCommon.Monitoring
private readonly PerformanceCounter _cpuCounter;
private readonly float[] _loadReadings = {0F, 0F, 0F, 0F};
- private float _load;
private int _interval;
private int _limit;
private int _behaving;
@@ -54,7 +53,7 @@ namespace ATxCommon.Monitoring
/// Current CPU load (usage percentage over all cores), averaged of the last four readings.
/// </summary>
/// <returns>The average CPU load from the last four readings.</returns>
- public float Load() => _load;
+ public float Load { get; private set; }
/// <summary>
/// How often (in ms) to check the CPU load.
@@ -140,7 +139,7 @@ namespace ATxCommon.Monitoring
// ConstrainedCopy seems to be the most efficient approach to shift the array:
Array.ConstrainedCopy(_loadReadings, 1, _loadReadings, 0, 3);
_loadReadings[3] = _cpuCounter.NextValue();
- _load = _loadReadings.Average();
+ Load = _loadReadings.Average();
if (_loadReadings[3] > _limit) {
if (_behaving > _probation) {
// this means the load was considered as "low" before, so raise an event:
--
GitLab