Ersatz für Class.forName

[sylo]

Hi zusammen

ich wollte wissen ob es eine "bessere" Möglichkeit gibt, als das Erstellen von Objekten mit Class.forName, damit es performanter wird.

Ich habe einen Jtree. Je nachdem welchen Knoten ich anklicke, müssen andere Objekte erzeugt werden. Um das Ganze eben dynamisch zu gestalten, benutze ich Class.forName. Dies erscheint mir aber etwas langsam.

Vielleicht kennt ja jemand eine bessere Lösung. Bei Bedarf kann ich ein wenig Code bieten.

Grüße
sylo

 

[SlaterB]

Code wäre gut,
hast du wirklich Performance-Probleme, erstellst du Millionen Objekte pro Sekunde?
falls die auch im JTree angezeigt werden, sollte alle graphische Arbeit weit mehr Probleme bereiten als Objekterzeugung,
falls nicht gerade die Konstruktoren mit viel Arbeit beladen sind, aber auch das wäre dann von Class.forName unabhängig,

allgemein kannst du dir zu jeder Klasse eine Factory erzeugen lassen/ vorgeben,
notfalls diese einmalig mit Class.forName erzeugen, was immer das bedeutet,
und dann sollte die Factory ganz normal passende Objekte liefern

 

[sylo]

Hi

ich versuch mal den Code kurz zu halten.

Es gibt ein JsplitPane. Auf der linken Seite wird ein Jtree angezeigt. Wenn ich auf einen Knoten klicke wird die Methode change() aufgerufen. Hier steht unteranderem:

Panel attributes = new Panel();
JPanel attributesPanel = attributes.getPanel(path, parser, textFieldStatusBar, this);
jScrollPaneAttributes = new JScrollPane(attributesPanel);
jSplitPane.setRightComponent(jScrollPaneAttributes());

In der eigenen Panel Klasse werden nun, je nach Knoten, verschiedene Elemente erzeugt

Class myClass;
                        Constructor myConstructor;
                        try {
                            myClass = Class.forName("plugin." + className);
                            Class[] class1 = new Class[9];
                            class1[0] = String.class;
                            class1[1] = String.class;
                            class1[2] = String[].class;
                            class1[3] = boolean.class;
                            class1[4] = Parser.class;
                            class1[5] = TreePath.class;
                            class1[6] = JTextField.class;
                            class1[7] = String.class;
                            class1[8] = GUIElements.class;
                            myConstructor = myClass.getConstructor(class1);

                            Object[] object1 = new Object[9];
                            object1[0] = nodeWithNameAttribute.getNodeValue();
                            object1[1] = currentValue;
                            object1[2] = enumValues;
                            object1[3] = required;
                            object1[4] = parser;
                            object1[5] = path;
                            object1[6] = statusBar;
                            object1[7] = defaultValue;
                            object1[8] = guiElements;
                            comp = ((JComponent) myConstructor.newInstance(object1));

und zurückgeliefert.
Jetzt nehmen wir an, es soll ein Label und Textfield erzeugt werden, dann wird an dieser Stelle der Konstruktor für die Klasse "Text" aufgerufen. Diese Klasse "Text" erbt nun von einer anderen Klasse die "Plugin" heißt.

Wenn ich auf einen Knoten klicke, wo er 15 mal ein Label und ein Textfield erzeugen muss, braucht er ca. 2 sekunden. Ist meiner Meinung nach etwas viel.

@SlaterB
Was genau meinst du mit der Factory und wo genau sollte ich die einsetzen?

Grüße
sylo

 

[SlaterB]

der obige Code sieht durchaus recht wüst aus, ob er Zeit benötigt darf aber bezweifelt werden,
viel eher sind immer noch die Komponenten an sich langsam

teste alternativ alles mit normalen Aufrufen
> import plugin.Sowieso;
> Sowieso s = new Sowieso(nodeWithNameAttribute.getNodeValue(), currentValue, ..);
geht es dann schneller?

--------

mit Factory meinte ich, dass der Ersteller des Plugins Sowieso auch eine weitere Plugin-Klasse SowiesoFactory erstellt,
wo dann exakt der Code
> Sowieso s = new Sowieso(nodeWithNameAttribute.getNodeValue(), currentValue, ..);
korrekt steht, ohne diese unnötigen Arrays,

ein Objekt der Factory-Klasse muss aufwendig erstellt werden, danach ist es dann ein normaler Methodenaufruf um Sowieso-Objekte zu erstellen:
> sFactory.createNewElement();
oder
> sFactory.createNewElement(this); // die Factory holt aus this alle benötigten Infos

ob das hier sinnvoll weiterhilft und ob die Factory an alle benötigten Parameter kommt, ist sicherlich eine offene Frage

 

[sylo]

Habs gerade getestet. Es ändert sich aber nichts an der Geschwindigkeit. Liegt es dann am Zeichnenvorgang?

 

[SlaterB]

worum geht es nochmal genau?,
drückst du auf ein JTree-Element und dann wird ein größeres Panel neu angelegt und irgendwo angezeigt?
2 sec kommt mir wirklich etwas lang vor, ja

mit System.currentTimeMillis() vor und nach Aktionen kann man gewisse Zeiten messen,
so kannst du vielleicht schauen, ob die Objekterzeugung an sich länger dauert (messbar) oder erst danach das Zeichnen (eher nicht messbar)

eine andere Richtung wäre, statt der komplizierten Komponenten einfache JLabel zu verwenden,
vielleicht auch erst nur eins statt 13, und so zu beobachten, was wie lange dauert

statt JTree tut es testweise auch ein simpler Button mit ActionListener, falls du hier mehr Code posten willst,
sonst hoffentlich keinen Einfluss, ein Test schadet aber nicht

 

[sylo]

Hi SlaterB

Das mit dem System.currentTimeMillis() war ein super Tipp. :applaus: Habe damit quasi die "FehlerQuelle" eingrenzen können.

Es lag an der Klasse Plugin. In dieser Klasse war global folgendes definiert:

private JFileChooser fileChooser = new JFileChooser;

d.h. es wurden bei einem Klick 13 Filechooser erzeugt. Das dauert halt

Vielen dank für deine Hilfe. Jetzt läufts wesentlich schneller.

Grüße
sylo

EDIT: Vorher hat das ganze 1844 millisekunden gedauert und jetzt 110 :autsch: