Einlesen der CSV-Datei zeilenweise ein und analysieren es

Ich habe eine CSV Datei, die ich brauche, um zu Lesen, Zeile für Zeile mit Hilfe eines Scanner und speichern nur Land-Namen in ein array von strings. Hier ist meine CSV-Datei:

World Development Indicators
Number of countries,4
Country Name,2005,2006,2007
Bangladesh,6.28776238,13.20573922,23.46762823
"Bahamas,The",69.21279415,75.37855087,109.340767
Brazil,46.31418452,53.11025849,63.67475185
Germany,94.55486999,102.2828888,115.1403608

Dies ist, was ich habe, so weit:

public String[] getCountryNames() throws IOException, FileNotFoundException{
    String[] countryNames = new String[3];
    int index = 0;
    BufferedReader br = new BufferedReader(new FileReader(fileName));
    br.readLine();
    br.readLine();
    br.readLine();
    String line = br.readLine();
    while((br.readLine() != null) && !line.isEmpty()){
        String[] countries = line.split(",");
        countryNames[index] = countries[0];
        index++;
        line = br.readLine();
    }
    System.out.println(Arrays.toString(countryNames));
    return countryNames;
}

Ausgabe:

[Bangladesh, Brazil, null]

Für einige Grund, es überspringt die "Bahamas" und kann nicht Lesen, Deutschland. Bitte helfen Sie mir, habe ich fest an diese Methode für die Stunden schon. Vielen Dank für Ihre Zeit und Mühe. Die Rückgabe sollte ein array von Strings (Land name).

  • Verwenden Sie eine geeignete parsing-Bibliothek, wie Apache Commons CSV oder OpenCSV
  • Wann immer Sie anrufen br.readline() Sie sind effektiv eine neue Zeile Lesen. Es spielt keine Rolle, wenn Sie tun, es innerhalb oder außerhalb der Schleife oder einfach nur zum Vergleich.
InformationsquelleAutor Dmytro Marych | 2018-01-29
  1. 0

    Gibt es zwei Probleme mit dem code für die Analyse dieser CSV-Datei. Ein paar Leute haben darauf hingewiesen, die Sie aufrufen readLine auf Ihre Leser zu viele Male, und verwirft den Ausgang. Jedes mal, wenn Sie aus dem stream gelesen, Sie verlieren den Zugriff auf alle Daten, bevor Sie die aktuelle lese-Punkt. So reader.readLine() != null, zum Beispiel, liest die neuen Daten aus dem stream, prüft, dass es nicht null ist, und bekommt dann sofort Los, da Sie noch nicht gespeichert sind, in eine variable. Das ist der wichtigste Grund, warum Sie sind, den Verlust von Daten während des Lesens.

    Das zweite Problem ist die split-Bedingung. Sie Spalten sich auf Kommas, das macht Sinn, da diese eine CSV-Datei, aber Ihre Daten enthält Kommas zu verwenden (zum Beispiel, "der Bahamas, Die"). Sie benötigen ein spezifischer split Zustand, wie beschrieben in dieser Beitrag.

    Hier ist ein Beispiel, wie dies Aussehen könnte (unter Verwendung einer Liste für die countryNames anstelle von einem array, denn das ist viel einfacher, mit zu arbeiten):

    private static final String csv = "World Development Indicators\n"
    + "Number of countries,4\n"
    + "Country Name,2005,2006,2007\n"
    + "Bangladesh,6.28776238,13.20573922,23.46762823\n"
    + "\"Bahamas,The\",69.21279415,75.37855087,109.340767\n"
    + "Brazil,46.31418452,53.11025849,63.67475185\n"
    + "Germany,94.55486999,102.2828888,115.1403608\n";

public static String[] getCountryNames() throws Exception {
    List<String> countryNames = new ArrayList<>();

    //BufferedReader br = new BufferedReader(new FileReader(fileName));
    BufferedReader br = new BufferedReader(new StringReader(csv));
    br.readLine();
    br.readLine();
    br.readLine();

    String line = br.readLine();
    while (line != null && !line.isEmpty()) {
        String[] countries = line.split(",(?=(?:[^\"]*\"[^\"]*\")*[^\"]*$)", -1);
        countryNames.add(countries[0]);
        line = br.readLine();
    }

    System.out.println(countryNames);
    return countryNames.toArray(new String[0]);
}
    • Das regex ist langsam, da es zum Scannen der rest der Zeile jedes mal, wenn es auf ein Komma. Der code wird auch nicht in den Griff zitiert eine multi-line-Wert. Wäre viel besser, eine aktuelle CSV-Parser. --- Upvote für die Befestigung while - Schleife und die Verwendung von List. Downvote für langsame bzw. unvollständige CSV-parsing. Keine Wahl, entweder Weg von mir.
    • Ja, mit einer tatsächlichen CSV-parser wäre die ideale alternative, aber ich dachte, der Hinweis würde nicht sehr hilfreich sein für die OP zu verstehen, die Probleme mit Ihren aktuellen code ein. Das regex wird von der post habe ich verknüpft (d.h. ich habe es nicht geschrieben), aber es funktioniert gut genug für den Anwendungsfall hier. Da gibt es nicht wahrscheinlich zu sein, jedem Land die Namen, die mehrere Linien, die Bilanzierung, die wahrscheinlich nicht notwendig oder relevant für diese Lösung. Vielen Dank für erwähnenswert, dass, obwohl!
    • StringReader nicht für mich arbeiten, aber FileReader hat. Vielen Dank! Und danke für die Erklärung, ich wirklich zu schätzen
    InformationsquelleAutor nbrooks
  2. 1

    Scheint es, dass Sie Lesen, zu viele Linien, wie unten gesehen:

    String line = br.readLine(); //Reads 1 line
while((br.readLine() != null) && !line.isEmpty()){ //Reads 1 line per iteration (and doesn't store it in a variable)
    String[] countries = line.split(",");
    countryNames[index] = countries[0];
    index++;
    line = br.readLine(); //Reads another line per iteration
}

    Die korrekte syntax für die while Schleife:

    String line;

while((line = br.readLine()) != null && !line.isEmpty() && index < countryNames.length) {
    String[] countries = line.split(",");
    countryNames[index++] = countries[0];
}

    Bemerken, wie line zugewiesen wird innerhalb der Bedingung statt innerhalb der Schleife.

    • Ich habe gelesen, dass viele Zeilen, um zu überspringen die ersten 3 Zeilen, die nicht enthalten Ländernamen.
    • Ich beziehe mich nicht auf die drei Aufrufe br.readLine(); vor der Schleife.
    InformationsquelleAutor Jacob G.
Schreibe einen Kommentar