Scanner erkennt keinen Text in Textdatei, obwohl welcher drinsteht

[_brina205]

Ich habe einen Pizza-Bestellsystem erzeugt für eine Hausaufgabe. Jetzt soll ich, wenn ich im Menü Bestellung auf Lesen klicke, die Textdatei ausgelesen werden. Allerdings erkennt der Scanner obwohl etwas in der Textdatei steht nichts und gibt dann im Dialogfenster nur ein leeres Array aus. Muss ich die Textdatei irgendwie noch freigeben oder so was?
Es passiert auch nur wenn ich das Programm starte und direkt auf Lesen klicke, wenn ich zuerst mit der Methode writeOrderCSV in die Datei schreibe, erkennt er es.

[CODE lang="java" title="Controller" highlight="27,28"]public class Controller {

private PizzaKonfigurator view;
private DataStorage dataStorage;
private Pizza pizza;
private Order order;


public static void main(String[] args) {
PizzaKonfigurator view = new PizzaKonfigurator();
new Controller(view);
view.setVisible(true);
}

public Controller(PizzaKonfigurator view) {
this.view = view;
order = new Order();
dataStorage = new DataStorage();
initController();
}

/**
* Initialisiert den Controller mit allen Aktionen
*/
public void initController() {
view.getMyMenuBar().getMenuItemBeenden().addActionListener(e -> System.exit(0));
view.getMyMenuBar().getMenuItemLesen().addActionListener(e -> {
JOptionPane.showMessageDialog(view, "Bestellung " + order.getOrderId() + order.getPizzas(), "Bestellung", JOptionPane.INFORMATION_MESSAGE);
});
view.getMyMenuBar().getMenuItemSpeichern().addActionListener(e -> dataStorage.writeOrderCSV(order));
view.getPizzaSize().addActionListener(e -> view.getPizzaSize().getSelectedItem());
for (JRadioButton topping : view.getToppings()) {
topping.addActionListener(e -> topping.getText());
}
Object[] options = {"Ja", "Nein"};
view.getFertig().addActionListener(e -> {
pizzaErzeugen();
JOptionPane.showMessageDialog(view, "Preis: " + pizza.getPrice(), "Bestellung", JOptionPane.INFORMATION_MESSAGE);
int n = JOptionPane.showOptionDialog(view, "Möchten Sie die Pizza zur Bestellung hinzufügen?", "Bestellung", JOptionPane.YES_NO_OPTION, JOptionPane.QUESTION_MESSAGE, null, options, options[0]);
if (n == JOptionPane.YES_OPTION) {
order.addPizza(pizza);
}
});
view.getBeenden().addActionListener(e -> {
int n = JOptionPane.showOptionDialog(view, "Möchten Sie die Anwendung wirklich beenden?", "Beenden", JOptionPane.YES_NO_OPTION, JOptionPane.QUESTION_MESSAGE, null, options, options[0]);
if (n == JOptionPane.YES_OPTION) {
System.exit(0);
}
});
}[/CODE]
[CODE lang="java" title="DataStorage" highlight="22"]public void writeOrderCSV(Order order) {
try {
pWriter = new PrintWriter(new BufferedWriter(new FileWriter("bestellung.txt")));
pWriter.println("Bestellung: ");
for (Pizza pizza : order.getPizzas()) {
pWriter.format("Größe: %s ; Beläge: %s ; Preis: %,.2f %n", pizza.getSize(), pizza.getToppings(), pizza.getPrice());
}
} catch (IOException e) {
JOptionPane.showMessageDialog(null, "Fehler" , "Fehler", JOptionPane.WARNING_MESSAGE);
} finally {
if (pWriter != null) {
pWriter.close();
}
}
}

public Order readOrderCSV() {
Order order = new Order();
try {
Scanner s = new Scanner(new BufferedReader(new FileReader("bestellung.txt")));
Pizza pizza = null;
while (s.hasNext(";")) {
s.useDelimiter(";");
//Größe auslesen
String size = s.next();
for (PizzaSize psize : PizzaSize.values()) {
if (size.contains(psize.toString())) {
pizza = new Pizza(psize);
}
}
//Toppings auslesen
String belaege = s.next();
for (PizzaTopping topping : PizzaTopping.values()) {
if (belaege.contains(topping.toString())) {
pizza.toppingHinzufügen(topping);
}
}
order.addPizza(pizza);
s.useDelimiter(System.lineSeparator());
}
} catch (IOException e) {
JOptionPane.showMessageDialog(null, "Die angegebene Datei existiert nicht" , "Datei", JOptionPane.WARNING_MESSAGE);
}
return order;

}[/CODE]

[CODE lang="java" title="Textdatei.txt"]Bestellung:
Größe: Klein ; Beläge: [Salami, Ananas, Käse, Schinken] ; Preis: 13,00
Größe: Klein ; Beläge: [Nutella, Ananas] ; Preis: 6,00[/CODE]

 

[Oneixee5]

Du kannst einfach mal ausgeben was in readOrderCSV() passiert, z.B.: System.out.println(size); und dann auch schrittweise, was in den Schleifen passiert.

 

[Oneixee5]

Wenn du mit einem Scanner eine Datei öffnest, dann ist es notwendig den Scanner zu schließen, wenn keine Dateizugriffe mehr benötigt werden.
Am einfachsten funktioniert das so:

try(Scanner scanner = new Scanner(...)) {
  ...
}

Es kann aber auch direkt #close() aufgerufen werden.

Scanner (Java Platform SE 7 )

docs.oracle.com

Ein Scanner wird normalerweise nicht geschlossen wenn von System.in gelesen wird.

 

[_brina205]

Das mit dem Scanner habe ich jetzt gemacht, aber trotzdem gleiches Ergebnis.
Beim Debuggen habe ich gesehen, dass es schon die while-Schleife überspringt, weil der Scanner die Textdatei als leer ansieht und kein Semikolon findet.

 

[Oneixee5]

Na da hast du ja schon einen Fehler gefunden. Jetzt musst du es nur noch richtig machen.

 

[_brina205]

Aber ich verstehe doch nicht, warum es die Textdatei als leer ansieht, das ist doch das Problem
Weil es steht eindeutig was drin, wenn ich die Textdatei öffne, bevor ich das Programm starte.

 

[Oneixee5]

ist denn in der ersten Zeile der Datei ein ; ?

 

[_brina205]

Nein, der Text sieht von der Struktur her immer genauso aus wie in der Beispieltextdatei oben

 

[kneitzel]

Du hast die Stelle, die sich anders verhält als erwartet, doch erkannt. Somit scheint hasNext etwas anderes zu machen, als du dachtest.

Daher wäre es doch nun naheliegend, dem nach zu gehen. Hast Du eine Idee, wo und wie Du die Dokumentation von der Klasse Scanner finden kannst? Und dann die Methode hasNext(String)?

 

[Kuryu]

Hey brina. Ich bin mir nicht mehr ganz sicher aber ich glaube das müsste so aussehen:

public Order readOrderCSV() {
        Order order = new Order();
        try {
            Scanner s = new Scanner(new BufferedReader(new FileReader("bestellung.txt")));
            Pizza pizza = null;
            s.useDelimiter(";");
            while (s.hasNext()) {
                
                //Größe auslesen
                String size = s.next();
                for (PizzaSize psize : PizzaSize.values()) {
                    if (size.contains(psize.toString())) {
                        pizza = new Pizza(psize);
                    }
                }
                //Toppings auslesen
                String belaege = s.next();
                for (PizzaTopping topping : PizzaTopping.values()) {
                    if (belaege.contains(topping.toString())) {
                        pizza.toppingHinzufügen(topping);
                    }
                }
                order.addPizza(pizza);
                s.useDelimiter(System.lineSeparator());
            }
        } catch (IOException e) {
            JOptionPane.showMessageDialog(null, "Die angegebene Datei existiert nicht" , "Datei", JOptionPane.WARNING_MESSAGE);
        }
        return order;

    }

Du solltest auf jeden Fall deinen Scanner am Ende closen. also s.close(); noch mit einbauen. Und ansonsten kannst du ja System.out.println()'s in die die Zwischenschritte mit einbauen um zu sehen welche Strings du gerade wo und wann ausliest.
Ich hoffe, dass ich dir weiterhelfen konnte.

 

[mihe7]

Du solltest auf jeden Fall deinen Scanner am Ende closen. also s.close(); noch mit einbauen.

Ja, aber bitte nicht explizit mit close(), sondern mit try-with-resources.