Worst-Performance-Award für Arbeiten mit ListModel

[hdi]

Hallo...

ich hab grad ne Methode geschrieben, und ich kuck sie mir an und denke "oh mein gott :roll:"
Leider stell ich mich beim Arbeiten mit arrays usw immer sehr doof an, vllt geh ich auch das komplett
falsch an.

Hab ne JList, mit einem String[] model. Da man bei einem ListModel weder ein "add" noch "remove" hat, muss
ich mir das komplette Model immer neu machen, wenn ich etwas lösche oder adde.

Im Anhang meine grandiose Methode, die Elemente aus der Liste löscht. Wäre nett, wenn ihr mir da
ne bessere Version vorschlagt, ich stell mich bei sowas wie gesagt immer recht doof an, ich machs immer
unglaublich kompliziert, aber mir fehtl die Übung mit sowas. Bitteschön:

/* Neue Liste = alte Liste - selektierte Einträge */
			int[] selectedIndices = list.getSelectedIndices(); 
			int oldSize = list.getModel().getSize();
			int newSize = oldSize - selectedIndices.length;
			String[] newListModel = new String[newSize];
			int newIndex = 0;
			/* Liste kopieren bis auf selektierte Einträge */
			for (int i = 0; i < oldSize; i++) {
				/* Soll dieser Eintrag kopiert werden ? */
				boolean toCopy = true;
				for (int j : selectedIndices) {
					if (i == j) {
						// nicht kopieren, Element überspringen
						toCopy = false;
						break;
					}
				}
				if (toCopy) {
					// kopieren, es ist keines der selektierten Elemente:
					newListModel[newIndex] = (String) list.getModel()
							.getElementAt(i);
					newIndex++;
				}
			}
			/* Liste durch neue ersetzen und wieder an das ScrollPane hängen */
			list = new JList(newListModel);
			listPane.setViewportView(list);

Danke :toll:

 

[stev.glasow]

schau dir mal java.util an das ist zwar "Bauklötze bauen für Spielkinder" mit bissel overhead aber wenn juckt das hierbei. find's lustig bei sowas auf performance zu achten, den unterschied merkt man eh nicht.

 

[hdi]

lol, beginnt jetz das Thread-übergreifende Stalking Ich hab auch nie gesagt, dass ich Bauklötze nich mag, sondern nur, dass es nun mal Fakt ist. Aber...hey... bitte, lassen Sie mich in Frieden, um Gottes Willen :cry:

edit: also ich bin etwas überfordert mit util. Was brauch ich hier? Ich hab n ListModel, dazu gibts nix in util, das einzige
was ich noch habe is ja ein Array. Aber irgendwie kann ich da keine Methode von brauchen.

Ich wollt's auch mit System.arraycopy machen, aber ich kann das ListModel nicht in ein Array casten. Sorry, dieser Tipp hilft mir grad irgendwie nicht weiter ???:L Ich brauch auch nich wirklich Methoden, die ne bessere Performance haben, auch wenn der Titel das sagt. Wär natürlich auch gut, aber vorallem sieht der code doch einfach total sch*** aus.
Diese Sache mit dem boolean dazwischen und so, ich weiss ja nicht.. Geht das nich kürzer?

 

[stev.glasow]

ich dachte jetzt so in der art:

List<Integer> selectedIndices = Arrays.asList(list.getSelectedIndices());
List<String> newListModel = new ArrayList<String>();
ListModel lmod = list.getModel();

for(int i = 0; oldSize.getSize(); ++i)
	if(!selectedIndices.contains(i)) 
		newListModel.add((String)lmod.getElementAt(i));

list = new JList(newListModel.toArray(new String[newListModel.size()]));

ist nicht getestet.

aber bau dir mal nen vernüftiges model - da die JList neu anzulegen verbraucht einiges mehr an ressourcen

 

[hdi]

List<Integer> selectedIndices = Arrays.asList(list.getSelectedIndices());

geht leider nicht, weil er nicht von List<int[]> nach List<Integer> casten kann. Hab auch versucht
die for-schleife irgendwie anzupassen, krieg's aber so nich hin.

edit: Was meinst du mit "vernünftiges" Model? Wie sieht ein vernünftiges Model aus?

 

[stev.glasow]

List<Integer> selectedIndices = Arrays.asList(list.getSelectedIndices());

geht leider nicht, weil er nicht von List<int[]> nach List<Integer> casten kann.

hättest jetzt auch einfach ne ArrayList<Integer> anlegen können und dann die werte selber eingefügt.

Hab auch versucht die for-schleife irgendwie anzupassen, krieg's aber so nich hin.

int[] selectedIndices = list.getSelectedIndices();
// Arrays.sort(selectedIndices); [edit] kann raus
		
List<String> newListModel = new ArrayList<String>();
ListModel lmod = list.getModel();
	
int s = 0;
for(int i = 0; lmod.getSize(); ++i)
   if(s < selectedIndices.length && i == selectedIndices[s])
	   ++s;
   else	   
      newListModel.add((String)lmod.getElementAt(i));

list = new JList(newListModel.toArray(new String[newListModel.size()]));

Was meinst du mit "vernünftiges" Model? Wie sieht ein vernünftiges Model aus?

das interface ListModel selber implementieren - dann kannst auch einfach bestimmte elemente aus dem model entfernen, etc. musst mal im forum oder mit google suchen.

 

[Marco13]

Ähm - warum eigentlich das gewurste mit einem eigenen ListModel und vor allem dem erstellen einer neuen JList :shock: ? Was spricht gegen ein DefaultMutableListModel? Da hat man alles, was man braucht, und braucht auch keine neue JList zu erstellen (was IMHO ziemlich murksig ist....)

 

[hdi]

Du meinst DefaultListModel oder? Mutable gibts nicht, aber es ist veränderbar. Hab das jetz auch genutzt, und schwuppst sieht meine ganze Methode nun so aus:

			DefaultListModel listModel = (DefaultListModel) list.getModel();
			int i = 0;
			while ((i = list.getSelectedIndex()) != -1) {
				listModel.remove(i);
			}

Das is doch mal was :toll: Danke