Convert shee celldata to JSON and load cell data from JSON

[hjrb]

I'm using this code to convert the Cell data to JSON and load cell data from JSON

`
record CellContent(int Row, int Col, object? Value, string? Formula, CellFormat? Format, string Type, CellValueType ValueType);

JsonSerializerOptions options = new JsonSerializerOptions
	{
		WriteIndented = true
	};

public static string ToJson(Sheet sheet, JsonSerializerOptions? options = null)
{
	var cells = sheet.Cells;
	var cellContents = new List<CellContent>();

	for (int row = 0; row < sheet.NumRows; row++)
	{
		for (int col = 0; col < sheet.NumCols; col++)
		{
			var cell=cells[row, col];
			if (cell.Value != null || cell.Formula != null)
			{
				var cellContent = new CellContent(row, col, cell.Value, cell.Formula, cell.Format.Clone(), cell.Type, cell.ValueType);
				cellContents.Add(cellContent);
			}
		}
	}
	return JsonSerializer.Serialize(cellContents, options);
}

private static Sheet LoadFromJson(string cellDataJson, JsonSerializerOptions? options = null)
{
	var cellContents = JsonSerializer.Deserialize<List<CellContent>>(cellDataJson, options);
	var rows= cellContents.MaxBy(a => a.Row).Row+1; if (rows < 10) rows = 10;
	var cols=cellContents.MaxBy(a => a.Col).Col+1; if (cols < 10) cols = 10;
	var sheet = new Sheet(rows, cols);
	var cells = sheet.Cells;
	foreach (var cellContent in cellContents)
	{
		var cell=sheet.Cells[cellContent.Row, cellContent.Col];
		cell.Type = cellContent.Type;
		if (cellContent.Formula is not null)
		{
			cell.Formula = cellContent.Formula;
		}
		else
		{
			if (cellContent.Value is not null) cell.Value = cellContent.Value.ToString();
		}
		if (cellContent.Format is not null) cell.Format = cellContent.Format;
	}
	return sheet;
}

`

Including the row and column as JSON properties has the advantage that only those cell be included that either has a value or a formula, which reduces data volume. The value 10 for the minimum number of columns and rows is abitrary and could be made an optional parameter.
Any comments appreciated.

[deanparker-web]

Who's this Jason guy that everyone keeps talking about?

Great readable code, lots of optimizations and I appreciate you sharing your code.

The smaller data would improve performance passing data via API. Which I will implement in my PWA app this week.

[anmcgrath]

Looks interesting @hjrb. I've coincidentally been working on some sheet serialization to JSON in the branch json-serialization