Methoden Binärbaum, Traversierung in Array speichern

jan_nick

Neues Mitglied
Guten Abend,

ich habe soeben dieses Forum entdeckt und möchte mich direkt mit einer Frage zur Optimierung meiner bisherigen Lösung an euch wenden:
Es geht um das Ausgeben eines Binären Baumes in richtiger (sortierter) Reihenfolge. Die Baumstruktur ist bereits vorhanden, sprich es geht nicht um das Einfügen oder Umändern der einzelnen Knoten des Binärbaumes, sondern wirklich nur um deren Ausgabe (Inorder). Meine bisherige Lösung funktioniert soweit, jedoch ist diese lediglich nur auf die in der Aufgabe beschriebenen 7 Knoten mit Werten von 1 bis 7 anwendbar und nicht flexibel auf beispielsweise negative Zahlen. Zudem soll die Anzahl der Knoten elegant ohne weitere globale Variablen außerhalb der Klasse ausgegeben werden, was bei mir lediglich mit einer globalen Variable und einem Zähler funktioniert.
Hat jemand Vorschläge zur Optimierung meiner Aufgabe?

Code:
import java.util.Arrays;

public class Aufgabe36 {
   
    /** 
     * Methode binSort wie in der Aufgabenstellung beschrieben
     * 
     * @param node der Wurzelknoten des betrachteten (Teil-)Baums
     * @param a int-array, in dem die Werte der Knoten abgelegt werden sollen (Achtung: Array muss so viele Eintraege haben wie Knoten im Baum sind)
     * @param pos aktuelle Array-Position (0 fuer Aufruf mit der Wurzel des gesamten Baums)
     * @return Anzahl Knoten des Baums
     */
    int knotenanzahl = 0;
   
    public int binSort(BinTreeNode node,int[] a,int pos) {
       
        //System.out.println(pos);
        knotenanzahl++;
       
        pos = node.getValue()-1;
       
        if(node.getLeftChild() != null){

            binSort(node.getLeftChild(), a, pos);

        }
       
        a[pos] = node.getValue();
       
        if(node.getRightChild() != null){

            binSort(node.getRightChild(), a, pos);

        }

            return knotenanzahl;    
    }
   
    /**
     * Main-Methode zum Testen der Methode binSort
     */
    public static void main(String[] args) {
        // Teste Implementierung
        // 1. Erzeuge einfachen Baum
        BinTreeNode root = new BinTreeNode(4);
       
        root.setLeftChild(new BinTreeNode(2));
        root.getLeftChild().setRightChild(new BinTreeNode(3));
        root.getLeftChild().setLeftChild(new BinTreeNode(1));
       
        root.setRightChild(new BinTreeNode(6));
        root.getRightChild().setRightChild(new BinTreeNode(7));
        root.getRightChild().setLeftChild(new BinTreeNode(5));
       
        // 2. Erstelle Array
        int result[] = new int[7];
       
        // 3. Erstelle Aufgabe36-Objekt und rufe binSort(...) auf
        Aufgabe36 myAufgabe = new Aufgabe36();
        int numNodes = myAufgabe.binSort(root, result, 0);
        System.out.println("Anzahl Knoten: " + numNodes);
        System.out.println(Arrays.toString(result));
    }
   
}

public class BinTreeNode {
   
    public BinTreeNode(int value) {
        // setze den Wert des Knotens
        this.value = value;
    }
   
    /* getter und setter */
   
    public BinTreeNode getLeftChild() { return leftChild; }
    public void setLeftChild(BinTreeNode left) { leftChild = left; }
   
    public BinTreeNode getRightChild() { return rightChild; }
    public void setRightChild(BinTreeNode right) { rightChild = right; }
   
    public int getValue() { return value; }
   
    /* member attribute */
    private int value;
    private BinTreeNode leftChild;
    private BinTreeNode rightChild;
   
}
 
Ich würde an deiner Stelle mit der Anzahl der Knoten anfangen - das soll sicherlich eine Methode von BinTreeNode sein, und nicht während des Ausgeben passieren.
Als Denkanstoß für die Anzahl: Ein Baum hat doch immer so viele Knoten, wie die Anzahl in den beiden Unter-Knoten plus 1
 
