Auslesen von Strings aus einer xls-Datei

[Andy A.]

Ich arbeite momentan an einem Programm, das Bestellungen aus einer xls-Datei auslesen soll. Dazu wird die vom Benutzer eingegebene Bestellnummer mit der Bestellnummer aus der Datei verglichen. Das Auslesen klappt soweit, allerdings gibt es Bestellungen mit der selben Bestellnummer und die equals()-Methode nimmt nur den ersten Datensatz aus der Tabelle mit der eingegebenen Bestellnummer. Kann mir vielleicht jemand sagen, wie man alle Datensätze mit der selben Bestellnummer auslesen kann?

Gruß Andy

 

[SlaterB]

> und die equals()-Methode nimmt [..]

redest du von irgendeinem Framework, zu dem man etwas wissen soll
oder hast du individuellen Code mit BufferedReader usw?
wie soll man den dann korrigieren, ohne ihn zu kennen?

 

[Gast]

if (eingabeauftr.equals(currentStateCell.toString())) {
System.out.print(currentRow.getRowNum() + 1 + " ");
for (int i = 0; i < currentRow.getLastCellNum(); i++){
System.out.print(currentRow.getCell((short) i).toString() + " ");
}
}

so, sieht der Code im Moment aus. Das Problem ist halt, dass die equals()-Methode nur den ersten Eintrag mit der Bestellnummer nimmt und dir anderen alle nicht beachtet.

 

[SlaterB]

aber warum?
für alles muss es doch im Code einen Grund geben,

dein Code zeigt, wie mit currentStateCell umgegangen wird,

es ist aber nicht zu sehen welche Datensätze derart überprüft werden,
ob alle oder einige, ob nach erfolgreichen Fund abgebrochen wird oder nicht usw.

wenn dieser Code für alle Datensätze durchlaufen wird, dann wird equals auch oft genug true liefern
oder die Strings stimmen doch nicht überein, was du mit System.out.printlns oder einem Debugger genauer untersuchen musst

 

[Andy A.]

Es funktioniert alles soweit gut, er liest auch den Datensatz mit der richtigen Bestellnummer aus, aber jedoch nur den ersten. So ist die equals()-Methode programmiert. Sie kann nur den ersten Datensatz auslesen, die anderen alle ignoriert sie. Deshalb suche ich jetzt nen Weg, wie man das anders machen kann, sodass alle Datensätze mit der eingegeben Bestellnummer ausgelesen werden können. Ich hoffe, es ist klarer geworden, was ich wirklich meine.

Gruß Andy

 

[SlaterB]

das equals kümmert sich nur darum, ob zwei Strings gleich sind,
es kann nicht darüber entscheiden, welche oder wieviele Datensätze gelesen werden,

es ist die Frage, woher du currentStateCell und currentRow herbekommst,
falls irgendwo etwas per BufferedReader eingelesen wird, so muss an diese Stelle eine Schleife,
bzw. klingt ja nach Apache POI für Excel-Dateien, aber auch da gibts Programmsteuerung per Schleifen

mehr Code..

 

[FArt]

... dann musst du halt über alles Einträge iterieren und jedes mal mit equals() prüfen, ob dich der Eintrag interessiert...

 

[Andy A.]

