CSV-Datei auslesen in Java

[Gaudimagspam]

Das ist für die 1a i). Wäre das so richtig? Ich gebe ja den Pfad an und lese die Datei.

 

[kneitzel]

Da sehe ich mehrere Punkte:

a) Generell bitte Code nicht als Bildschirmfoto sondern per Copy & Paste in code Tags posten.

b) Bitte versuchen, genaue Fragen zu stellen. Die Frage "Wäre das so richtig?" würde ich prinzipiell mit einer Gegenfrage beantworten: Hast Du es ausprobiert? In b) werden ja Unit Tests erwähnt: Laufen die durch?

Dann zu dem bisschen Code fällt mir direkt auf:

c) Dein Code dürfte sich nicht übersetzen lassen. new FileReader kann IOException werfen und die fängst Du nicht und Deine Methode hat auch kein throws IOException.

d) Du nutzt Klassen die AutoClosable implementieren für lokale Variablen -> Da sollte (in der Regel) immer try with resources verwendet werden!

e) Namen! Sinnvolle Namen vergeben. Die Methoden sollten so benannt sein, dass klar ist, was sie machen. Dein Methode sagt "getFileName" aber sie gibt nicht nur einen FileName, die macht ganz viele andere Dinge ... Das sieht nicht bezüglich Clean Code nicht korrekt aus.

f) Auf Grund des kurzen Ausschnitts schwer zu sagen, aber vermutlich hast Du wie in Scripts üblich, die Variablen alle am Methodenanfang deklariert. Deklaration und Nutzung sollten dicht beisammen sein und auch von der Lebenszeit auf ihren Block begrenzt sein so möglich.

Das wäre so mein Feedback, das mir da einfallen würde ...

 

[kneitzel]

Ach so - hatte nicht gut genug gelesen - der Punkt b) ist natürlich Quatsch. Deine Frage ist sehr konkret - ich habe nur die Aufgabe 1 a) i nicht gut genug gelesen.

Konkrete Antwort: Nein, das ist falsch.
- getFileName soll nur einen Filenamen zurück geben. Von Datei einlesen steht da nichts. Das gehört da also nicht rein!
- Absolute und relative Pfade - sagen Dir die Begriffe etwas? Relative Pfade sind immer relativ zum aktuellen Pfad, d.h. sie beziehen sich auf den aktuellen Pfad. Das sind dann also Pfade wie "./someFolder/file.txt". Absolute Pfade gelten absolut, d.h, sie sind unabhängig davon, wo Du gerade bist. Das ist dann also etwas wie "c:/users/konrad/file.txt"

An meinen Pfaden siehst Du: Pfade können in Java generell mit / angegeben werden. Das Funktioniert dann auch unter Windows, auch wenn Windows selbst eigentlich \ verwendet.

Somit ist 1 a i aus der Aufgabe sehr einfach zu lösen. Es gibt nur eine Zeile mit return "...."; Durch was die ... ersetzt werden müssen, wird in dem Text der Aufgabe klar gesagt.

 

[M.L.]

Der Einsatz von OpenCSV könnte übrigens auch einige Probleme umgehen.

 

[Gaudimagspam]

Das passt eher a ii)

 

[Gaudimagspam]

Hier der Code:

public List<Double> getTemperatureMeanValues(String fileName) {
String fileName= "C:\\Users\\pirat\\Music\\assignment_1 (1)\\assignment_1\\temperature\\data";
//create BufferedReader to read csv file
BufferedReader br = new BufferedReader(new FileReader(strFile));
String strLine = "";
StringTokenizer st = null;
int lineNumber = 0, tokenNumber = 0;
//read comma separated file line by line
while((strLine = br.readLine()) != null)
{
lineNumber++;

String[] array = strLine.split(",");
String city = array[0];
double[] array_float = new double[array.length-1];
for (int i = 1; i <array.length ; i++) {
array_double[i-1] = double.parsedouble(array);
}

 

[kneitzel]

Probierst du den Code direkt aus? Das wäre meine Empfehlung um Schreibfehler zu vermeiden und Fehlermeldungen angezeigt zu bekommen ...

Ehe ich auf den Code eingehe noch ein Hinweis zum Code posten: über dem Eingabefeld ist die Zeile mit Buttons für die Formatierung. Der erste ist </> - über diesen mitte Code einfügen... Denn so wertet das Forum gewisse Dinge als Formatierungshinweis...

Das Problem mit der IOException aus meiner ersten Antwort existiert noch meine ich.
Manche Dinge haben falsche Gross/Kleinschreibung (Double.parseDouble dürfte z.B. die gewünschte Methode zum Parsen sein.)

Dann liest du Zeile für Zeile aus, machst den Split und parst die Zahlen. Das sieht prinzipiell ok aus. Nur mit den Daten passiert dann noch nichts....
Sprich: du bist noch nicht fertig.

Und die Methode soll etwas zurück geben - das tut sie bisher noch nicht.

Das wären auf die Schnelle jetzt meine Hinweise.

 

[krgewb]

Bitte immer in code-Tags posten.

public List<Double> getTemperatureMeanValues(String fileName) {
    String fileName= "C:\\Users\\pirat\\Music\\assignment_1 (1)\\assignment_1\\temperature\\data";
    //create BufferedReader to read csv file
    BufferedReader br = new BufferedReader(new FileReader(strFile));
    String strLine = "";
    StringTokenizer st = null;
    int lineNumber = 0, tokenNumber = 0;
    //read comma separated file line by line
    while((strLine = br.readLine()) != null)
    {
        lineNumber++;

        String[] array = strLine.split(",");
        String city = array[0];
        double[] array_float = new double[array.length-1];
        for (int i = 1; i <array.length ; i++) {
            array_double[i-1] = double.parsedouble(array[i]);
        }