Warum wartet Java nicht bis die for schleife zu ende ist

[ToubeX]

Folgender:

		for(int i = 0; i < alleDatein.size(); i++) {
			groesseArray = i + 1;
		}
		System.out.println("test");
		System.out.println(groesseArray);

Ausgegeben komm ich :
2
test
1
test
3

Warum bekomm ich zwischenzeitlich schon das System.out.Print ausgegeben wohl die schleife noch nicht beendet ist?
wie bekomm ich java dazu das er das beenden der schleife abwartet?
Vielleicht steh ich gerade auch nur voll aufm schlauch ...

 

[dzim]

Ich vermute mal, dass er so schnell durch ist, dass er den Puffer (System.out) noch befüllte. Ist aber bei dem Code-Schnipsel auch schwierig zu sagen. Mir fällt so ein Verhalten meist dann auf, wenn ich System.out und System.err mische...

 

[ToubeX]

Ich vermute mal, dass er so schnell durch ist, dass er den Puffer (System.out) noch befüllte. Ist aber bei dem Code-Schnipsel auch schwierig zu sagen. Mir fällt so ein Verhalten meist dann auf, wenn ich System.out und System.err mische...

Das komische ist ... die 3 int Werte sollen eigentlich einer sein.
also ingesamt 6 sollte rauskommen

 

[dzim]

Wie gesagt: Kenn deinen Code ringsrum ja nicht.

 

[ToubeX]

package gui;

import java.awt.BorderLayout;
import java.awt.Dimension;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.File;
import java.util.ArrayList;

import javax.swing.BoxLayout;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTabbedPane;
import javax.swing.JTable;
import javax.swing.table.DefaultTableModel;
import overrideKlassen.JButtonEinstellungen;

public class startframe extends JFrame implements ActionListener  {
	
	/**
	 * 
	 */
	private static final long serialVersionUID = 1L;
	JFrame hauptframe;
			JPanel hauptpanel;
				JTabbedPane showpanel;
					JPanel ordnerpanel;
						JTable dateiListe;
					JPanel logpanel;
				JPanel buttonPanel;
					JButtonEinstellungen dateiHinzufuegenButton;
					JButtonEinstellungen zeigelog;
					
	String[] spaltenBezeichnung = {"Datei","Pfad"};
	public startframe(String cloudPfad) {
		
		hauptframe = new JFrame("TFE - TFileEncrypt");
		hauptframe.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
			
			hauptpanel = new JPanel();
			hauptpanel.setLayout(new BorderLayout());
			//ShowPanel
				showpanel = new JTabbedPane();
				showpanel.setPreferredSize(new Dimension(700,400));
					
						ordnerpanel = new JPanel();
						ordnerpanel.setLayout(new java.awt.GridLayout(1,1));
						//Alle Files auflisten Start
							String[][] tabellenInhalt = listeAlleDatein(cloudPfad);
							
							DefaultTableModel dtm = new DefaultTableModel(spaltenBezeichnung,tabellenInhalt.length);
							
								for(int i=0; i < tabellenInhalt.length; i++) {
									dtm.setValueAt(tabellenInhalt[i][0], i, 0);
									dtm.setValueAt(tabellenInhalt[i][1], i, 1);
								}
							
							@SuppressWarnings("serial")
							JTable dateiListe = new JTable(dtm) {
							    @Override
						        public boolean isCellEditable(int rowIndex, int columnIndex) {
						            return false;
						        }
							};
							
							
						//Alle Files auflisten Ende
						ordnerpanel.add(new JScrollPane(dateiListe));
						
						logpanel = new JPanel();
					
					showpanel.addTab("Datei\u00FCbersicht",ordnerpanel);
					showpanel.addTab("Log",logpanel);

				
				//ButtonPanel
				buttonPanel = new JPanel();
				buttonPanel.setLayout(new BoxLayout(buttonPanel, BoxLayout.Y_AXIS));
				
					dateiHinzufuegenButton = new JButtonEinstellungen("Datei hinzuf\u00FCgen");
				
				buttonPanel.add(dateiHinzufuegenButton);
					
				
			hauptpanel.add(showpanel, java.awt.BorderLayout.WEST);	
			hauptpanel.add(buttonPanel, java.awt.BorderLayout.EAST);
			
		hauptframe.add(hauptpanel);
		hauptframe.setResizable(false);
		hauptframe.pack();
		hauptframe.setVisible(true);
		
	}

