关于你的代码:
public static int factor(int m) {
for(int y=2 ; y <= m/2 ; y++) {
if(m%y==0) {
return y;
continue;
}
}
return y;
}
在最终返回y时,y不存在.它的范围仅限于for语句的内部,因为这是您创建它的地方.这就是你得到未定义变量的原因.
在任何情况下,当你找不到一个因子时返回y是完全错误的事情,因为如果你传入(例如)47,它会给你回24(47/2 1),尽管它不是一个因素.
在你返回后尝试继续循环也没什么意义:-)而且,为了提高效率,你只需要达到m的平方根而不是它的一半.
因此,我将以此为出发点:
public static int factor (int num) {
for (int tst = 2 ; tst * tst <= num ; tst++)
if (num % tst == 0)
return tst;
return num;
}
这具有使用素数的优点,因为素数的第一个因素是素数本身.而且,如果你愚蠢地传递一个负数(或少于两个,你也会得到你传入的数字.如果你想要不同的行为,你可能想要为代码添加一些额外的检查.
而且你可以更快地做到这一点,例如:
public static int factor (int num) {
if (num % 2 == 0) return 2;
for (int tst = 3 ; tst * tst <= num ; tst += 2)
if (num % tst == 0)
return tst;
return num;
}
这会对前面的2进行检查,然后只使用奇数进行余数检查.因为你已经检查过2,你知道它不能是任何偶数的倍数,所以你可以通过只检查奇数来大致加倍速度.
如果你想让它更快(可能,虽然你应该检查它并记住代码可能更难理解),你可以在评论中使用Will指出的聪明方案.
如果您考虑上面的循环使用的奇数和一些注释,您可以看到您定期得到三个的倍数:
5
7
9 = 3 x 3
11
13
15 = 3 x 5
17
19
21 = 3 x 7
23
25
27 = 3 x 9
当您意识到每个带注释的数字比前一个带注释的数字多六(3 x 2)时,这在数学上是显而易见的.
因此,如果从5开始并交替添加2和4,您将跳过三个的倍数以及两个的倍数:
5, +2=7, +4=11, +2=13, +4=17, +2=19, +4=23, ...
这可以使用以下代码完成:
public static long factor (long num) {
if (num % 2 == 0) return 2;
if (num % 3 == 0) return 3;
for (int tst = 5, add = 2 ; tst * tst <= num ; tst += add, add = 6 - add)
if (num % tst == 0)
return tst;
return num;
}
你必须在3前面添加测试,因为它违反了2,4,2规则(序列3,5,7有两个连续的两个间隙),但这可能是为了大约再减少25%而付出的小代价来自原始搜索空间(超过50%已经通过跳过所有偶数来实现).
设置添加到2然后使用add = 6更新 – 添加是一种让它在2和4之间交替的方法:
6 - 2 -> 4
6 - 4 -> 2
正如我所说的,这可能会提高速度,特别是在模数比简单减法更昂贵的环境中,但你可能希望实际对其进行基准测试.我只是将它作为另一种可能的优化提供.