HashSets的ArrayList迭代索引我没有指定?

时间:2015-07-21 15:26:46

标签: java primes factoring

isPrime()检查数字是否为素数,getPrimes(int upper)获取所有素数,包括上限。我希望sievePrimeFactorSets创建每个数字的所有素因子(无重复)的HashSet,并将该HashSet存储在给定值,例如primeFactors.get(20) = [2,5]处的HashSet。

现在它将每个主要添加到每个值,因此primeFactors.get(20) = [2,3,5,7,11,13,etc]。为什么会这样?

public ArrayList<HashSet<Integer>> sievePrimeFactorSets(int upper)
{
    ArrayList<HashSet<Integer>> primeFactors = new ArrayList<HashSet<Integer>>();
    HashSet<Integer> empty = new HashSet<Integer>();
    for (int i = 0; i <= upper; i++)
    {
        primeFactors.add(empty);
    }
    ArrayList<Integer> primes = getPrimes(upper);
    for (Integer p : primes)
    {
        for (int j = p; j <= upper; j+=p)
        {
            primeFactors.get(j).add(p);
        }
    }
    return primeFactors;
}

public ArrayList<Integer> getPrimes (int upper)
{
    ArrayList<Integer> primes = new ArrayList<Integer>();
    primes.add(2);
    for (int i = 3; i <= upper; i++)
    {
        if (isPrime(i))
        {
            primes.add(i);
        }
    }
    return primes;
}

1 个答案:

答案 0 :(得分:3)

这一行:

primeFactors.add(empty);

将相同的空哈希集添加到数组的每个元素。因此,每个元素共享相同的哈希集,并且您认为您对其中的更改实际上是对所有元素进行的。

只需替换为:

primeFactors.add(new HashSet<>());
相关问题