	public void actionPerformed(ActionEvent ae) {
//		if(ae.getSource() == this.exit) {
//			hauptframe.dispose();
//		}
		
	}
	
	private String[][] listeAlleDatein(String cloudPfad) {
		File ordner = new File(cloudPfad);
		File[] alles = ordner.listFiles();
		ArrayList<String> alleDatein = new ArrayList<String>();
		
		for (int i=0; i < alles.length; i++) {
			if(alles[i].isDirectory()) {
				listeAlleDatein(alles[i].getAbsolutePath());
			}
			
			if(alles[i].isFile()) {
				String stringAlleDatein = alles[i].getAbsolutePath();
				alleDatein.add(stringAlleDatein);
			}
		}
		//ArrayList aufteilen
		
		String[][]zweiDArray = mehrdimensionalArrayerstellen(alleDatein);
		//Variablen und Arrays auf null setzen
		ordner = null;
		alles = null;
		alleDatein.clear();
		
		return zweiDArray;
	}
	
	private String[][] mehrdimensionalArrayerstellen(ArrayList<String> alleDatein) {
		int groesseArray = 0;
		
		for(int i = 0; i < alleDatein.size(); i++) {
			groesseArray = i + 1;
		}
		System.out.println("test");
		System.out.println(groesseArray);
		
		String[] alleDateinArray = alleDatein.toArray(new String[alleDatein.size()]);
		String[][] dateizweiDArray = new String [alleDateinArray.length][2];
		
		for(int i=0; i < dateizweiDArray.length; i++) {
			String dateiName = alleDateinArray[i].substring(alleDateinArray[i].lastIndexOf("\\") + 1);
			dateizweiDArray[i][0] = dateiName;
			dateizweiDArray[i][1] = alleDateinArray[i];
		}

		//Variablen und Arrays auf null setzen
		alleDateinArray = null;
		alleDatein.clear();
		
		return dateizweiDArray;
	}
}

 

[dzim]

Jetzt tippe ich auf Stinknormale Rekursion: listeAlleDatein ruft sich auch selbst aus (was klar ist, wenn du den Dateibaum durchläufst), aber dadurch kommen eben auch die mehrfachen Ausgaben auf dei Konsole, denn listeAlleDatein ruft mehrdimensionalArrayerstellen auf, welche Wiederum *immer* die System.out-Calls macht.

 

[ToubeX]

Jetzt tippe ich auf Stinknormale Rekursion: listeAlleDatein ruft sich auch selbst aus (was klar ist, wenn du den Dateibaum durchläufst), aber dadurch kommen eben auch die mehrfachen Ausgaben auf dei Konsole, denn listeAlleDatein ruft mehrdimensionalArrayerstellen auf, welche Wiederum *immer* die System.out-Calls macht.

Aber die Rekursion sollte schon abgeschlossen sein wenn ich die Methode mehrdimensionalArrayerstellen aufrufe wo das Problem auftritt.

 

[dzim]

Häh? Nein! Natürlich nicht! Nicht weil du es in einer eigenen Methode hast, ist es ausserhalb der Rekursion!

 

[ToubeX]

Häh? Nein! Natürlich nicht! Nicht weil du es in einer eigenen Methode hast, ist es ausserhalb der Rekursion!

Sollte der Code net eigentlich nur solange die Rekursion ausführen bis keine If schleife zu zutrifft und DANACH erst danach wird die neue Methode aufgerufen

 

[dzim]

Nein.

 