Ich würde an deiner Stelle mit der Anzahl der Knoten anfangen
Ich würde an deiner Stelle mal damit anfangen deine Klasse ein bisschen umzubauen. Bisher funktioniert das ja eigentlich nicht wirklich sondern du stellst durch die richtige Reihenfolge der Code-Zeilen in main sicher dass deine Knoten (natürlich auch nur mit den hart kodierten Werten) an den richtigen Stellen im Baum landen. Wie du richtig erkannt hast ist das nicht besonders "flexibel". Du solltest dir eine einfuegen-Methode machen die einen neuen Wert an die richtige Stelle im Baum einsortiert. Besser wäre sowas:
Code:
public class BinBaum1 {

    BinBaum1 links;   // linker Teilbaum
    BinBaum1 rechts;  // rechter Teilbaum
    int inhalt;      // Inhalt soll int sein.

    // Konstruktor
    BinBaum1(int zahl) {
        this.links = null;
        this.rechts = null;
        this.inhalt = zahl;
    }

    void fuegeEin(int wert) {
        if (wert < inhalt) { // in linken Teilbaum einfuegen
            if (links == null) {
                links = new BinBaum1(wert);
            } else {
                links.fuegeEin(wert);
            }
        } else { // in rechten Teilbaum einfuegen
            if (rechts == null) {
                rechts = new BinBaum1(wert);
            } else {
                rechts.fuegeEin(wert);
            }
        }
    }

    void binSort() {
        if (links != null) {
            links.binSort();
        }
        System.out.print(inhalt);
        System.out.println();
        if (rechts != null) {
            rechts.binSort();
        }
    }
}
Und der nächste Schritt wäre jetzt das Zählen der Knoten. (wie eine main zum testen des ganzen aussehen müsste weisst du sicher ?)
 
Und laut dem nächstem Satz ist es nicht mehr notwendig...
Die nächsten Sätze von ihm sind eigentlich nichtssagend denn er hat ja gar keine Baumstruktur, kein Einfügen usw. Wenn er in seiner main-Methode auch nur zwei Zeilen vertauscht oder 2 Werte verdreht würde sein ganzes schönes binSort nicht mehr funktionieren. Hat er ja selber schon gemerkt.
 
Die nächsten Sätze von ihm sind eigentlich nichtssagend denn er hat ja gar keine Baumstruktur, kein Einfügen usw. Wenn er in seiner main-Methode auch nur zwei Zeilen vertauscht oder 2 Werte verdreht würde sein ganzes schönes binSort nicht mehr funktionieren. Hat er ja selber schon gemerkt.
Deshalb: die die genaue und ganze Aufgabenstellung ist nötig.

Eine Baumstruktur hat er natürlich schon - das ist übrigens die Klasse am Ende des Codes - und einfügen kann man da auch - das sind btw die Getter und Setter - ohne könnte er ja kaum den Baum in der main erstellen.
binSort funktioniert sowieso nicht, aus ganz anderen Gründen.
Aber für die Implementierung von binSort, was vom Namen her das sortieren beinhaltet, ist kein sortierter Binärbaum nötig - man kann das auch wunderbar sortiert ausgeben, ohne das der Baum vorher sortiert ist.
 
Er hat einen Baum aber keine Baumstruktur.
Ein Baum hat also nicht die Struktur eines Baumes?
Was hat er denn dann, die Struktur einer Ananas?

Und der TE hat das schon besser erkannt als du:
Und weißt du woran das nicht liegt? An dem fehlenden Einfügen des Baums.
Im Gegensatz zu dir hat der TE wenigstens aber auch erkannt, dass es nicht daran liegt, sondern an der binSort. Deren Problem mit negativen Zahlen oder Lücken wird sicherlich nicht behoben, wenn man Werte passend in den Baum einfügen kann...
 
Nicht "des Baums" sondern "in den Baum".
Nein, das Einfügen des Baumes ist völlig richtig - es ist die Funktion des Baums.

Na dann viel Spass beim ändern der binSort ... hättest du übrigens schon lang mal tun können. Obwohl ... die Lösung von dir kenne ich eigentlich schon.
Schon dass du sie kennst 😉 Aber wenn du mich schon so gut kennst, solltest du ja wissen, dass von mir nicht einfach so fertige Lösungen kommen...

Zeig doch mal deine, die kenn ich noch nicht...
 
Ja klar. Mit voller Absicht. Ich liefere ja keine fertigen Lösungen. den Rest muss der TE machen. Stand auch als Bemerkung am Ende von Post #5 dass der nächste Schritt jetzt das Zählen wäre.
 

Neue Themen


Zurück
Oben