ImageIcon: ZipException bei ungültiger PNG-Datei

[Joe77]

Hallo,

ich habe ein Programm, welches PNG-Bilder von verschiedenen URLs runterlädt.
Wird das Bild gefunden, wird es gespeichert ansonsten wird die nächste URL probiert.
Das funktioniert auch problemlos.

Manchmal kommt es aber vor, dass das Bild bei einer URL zwar gefunden wird, aber eigentlich keine gültige PNG-Datei ist (weil jemand dort eine ungültige Datei zur Verfügung gestellt hat).

Beim Laden einer ungültigen Datei in ein

ImageIcon

tritt eine

ZipException

auf, welche aber nicht abgefangen werden kann.
Meine Vermutung: Die Exception wird bereits in

ImageIcon

(oder woanders im Call-Stack) abgefangen und über

printStackTrace()

ausgegeben.

Alles was ich will ist, auf eine ungültige Datei reagieren zu können und ggf. mit der nächsten URL weitermachen.

Mit dem folgenden Test-Programm kann das ganze getestet werden. Auf einen Button-Druck hin versucht es die Dateien

c:\valid.png

und

c:\corrupt.png

zu öffnen und anzuzeigen.
Beim Öffnen von

c:\corrupt.png

tritt dann in Zeile 98 die Exception auf und wird auf der Konsole ausgegeben.
Wenn ich versuche die ZipException abzufangen, schimpft der Java-Compiler:

exception java.util.zip.ZipException is never thrown in body of corresponding try statement

Das abfangen einer allgemeinen Exception bringt auch nichts. Der entsprechende Code-Teil wird nicht betreten.

In der angehängten Datei

Test.zip

die beiden PNG-Bilder, die JAVA-Datei und eine JAR-Datei enthalten.

Noch mal meine Frage: wie kann ich auf diese Exception reagieren bzw. wie kann ich sonst überprüfen, ob die PNG-Datei gültig ist.

Schon mal Danke im voraus
Joe

import java.awt.Color;
import java.awt.Dimension;
import java.awt.FlowLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.File;
import java.net.MalformedURLException;
import java.util.zip.ZipException;

import javax.swing.BoxLayout;
import javax.swing.ImageIcon;
import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JPanel;
import javax.swing.JTextArea;
import javax.swing.border.LineBorder;

public class Test extends JFrame {

	private JLabel validImage;
	private JLabel corruptImage;
	private JTextArea error;

	public static void main(String[] args) {
		new Test();
	}

	public Test() {
		super();
		this.setDefaultCloseOperation(EXIT_ON_CLOSE);
		this.setSize(400, 500);
		this.setTitle("Test");

		JPanel pnImages = new JPanel(new FlowLayout());
		validImage = new JLabel();
		validImage.setBorder(new LineBorder(Color.BLACK));
		validImage.setPreferredSize(new Dimension(170, 235));
		corruptImage = new JLabel();
		corruptImage.setBorder(new LineBorder(Color.BLACK));
		corruptImage.setPreferredSize(new Dimension(170, 235));
		pnImages.add(validImage);
		pnImages.add(corruptImage);

		JPanel pnButton = new JPanel(new FlowLayout());
		JButton button = new JButton("Load images");
		button.addActionListener(new ActionListener() {
			public void actionPerformed(ActionEvent ev) {
				loadImages();
			}
		});
		pnButton.add(button);

		JPanel pnError = new JPanel(new FlowLayout());
		error = new JTextArea();
		error.setPreferredSize(new Dimension(350, 100));
		error.setBorder(new LineBorder(Color.BLACK));
		error.setFocusable(false);
		pnError.add(error);

		JPanel panel = new JPanel();
		panel.setLayout(new BoxLayout(panel, BoxLayout.Y_AXIS));
		panel.add(pnImages);
		panel.add(pnButton);
		panel.add(pnError);

		this.setContentPane(panel);

		this.setVisible(true);
	}

	private void loadImages() {
		validImage.setIcon(loadImage("c:\\valid.png"));
		corruptImage.setIcon(loadImage("c:\\corrupt.png"));
	}