[ToubeX]

   private String[][] listeAlleDatein(String cloudPfad) { [COLOR="Red"]2) Dann springta dahin und prüft ob das 1. oder 2. if zutrifft, wenn nicht dann macthta ab den Kommentar //ArrayList aufteilen weiter oder hab ich da schon an Denkfehler?[/COLOR]
        File ordner = new File(cloudPfad);
        File[] alles = ordner.listFiles();
        ArrayList<String> alleDatein = new ArrayList<String>();
       
        for (int i=0; i < alles.length; i++) {
            if(alles[i].isDirectory()) {
                listeAlleDatein(alles[i].getAbsolutePath()); [COLOR="Red"] 1) Hier ruf ich doch die rekursion auf[/COLOR]
            }
           
            if(alles[i].isFile()) {
                String stringAlleDatein = alles[i].getAbsolutePath();
                alleDatein.add(stringAlleDatein);
            }
        }
        //ArrayList aufteilen
       
        String[][]zweiDArray = mehrdimensionalArrayerstellen(alleDatein);
        //Variablen und Arrays auf null setzen
        ordner = null;
        alles = null;
        alleDatein.clear();
       
        return zweiDArray;
    }
   
    private String[][] mehrdimensionalArrayerstellen(ArrayList<String> alleDatein) {
        int groesseArray = 0;
       
        for(int i = 0; i < alleDatein.size(); i++) {
            groesseArray = i + 1;
        }
        System.out.println("test");
        System.out.println(groesseArray);
       
        String[] alleDateinArray = alleDatein.toArray(new String[alleDatein.size()]);
        String[][] dateizweiDArray = new String [alleDateinArray.length][2];
       
        for(int i=0; i < dateizweiDArray.length; i++) {
            String dateiName = alleDateinArray[i].substring(alleDateinArray[i].lastIndexOf("\\") + 1);
            dateizweiDArray[i][0] = dateiName;
            dateizweiDArray[i][1] = alleDateinArray[i];
        }
 
        //Variablen und Arrays auf null setzen
        alleDateinArray = null;
        alleDatein.clear();
       
        return dateizweiDArray;
    }
}

 

[dzim]

Du rufst immer noch mehrdimensionalArrayerstellen innerhalb von listeAlleDatein auf. Rekursion ist nicht Paralellisierung, also werden Verzeichnisse immer noch sequentiell abgearbeitet. Kann also immer noch passieren (immer auf jeder Verzeichnis-Ebene).

vielleicht solltest du mal erklären, was deine schräge Matrix von Strings am Ende machen soll (ich hab gerade keinen Bock den Swing-Teil des Codes durchzulesen).

BTW: Erwarte heute keine Antworten mehr, ich mach jetzt heim...

 

[ToubeX]

Die Methode listeAlleDatein soll alle Datein aus dem und allen Unterordner auslesen und mehrdimensionalArrayerstellen soll hergehen soll die Pfade nochmal zerlegen in nen Dateiname und dann in einem 2D Array abspeichern und dann zurück an die Main übergeben werden, welche die listeAlleDatein aufgerufen hatte

 

[Flown]

Darf ich dir ein paar gute Ratschläge erteilen?

Also:
- Man leitet nicht von JFrame ab, außer man ändert dessen Funktionalität
- Man schreibt Variablen, etc. in camelCase
- Man versucht alle Bezeichner (Klassen, Variablen, ...) in Englisch zu halten
- Man verwendet sprechende Namen für Bezeichner (z.B.: Methodennamen)
- Man soll Konstanten verwenden (wie z.B. Spaltennamen) und richtig deklarieren (public/private static final ...)
- Man soll die API verwenden (java.nio -> FileVisitor -> durch den Filetree wandern) -> vorausgesetzt Java 8

Dann sollte der Code so aussehen:

Spoiler: New Code

import java.awt.BorderLayout;
import java.awt.Dimension;
import java.awt.GridLayout;
import java.io.IOException;
import java.nio.file.FileVisitResult;
import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.Paths;
import java.nio.file.SimpleFileVisitor;
import java.nio.file.attribute.BasicFileAttributes;
import java.util.ArrayList;
import java.util.List;
import java.util.stream.Collectors;

