问题描述
我试图遍历Integer哈希集以找到前两个最大元素。我的第二大元素位于哈希集的末尾,我刚刚发现foreach循环正在跳过最后一个元素。为什么会这样?
for(int n:set)
{
if(n>max1)
max1=n;
else if(n==max1)
max2=n;
else if(n>max2)
max2=n;
}
解决方法
问题不是循环,而是您在每次迭代中都未正确更新max1
和max2
的事实:
for(int n : set) {
if (n > max1) {
max2 = max1;
max1 = n;
} else if (n > max2) {
max2 = n;
}
}
请注意,由于它是一个集合,并且值是唯一的,因此n
等于max1
或max2
时就不会出现这种情况。