Ein sort problem

[Permafrost]

irgendwie finde ich einen fehler nicht evtl sieht ihr ihn ja.
ich habe noch eine eingabe liste in der ich die typen aussuchen kann und wenn ich welche ausgesucht habe und namen festgelegt habe und es mir dann anzeigen lassen möchte dann bleibt das programm hängen.
wenn ihr mehr quellcode braucht einfach fragen wollte nicht alles zukleistern.

    public static String[] setSort(String[] myArray) {


        String tempValue;
        boolean sorted = false;

        while (!sorted) {
            for (int i = 0; i < myArray.length - 1; i++) {
                sorted = true;
                if (myArray[i].compareToIgnoreCase(myArray[i + 1]) > 0) {
                    tempValue = myArray[i];
                    myArray[i] = myArray[i + 1];
                    myArray[i + 1] = tempValue;
                    sorted = false;
                }
            }
        }
        return myArray;
    }
}

    private static boolean printAnimalNames(List<Animal> oZoo) {
        List<String> oTypes = new LinkedList<String>();
        oTypes.addAll(getAnimalTypes(oZoo));
        for (String sTypes : oTypes) {
            System.out.println(" ");
            System.out.println(sTypes);
            List<String> oNicknames = new LinkedList<String>();
            oNicknames.addAll(getAnimalNames(oZoo, sTypes));
            //for(String sNicknames: oNicknames)
            //  System.out.println(sNicknames);

            sort = setSort(oNicknames.toArray(new String[0]));

            for (int i = 0; i < sort.length; ++i) {

                System.out.print(sort[i] + "\n");
                System.out.println();
            }
        }

        return true;
    }

    private static List<String> getAnimalTypes(List<Animal> oZoo) {
        List<String> oTypes = new LinkedList<String>();
        for (Animal oAnimal : oZoo) {
            boolean bEnde = false;
            for (String sType : oTypes) {

                if (oAnimal.sName.equalsIgnoreCase(sType)) {
                    bEnde = true;
                    break;
                }
            }
            if (!bEnde) {
                oTypes.add(oAnimal.sName);
            }

        }
        return oTypes;
    }

    private static List<String> getAnimalNames(List<Animal> oZoo, String sName) {
        List<String> oNicknames = new LinkedList<String>();
        for (Animal oAnimal : oZoo) {
            if (oAnimal.sName.equalsIgnoreCase(sName)) {
                oNicknames.add(oAnimal.msNickname + " " + oAnimal.msoBd.getDatum());
            }
        }
        return oNicknames;
    }

 

[goodcat]

Hi,

mal ein Schuss ins blaue... hört sich irgendwie nach einer Endlosschleife an.
Eine Exception bekommst Du nicht?

Schreib doch mal System.out.println() in deine Schleifen um zu sehen wo er hängen bleibt... bzw. starte doch einfach mal den Debugger...

 

[pl4gu33]

Hi,

mal ein Schuss ins blaue... hört sich irgendwie nach einer Endlosschleife an.
Eine Exception bekommst Du nicht?

Schreib doch mal System.out.println() in deine Schleifen um zu sehen wo er hängen bleibt... bzw. starte doch einfach mal den Debugger...

aber am Besten schreibst du in dein Syso verschiedene Sachen rein
und nicht "System.out.println()" sonst bringt dass auch net viel....

aber hört sich wirklich nach Endlosschleife an

 

[goodcat]

Hehe dachte eigentlich das wäre glasklar das in System.out.println("auch was sinnvolles reingehört")

 

[Permafrost]

hab den fehler dank eines kolegen gefunden war eine endlosschleife^^ drinnen
aber auch vielen dank für eure hilfe^^

public static String[] setSort(String[] myArray)
    {
        if(myArray.length < 2)
            return myArray;

        String tempValue;
        boolean sorted = false;

        while (!sorted) {
            
            for (int i = 0; i < myArray.length - 1; i++) {
                sorted = true;
                if (myArray[i].compareToIgnoreCase(myArray[i + 1]) > 0) {
                    tempValue = myArray[i];
                    myArray[i] = myArray[i + 1];
                    myArray[i + 1] = tempValue;
                    sorted = false;
                }
                
            }
        }
        return myArray;
    }

 

[Permafrost]

und ich hab wieder ein problem undzwar möchte ich jetzt das Array zu eine liste machen nur habe ich
probleme mit dem i+1 könntet ihr mir noch mal helfen wehre echt lieb von euch

 public static List<String> setSort(List<String> sList)
    {
        if(sList.size() < 2)
            return sList;

        String tempValue;
        boolean sorted = false;

        while (!sorted) {
            
            for (int i = 0; i <sList.size() - 1; i++) {
                sorted = true;
                if (sList.get(i).compareToIgnoreCase(sList(i + 1)) > 0) { //hier
                    tempValue = sList.get(i);
                    sList.get(i) = sList.get(i + 1); //hier
                    sList.get(i + 1) = tempValue; //hier
                    sorted = false;
                }
                
            }
        }
        return sList;
    }

 

[goodcat]

Hi,

also ich verstehe zwar deine Methode nicht ganz, aber egal.
Aber wo genau liegt denn dein Problem mit dem i+1?

Was mir zu deinem Code auf die schnelle auffällt...

[JAVA=15]
sList.get(i) = sList.get(i + 1); //hier
sList.get(i + 1) = tempValue; //hier
[/code]

Wenn Du einer ArrayList etwas zuweisen möchtest kannst Du kein sList.get() benutzten. Sondern du musst entweder sList.set(0, "String hier rein!") benutzen um auf den Index 0 etwas zu schreiben oder Du machst es mit s.List.add("String hier rein!"), wodruch der String in der ArrayList angehängt wird...

Also...

sList.set(i, sList.get(i + 1));
sList.set((i + 1), tempValue);