Txt Datei einlesen

[sserio]

Also ich habe das Problem, dass wenn ich eine TXT Datei in eine Methode als Parameter rein gebe meine Exception geschmissen wird,
[CODE lang="java" title="Funktioniert nicht"]package ProjectEuler22;

import java.io.File;
import java.io.FileNotFoundException;
import java.util.ArrayList;
import java.util.List;
import java.util.Scanner;

public class Main {
public static void main(String[] args) {
for(String s:getTxtItems("C:/Users/sergi/OneDrive/Desktop/Data/namen.txt")){
System.out.println(s);
}

}

public static ArrayList<String> getTxtItems(String txtLink) {
var data = new File("txtLink");
var dataList = new ArrayList<String>();
Scanner scanner = null;
try {
scanner = new Scanner(data);
} catch (FileNotFoundException e) {
System.out.println("File not found!");
}
while (scanner.hasNext()) {
dataList.add(scanner.next());
}
return dataList;
}
}[/CODE]
[CODE lang="java" title="Funktioniert"]package ProjectEuler22;

import java.io.File;
import java.io.FileNotFoundException;
import java.util.ArrayList;
import java.util.List;
import java.util.Scanner;

public class Main {
public static void main(String[] args) {
for(String s:getTxtItems()){
System.out.println(s);
}

}

public static ArrayList<String> getTxtItems() {
var data = new File("C:/Users/sergi/OneDrive/Desktop/Data/namen.txt");
var dataList = new ArrayList<String>();
Scanner scanner = null;
try {
scanner = new Scanner(data);
} catch (FileNotFoundException e) {
System.out.println("File not found!");
}
while (scanner.hasNext()) {
dataList.add(scanner.next());
}
return dataList;
}
}[/CODE]
aber wenn ich die Methode ohne Eingabeparameter schreibe und den txtLink direkt in den File packe alles funktioniert. Die beiden Codes sind down below...

 

[sserio]

Also ich habe das Problem, dass wenn ich eine TXT Datei in eine Methode als Parameter rein gebe meine Exception geschmissen wird,
[CODE lang="java" title="Funktioniert nicht"]package ProjectEuler22;

import java.io.File;
import java.io.FileNotFoundException;
import java.util.ArrayList;
import java.util.List;
import java.util.Scanner;

public class Main {
public static void main(String[] args) {
for(String s:getTxtItems("C:/Users/sergi/OneDrive/Desktop/Data/namen.txt")){
System.out.println(s);
}

}

public static ArrayList<String> getTxtItems(String txtLink) {
var data = new File("txtLink");
var dataList = new ArrayList<String>();
Scanner scanner = null;
try {
scanner = new Scanner(data);
} catch (FileNotFoundException e) {
System.out.println("File not found!");
}
while (scanner.hasNext()) {
dataList.add(scanner.next());
}
return dataList;
}
}[/CODE]
[CODE lang="java" title="Funktioniert"]package ProjectEuler22;

import java.io.File;
import java.io.FileNotFoundException;
import java.util.ArrayList;
import java.util.List;
import java.util.Scanner;

public class Main {
public static void main(String[] args) {
for(String s:getTxtItems()){
System.out.println(s);
}

}

public static ArrayList<String> getTxtItems() {
var data = new File("C:/Users/sergi/OneDrive/Desktop/Data/namen.txt");
var dataList = new ArrayList<String>();
Scanner scanner = null;
try {
scanner = new Scanner(data);
} catch (FileNotFoundException e) {
System.out.println("File not found!");
}
while (scanner.hasNext()) {
dataList.add(scanner.next());
}
return dataList;
}
}[/CODE]
aber wenn ich die Methode ohne Eingabeparameter schreibe und den txtLink direkt in den File packe alles funktioniert. Die beiden Codes sind down below...

Habe das Problem gefunden, wie kann man einen Beitrag löschen ?

 

[fhoffmann]

Habe das Problem gefunden

