Java xlsx zu csv

[Lia]

Hi

Versuche eben eine xlsx zu einer Csv umzuwandeln.

Beim umwandeln werden aber alle leeren Zellen ignoriert. Dadurch tut er es nicht richtig umwandeln, weil er die Zellen nach links verschiebt und dadurch eine falsche Ausgabe generiert.

Habe in der Spalte

switch(cellType){

...

case : BLANK
sb.append(";;");
}

Ignoriert das total.

 

[kneitzel]

Kannst Du da mehr Details nennen? Was für Libraries nutzt Du?

Und wenn er dies ignoriert: Mal im Debugger geschaut? Evtl. ist cellType nicht BLANK sondern irgend etwas anderes?

 

[mihe7]

@Lia guckst Du z. B. hier https://gist.github.com/robertchong/11071949 - müsstest Du wahrscheinlich ein wenig aktualisieren.

 

[Lia]

Hi @kneitzel 🙂

ne es ist leer also blank habe auch Mal eine xlsx mit leeren Zellen erstellt. Und ein System.out eingebaut aber er geht da erst gar nicht rein.

 

[kneitzel]

@Lia Damit hast Du aber weiter keine weiteren Details genannt. Wir wissen somit weiterhin nicht, was cellType genau ist. Wenn er aber nicht bei case: BLANK rein geht, dann ist es mit relativ große Wahrscheinlichkeit wohl nicht BLANK. Da kann man aber im Debugger meist recht gut nachvollziehen.

Zeig doch den ganzen Code, der da wichtig ist, so dass wir Dir helfen können. So mit Glaskugel und so funktioniert das hier leider nicht.

Oder Du versuchst Dich an einer fertigen Lösung wie z.B. von @mihe7 vorgeschlagen.

 

[Xyz1]

Siehst Du einmal hier:

HSSF and XSSF Examples

poi.apache.org

                // Open the workbook
                this.openWorkbook(excelFile);
    
                // Convert it's contents into a CSV file
                this.convertToCSV();
    
                // Build the name of the csv folder from that of the Excel workbook.
                // Simply replace the .xls or .xlsx file extension with .csv
                destinationFilename = excelFile.getName();
                destinationFilename = destinationFilename.substring(
                        0, destinationFilename.lastIndexOf(".")) +
                        ToCSV.CSV_FILE_EXTENSION;
    
                // Save the CSV file away using the newly constricted file name
                // and to the specified directory.
                this.saveCSVFile(new File(destination, destinationFilename));

Umgekehrt ist es viel schwerer... klar!

 

[Lia]

@kneitzel

while(cellIterator.hasNext()){
Cell cell=cellIterator.next();
switch(cell.getCellType()){
case STRING:...

case BLANK:
sb.append(";");
}
}

So sieht der Abschnitt aus.

 

[mihe7]

Sowohl in der Antwort von z. B. @Tobias-nrw als auch von mir wird nicht via CellIterator über die Zellen iteriert. Probier es doch mal so:

            int lastCellNum = row.getLastCellNum();
            for(int i = 0; i <= lastCellNum; i++) {
                cell = row.getCell(i);
                if (cell == null) {
                    sb.append(";");
                } else switch(cell.getCellType()) {
                     case STRING: ...
                }
            }

 

[Lia]

Danke @mihe7
Ach mennno 🙁 das funktioniert auch nicht :/

 

[Xyz1]

@mihe7 Sind jetzt switches über Types erlaubt?

Bearbeitung: Nee, gerad nachgelesen: https://stackoverflow.com/a/29570906

 

[kneitzel]

Also gut, es scheint also so zu sein, dass Du die POI API von Apache verwendest. (Eine Aussage, die hilfreich gewesen wäre).

Bezüglich Iterator:
http://poi.apache.org/components/spreadsheet/quick-guide.html#Iterator
"Note that a rowIterator and cellIterator iterate over rows or cells that have been created, skipping empty rows and cells."

Und direkt danach dann ein Absatz, wie man auch leere Zellen bekommt.

Der darin wichtige Punkt ist, dass man getCell(i, MissingCellPolicy) benutzt, um eine leere Zelle zu bekommen. Aber die leeren Zellen musst Du ja nicht prüfen, denn das cell==null sollte schon die notwendige Prüfung sein.

 

[kneitzel]

@mihe7 Sind jetzt switches über Types erlaubt?

CellType ist ein enum, oder welches switch meintest Du?

 

[Xyz1]

Wie viele leere Zellen hat eigentlich ein unendliche Tabelle? 😀 😀

 

[Xyz1]

CellType ist ein enum, oder welches switch meintest Du?

Du warst nicht angesprochen...

 

[kneitzel]

Du warst nicht angesprochen...