	private ImageIcon loadImage(String filename) {
		File file = new File(filename);
		ImageIcon icon = null;

		if (file.exists()) {
			try {
				// With corrupt.png the next statement prints out:
				// java.util.zip.ZipException: incorrect header check
				// at
				// java.util.zip.InflaterInputStream.read(InflaterInputStream.java:164)
				// at
				// java.io.BufferedInputStream.fill(BufferedInputStream.java:235)
				// at
				// java.io.BufferedInputStream.read(BufferedInputStream.java:254)
				// at
				// sun.awt.image.PNGImageDecoder.produceImage(PNGImageDecoder.java:362)
				// at
				// sun.awt.image.InputStreamImageSource.doFetch(InputStreamImageSource.java:264)
				// at
				// sun.awt.image.ImageFetcher.fetchloop(ImageFetcher.java:189)
				// at sun.awt.image.ImageFetcher.run(ImageFetcher.java:153)
				icon = new ImageIcon(file.toURI().toURL());
			} catch (MalformedURLException e) {
				error.setText(error.getText() + "\n"
						+ "MalformedURLException!!!");
			} catch (Exception e) {
				error.setText(error.getText() + "\n" + "Exception!!!");
			}
			// Can't catch ZipException because compiler says: exception
			// java.util.zip.ZipException is never thrown in body of
			// corresponding try statement
			// } catch (ZipException e) {
			// System.out.println("ZipException!!!");
		} else {
			error.setText(error.getText() + "\n" + "File not found: "
					+ filename);
		}

		return icon;
	}
}

 

[SlaterB]

du kannst da genauso gut reagieren wie wenn die Bank während einer Online.Überweisung Geld von deinem Konto abzieht, nämlich gar nicht,
eine Methode hat ein Interface mit Parametern, Rückgabewert und wirft die Exceptions oder nicht,
was im innereren passiert kannst du nicht beeinflussen,

von Überschreiben der Klasse abgesehen, dazu detailliert den Quellcode anschauen, die Fehlerstelle finden und soweit möglich eine eigene Klasse verwenden mit geänderten Ablauf oder die Java-API verändern..

---

wenn du direkt an der Datei erkennen kannst, ob sie korrekt oder korrupt ist, dann das vorher selber testen

-----

was kommt eigentlich letztlich an deinen Aufrufer zurück, null-Wert oder eine andere RuntimeException?

 

[Joe77]

von Überschreiben der Klasse abgesehen, dazu detailliert den Quellcode anschauen, die Fehlerstelle finden und soweit möglich eine eigene Klasse verwenden mit geänderten Ablauf oder die Java-API verändern..

Nachdem die Exception in

InflaterInputStream.read

auftritt hab ich mal probiert selber diesen Stream aufzumachen:

try
{
  fstream = new FileInputStream(file);
  inflater = new InflaterInputStream(fstream);
  byte[] bytes = new byte[64000];
  // FIXME Always throws ZipException, also on valid images.
  inflater.read(bytes);
  result = true;
}
catch (Exception e)
{
  result = false;
}

Ich kann dann tatsächlich die Exception abfagen, allerdings trat sie auch beim gültigen Bild auf. D.h. für mich, bei gültigen Bildern kommt er bei der Original-Java-Klasse eigentlich gar nicht zum InflaterInputStream.

Wenn ich mir den Call-Stack bei der Exception so anschaue, müsste ich eigentlich dann folgende Klassen überschreiben:

• ImageIcion
• ImageFetcher
• InputStreamImageSource
• PNGImageDecoder
• BufferedInputStream
• InflaterInputStream

----------------------------------

wenn du direkt an der Datei erkennen kannst, ob sie korrekt oder korrupt ist, dann das vorher selber testen

Das weiß ich leider nicht wie ich das machen soll.
----------------------------------

was kommt eigentlich letztlich an deinen Aufrufer zurück, null-Wert oder eine andere RuntimeException?

Das erstellte ImageIcon ist dann seltsamerweise eigentlich ok. Ich die Width, Height und Description auslesen und die Werte sind richtig. Andere Properties (falls überhaupt vorhanden) hab ich noch abgefragt. Es wird aber nichts angezeigt.
Von dem her kann ich auf Grund des ImageIcon nicht unterscheiden, ob das Bild gültig ist oder nicht.