Δημοσιεύτηκε: 16 Απρ 2012, 16:19
από giannosfor
Οκ εντάξει,αν και αναλόγως πώς θα την χρησιμοποιήσεις έχει κάποια λαθάκια ,κυρίως στη λογική.
Αν την χρησιμοποιούσα θα την έγραφα κάπως έτσι.

Μορφοποιημένος Κώδικας: Επιλογή όλων
import java.io.File;
import java.io.FileNotFoundException;
import java.util.ArrayList;
import java.util.Formatter;
import java.util.Scanner;

public class Save_Load_Library {

public static void WriteFile(String file, String[] array) {
try {
Formatter output = new Formatter(file);
for (int i = 0; i < array.length; i++) {
output.format("%s \n", array[i]);
}
output.close();
} catch (SecurityException se) {
System.err.println("You dont have the permission to open the file.");
} catch (FileNotFoundException fe) {
System.err.println("File not found.");
}
}

public static String[] readFile(String path) throws FileNotFoundException {
ArrayList< String> rarray = new ArrayList();
Scanner input = new Scanner(new File(path));
while (input.hasNext()) {
rarray.add(input.nextLine());
}
input.close();
return (String[]) rarray.toArray();
}
}

Πρώτα θα τις έκανα static ,δε χρειάζεται να δημιουργήσεις καινούργιο αντικείμενο αφού περνάς εσύ τα στοιχεία στις παραμέτρους.
Η τελευταία μέθοδο θεωρώ ότι δε χρειάζεται και τον έλεγχο για FileException θα τον έκανα στην κλάση που θα καλούσε τη μέθοδο.