import javax.swing.BoxLayout;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTabbedPane;
import javax.swing.JTable;
import javax.swing.table.DefaultTableModel;

public class StartFrame {
	private JFrame mainFrame;
	private JTabbedPane tabPane;
	private JPanel mainPanel;
	private JPanel directoryPanel;
	private JPanel logPanel;
	private JPanel buttonPanel;
	private JTable fileTable;
	private JButton addFileBtn;

	private static final String[] COLUMN_NAMES = { "Datei", "Pfad" };
	
	public static void main(String[] args) {
		new StartFrame(".");
	}

	public StartFrame(String cloudPfad) {

		mainFrame = new JFrame("TFE - TFileEncrypt");
		mainFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

		mainPanel = new JPanel();
		mainPanel.setLayout(new BorderLayout());
		// ShowPanel
		tabPane = new JTabbedPane();
		tabPane.setPreferredSize(new Dimension(700, 400));

		directoryPanel = new JPanel();
		directoryPanel.setLayout(new GridLayout(1, 1));
		// List all files
		String[][] tableContent = dividePaths(listAllFiles(Paths.get(cloudPfad)));

		DefaultTableModel dtm = new DefaultTableModel(tableContent, COLUMN_NAMES) {
			private static final long serialVersionUID = -8695264333573532007L;

			@Override
			public boolean isCellEditable(int row, int column) {
				return false;
			}
		};

		fileTable = new JTable(dtm);

		directoryPanel.add(new JScrollPane(fileTable));

		logPanel = new JPanel();

		tabPane.addTab("Datei\u00FCbersicht", directoryPanel);
		tabPane.addTab("Log", logPanel);

		// ButtonPanel
		buttonPanel = new JPanel();
		buttonPanel.setLayout(new BoxLayout(buttonPanel, BoxLayout.Y_AXIS));

		addFileBtn = new JButton("Datei hinzuf\u00FCgen");

		buttonPanel.add(addFileBtn);

		mainPanel.add(tabPane, BorderLayout.WEST);
		mainPanel.add(buttonPanel, BorderLayout.EAST);

		mainFrame.add(mainPanel);
		mainFrame.setResizable(false);
		mainFrame.pack();
		mainFrame.setVisible(true);

	}

	private List<Path> listAllFiles(Path path) {
		List<Path> allFiles = new ArrayList<>();
		try {
			Files.walkFileTree(path, new SimpleFileVisitor<Path>() {
				@Override
				public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IOException {
					allFiles.add(file);
					return super.visitFile(file, attrs);
				}
			});
		}
		catch (IOException e) {
			e.printStackTrace();
		}
		return allFiles;
	}
	
	private String[][] dividePaths(List<Path> paths) {
		List<String[]> s = paths.stream().map(p -> {
			String absolute = p.normalize().toAbsolutePath().toString();
			return new String[] {absolute.substring(absolute.lastIndexOf("\\") + 1), absolute};
		}).collect(Collectors.toCollection(ArrayList::new));
		return s.toArray(new String[s.size()][]);
	}
}

 

[dzim]

Danke @Flown: Ich hatte eigentlich auch keinen Bock, die Methode zu korigieren... :-D

 

[Joose]

keine If schleife zu zutrifft

Thema if-Schleife

 

[ToubeX]

Darf ich dir ein paar gute Ratschläge erteilen?

Also:
- Man leitet nicht von JFrame ab, außer man ändert dessen Funktionalität
- Man schreibt Variablen, etc. in camelCase
- Man versucht alle Bezeichner (Klassen, Variablen, ...) in Englisch zu halten
- Man verwendet sprechende Namen für Bezeichner (z.B.: Methodennamen)
- Man soll Konstanten verwenden (wie z.B. Spaltennamen) und richtig deklarieren (public/private static final ...)
- Man soll die API verwenden (java.nio -> FileVisitor -> durch den Filetree wandern) -> vorausgesetzt Java 8

Dann sollte der Code so aussehen:

Spoiler: New Code