public String[] getAuftrag(String eingabeauftr, String filename) {

try {

fs = new POIFSFileSystem(new FileInputStream(filename));
hssfworkbook = new HSSFWorkbook(fs);

sheet = hssfworkbook.getSheetAt(0);

currentStateCell = null;
currentRow = sheet.getRow(0);

String[] feld = new String[currentRow.getLastCellNum()];

while (rowCount < sheet.getLastRowNum() + 1) {

try {

currentRow = sheet.getRow(rowCount++);
currentStateCell = currentRow.getCell(0);

if (eingabeauftr
.equals(currentStateCell.toString()))) {
for (int i = 0; i < currentRow.getLastCellNum(); i++) {

feld = currentRow.getCell(i)
.toString();
}
tab = 0;
feld1 = feld;
break;
}
if (currentRow.getRowNum() == sheet
.getLastRowNum()) {

sheet = hssfworkbook.getSheetAt(tab++);

if (sheet.getLastRowNum() == 0) {
JOptionPane.showMessageDialog(null,
"Die Suche ergab keinen Treffer!");
tab = 0;

break;
}

rowCount = 0;

}

} catch (java.lang.NullPointerException f) {
JOptionPane.showMessageDialog(null,
"Die Suche ergab keinen Treffer!");
break;
} catch (java.lang.NumberFormatException g) {
System.out.println(g.getMessage());
}
}
} catch (java.io.FileNotFoundException e) {
JOptionPane.showMessageDialog(null, "Datei wurde nicht gefunden!");
} catch (IOException e) {
JOptionPane.showMessageDialog(null, "Datei wurde nicht gefunden!");
}
return feld1;
}


So, hier ist der Code, hoffe, ihr könnt mir helfen .

 

[SlaterB]

eine Schleife hast du anscheinend schon:
> while (rowCount < sheet.getLastRowNum() + 1) {

jetzt gib doch mal VOR dem
if (eingabeauftr
.equals(currentStateCell.toString()))) {

currentStateCell pro Zeile aus, schau dir die Werte an, ob alle drankommen, die du haben möchtest

 

[SlaterB]

Befehle wie
> break;
oder
> rowCount = 0;
sind natürlich auch nicht förderlich für eine korrekte normale Abarbeitung der Schleife

try/ catch NullPointerException ist ganz schlimm

 

[Andy A.]

Also, ich habs gemacht (Ich hoffe, ich habe dich richtig verstanden) und sobald er einen Treffer beim Vergleichen landet, vergleicht er die anderen Datensätze nicht mehr. Er bricht einfach an der Stelle ab.

 

[SlaterB]

siehe mein letztes Post,
wenn du nicht weißt was dein Code macht, dann ist das natürlich schlecht,

 

[Andy A.]

Naja, also "rowCount = 0" ist schon notwendig, weil er mehrere Tabellen in der Datei durchlaufen muss. Dies muss er natürlich von der ersten Zeile aus machen.

 

[SlaterB]

auf jeden Fall steht am Ende von
if (eingabeauftr.equals(currentStateCell.toString()))) {
eindeutig ein break;
und daher ist die Schleife zu Ende

das ist doch leicht einzusehen?
hättest du evtl. ganz oben im dritten Posting drin lassen sollen,
dann wäre das schnell klar gewesen

 

[Andy A.]

Ohne "break" befindet sich das Prog. in einer Endlos-Schleife und die
"try/ catch NullPointerException" ist dafür da, falls sich in der Datei eine leere Zeile befindet.

 

[SlaterB]

> Ohne "break" befindet sich das Prog. in einer Endlos-Schleife

ich behaupte nein, denn rowCount++ wird immer um 1 erhöht,
falls das nicht gerade durch eine geniale Exception-Lösung übersprungen wird oder, was bei dir wahrscheinlich passiert, ständig wieder auf 0 gesetzt wird..

und while (rowCount < sheet.getLastRowNum() + 1) {
liefert dann false

> try/ catch NullPointerException" ist dafür da, falls sich in der Datei eine leere Zeile befindet.

wie wärs mit if (x == null)?


egal, es geht ja jetzt um das Lesen weiterer Datensätze,
und dazu ist mein Tipp, dass dieses break weg muss und etwaige Probleme anders gelöst werden müssen,
kannst du so annehmen oder auch ignorieren, wie du willst

 

[Andy A.]

Ja da hast du recht. Ich kann "break" und "rowCount = 0" auch weglassen, das löst das Problem aber nicht, das ist eine zusätzliche Barriere, denn dann liest er die anderen Tabellen der Datei nicht aus. Aber über andere Lösung nachzudenken, ist natürlich ne Möglichkeit. Trotzdem danke!

Gruß Andy