JTextArea und Threads

[Antineutrino]

Hallo,

ich habe ein Programm geschrieben, das regelmäßig eine Log-Datei ausliest und in einer JTextArea anzeigt. Leider friert das Proframm früher oder später bei

int i = text.getSelectionStart();
int j = text.getSelectionEnd();
text.setText(s);
text.setSelectionStart(Math.max(i, j));
text.setSelectionEnd(Math.min(i, j));

ein.
Es funktioniert aber, wenn ich

text.setEnabled(false);

sage und die set/getSelection weglasse. Dann kann ich ja aber nichts mehr mit der Maus markieren.

Woran kann das liegen?

mfg

 

[Antineutrino]

Das Programm friert ein, wenn der Thread zu kurz schläft. Bei 1000ms z.B. läuft es, allerdings auch nur, wenn ich das get/setSelection weglasse. Dann verascwindet aber ständig die aktuelle Markierung.

 

[SlaterB]

wenn du ein vollständiges Programm postest, könnte man daran knabbern,
so sehe zumindest ich keinen bekannten Anhaltspunkt,
und vermute stark, dass es mit irgendwas anderem zu tun hat

 

[Antineutrino]

Hier wird der Text in die TextArea eingetragen:

public synchronized void setText(String s) {
		text.setText(s);
	}

Hier ist der Thread:

	public void run() {

		while (true) {
			try {

				FileReader fr = new FileReader(f.getAbsolutePath());
				BufferedReader br = new BufferedReader(fr);
				StringBuffer sb = new StringBuffer();
				String inhalt;

				while (br.ready()) {
					String line = br.readLine();
					sb.append(line);
					sb.append("\n");
				}

				inhalt = sb.toString();
				String view = inhalt.substring(alreadyViewed);

				if (g.getText().equals(view)) {
					somethingNew = false;
				} else {
					somethingNew = true;
				}

				if (somethingNew) {
					g.setText(view);
					System.out.println("scrolle herunter...");
					g.scrollDown();
				}

				br.close();
				fr.close();

				if (ende) {
					break;
				}

				Thread.sleep(1000);

			} catch (FileNotFoundException e) {
				e.printStackTrace();
			} catch (IOException e) {
				e.printStackTrace();
			} catch (InterruptedException e) {
				e.printStackTrace();
			}
		}

	}

[/quote]

 

[SlaterB]

mit 'vollständiges Programm' meinte ich tatsächlich ein vollständiges Programm
aber soll keine Kritik sein, vielleicht hilft es ja wem anders, klarer zu sehen

--------

synchronized ist allerdings ein guter Grund, warum ein Java-Programm sich mal ohne Exception aufhängen sollte
insbesondere sind solche DeadLocks oft abhängig von Wartezeiten, was es noch wahrscheinlicher macht:

Thread A ist in Monitor 1 und wartet auf Zugriff zu Monitor 2
Thread B ist in Monitor 2 und wartet auf Zugriff zu Monitor 1


wenn du dir die Mühe machst da mitzuloggen
(a la System.out.println("Thread x steht vor synchronized-Block y"))
könntest du solche Fehler vielleicht finden,

ist aber aufwendig und vielleicht gar nicht dein Problem

 

[Antineutrino]

Ich habe nur einen ActionListener, der *einen* Beobachter-Thread (s.o.) startet. Da sollte doch eigentlich kein Deadlock entstehen, oder? Wenn eine neue Datei geöffnet wird, wird ein Flag gesetzt, damit der Thread sich beendet.

Sehr viel länger ist das Programm auch nicht.

if (e.getActionCommand().equals("Open")) {

			JFileChooser auswahl = new JFileChooser();
			auswahl.setFileFilter(new LOGFilter());
			auswahl.setFileSelectionMode(JFileChooser.FILES_AND_DIRECTORIES);
			auswahl.showOpenDialog(null);
			File file = auswahl.getSelectedFile();

			if (file != null) {

				if (fw != null) {
					fw.end();
				}

				String path = file.getAbsolutePath();
				g.setTitle(path + " - " + "LogViewer");

				try {

					fw = new FileWatcher(file, g);

					BufferedReader br = new BufferedReader(new FileReader(path));

					StringBuffer sb = new StringBuffer();

					while (br.ready()) {
						sb.append(br.readLine());
						sb.append("\n");
					}

					g.setText(sb.toString());
					br.close();
					fw.start();
					g.scrollDown();

				} catch (FileNotFoundException e1) {
					JOptionPane.showInternalMessageDialog(null, "Datei nicht gefunden.", "Fehler!", JOptionPane.ERROR_MESSAGE);
					System.out.println("Datei konnte nicht gelesen werden.");
				} catch (IOException e2) {
					JOptionPane.showInternalMessageDialog(null, "Fehler beim Lesen der Datei.", "Fehler!", JOptionPane.ERROR_MESSAGE);
					System.out.println("Fehler beim Einlesen der Datei.");
				} catch(StringIndexOutOfBoundsException e3){
					System.out.println("");
				}

			}

		}

 

[SlaterB]

du hast den Event-Thread und deinen Thread und du weißt gar nicht,
was alles intern vielleicht als synchronisiert markiert ist

aber es ist nur eine Ahnung und ich kann da auch nix weiter zu sagen

 

[André Uhres]

Hallo,

ich habe ein Programm geschrieben, das regelmäßig eine Log-Datei ausliest und in einer JTextArea anzeigt. Leider friert das Proframm früher oder später bei

int i = text.getSelectionStart();
int j = text.getSelectionEnd();
text.setText(s);
text.setSelectionStart(Math.max(i, j));
text.setSelectionEnd(Math.min(i, j));

ein.
Es funktioniert aber, wenn ich

text.setEnabled(false);

sage und die set/getSelection weglasse. Dann kann ich ja aber nichts mehr mit der Maus markieren.

Woran kann das liegen?

mfg

Die meisten Swing Methoden sind nicht "thread safe".
Weil das "Swing event handling" in einem speziellen Thread, dem "event dispatch thread", läuft,
müssen auch alle anderen Swing Komponentenmethoden auf dem "event dispatch thread" aufgerufen werden.
Programme, die diese Regel ignorieren, könnten zwar die meiste Zeit korrekt funktionieren,
sind aber unvorhersehbaren Fehlern unterworfen, die schwer nachvollziehbar sind.
Wenn der Code nicht im "event dispatch thread" läuft, benutzt man die Methode "SwingUtilities.invokeLater",
damit der Code im "event dispatch thread" ausgeführt wird:

       SwingUtilities.invokeLater(new Runnable() {
            public void run() {
                int i = text.getSelectionStart();
                int j = text.getSelectionEnd();
                text.setText(s);
                text.setSelectionStart(Math.max(i, j));
                text.setSelectionEnd(Math.min(i, j));
            }
        });

 

[Ralf Ueberfuhr]

Ergänzung: Möchtest du den Aufruf synchron haben, also warten, bis der Wert im Textfeld gesetzt ist, dann nimmt die Methode SwingUtilities#invokeAndWait(...)

 

[Antineutrino]

Vielen Dank für die Tips!