Swing App reagiert nicht

[sirbender]

Hi,

ich habe eine simple JList der ich einen ListSelectionListener hinzugefuegt habe. Mit dem listener lese ich abhaengig vom selektierten Item ein XML Dokument ein und lese einige Elemente aus:

		DOMParser parser = new DOMParser();
		parser.parse(file);
		Document doc = parser.getDocument();
...
...

Reproduzierbar (es handelt sich also nicht um ein Problem mit der Event-Dispatch Queue) friert mir die UI ein. Es wird keine Exception geworfen.

Wenn ich die Aktivitaet des ListSelectionListener auskommentiere friert allerdings nichts ein. Wie kann es sein, wenn beim XML Einlesen keine Exception geworfen wird, dass mir die UI einfriert? Trotz jahrelanger Erfahrung mit Swing steh ich auf dem Schlauch

Danke,
sb

 

[Marco13]

Ich nehme mal an, dass das Parsen nicht so lange dauert, dass du das Blockieren während des Parsens als ein "einfrieren" wahrnimmst. Dann würden System.out's, Debugging und für's Forum natürlich ein KSKB helfen. Ansonsten: Was passiert z.B. bei einer leeren Eingabedatei?

 

[sirbender]

Hmmm...eigentlich sind meine XML files sehr klein. Und die meisten werden auch schnell eingelesen. Aber ich hab jetzt gerade mal getestet um sicher zu gehen. Einige XMLs werden in der Tat schleichend langsam eingelesen. In meinen Tests 30 sec. Das erklaert, warum es einfriert. Wie gesagt...bei den meisten XMLs dauert es 10 ms...deswegen hatte ich es direkt im EDT laufen.

Wie kommt es das der Parser sooo lange braucht? Wie kann ich es schneller machen?

Ich hab mal ein Beispiel XML angehaengt, wo es ewig dauert: Bee clip art - vector clip art online, royalty free & public domain
Einfach auf Download und SVG Datei runterladen.

Die Datei ist sehr kurz. Wie kann das sein? Was muss ich an meinem XML-Parser code von oben aendern, damit es schneller geht?

Danke,
sb

 

[Marco13]

Wenn sie so klein ist kannst du sie auch inline hier posten....

 

[sirbender]

Ok. Siehe Beispiel unten.

