.NET OpenXML-performance-Probleme

Ich bin versucht zu schreiben, eine Excel-Datei aus einer ASP.NET web-server mithilfe von OpenXML. Ich habe über 2100 Datensätze und das dauert etwa 20-30 Sekunden, dies zu tun. Irgendeiner Weise ich kann, damit es schneller geht? Abrufen der 2100 Zeilen aus der db dauert den Bruchteil einer Sekunde. Nicht sicher, warum die Manipulation in den Speicher nehmen würde, nicht mehr.

Hinweis: ExcelWriter ist unsere individuelle Klasse, aber alle seine Methoden sind direkt von der code in diesem link, http://msdn.microsoft.com/en-us/library/cc861607.aspx

   public static MemoryStream CreateThingReport(List<Thing> things, MemoryStream template)
    {
        SpreadsheetDocument spreadsheet = SpreadsheetDocument.Open(template, true);
        WorksheetPart workSheetPart = spreadsheet.WorkbookPart.WorksheetParts.First();

        SharedStringTablePart sharedStringPart = spreadsheet.WorkbookPart.GetPartsOfType<SharedStringTablePart>().First();

        Cell cell = null;
        int index = 0;

        //create cell formatting for header text
        Alignment wrappedAlignment = new Alignment { WrapText = true };
               uint rowOffset = 2;

  foreach (Thing t in things)
        {
            //Received Date
            cell = ExcelWriter.InsertCellIntoWorksheet("A", rowOffset, workSheetPart);
            index = ExcelWriter.InsertSharedStringItem(t.CreateDate.ToShortDateString(), sharedStringPart);
            cell.CellValue = new CellValue(index.ToString());
            cell.DataType = new DocumentFormat.OpenXml.EnumValue<CellValues>(CellValues.SharedString);

            //Car Part Name
            cell = ExcelWriter.InsertCellIntoWorksheet("B", rowOffset, workSheetPart);
            index = ExcelWriter.InsertSharedStringItem(t.CarPart.Name, sharedStringPart);
            cell.CellValue = new CellValue(index.ToString());
            cell.DataType = new DocumentFormat.OpenXml.EnumValue<CellValues>(CellValues.SharedString);

  rowOffset++; 
   }

 workSheetPart.Worksheet.Save();

        spreadsheet.WorkbookPart.Workbook.Save();
        spreadsheet.Close();

        return template;

InformationsquelleAutor The Internet | 2012-05-10

.net asp.net-mvc-3 c#openxml openxml-sdk

So sieht es aus wie jemand Sie in der MSDN-community docs lief in ähnliche Auswirkungen auf die Leistung. Der code unten ist sehr ineffizient. Mir wurde empfohlen, eine hash-Tabelle.

Für unsere Lösung, die wir gerade entfernt haben, die Einfügung der shared strings zusammen und gingen von 1:03 Sekunden auf 0:03 Sekunden download-Zeit.

//Old: (1:03)
            cell = ExcelWriter.InsertCellIntoWorksheet("A", rowOffset, workSheetPart);
            index = ExcelWriter.InsertSharedStringItem(thing.CreateDate.ToShortDateString(), sharedStringPart);
            cell.CellValue = new CellValue(index.ToString());
            cell.DataType = new DocumentFormat.OpenXml.EnumValue<CellValues>(CellValues.SharedString);

 //New: (0:03)
             cell = ExcelWriter.InsertCellIntoWorksheet("A", rowOffset, workSheetPart);
             cell.CellValue = new CellValue(thing.CreateDate.ToShortDateString());
              cell.DataType = new DocumentFormat.OpenXml.EnumValue<CellValues>(CellValues.String);

MSDN Docs (slow-Lösung, sollten Sie verwenden eine Hash-Tabelle statt)

      private static int InsertSharedStringItem(string text, SharedStringTablePart         shareStringPart)
  {
//If the part does not contain a SharedStringTable, create one.
if (shareStringPart.SharedStringTable == null)
{
    shareStringPart.SharedStringTable = new SharedStringTable();
}

int i = 0;

//Iterate through all the items in the SharedStringTable. If the text already exists, return its index.
foreach (SharedStringItem item in shareStringPart.SharedStringTable.Elements<SharedStringItem>())
{
    if (item.InnerText == text)
    {
        return i;
    }

    i++;
}

//The text does not exist in the part. Create the SharedStringItem and return its index.
shareStringPart.SharedStringTable.AppendChild(new SharedStringItem(new DocumentFormat.OpenXml.Spreadsheet.Text(text)));
shareStringPart.SharedStringTable.Save();

return i;
 }

Ich stehe vor demselben Problem... muss ich schreiben, 1000+ Zeilen und in einigen Fällen 10000+ Zeilen und es ist langsam wie die Hölle... Hier sagte Sie, dass Sie verwenden können hash-Tabelle können Sie geben ein Beispiel, wie? oder wenn irgendeine andere Sache, die Sie verwendet, um die Leistung zu verbessern...
Ich freue mich auf 500K Zeilen. Haben Sie andere Verbesserungen, da diese post, die Sie teilen konnte? Ich zog nach der SAX-Methode, die Speichernutzung zu minimieren. Und ich über 1000 Zeilen pro ~1,1 Sekunden. Wenn Sie schneller als diese bitte teilen.

InformationsquelleAutor The Internet

5

@Internet

Beachten Sie, dass String-Datentyp ist eigentlich für die Formeln für text verwenden sollte InlineString.
Sehen 17.18.11 ST_CellType (Cell-Typ):
- inlineStr (Inline String) - Zelle mit (inline -) reiche string,
  d.h., einen, der nicht in der shared Strings-Tabelle. Wenn diese Zelle Typ verwendet wird,
  dann ist die Zelle Wert in der ist-element anstatt dem v-element in
  die Zelle (c-element).
- str (String) - Zelle mit einer Formel-string.
InformationsquelleAutor user3285954
4

Die große Verbesserung ist mehr Save () - Funktion aus der Schleife
```
 //Save data
        shareStringPart.SharedStringTable.Save();
        worksheetPart.Worksheet.Save();
```
Für 500 Datensätze, für mich ist es ein Wechsel von 10 Minuten auf 1 min.
- Dies ist ein wirklich wichtiger Teil der Arbeit mit OpenXML - da viele der Bestandteile erfordern eine sparen, wir legen eher den speichern im inneren einzigartige action-Methoden oder Schleifen, anstatt im außen Bereich, wo der save ist schneller.
- Entfernen übermäßige 'Save ()' - Anrufe hat die Leistung verbessern, aber ich fand unter Umgehung der Shared Strings-Tabelle zusammen und speichern von text als 'InLineString' der Schnellste zu sein, für ein paar tausend Zeilen. Drei Stunden auf weniger als eine minute.
InformationsquelleAutor zquanghoangz

@kunjee

Wenn Sie wollen Leistung erstellen Sie alle erforderlichen Objekte im Voraus, so dass nicht überprüft bei jedem Aufruf dieser Methode. Dies ist der Grund, warum SharedStringTable übergeben wird als parameter anstelle von dem Teil.

Wörterbücher sind für die schnelle, indizierte lookup, haben eine bessere Leistung als eine for-Schleife. Sind bisschen schneller als hashtables, da sind stark typisiert, so ist nicht erforderlich-Boxen. Stark typisierte ist ein großer Vorteil sowieso.

private static int InsertSharedStringItem(string sharedString, SharedStringTable sharedStringTable, Dictionary<string, int> sharedStrings)
{
    int sharedStringIndex;

    if (!sharedStrings.TryGetValue(sharedString, out sharedStringIndex))
    {
        //The text does not exist in the part. Create the SharedStringItem now.
        sharedStringTable.AppendChild(new SharedStringItem(new Text(sharedString)));

        sharedStringIndex = sharedStrings.Count;

        sharedStrings.Add(sharedString, sharedStringIndex);
    }

    return sharedStringIndex;
}

InformationsquelleAutor user3285954

Wie bereits erwähnt, durch Das Internet, Sie hätten einen Hashtable-und, wie vorgeschlagen, durch zquanghoangz die Sie haben sollten, verschoben die Save() aus der Schleife.

InlineString funktioniert, aber es gibt MS-Excel-Kopfschmerzen beim öffnen der erzeugten Datei mit sinnlose Fehlermeldungen, die repariert werden können, aber gibt noch ein nerviges pop-up.

static Cell AddCellWithSharedStringText(
    [NotNull]string text, 
    [NotNull]Hashtable texts, 
    [NotNull]SharedStringTablePart shareStringPart
)
{
    if (!texts.ContainsKey(text))
    {
        shareStringPart.SharedStringTable.AppendChild(new SharedStringItem(new Text(text)));
        texts[text] = texts.Count;
    }
    var idx = (int)texts[text];
    Cell c1 = new Cell();
    c1.DataType = CellValues.SharedString;
    c1.CellValue = new CellValue(idx.ToString());
    return c1;
}

Diese Lösung brachte der export-down Zeit von ~5 Minuten und 6 Sekunden auf eine [9880 x 66] grid.

InformationsquelleAutor Karel Wesseling

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.