From 770e9488fad76cdf987e6992f84ff4064b5afcfb Mon Sep 17 00:00:00 2001
From: wanshenmean <cathay_xy@163.com>
Date: 星期五, 13 三月 2026 13:28:54 +0800
Subject: [PATCH] fix: address code quality issues in I/Q/T/C regions
---
Code/WCS/WIDESEAWCS_S7Simulator/WIDESEAWCS_S7Simulator.Core/Memory/CRegion.cs | 53 +++++++++++++++++++++++++++--------------------------
1 files changed, 27 insertions(+), 26 deletions(-)
diff --git a/Code/WCS/WIDESEAWCS_S7Simulator/WIDESEAWCS_S7Simulator.Core/Memory/CRegion.cs b/Code/WCS/WIDESEAWCS_S7Simulator/WIDESEAWCS_S7Simulator.Core/Memory/CRegion.cs
index f5a7b62..b0aa20f 100644
--- a/Code/WCS/WIDESEAWCS_S7Simulator/WIDESEAWCS_S7Simulator.Core/Memory/CRegion.cs
+++ b/Code/WCS/WIDESEAWCS_S7Simulator/WIDESEAWCS_S7Simulator.Core/Memory/CRegion.cs
@@ -11,11 +11,6 @@
public class CRegion : MemoryRegion, IMemoryRegion
{
/// <summary>
- /// 鏍囪瘑瀵硅薄鏄惁宸茶閲婃斁
- /// </summary>
- private bool _disposed = false;
-
- /// <summary>
/// 鍖哄煙绫诲瀷
/// </summary>
public override string RegionType => "C";
@@ -26,11 +21,24 @@
private const int CounterSize = 2;
/// <summary>
+ /// 璁℃暟鍣ㄦ暟閲�
+ /// </summary>
+ public int CounterCount => Size / CounterSize;
+
+ /// <summary>
/// 鏋勯�犲嚱鏁�
/// </summary>
/// <param name="counterCount">璁℃暟鍣ㄦ暟閲�</param>
+ /// <exception cref="ArgumentOutOfRangeException">褰揷ounterCount灏忎簬鎴栫瓑浜�0鏃舵姏鍑�</exception>
+ /// <exception cref="OverflowException">褰揷ounterCount * CounterSize瓒呭嚭int鑼冨洿鏃舵姏鍑�</exception>
public CRegion(int counterCount) : base(counterCount * CounterSize)
{
+ if (counterCount <= 0)
+ throw new ArgumentOutOfRangeException(nameof(counterCount), "璁℃暟鍣ㄦ暟閲忓繀椤诲ぇ浜�0");
+
+ // 楠岃瘉娌℃湁婧㈠嚭
+ if (counterCount > Size / CounterSize)
+ throw new OverflowException($"璁℃暟鍣ㄦ暟閲弡counterCount}杩囧ぇ锛岃秴鍑哄唴瀛橀檺鍒�");
}
/// <summary>
@@ -38,10 +46,15 @@
/// </summary>
/// <param name="counterNumber">璁℃暟鍣ㄥ彿锛堜粠1寮�濮嬶級</param>
/// <returns>璁℃暟鍣ㄥ��</returns>
+ /// <exception cref="ArgumentOutOfRangeException">褰揷ounterNumber涓�0鎴栬秴鍑鸿鏁板櫒鏁伴噺鏃舵姏鍑�</exception>
public ushort ReadCounter(ushort counterNumber)
{
- if (_disposed)
- throw new ObjectDisposedException(nameof(CRegion));
+ if (counterNumber == 0)
+ throw new ArgumentOutOfRangeException(nameof(counterNumber), "璁℃暟鍣ㄥ彿蹇呴』澶т簬0锛堜粠1寮�濮嬶級");
+
+ if (counterNumber > CounterCount)
+ throw new ArgumentOutOfRangeException(nameof(counterNumber),
+ $"璁℃暟鍣ㄥ彿{counterNumber}瓒呭嚭鑼冨洿锛屽綋鍓嶈鏁板櫒鎬绘暟涓簕CounterCount}");
ushort offset = (ushort)((counterNumber - 1) * CounterSize);
var data = Read(offset, CounterSize);
@@ -53,31 +66,19 @@
/// </summary>
/// <param name="counterNumber">璁℃暟鍣ㄥ彿锛堜粠1寮�濮嬶級</param>
/// <param name="value">璁℃暟鍣ㄥ��</param>
+ /// <exception cref="ArgumentOutOfRangeException">褰揷ounterNumber涓�0鎴栬秴鍑鸿鏁板櫒鏁伴噺鏃舵姏鍑�</exception>
public void WriteCounter(ushort counterNumber, ushort value)
{
- if (_disposed)
- throw new ObjectDisposedException(nameof(CRegion));
+ if (counterNumber == 0)
+ throw new ArgumentOutOfRangeException(nameof(counterNumber), "璁℃暟鍣ㄥ彿蹇呴』澶т簬0锛堜粠1寮�濮嬶級");
+
+ if (counterNumber > CounterCount)
+ throw new ArgumentOutOfRangeException(nameof(counterNumber),
+ $"璁℃暟鍣ㄥ彿{counterNumber}瓒呭嚭鑼冨洿锛屽綋鍓嶈鏁板櫒鎬绘暟涓簕CounterCount}");
ushort offset = (ushort)((counterNumber - 1) * CounterSize);
var data = new byte[] { (byte)(value >> 8), (byte)(value & 0xFF) };
Write(offset, data);
- }
-
- /// <summary>
- /// 閲婃斁璧勬簮
- /// </summary>
- /// <param name="disposing">鏄惁姝ｅ湪閲婃斁鎵樼璧勬簮</param>
- protected override void Dispose(bool disposing)
- {
- if (!_disposed)
- {
- if (disposing)
- {
- // 閲婃斁鎵樼璧勬簮
- }
- _disposed = true;
- }
- base.Dispose(disposing);
}
}
}
--
Gitblit v1.9.3