[XML]
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
<!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 20010904//EN"
"http://www.w3.org/TR/2001/REC-SVG-20010904/DTD/svg10.dtd">
<!-- Created with Inkscape (Inkscape. Draw Freely.) --><svg height="400pt" id="svg548" inkscape:version="0.38.1" sodipodi:docbase="/var/www/html/svg_gallery/svg/misc" sodipodi:docname="mushroom.svg" sodipodi:version="0.32" width="400pt" xmlns="http://www.w3.org/2000/svg" xmlns:inkscape="http://www.inkscape.org/namespaces/inkscape" xmlns:sodipodi="http://sodipodi.sourceforge.net/DTD/sodipodi-0.dtd" xmlns:xlink="http://www.w3.org/1999/xlink">
<metadata>
<rdf:RDF xmlns:cc="http://web.resource.org/cc/" xmlns:dc="http://purl.org/dc/elements/1.1/" xmlns:rdf="http://www.w3.org/1999/02/22-rdf-syntax-ns#">
<cc:Work rdf:about="">
<dc:title>Clipart by Nicu Buculei - bee</dc:title>
<dc:description></dc:description>
<dc:subject>
<rdf:Bag>
<rdf:li></rdf:li>
<rdf:li>plant</rdf:li>
</rdf:Bag>
</dc:subject>
<dcublisher>
<cc:Agent rdf:about="http://www.openclipart.org">
<dc:title>Nicu Buculei</dc:title>
</cc:Agent>
</dcublisher>
<dc:creator>
<cc:Agent>
<dc:title>Nicu Buculei</dc:title>
</cc:Agent>
</dc:creator>
<dc:rights>
<cc:Agent>
<dc:title>Nicu Buculei</dc:title>
</cc:Agent>
</dc:rights>
<dc:date></dc:date>
<dc:format>image/svg+xml</dc:format>
<dc:type rdf:resource="http://purl.org/dc/dcmitype/StillImage"/>
<cc:license rdf:resource="http://web.resource.org/cc/PublicDomain"/>
<dc:language>en</dc:language>
</cc:Work>
<cc:License rdf:about="http://web.resource.org/cc/PublicDomain">
<ccermits rdf:resource="http://web.resource.org/cc/Reproduction"/>
<ccermits rdf:resource="http://web.resource.org/cc/Distribution"/>
<ccermits rdf:resource="http://web.resource.org/cc/DerivativeWorks"/>
</cc:License>
</rdf:RDF>
</metadata>
<defs id="defs550"/>
<sodipodi:namedview id="base" inkscape:cx="200.00000" inkscape:cy="290.15411" inkscape:window-height="718" inkscape:window-width="675" inkscape:window-x="25" inkscape:window-y="24" inkscape:zoom="1.0000000" showgrid="true"/>
<g id="g837">
<path d="M 215 202.5 C 215 202.5 226.25 328.75 220 377.5 C 213.75 426.25 210 443.75 223.75 461.25 C 252.5 468.75 345 478.75 366.25 446.25 C 365 403.75 315 345 306.25 297.5 C 297.5 250 335.65939 203.89563 278.15939 192.64563 C 220.65939 181.39563 213.75 205 215 202.5 z " id="path785" sodipodi:nodetypes="ccccccc" style="fill:#fafada;fill-rule:evenodd;stroke:black;stroke-opacity:1;stroke-width:10.000000;stroke-linejoin:round;stroke-linecap:butt;fill-opacity:1;stroke-dasharray:none;"/>
<path d="M 215 211.25 C 215 211.25 218.75 262.5 203.75 301.25 C 230 320 308.75 297.5 327.5 297.5 C 302.5 258.75 312.5 223.75 312.5 216.25 C 312.5 208.75 212.5 208.75 215 211.25 z " id="path786" sodipodi:nodetypes="ccccc" style="fill:#f0f0df;fill-rule:evenodd;stroke:black;stroke-opacity:1;stroke-width:10.000000;stroke-linejoin:round;stroke-linecap:butt;fill-opacity:1;stroke-dasharray:none;"/>
<path d="M 20 271.25 C 31.25 182.5 161.25 37.5 262.5 33.75 C 378.75 16.25 481.25 145 493.75 191.25 C 462.5 210 311.25 221.25 273.75 237.5 C 236.25 261.25 41.25 273.75 20 271.25 z " id="path784" sodipodi:nodetypes="ccccc" style="fill:#fa0000;fill-rule:evenodd;stroke:#000000;stroke-opacity:1;stroke-width:10.000000;stroke-linejoin:round;stroke-linecap:butt;fill-opacity:1;stroke-dasharray:none;"/>
<path d="M 255 118.75 C 200 141.25 212.5 226.25 268.75 217.5 C 360 201.25 293.75 108.75 255 118.75 z " id="path787" sodipodi:nodetypes="ccc" style="fill:#faffff;fill-rule:evenodd;stroke:none;stroke-opacity:1;stroke-width:1pt;stroke-linejoin:miter;stroke-linecap:butt;fill-opacity:1;"/>
<path d="M 32.271645,254.45433 C 26.021645,244.45433 58.521645,161.95433 93.521645,176.95433 C 137.27164,198.20433 72.271645,285.70433 32.271645,254.45433 z " id="path788" sodipodi:nodetypes="ccc" style="fill:#faffff;fill-rule:evenodd;stroke:none;stroke-opacity:1;stroke-width:1pt;stroke-linejoin:miter;stroke-linecap:butt;fill-opacity:1;"/>
<path d="M 248.36299,38.521645 C 328.36299,27.271645 274.61299,66.021645 247.11299,73.521645 C 223.36299,78.521645 142.11299,66.021645 248.36299,38.521645 z " id="path789" sodipodi:nodetypes="ccc" style="fill:#faffff;fill-rule:evenodd;stroke:none;stroke-opacity:1;stroke-width:1pt;stroke-linejoin:miter;stroke-linecap:butt;fill-opacity:1;"/>
<path d="M 142.5 102.5 C 173.75 91.25 230 153.75 162.5 178.75 C 93.75 187.5 107.5 117.5 142.5 102.5 z " id="path790" sodipodi:nodetypes="ccc" style="fill:#faffff;fill-rule:evenodd;stroke:none;stroke-opacity:1;stroke-width:1pt;stroke-linejoin:miter;stroke-linecap:butt;fill-opacity:1;"/>
<path d="M 430.93268,107.65866 C 398.43268,81.408658 352.18268,88.908658 363.43268,146.40866 C 389.68268,200.15866 459.68268,153.90866 430.93268,107.65866 z " id="path791" sodipodi:nodetypes="ccc" style="fill:#faffff;fill-rule:evenodd;stroke:none;stroke-opacity:1;stroke-width:1pt;stroke-linejoin:miter;stroke-linecap:butt;fill-opacity:1;"/>
<path d="M 327.18268,42.817316 C 297.18268,59.067316 332.18268,117.81732 383.43268,70.317316 C 373.43268,59.067316 335.93268,45.317316 327.18268,42.817316 z " id="path792" sodipodi:nodetypes="ccc" style="fill:#faffff;fill-rule:evenodd;stroke:none;stroke-opacity:1;stroke-width:1pt;stroke-linejoin:miter;stroke-linecap:butt;fill-opacity:1;"/>
<path d="M 488.27403,187.88701 C 483.27403,175.38701 470.61537,131.09134 443.11537,167.34134 C 413.50238,223.66104 488.27403,187.88701 488.27403,187.88701 z " id="path793" sodipodi:nodetypes="ccc" style="fill:#faffff;fill-rule:evenodd;stroke:none;stroke-opacity:1;stroke-width:1pt;stroke-linejoin:miter;stroke-linecap:butt;fill-opacity:1;"/>
<path d="M 296.1165 307.44337 C 302.58899 317.15211 352.75081 414.23948 354.36893 432.03883 C 355.98706 449.83819 338.18771 456.31068 328.47897 453.07443 C 318.77023 449.83818 284.78964 318.77022 296.1165 307.44337 z " id="path1193" sodipodi:nodetypes="cccc" style="fill:#e6e4df;fill-rule:evenodd;stroke:none;stroke-opacity:1;stroke-width:1pt;stroke-linejoin:miter;stroke-linecap:butt;fill-opacity:1;"/>
<path d="M 289.64401 37.216828 C 208.73786 30.744336 72.815534 116.50486 29.126214 260.5178 C 42.071197 265.37217 33.980582 273.46278 74.433661 260.5178 C 127.83171 69.579284 249.19093 63.106796 289.64401 37.216828 z " id="path1194" sodipodi:nodetypes="cccc" style="fill:#ffffff;fill-rule:evenodd;stroke:none;stroke-opacity:1;stroke-width:1pt;stroke-linejoin:miter;stroke-linecap:butt;fill-opacity:0.433333;"/>
</g>
</svg>
[/XML]

 

