在java中启动对象的位置

Question

这个问题与我对另一个问题的回答有关。 原问题是here

谁能解释为什么错误代码会按照 cmets 中解释的方式失败（因为它是伪代码）

 // bad code
        ResultSet rs = getDataFromDBMS("Select * from [tableName];");

            //temp uses a collection member within it to hold a list of column names to data value pairs (hashmap<String,String>)
        Object temp = new objectToHoldInfoFromResultSet();

        // loop over the result set
        while (rs.next)// for each row in the result set
        {
            for (int i = 1; i <= rs.getNumberColums; i++) {
                temp.add(infoAboutColumn);
            }
            temp.printInfo();// always prints correct (ie current) info
                    //the print just takes the hashmap member and asks for a       
                    //toString() on it
            anotherObject(makeUseOf(temp));// always uses info from first
                   //iteration. Essentially grabs the hashMap member of temp, and puts
                   //this into another collection of type 
                   //HashMap< HashMap<String,String>, temp> see the linked question
                   //for more detail.

        }

        // Seemingly each loop into the while the temp.doSomethingToData(); uses
        // the temp object created in the first iteration

        // good code
        ResultSet rs = getDataFromDBMS("Select * from [tableName];");

        // loop over the result set
        while (rs.next)// for each row in the result set
        {
            Object temp = new objectToHoldInfoFromResultSet();// moving
                                                                // declaration
                                                                // of temp into
                                                                // the while
                                                                // loop solves
                                                                // the problem.
            for (int i = 1; i <= rs.getNumberColums; i++) {
                temp.add(infoAboutColumn);
            }
            temp.printInfo();// always prints correct (ie current) info

            anotherObject(makeUseOf(temp));// also uses the correct (ie current)
                                            // info.

        }

Answer 1

如果不知道 temp.printInfo() 和 makeUseOf() 在做什么，我们无法可靠地回答这个问题。不过，按照您描述的方式实现它们很容易。

当您在循环外实例化temp 时，您将在循环的所有迭代中使用相同的对象。因此，它可以从每次迭代中收集数据（例如，收集到一个集合中）。然后，方法可以获取当前迭代中累积的数据，以及任何先前迭代中累积的数据，如果不是故意的，这可能会导致问题。

所以让我们假设temp 包含一个集合，并且在每次迭代中，结果集中的一列被添加到其中。现在，如果实现temp.printInfo() 来打印关于此集合中last 元素的信息，而实现makeUseOf() 来处理集合中的first 元素，你得到你观察到的行为。

OTOH 当您在循环内实例化 temp 时，您将在每次迭代中获得一个新的、不同的对象，它不会“记住”来自早期迭代的任何数据。因此，即使使用上述temp.printInfo() 和makeUseOf() 的实现，您也会得到正确的结果。

Answer 2

我不知道为什么一个好一个坏。我的猜测（不知道 objectToHoldInfoFromResultSet 和其他方法的行为是什么）如下：

在第一种情况下，“objectToHoldInfoFromResultSet”（应该大写）每次都创建一次

temp.add(infoAboutColumn);

被调用，新的记录数据被添加到对象中。我猜应该为每条记录清除此信息……否则你会得到很多重复。通过重新初始化持有者对象来处理复制。 IE。 (1 2 3 4 5 6) 而不是 (1 1 2 1 2 3 1 2 3 4 1 2 3 4 5 1 2 3 4 5 6)。

不知道各种礼物，我也无话可说。