Question

我写了一些代码，它在for循环中逐行读取文本文件;每一行都是用于在JPanel上绘制新形状的命令。我有一个用于存储名称和形状的HashMap。

Map<String, GeoShape> geoObj = new HashMap<String, GeoShape>();

示例输入是这样的：添加P1点蓝色50 50 10

cmd是一个String []，它包含splitted输入命令。问题出在这里：

if (cmd[2].equalsIgnoreCase("Point")) 
    geoObj.put(cmd[1], new Point(cmd, graph));

它陷入无休止的循环中。我在调试模式下运行代码，类Point没什么问题。

非常感谢任何帮助！

编辑：这是Class Point

public class Point extends Segment {
    private Line2D point;
    private Stroke thickness;
    private Color fColor;
    private double x;
    private double y;

    public Point(String[] cmd, Graphics2D graph) {
        System.out.println("point");
        x = Double.parseDouble(cmd[4]);
        y = Double.parseDouble(cmd[5]);
        try {
            fColor = setColor(cmd[3]);
        } catch (Exception e) {
            e.printStackTrace();
        }
        thickness = new BasicStroke(Integer.parseInt(cmd[6]));

        paint(graph);
    }

    @Override
    public void paint(Graphics2D graph) {
        point = new Line2D.Double(x, y, x, y);

        graph.setStroke(thickness);
        graph.setColor(fColor);
        graph.draw(point);
    }
}

for循环：

FileReader fr = new FileReader(file);
BufferedReader br = new BufferedReader(fr);

int linesNum = Integer.parseInt(br.readLine());
String line = br.readLine();
for (int i = 1; i <= linesNum; i++) {
     while (line != null) {
        cmd = line.split(" ");
        if (cmd[0].equalsIgnoreCase("ADD")) {
        if (cmd[2].equalsIgnoreCase("Point")) 
        geoObj.put(cmd[1], new Point(cmd, graph));
            // some else if with the same structure for other shapes
//at the end of for loop I have this:  line = br.readLine();

Answer 1

您是说单次调用put会导致无限循环吗？

理论上这是可能的，但前提是HashMap在没有正确同步的情况下由另一个线程更新。（您需要对HashMap类进行详细分析，以确定这是否实际可行/可能，但如果两个线程同时读取和写入HashMap而没有正确同步，那么哈希链可能会被破坏，当get调用尝试搜索损坏的链时会导致无限循环。）

如果那不是问题，那么问题在于你没有向我们展示过的代码。

在将新对象放入HashMap时陷入无限循环

1 个答案: