From 3d4a97b4e9a4b0d3dedd400d497ee303cb7ad181 Mon Sep 17 00:00:00 2001
From: nikzori <kzr.niki@gmail.com>
Date: Thu, 27 Mar 2025 12:57:40 +0300
Subject: [PATCH] fixed and optimized port data processing

---
 Gidrolock Modbus Configurator.csproj |  2 +-
 Modbus.cs                            | 45 +++++++++++++---------------
 Properties/Resources.Designer.cs     |  4 +--
 Properties/Settings.Designer.cs      |  2 +-
 4 files changed, 24 insertions(+), 29 deletions(-)

diff --git a/Gidrolock Modbus Configurator.csproj b/Gidrolock Modbus Configurator.csproj
index 4e122e8..4c56a59 100644
--- a/Gidrolock Modbus Configurator.csproj	
+++ b/Gidrolock Modbus Configurator.csproj	
@@ -6,7 +6,7 @@
     <Platform Condition=" '$(Platform)' == '' ">AnyCPU</Platform>
     <ProjectGuid>{47A399D3-30C1-4EF4-9E94-F3518919A59C}</ProjectGuid>
     <OutputType>WinExe</OutputType>
-    <RootNamespace>Gidrolock_Modbus_Configurator</RootNamespace>
+    <RootNamespace>Gidrolock_Modbus_Scanner</RootNamespace>
     <AssemblyName>Gidrolock Modbus Configurator</AssemblyName>
     <TargetFrameworkVersion>v4.8</TargetFrameworkVersion>
     <FileAlignment>512</FileAlignment>
diff --git a/Modbus.cs b/Modbus.cs
index 716b8ce..ec3b7f7 100644
--- a/Modbus.cs
+++ b/Modbus.cs
@@ -14,7 +14,7 @@ namespace Gidrolock_Modbus_Scanner
     public static class Modbus
     {
         public static SerialPort port = new SerialPort();
-       
+
         public static event EventHandler<ModbusResponseEventArgs> ResponseReceived = delegate { };
 
         public static void Init()
@@ -276,38 +276,34 @@ namespace Gidrolock_Modbus_Scanner
         static int count = 0;
         static void PortDataReceived(object sender, EventArgs e)
         {
+
             //reset values on every event call;
             buffer = new byte[255];
             offset = 0;
             try
             {
-                //reset values on every event call;
-                buffer = new byte[255];
-                offset = 0;
-                try
+                stopwatch.Restart();
+                while (stopwatch.ElapsedMilliseconds < 20)
                 {
-                    stopwatch.Restart();
-                    while (stopwatch.ElapsedMilliseconds < 20)
+                    if (port.BytesToRead > 0)
                     {
-                        if (port.BytesToRead > 0)
-                        {
-                            stopwatch.Restart();
-                            count = port.BytesToRead;
-                            port.Read(buffer, offset, port.BytesToRead);
-                            offset += count;
-                        }
+                        stopwatch.Restart();
+                        count = port.BytesToRead;
+                        port.Read(buffer, offset, port.BytesToRead);
+                        offset += count;
                     }
-                    // assume that the message ended
+                }
+                // assume that the message ended
 
-                    List<byte> message = new List<byte>();
-                    for (int i = 0; i < offset; i++)
-                    {
-                        message.Add(buffer[i]);
-                    }
-                    if (message.Count == 0)
-                        return;
+                List<byte> message = new List<byte>();
+                for (int i = 0; i < offset; i++)
+                {
+                    message.Add(buffer[i]);
+                }
+                if (message.Count == 0)
+                    return;
 
-                    Console.WriteLine("Incoming message: " + ByteArrayToString(message.ToArray(), false));
+                Console.WriteLine("Incoming message: " + ByteArrayToString(message.ToArray(), false));
                 /*
                 if (!CheckResponse(message.ToArray()))
                 {
@@ -342,7 +338,6 @@ namespace Gidrolock_Modbus_Scanner
         }
 
     }
-
     public class ModbusResponseEventArgs : EventArgs
     {
         public byte[] Message { get; set; }
@@ -362,7 +357,7 @@ namespace Gidrolock_Modbus_Scanner
                 }
                 //Console.WriteLine("Read data: " + Modbus.ByteArrayToString(Data, false));
             }
-            else Data = new byte[1] {0x0F};
+            else Data = new byte[1] { 0x0F };
         }
     }
 
diff --git a/Properties/Resources.Designer.cs b/Properties/Resources.Designer.cs
index b9a4de3..2198345 100644
--- a/Properties/Resources.Designer.cs
+++ b/Properties/Resources.Designer.cs
@@ -8,7 +8,7 @@
 // </auto-generated>
 //------------------------------------------------------------------------------
 
-namespace Gidrolock_Modbus_Configurator.Properties {
+namespace Gidrolock_Modbus_Scanner.Properties {
     using System;
     
     
@@ -39,7 +39,7 @@ namespace Gidrolock_Modbus_Configurator.Properties {
         internal static global::System.Resources.ResourceManager ResourceManager {
             get {
                 if (object.ReferenceEquals(resourceMan, null)) {
-                    global::System.Resources.ResourceManager temp = new global::System.Resources.ResourceManager("Gidrolock_Modbus_Configurator.Properties.Resources", typeof(Resources).Assembly);
+                    global::System.Resources.ResourceManager temp = new global::System.Resources.ResourceManager("Gidrolock_Modbus_Scanner.Properties.Resources", typeof(Resources).Assembly);
                     resourceMan = temp;
                 }
                 return resourceMan;
diff --git a/Properties/Settings.Designer.cs b/Properties/Settings.Designer.cs
index 6b9c2f5..df0c1cb 100644
--- a/Properties/Settings.Designer.cs
+++ b/Properties/Settings.Designer.cs
@@ -8,7 +8,7 @@
 // </auto-generated>
 //------------------------------------------------------------------------------
 
-namespace Gidrolock_Modbus_Configurator.Properties {
+namespace Gidrolock_Modbus_Scanner.Properties {
     
     
     [global::System.Runtime.CompilerServices.CompilerGeneratedAttribute()]