import java.awt.BorderLayout;
import java.awt.Dimension;
import java.awt.GridLayout;
import java.io.IOException;
import java.nio.file.FileVisitResult;
import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.Paths;
import java.nio.file.SimpleFileVisitor;
import java.nio.file.attribute.BasicFileAttributes;
import java.util.ArrayList;
import java.util.List;
import java.util.stream.Collectors;

import javax.swing.BoxLayout;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.JTabbedPane;
import javax.swing.JTable;
import javax.swing.table.DefaultTableModel;

public class StartFrame {
	private JFrame mainFrame;
	private JTabbedPane tabPane;
	private JPanel mainPanel;
	private JPanel directoryPanel;
	private JPanel logPanel;
	private JPanel buttonPanel;
	private JTable fileTable;
	private JButton addFileBtn;

	private static final String[] COLUMN_NAMES = { "Datei", "Pfad" };
	
	public static void main(String[] args) {
		new StartFrame(".");
	}

	public StartFrame(String cloudPfad) {

		mainFrame = new JFrame("TFE - TFileEncrypt");
		mainFrame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

		mainPanel = new JPanel();
		mainPanel.setLayout(new BorderLayout());
		// ShowPanel
		tabPane = new JTabbedPane();
		tabPane.setPreferredSize(new Dimension(700, 400));

		directoryPanel = new JPanel();
		directoryPanel.setLayout(new GridLayout(1, 1));
		// List all files
		String[][] tableContent = dividePaths(listAllFiles(Paths.get(cloudPfad)));

		DefaultTableModel dtm = new DefaultTableModel(tableContent, COLUMN_NAMES) {
			private static final long serialVersionUID = -8695264333573532007L;

			@Override
			public boolean isCellEditable(int row, int column) {
				return false;
			}
		};

		fileTable = new JTable(dtm);

		directoryPanel.add(new JScrollPane(fileTable));

		logPanel = new JPanel();

		tabPane.addTab("Datei\u00FCbersicht", directoryPanel);
		tabPane.addTab("Log", logPanel);

		// ButtonPanel
		buttonPanel = new JPanel();
		buttonPanel.setLayout(new BoxLayout(buttonPanel, BoxLayout.Y_AXIS));

		addFileBtn = new JButton("Datei hinzuf\u00FCgen");

		buttonPanel.add(addFileBtn);

		mainPanel.add(tabPane, BorderLayout.WEST);
		mainPanel.add(buttonPanel, BorderLayout.EAST);

		mainFrame.add(mainPanel);
		mainFrame.setResizable(false);
		mainFrame.pack();
		mainFrame.setVisible(true);

	}

	private List<Path> listAllFiles(Path path) {
		List<Path> allFiles = new ArrayList<>();
		try {
			Files.walkFileTree(path, new SimpleFileVisitor<Path>() {
				@Override
				public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IOException {
					allFiles.add(file);
					return super.visitFile(file, attrs);
				}
			});
		}
		catch (IOException e) {
			e.printStackTrace();
		}
		return allFiles;
	}
	
	private String[][] dividePaths(List<Path> paths) {
		List<String[]> s = paths.stream().map(p -> {
			String absolute = p.normalize().toAbsolutePath().toString();
			return new String[] {absolute.substring(absolute.lastIndexOf("\\") + 1), absolute};
		}).collect(Collectors.toCollection(ArrayList::new));
		return s.toArray(new String[s.size()][]);
	}
}

Vielen Dank ich werde in Zukunft deine Ratschläge befolgen.

 

[Flown]

Neue Verbesserung der dividePaths Methode (meinen eigenen Rat folgen: Man soll die API verwenden -> substring -> getFileName()):

private String[][] dividePaths(List<Path> paths) {
    List<String[]> s = paths.stream().map(p -> new String[] {p.getFileName().toString(), p.normalize().toAbsolutePath().toString()}
    ).collect(Collectors.toList());
    return s.toArray(new String[s.size()][]);
}

 

[dzim]

Ah. Cool. Streaming-API... Zu den tollen neuen Java8-Features bin ich primär dank Android noch nicht durchgedrungen :-(