Was für eine tolle Aussage von Dir schon wieder. Ist es Dir jetzt peinlich, dass Du dies nicht selbst gesehen hast? Und dass ausgerechnet ich Dich darauf aufmerksam machen musste? Oder was ist das jetzt für eine tolle Reaktion?

Wenn Du nur eine Antwort von @mihe7 haben willst, dann schreib ihm eine PN. Oder lebe schlicht damit, dass auch Andere Dir antworten. (Und nein, etwas zu übersehen muss einem nicht peinlich sein. So Kleinkind-Verhalten aber schon!)

 

[Lia]

Habe ich etwas falsches gesagt @kneitzel vielen Dank für den Tipp! Und allen anderen auch. Ich probiere es gleich Mal aus 🙂

 

[kneitzel]

Habe ich etwas falsches gesagt

Nein, alles ok. Für mich war einfach nur nicht sofort klar, welche Library Du nutzt. Apache POI ist wohl eine sehr verbreitete Library, aber es gibt eben auch noch andere. Daher hatte ich da etwas nachgehakt und es dann aus dem Kontext am Ende geschlossen.

Und irgendwelche "Zickereien" zwischen @Tobias-nrw und mir ignorierst Du am Besten. Da geht es auch nicht um Dich oder Deine Themen...

 

[Lia]

Achso ok alles klar ich probiere es später Mal mit cell==null zu implementieren und schicke es Mal hier rein wenn es wieder nicht funktioniert 😳😀

 

[Lia]

@kneitzel habe es damit probiert :

wb.setMissingCellPolicy(MissingCellPolicy.RETURNNULLANDBLANK);

und habe den Zelltyp BLANK trotzdem ganz normal implementiert aber trotzdem ignoriert der Compiler die leere Zelle und verschiebt die Zellen ..

 

[kneitzel]

In dem Link stand es aber auch anders 🙂

Also in der Dokumentation, die ich verlinkt habe, wurde die Policy direkt im getCell benutzt und kein CellIterator verwendet. Der CellIterator ist diesbezüglich limitiert ("The CellIterator will only return the cells defined in the file, which is largely those with values or stylings, but it depends on Excel.")

Ich habe mal den Code, so wie er in der Dokumentation war, genommen und habe da einen Part noch angepasst, da Du ja scheinbar kein null als Cell haben willst (zweite Schleife):

package de.kneitzel.apachepoi;

import org.apache.poi.ss.usermodel.Cell;
import org.apache.poi.ss.usermodel.Row;
import org.apache.poi.ss.usermodel.Sheet;
import org.apache.poi.xssf.usermodel.XSSFWorkbook;

import java.io.IOException;
import java.util.Iterator;

public class XlsxTest {
    public static void main(String[] args) throws IOException {
        XSSFWorkbook workbook = new XSSFWorkbook("c:/Projects/javatests/test.xlsx");
        Sheet datatypeSheet = workbook.getSheetAt(0);

        // Decide which rows to process
        int rowStart = datatypeSheet.getFirstRowNum();
        int rowEnd = datatypeSheet.getLastRowNum();
        System.out.println("Rows: " + rowStart + " - " + rowEnd);

        //  Handle empty cells with null
        System.out.println("Using RETURN_BLANK_AND_NULL");
        for (int rowNum = rowStart; rowNum <= rowEnd; rowNum++) {
            Row r = datatypeSheet.getRow(rowNum);
            if (r == null) {
                // This whole row is empty
                // Handle it as needed
                continue;
            }

            int lastColumn = Math.max(r.getLastCellNum(), 1);

            for (int cn = 0; cn < lastColumn; cn++) {
                Cell c = r.getCell(cn, Row.MissingCellPolicy.RETURN_BLANK_AS_NULL);
                if (c == null) {
                    System.out.println("Empty Cell!");
                } else {
                    System.out.println(c.getAddress().toString() + ": " + c.getCellType().toString());
                }
            }
        }

        // Handle empty cells als BLANK cell.
        System.out.println("Using CREATE_NULL_AS_BLANK");
        for (int rowNum = rowStart; rowNum <= rowEnd; rowNum++) {
            Row r = datatypeSheet.getRow(rowNum);
            if (r == null) {
                // This whole row is empty
                // Handle it as needed
                continue;
            }

            int lastColumn = Math.max(r.getLastCellNum(), 1);

            for (int cn = 0; cn < lastColumn; cn++) {
                Cell c = r.getCell(cn, Row.MissingCellPolicy.CREATE_NULL_AS_BLANK);
                System.out.println(c.getAddress().toString() + ": " + c.getCellType().toString());
            }
        }
    }
}

Der erste Part entspricht damit auch dem, was @mihe7 geschrieben hat. Leere Cell wird halt als null zurück gegeben. So man da aber eine BLANK Cell haben will, dann kann man halt bei getCell die RowMissingCellPolicy.CREATE_NULL_AS_BLANK verwenden.