简单的迭代器循环会产生意想不到的结果

时间:2009-05-03 06:48:39

标签: java iterator logic while-loop

我确信这是我逻辑上的一个明显错误,但我似乎无法弄清楚我做错了什么。很简单,我有一个安全代码的数组列表,我想计算每个安全代码组合之间的相关性。我的代码如下:

private void crossCorr(ArrayList<String> codes, HashMap<String, Stock> stockMap){

    // iterate through the codes list and get all the cross correlations:

    Iterator<String> itr1 = codes.iterator();
    while(itr1.hasNext()){
        String code1 = itr1.next();
System.out.println("  -----  " +code1);
        Iterator<String> itr2 = codes.iterator();
        ArrayList<Double> corrs = new ArrayList<Double>();
        HashMap<String, Double> codeCorr = new HashMap<String, Double>();
        String code2="";
        double corr=-2;
        while(itr2.hasNext()){
            code2 = itr2.next();
            Maths calcs = new Maths();      
            corr = calcs.getCorrelation(stockMap.get(code1).getRorHistory(), stockMap.get(code2).getRorHistory());
            corrs.add(corr);    // we order this list and then use this ordered list
                                // to find the codes for the stocks with the highest
                                // correlation for any given stock
            codeCorr.put(code2, corr);
            System.out.println(code1+" - "+code2+" - Correlation is "+corr);
        }
        orderCorrs(6, codeCorr, corrs, code1);
    }

}

我得到的输出是 -

----- GOOG
GOOG - GOOG - 相关系数为1.0000000000000002 GOOG - YHOO - 相关系数为0.6986623807707519
GOOG - MSFT - 相关系数为0.7275411317567286
GOOG - CSCO - 相关系数为0.8122979333663799
GOOG - AAPL - 相关系数为0.8217785260604609
GOOG - ADBE - 相关系数为0.6102679356472099
GOOG - DISH - 相关系数为0.644852624453125
GOOG - NSOL - 相关系数为0.11600387177879072
GOOG - SBUX - 相关系数为0.6694306410719489
GOOG - PSFT - 相关系数为-0.09921822861087544
GOOG - XOM - 相关系数为0.6728272039489009
GOOG - WMT - 相关系数为0.4004364090315347
GOOG - IBM - 相关系数为0.7559988282095168
GOOG - JPM - 相关系数为0.7085525367336528
GOOG - DNA - 相关系数为0.13628949379947575
GOOG - HPQ - 相关系数为0.7819350018750656
GOOG - KO - 相关系数为0.5700932682157461
GOOG - VZ - 相关系数为0.737881573641585
GOOG - INTC - 相关系数为0.7654127298771953
GOOG - SPY - 相关系数为0.8042488406758052
GOOG - PEP - 相关系数为0.6297924741882344
GOOG - WFC - 相关系数为0.5064491351161948
GOOG - ABT - 相关系数为0.238752389446595
GOOG - QCOM - 相关系数为0.726356709262025
GOOG - COP - 相关系数为0.570604981251932
GOOG - MCD - 相关系数为0.5434872575410538

但我永远不会得到

YHOO - GOOG - 相关性......等等。

我确信这是一个简单的错误,出于某种原因我不会接受它。

我知道我正在进行两次相关计算,一旦我按照预期工作,我会解决这个问题。

如果我需要提供更多信息,请告诉我。

1 个答案:

答案 0 :(得分:3)

快速测试应用程序删除了对我们不知道的代码的引用对我来说很好。

是否有可能某些东西会抛出您未处理的未经检查的异常?如果您可以发布一个简短但完整的程序来证明问题,那将极大地帮助您。获取完整项目的副本,然后删除位(例如硬编码数据和返回值),直到您无法取消任何其他内容而不会消除问题。当然,你可能已经找到了自己的问题。

顺便说一下,如果你使用Java 1.5或更高版本,使用扩展for循环比使用迭代器显式更简单。你会这样做:

for (String code1 : codes) {
    // ...
    for (String code2 : codes) {
        // ...
    }
}