[Marco13]

(*planlosrat*) Wie sieht's denn aus, wenn man den metadata-Teil rausnimmt?

 

[nillehammer]

DOMParser parser = new DOMParser();
parser.parse(file);
Document doc = parser.getDocument();

Deine Art zu parsen ist -sagen wir mal- etwas ungewöhnlich. Du arbeitest direkt mit Klassen aus einem internal Package. Das macht man eigentlich nicht. Richtig wäre es so:

DocumentBuilderFactory dbf = DocumentBuilderFactory.newInstance();

// Du benutzt Namespaces, Deine Parser müssen also namespace aware sein
dbf.setNamespaceAware(true);
    
DocumentBuilder db = dbf.newDocumentBuilder();

Document doc = db.parse(...);

Das mal vorab. Mit dem setNamespaceAware könnte sich möglicherweise Dein Probem lösen. Bei einem Test ist mir auch noch aufgefallen, dass Java versucht, World Wide Web Consortium (W3C) aufzulösen. Das hat bei meiner Umgebung nicht funktioniert. Deswegen konnte ich es nicht vollständig austesten. Aber vielleicht führt die Verbindung dort hin zu einer Verzögerung. Leider auch nur etwas schlaues Gerate, aber vielleicht hilft's ja.

 

[sirbender]

Nein. Hilft leider nicht. Ich habe das XML auch schon als String eingelesen um das Einlesen auszuschliessen. Dauert immer noch 30 sec.

 

[Marco13]

Ja, hab's gerade mal schnell durch den Debugger laufen lassen: Er verschluckt sich wohl beim Öffnen des InputStreams von w3.org. Wenn man das hier
[xml]
<!DOCTYPE svg PUBLIC "-//W3C//DTD SVG 20010904//EN"
"http://www.w3.org/TR/2001/REC-SVG-20010904/DTD/svg10.dtd">
[/xml]
rausnimmt, geht's schnell.

 

[sirbender]

Wow...ganz schoen krass. Also ich finde das ist ein Java Bug. Tests auf dtds sollte man explizit anschalten muessen wenn sie zu so krassem Verhalten fuehren. Was denkt ihr? Ist es ein Bug? Kann ja nicht sein, dass man valides XML vorprozessieren muss, oder?

 

[Marco13]

Hmja... scheint als würde die Seite schlicht so lange brauchen :bahnhof: Hab' mal kurz gewebsucht, und z.B. auf java - Make DocumentBuilder.parse ignore DTD references - Stack Overflow ein paar Hinweise gefunden. Das erste

        db.setEntityResolver(new EntityResolver() {
            @Override
            public InputSource resolveEntity(String publicId, String systemId)
                    throws SAXException, IOException {
                if (systemId.contains(".dtd")) {
                    return new InputSource(new StringReader(""));
                } else {
                    return null;
                }
            }
        });

funktioniert, aber das mit dem 'contains' sieht hackig aus, ggf. nochmal gegenchecken WAS genau da gemacht wird, und wie die Alternativen sind.

 

[nillehammer]

Er verschluckt sich wohl beim Öffnen des InputStreams von w3.org. Wenn man das hier

War auch meine Vermutung. Konnte ich nur nicht testen, weil unser interner DNS das nicht aufgelöst hat.

ggf. nochmal gegenchecken WAS genau da gemacht wird, und wie die Alternativen sind.

Der Parser muss sich die DTD herunterladen, um dagegen validieren zu können. Ich würde mir die Datei herunterladen und als Bestandteil des Programms mit ausliefern. Machen viele XML-basierte Anwendungen so. Die URL dann entsprechend anpassen.

 

[Marco13]

Die URL dann entsprechend anpassen.

Gerade da bin ich mir nicht sicher: Wenn man damit "irgendeine" Datei parsen will, die man ggf. gerade erst von einer URL geladen hat, kann man sie ja schlecht ändern. Da muss man wohl irgendwo in den Innereien des Parsers wühlen.