Du hättest uns auch mitteilen können, dass du versehentlich den Variablennamen in Anführungszeichen gesetzt hast.

 

[Oneixee5]

Nur noch mal als Hinweis: Scanner muss man schließen, wenn man sie mit Dateien verwendet. Der Code ist auch etwas umständlich. Ich schlage dir deshalb folgendes vor:

Path path = Paths.get("C:/Users/sergi/OneDrive/Desktop/Data/namen.txt");
List<String> list = Files.readAllLines(path /*, possible charset param?*/);
list.forEach(System.out::println);

Den Pfad habe ich einfach kopiert, ich habe kein Windows - deshalb nicht getestet.

 

[sserio]

Nur noch mal als Hinweis: Scanner muss man schließen, wenn man sie mit Dateien verwendet. Der Code ist auch etwas umständlich. Ich schlage dir deshalb folgendes vor:

Path path = Paths.get("C:/Users/sergi/OneDrive/Desktop/Data/namen.txt");
List<String> list = Files.readAllLines(path /*, possible charset param?*/);
list.forEach(System.out::println);

Den Pfad habe ich einfach kopiert, ich habe kein Windows - deshalb nicht getestet.

Das heißt, dass ich mir die ganze Sache mit dem TryCatch auch einfach sparen kann, sowie mit dem Scanner. Aber außerdem auch coole und für mich neue Art, über eine Liste zu iterrieren ( und dann auszugeben ). Danke

 

[Oneixee5]

Das heißt, dass ich mir die ganze Sache mit dem TryCatch auch einfach sparen kann

Nein, beim Zugriff auf Dateien sind idR. IOExceptions abzufangen. Du kannst das aber für Übungen vereinfachen:

    public static void main(final String[] args) throws IOException {
        final Path path = Paths.get("C:/Users/sergi/OneDrive/Desktop/Data/namen.txt");
        final List<String> list = Files.readAllLines(path /*, possible charset param?*/);
        list.forEach(System.out::println);
    }

Ein throws IOException an die main-Methode anhängen ist für einfache Tests ausreichend.

 

[sserio]

Nein, beim Zugriff auf Dateien sind idR. IOExceptions abzufangen. Du kannst das aber für Übungen vereinfachen:

    public static void main(final String[] args) throws IOException {
        final Path path = Paths.get("C:/Users/sergi/OneDrive/Desktop/Data/namen.txt");
        final List<String> list = Files.readAllLines(path /*, possible charset param?*/);
        list.forEach(System.out::println);
    }

Ein throws IOException an die main-Methode anhängen ist für einfache Tests ausreichend.

Ok

 

[Oneixee5]

Deine Methode würde ich so abändern:

    public static List<String> getTxtItems() throws IOException {
        final var data = new File("C:/Users/sergi/OneDrive/Desktop/Data/namen.txt");
        final var list = new ArrayList<String>();
        try (var scanner = new Scanner(data)) {
            while (scanner.hasNext()) {
                list.add(scanner.next());
            }
        }
        return list;
    }

Der Scanner wird automatisch geschlossen, auch im Fehlerfall, und die Exception wird an den Aufrufer weitergegeben.
Edit: var scanner habe ich noch mal geändert, da du offensichtlich diese Syntax verwenden möchtest.

 

[Oneixee5]

Da dir die "coole und für mich neue Art" so gefällt:

    public static List<String> getTxtItems() throws IOException {
        final var data = new File("C:/Users/sergi/OneDrive/Desktop/Data/namen.txt");
        final var list = new ArrayList<String>();
        try (var scanner = new Scanner(data)) {
            scanner.forEachRemaining(list::add);
        }
        return list;
    }

 

[M.L.]

ProjectEuler22

Weiterhin kann man Ideen entwickeln, wie die n-te Position zu bestimmen ist (z.B. Anzahl der "," ), oder man könnte Teil-Vornamen (MARY, FRAN,..) auf Vorrat berrechnen