String name mit fortlaufender Nummer

[Otz]

hallo ich brauche mal wieder etwas hilfestelltung:

und zwar:

ich habe eine collection mit elementen der klasse server

jetzt will ich eines dieser elemente kopieren und ebenfalls in die collection einfügen.
natürlich soll sich hierzu der namen ändern. ich dachte an soetwas:

server1
server1_(1)

sollte bereits ein server1_(1) in der collection sein, soll es natürlich automatisch auf server1_(2) verändert werden.

diese aufgabe stellt mich aber vor größere probleme als gedacht.

mein lösungsansatz:

[Java]

Collection<Server> collection = cont.getAllItemIds(); //alle server objekte in collection laden

for (int i = 1; i <= collection.size(); i++) {

for (Iterator<Server> iter = collection.iterator(); iter.hasNext(); ) //Iterator zum durchgehen des containers
{Server s = iter.next();

String test= (cont.getItem(target).getBean().getName())+"_("+i+")"; //bau des strings mit dem verglichen werden soll.

if (test.equals(s.getName()+ "_(" + i + ")")) { //wenn der string bereits im container wird schleife abgebrochen
//und mit i++ erneut propiert. solange bis es die kopie noch nicht gibt. bsp: server1_(1) gibt es schon, dann break und mit server1_(2) erneut
break;
} else if(!cont.getItem(target).getBean().getName().equals(s.getName()+ "_(" + i + ")")){ //kann eingefügt werden
Server s1 = new Server();
s1.setName(cont.getItem(target).getBean().getName() + "_(" + i + ")");
s1.setStatus(cont.getItem(target).getBean().getStatus());
cont.addItem(s1);
}
[/Java]

hoffe ihr versteht mein problem

vielen dank schonmal

 

[Network]

Ich verstehe dein Problem voll und ganz: http://www.techfak.uni-bielefeld.de/ags/pi/lehre/AuDIISS01/style_guide.pdf

[STRIKE]Überprüfen ob ein Element bereits in der Collection vorhanden ist geht meines Wissens nach am besten mit ".contains( E )"[/STRIKE]

Gruß
Net

 

[Otz]

ich will ja nicht überprüfen ob ein element in der collection vorhanden ist, sondern ein element mit einem bestimmten namen.
das ist doch nicht das selbe...

 

[ARadauer]

teile und hersche... mach dir methoden für die einzelnen aufgaben, schaun ob der name in der liste vorhanden ist, name anahnd von nummer machen.

while( solange name in liste){
nummer+;
name = neuenNamen machen();

 

[nillehammer]

Muss es denn eine Collection sein? Du könntest Deine Server als values in eine Map schmeißen und als Keys deren Namen benutzen:

Map<String,Server> servers = new HashMap<>();

String serverName = <zuSuchenderName>;
if(servers.containsKey(serverName) {
  serverName = buildNewName();
}
servers.put(serverName, server);

Oder falls es bei Collection bleiben muss, implementiere in der Klasse Server die Methoden equals() und hashCode() so, dass sie den Servernamen einbeziehen. Dann funktioniert auch die contains(server)-Methode.

 

[Otz]

ahhh ich habs. habe einfach zu kompliziert gedacht^^

ich hatte heute morgen irgendwie die idee, dass die angehängte zahl immer die höchste sein muss.
beispiel:
im container sind: server1_(2) und server1_(3)

jetzt dachte ich es muss der server1_(4) kommen, was aber falsch ist. es muss ja server1_(1) kommen (der evtl. zuvor gelöscht wurde).

habe mich da etwas zu sehr darauf konzentriert und es mir unnötig schwer gemacht


so läufts jetzt:
[Java]
for (int i = 1; i <= collection.size()+1; i++) {

String einzufuegen = (cont.getItem(target).getBean().getName())+"_("+i+")";

System.out.println(einzufuegen);



for (Iterator<Server> iter = collection.iterator(); iter.hasNext(); ) {
Server s = iter.next();

if (einzufuegen.equals(s.getName())) {
temp = i;
break;
} else if(!iter.hasNext()){
Server s1 = new Server();
s1.setName(einzufuegen);
s1.setStatus(cont.getItem(target).getBean().getStatus());
cont.addItem(s1);
}
[/Java]


jetzt wirft er mir nur noch folgenden fehler: java.util.ConcurrentModificationException

 

[nillehammer]

jetzt wirft er mir nur noch folgenden fehler: java.util.ConcurrentModificationException

Klassiker. Du kannst nicht mit einem Iterator iterieren und während der Iteration Modifikationen auf der dahinter liegenden Collection vornehmen. Merke Dir die einzufügenden Elemente in einer temporären Collection und mache nach der Iteration ein addAll().

 

[nillehammer]

Andere Idee. Wenn Du aus Deiner Collection eine List machst, kannst Du mit ListIterator (ListIterator (Java Platform SE 7 )) iterieren. Der stellt Dir direkt eine add-Methode zur Verfügung. Dann entfällt die temporäre Collection und das addAll, weil Du während der Iteration über die add-Methode des ListIterators adden kannst.

 

[Otz]

[Java]
[TIPP]List[/TIPP]<Server> collection = cont.getAllItemIds();

for (int i = 1; i <= collection.size()+1; i++) {
String einzufuegen = (cont.getItem(target).getBean().getName())+"_("+i+")";
for ([TIPP]ListIterator[/TIPP]<Server> iter = collection.[TIPP]listIterator[/TIPP](); iter.hasNext(); ) {
Server s = iter.next();
if (einzufuegen.equals(s.getName())) {
break;
} else if(!iter.hasNext()){
Server s1 = new Server();
s1.setName(cont.getItem(target).getBean().getName()+"_("+i+")");
s1.setStatus(cont.getItem(target).getBean().getStatus());
iter.add(s1);
cont.addItem(s1);
}

}

}

}

}
[/Java]
jetzt kommt zwar kein fehler mehr aber er fügt acuh keine neuen elemente mehr ein.

was ist falsch?

 

[nillehammer]

Du verwendest die richtigen Methoden. Das ist schonmal gut. Der genaue Ablauf in Deinem Code ist für mich (zumal ohne Kontext) leider nicht nachzuvollziehen (und für Dich offensichtlich auch nicht ).
- Vielleicht sind die beiden Schleifen genau falsch rum geschachtelt
- Vielleicht soll es kein break, sondern ein continue sein
- Vielleicht passen die if-else-Verzweigungen nicht
- Vielleicht sieht jemand anders den Fehler.

Ich kann Dir nur den allgemeinen Rat geben, den ARadauer auch schon angedeutet hat. Pule Deine 4fache Verschachtelung mal in nette kleine Methoden auseinander. Dann siehst Du den Fehler vielleicht schon.

 

[Otz]

was ich nicht verstehe warum ich das hier doppelt machen muss?

[Java] iter.add(s1);
cont.addItem(s1); [/Java]

am anfang lese ich doch aus cont alle items aus und füge sie in die liste mit der ich hier arbeiten

[Java] List<Server> collection = cont.getAllItemIds();[/Java]

und mit dem cont.addItem() mach ich ja eigentlich wieder genau das gleiche was die exception auslöst oder nich?

 

[bone2]

sowas hier?

    public static void main(String[] args) throws ClassNotFoundException {
        List<String> serverList = new ArrayList<String>();
        serverList.add("Server1");
        serverList.add("Server2");
        serverList.add("Server2_(1)");
        serverList.add("Server2_(2)");
        serverList.add("Server3");

        System.out.println("prepared ServerList: " + serverList);

        String cloneTarget = "Server2";
        boolean foundPossibleName = false;
        int nameCounter = 0;
        String tmpName = cloneTarget;

        while (! foundPossibleName) {
            boolean nameIsBlocked = false;
            System.out.println("looking for name " + tmpName);

            for (String server : serverList) {
                if (server.equals(tmpName)) {
                    nameIsBlocked = true;
                    break;
                }
            }

            if (nameIsBlocked) {
                nameCounter++;
                System.out.println("name " + tmpName + "is already in use");
                tmpName = cloneTarget + "_(" + nameCounter + ")";
            }
            else {
                foundPossibleName = true;
                System.out.println(tmpName + " is free");
            }
        }

        serverList.add(tmpName);
        System.out.println("finished ServerList: " + serverList);
    }

prepared ServerList: [Server1, Server2, Server2_(1), Server2_(2), Server3]
looking for name Server2
name Server2is already in use
looking for name Server2_(1)
name Server2_(1)is already in use
looking for name Server2_(2)
name Server2_(2)is already in use
looking for name Server2_(3)
Server2_(3) is free
finished ServerList: [Server1, Server2, Server2_(1), Server2_(2), Server3, Server2_(3)]

cloneTarget = cont.getItem(target).getBean().getName()

und halt alle Strings in listen und schleifen passend zu server.getName() ersetzen

 

[Otz]

jaaaa, sieht nicht schlecht aus. ich versuch es mal anzupassen. vielen dank!!!

 

[hüteüberhüte]

Hab da mal was:

    public static void main(String[] args) {
        HashMap<String, Object> map = new HashMap<String, Object>();

        // 10 zufällige Elemente einfügen
        Random r = new Random();
        for (int i = 0; i < 10; i++) {
            String s;
            while (map.containsKey(s = Integer.toString(r.nextInt(20))));
            map.put(s, null);
        }

        // 10 Elemente erneut einfügen
        for (int i = 0; i < 10; i++) {
            putMap(map, map.keySet().toArray(new String[0])[r.nextInt(map.size())]);
        }

        // Ausgeben
        String[] a = map.keySet().toArray(new String[0]);
        Arrays.sort(a);
        System.out.println(Arrays.toString(a));
    }

    public static void putMap(Map<String, Object> map, String s) { // + value parameter
        int idx = 0;
        if (s.matches("^\\d+_\\d+$")) {
            int indexOf_ = s.indexOf('_');
            idx = Integer.parseInt(s.substring(indexOf_ + 1));
            s = s.substring(0, indexOf_);
        }
        String sNew;
        while (map.containsKey(sNew = s + "_" + ++idx));
        map.put(sNew, null);
    }

[0, 11, 12, 12_1, 12_2, 13, 13_1, 14, 14_1, 15, 17, 17_1, 19, 8, 8_1, 9, 9_1, 9_2, 9_3, 9_4]

Das Entfernen ist aber nicht so einfach wie das Einfügen, wenn es keine "Lücken" geben soll

 

[Otz]

mit einer mischung aus bone2´s lösung und meinen ansätzen habe ich es zum laufen bekommen.

[Java]
List<Server> collection = cont.getAllItemIds();
String einzufuegen = (cont.getItem(target).getBean().getName())+ "_(1)";

boolean foundPossibleName = false;
int nameCount = 1;

while (!foundPossibleName) {
boolean nameBereitsVergeben = false;

for (Server server : collection) {

if (server.getName().equals(einzufuegen)) {
System.out.println(server.getName());

nameBereitsVergeben = true;
break;
}
}
if (nameBereitsVergeben) {
nameCount++;
einzufuegen = (cont.getItem(target).getBean().getName())+ "_(" + nameCount + ")";
} else {
foundPossibleName = true;
}
}

Server s1 = new Server();
s1.setName(einzufuegen);
s1.setStatus(cont.getItem(target).getBean().getStatus());
cont.addItem(s1);

}
[/Java]

vielen dank für die Unterstützung