Question

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;
}

Answer 1

这一行：

primeFactors.add(empty);

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

只需替换为：

primeFactors.add(new HashSet<>());

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

1 个答案: