serial port memmory problem

Question

guys i have make some other post before few months! i cand find solution please help me if someone know any solution!!

im reading with serial port from a micro controller board and i have a problem!
the program works OK but the memory usage increasing! begins with 13mb and goes to 100mb in an hour

i have try to use
System.GC.Collect();
System.GC.WaitForPendingFinalizers();

in the

private void serialPort1_DataReceived(object sender, System.IO.Ports.SerialDataReceivedEventArgs e)
{

RxString = serialPort1.ReadLine();

this.Invoke(new EventHandler(DisplayText));
System.GC.Collect();
System.GC.WaitForPendingFinalizers();

}

but still nothing!! any idea?

Ryan Alford · Answer

use BeginInvoke instead of Invoke. can you also show the code that you are using?

Bechir Bejaoui · Answer

you can wrapp your code in a try catch blocs to prevent the application from interruption try{ //code } catch(Exception caught){}

george · Answer

guys i found a way and works but isome times i get in this line label1.Invoke(tickerDelegate1 new object[] { _serialPort.ReadLine() }); the error input string is not in correct format! but not always!! is any way to avoid this? thanks

Bechir Bejaoui · Answer

The problem of the micro controler is that there is not sufficient memory cache for example 6k or at least 20k it realy problem

Ryan Alford · Answer

However the code is unnecessarily bloated.

Bechir Bejaoui · Answer

No rayan the problem is not residing in communicating via port the problem is that george deals with a micro controler and not an entreprise application the microcontroler this is the constraint

Ryan Alford · Answer

Your code is so bloated no wonder you are having memory issues. Here is a better and easier way of communicating to a serial port: ============================================= public partial class Form1 : Form { SerialPort _serialPort; // delegate is used to write to a UI control from a non-UI thread private delegate void SetTextDeleg(string text); private void Form1_Load(object sender EventArgs e) { _serialPort = new SerialPort(COM5 19200 Parity.None 8 StopBits.One Handshake.None); _serialPort.DataReceived += new SerialDataReceivedEventHandler(sp_DataReceived); _serialPort.ReadTimeout = 500; _serialPort.WriteTimeout = 500; _serialPort.Open(); } void sp_DataReceived(object sender SerialDataReceivedEventArgs e) { Thread.Sleep(500); string data = _serialPort.ReadLine(); this.BeginInvoke(new SetTextDeleg(si_DataReceived) new object[] { data }); } private void si_DataReceived(string data) { textBox1.Text = data.Trim(); } } ============================================= 1. There is need to create string objects for the SerialPort's settings. The values can be passed to the constructor of the SerialPort class 2. There is absolutely no need explicitly create a new Thread since the SerialDataReceivedEventHandler occurs on a different thread than the UI thread.

Bechir Bejaoui · Answer

I talked about your problem with an freind who is an electronic engineer he said to me that the microcontroller programmer co worker has tried to flash a program leveraged with delphi in a given micro controler and he encountred the same problem as you because the oriented object languages in general are gourmant in terms of memory and there didn't give a good performance with the small memory capacity micro controler then it is worth to leverage that using C even C++ could produce some secondary effects it is tried with the central car lock system and it causes some problems

george · Answer

IL try to change my software! so searching in INTERNET i found this code! displays the data from serial port to command line console!!! this code works and takes only 6mb in ram and never increasing! but i have try to insert the code in a form but i cant make this!! IL post the code working to console mode and my trying to change to win32 form //////////////////////////// console mode working using System; using System.IO.Ports; using System.Threading; public class PortChat { static bool _continue; static SerialPort _serialPort; public static void Main() { string name; string message; string portName; string baudRate; string parity; string dataBits; string stopBits; string handshake; portName = COM5; baudRate = 19200; parity = None; dataBits = 8; stopBits = One; handshake = None; StringComparer stringComparer = StringComparer.OrdinalIgnoreCase; Thread readThread = new Thread(Read); // Create a new SerialPort object with default settings. _serialPort = new SerialPort(); // Allow the user to set the appropriate properties. _serialPort.PortName = portName; _serialPort.BaudRate = Convert.ToInt16(baudRate); _serialPort.Parity = (Parity)Enum.Parse(typeof(Parity) parity); _serialPort.DataBits = int.Parse(dataBits); _serialPort.StopBits = (StopBits)Enum.Parse(typeof(StopBits) stopBits); _serialPort.Handshake = (Handshake)Enum.Parse(typeof(Handshake) handshake); // Set the read/write timeouts _serialPort.ReadTimeout = 500; _serialPort.WriteTimeout = 500; _serialPort.Open(); _continue = true; readThread.Start(); Console.Write(Name: ); name = Console.ReadLine(); Console.WriteLine(Type QUIT to exit); while (_continue) { message = Console.ReadLine(); if (stringComparer.Equals(quit message)) { _continue = false; } else { _serialPort.WriteLine( String.Format( : {1} name message)); } } readThread.Join(); _serialPort.Close(); } public static void Read() { while (_continue) { try { string message = _serialPort.ReadLine(); Console.WriteLine(message); } catch (TimeoutException) { } } } } ///////////////////////////////win32 form problem with te separate thread //////// i cant see any data in textbox1 if i don't add the Serial(); inside public Form1() but the form loads normally without an error //////// and if i add Serial(); inside public Form1 as below!!! the form never loads but the serial communication begins and works but i get no errors /// i believe is the thread but i cant find a way using System; using System.Collections.Generic; using System.ComponentModel; using System.Data; using System.Drawing; using System.Linq; using System.Text; using System.Windows.Forms; using System.IO.Ports; using System.Threading; namespace test_pci { public partial class Form1 : Form { public bool _continue; public SerialPort _serialPort; public string name; public string message; public string portName; public string baudRate; public string parity; public string dataBits; public string stopBits; public string handshake; public Form1() { InitializeComponent(); portName = COM5; baudRate = 19200; parity = None; dataBits = 8; stopBits = One; handshake = None; Serial(); } public void Serial() { StringComparer stringComparer = StringComparer.OrdinalIgnoreCase; Thread readThread = new Thread(Read); // Create a new SerialPort object with default settings. _serialPort = new SerialPort(); // Allow the user to set the appropriate properties. _serialPort.PortName = portName; _serialPort.BaudRate = Convert.ToInt16(baudRate); _serialPort.Parity = (Parity)Enum.Parse(typeof(Parity) parity); _serialPort.DataBits = int.Parse(dataBits); _serialPort.StopBits = (StopBits)Enum.Parse(typeof(StopBits) stopBits); _serialPort.Handshake = (Handshake)Enum.Parse(typeof(Handshake) handshake); // Set the read/write timeouts _serialPort.ReadTimeout = 500; _serialPort.WriteTimeout = 500; _serialPort.Open(); _continue = true; readThread.Start(); Console.Write(Name: ); name = Console.ReadLine(); //Console.WriteLine(Type QUIT to exit); while (_continue) { //message = Console.ReadLine(); if (stringComparer.Equals(quit message)) { _continue = false; } else { _serialPort.WriteLine( String.Format( : {1} name message)); } } readThread.Join(); _serialPort.Close(); } private void Form1_Load(object sender EventArgs e) { } public void Read() { while (_continue) { try { //string message = _serialPort.ReadLine(); //Console.WriteLine(message); textBox1.Text = _serialPort.ReadLine(); } catch (TimeoutException) { } } } } }

Ryan Alford · Answer

There has got to be something else you are doing. can you post the code of the DisplayText method? You should NEVER call the Garbage Collector. It is a HUGE performance hit.

Bechir Bejaoui · Answer

Ok george the problem is that C# is used within chipsets like ARM processors or equivalent the problem that C# couldn't be used especially when the needed memory is less than 100 mb but nevertheless there is a platform sepecialized to that kind of system called The Microsoft .NET Micro Framework I can't guarantee that could solve your problem but you can always test it