0

我正在尝试将树的所有节点递归地绘制到 JPanel 上。

它应该如下所示：

在此处输入图像描述

其中, B, C, D, 是 的子级A。

并且, E,F是B

父子关系的输出是准确的：

PARENT: A | x,y: 180, 100...
 CHILD: B | x,y: 205, 150
PARENT: B | x,y: 205, 150...
 CHILD: E | x,y: 230, 200
PARENT: B | x,y: 205, 150...
 CHILD: F | x,y: 230, 200
end
PARENT: A | x,y: 180, 100...
 CHILD: C | x,y: 205, 150
PARENT: A | x,y: 180, 100...
 CHILD: D | x,y: 205, 150

但是x, y坐标不是......每个孩子的位置都x与另一个孩子的位置重叠......所以它只显示每个父节点 1 个子节点：

在此处输入图像描述

我认为我必须做的是找到一种方法，为y每个新的子行增加一次......并x为父母的每个孩子增加一次，这样它们就可以很好地放在一行上......但是节点的坐标正在重置。

有什么想法吗？

代码：

public void paintComponent(Graphics g) {
    g.setColor(Color.BLACK);
    g.fillRect(0, 0, width, height);
    g.setColor(Color.CYAN);
    g.fillRect(rootNode.getX(), rootNode.getY(), rootNode.getWidth(), rootNode.getHeight());
    paintComponent(g, rootNode);

}

public void paintComponent(Graphics g, Nodes parentNode) {  
    //keep generating new nodePrintList to load with new Children
    ArrayList<Nodes> nodePrintList = new ArrayList<Nodes>();    

    //base case: end of nodeList
    if (nodeList.indexOf(parentNode)==nodeList.size()-1) {
        System.out.println("\nend");
    }
    else {  
    //traverse nodeList recursively 
        nodePrintList = getChildren(parentNode);    
        //loop through and print all children of node n
        System.out.println();

        for (Nodes child : nodePrintList) {             
            g.setColor(Color.GREEN);
            child.setX(parentNode.getX()+25);
            child.setY(parentNode.getY()+50);
            g.fillRect(child.getX(), child.getY(), child.getWidth(), child.getHeight());
            System.out.print("PARENT: " + parentNode.getValue() + " | x,y: " + parentNode.getX() + ", " + parentNode.getY() + "...\n CHILD: " + child.getValue() + " | x,y: " + child.getX() + ", " + child.getY());    
            paintComponent(g, child);
        }           
    }
}

getChildren()返回每个父母的孩子列表的方法：

//need to pass a new index to getChildren once current node has no more children
public ArrayList<Nodes> getChildren (Nodes n) {
    ArrayList<Nodes> childrenList;
    childrenList = new ArrayList<Nodes>();  
    int index = nodeList.indexOf(n);
    int col = 0;

    while (col < size) {
        if (adjMatrix[index][col] == 1) {
            childrenList.add(nodeList.get(col));
        }
        col++;
    }
    return childrenList;
}
4

2 回答 2

2

问题就在这里：

for (Nodes child : nodePrintList) {             
    //...
    child.setX(parentNode.getX()+25);
    //...
}

您的程序代码始终将子节点的位置设置为与父节点的固定 x 和 y 距离，无论它是哪个节点。

对此的一种解决方案：

for (int idx = 0; idx < nodePrintList.size(); idx++) {
    Nodes node = nodePrintList.get(idx);             
    //...
    child.setX(parentNode.getX()+ 25 * ((float)(nodePrintList.size()-1)/2 - idx);
    //...
}

虽然我的数学可能有点偏离，但这应该将所有子节点直接放在父节点下方，每个子节点与下一个节点相距 25 个单位。

于 2013-06-19T18:20:41.753 回答
2

主要问题似乎for在您的paintComponent方法中。在这里，您为每个孩子使用相同的 x 坐标。你可以像这样修复它：

int x = parentNode.getX()+25; // x position of first child
for (Nodes child : nodePrintList) {
    child.setX(x);
    child.setY(parentNode.getY()+50); // y position is same for each child
    // [...] fill rect, print, recursive call, etc.
    x -= 50; // in each iteration, move nodes further to the left
}

然而，这只是一个问题。另一个问题是，为了正确布置节点，您必须考虑每个分支的总宽度。解决此问题的一种方法可能是返回每个分支的总宽度，具有paintComponent(g, nodes)return x，并将其（加上一些偏移量）用于下一个孩子。上面的代码可能看起来像这样：

int x = parentNode.getX()+25; // x position of first child
for (Nodes child : nodePrintList) {
    child.setX(x);
    child.setY(parentNode.getY()+50); // y position is same for each child
    // [...] fill rect, print, etc.
    x = paintComponent(g, child) - 50; // next x is left of the total width of the child
}
return x; // return x (change methods signature accordingly)

这可能仍然需要一些修补，但它应该给你的想法。

于 2013-06-19T18:17:32.270 回答