memory usage increase a lot when calling Sheet.Rows.Length

[Panpan03]

Hi there,

I found a problem that memory usage increase a lot when calling Sheet.Rows.Length. I am wondering is it memory leak or by design?

Test code shows as below

public MainWindow()
{
InitializeComponent();

var workbook = new Spire.Xls.Workbook();
workbook.LoadFromFile("test.xlsx");
var sheet1 = workbook.Worksheets[0];
for (int i = 0; i < sheet1.Rows.Length; i++) //memory usage increases more tha 10MB each loop.
{
//do nothing
}
}

thanks,
Pan

[Triste.Dai]

Hi,

Thanks for your feedback.
I have tested the latest version 13.9.1 of our product, but I was unable to reproduce the problem you described. Typically, there should be no memory increase if no variables are being manipulated within the loop.

To further investigate this matter, could you please confirm whether you are using the latest commercial version of our product? If not, we kindly suggest updating to the latest version and retrying the test. Additionally, if the issue persists after the update, it would be helpful if you could provide us with the test document and details about your testing environment.

We appreciate your cooperation in resolving this matter. Should you have any further questions or concerns, please feel free to contact us. We are here to assist you.

Best regards,
Triste
E-iceblue support team

[Panpan03]

HI Triste,

Thanks for your quick verification, I am currently using version 13.9.1.0, and the code was built by Visual studio 2019.

You can watch the process memory by debugging the code.

Please find the attached test code and excel document.

thanks,
Pan

[Triste.Dai]

Hi Pan,

Thank you for your feedback.
I have successfully reproduced the issue you reported and consulted our development team regarding this matter. They explained that when using the sheet1.Rows.Length property, the CellRange array inside it is rebuilt every time. Considering the large number of rows in your document, this results in increased memory usage with each iteration of the for loop.

To optimize the process, we suggest using one of the following approaches:

Option 1:

Code: Select all

CellRange[] rows = sheet1.Rows;
for (int i = 0; i < rows.Length; i++)
{
    // Your code here
}

Option 2:

Code: Select all

int rowCount = sheet1.Rows.Length;
for (int i = 0; i < rowCount; i++)
{
    // Your code here
}

By implementing either of these approaches, you can avoid unnecessary memory consumption during each iteration. Please let us know if you have any further questions or concerns.

Thank you for bringing this to our attention, and we appreciate your patience and understanding.

Best regards,
Triste
E-iceblue support team

[Panpan03]

Get it，thanks a lot！

[Triste.Dai]

Hi Pan,

Thanks for your feedback. If you have any other questions, just feel free to contact us.

Have a nice day!

Best regards,
Triste
E-iceblue support team