# euler 23 revisited

posted 3 years ago

A perfect number is a number for which the sum of its proper divisors is exactly equal to the number. For example, the sum of the proper divisors of 28 would be 1 + 2 + 4 + 7 + 14 = 28, which means that 28 is a perfect number.

A number n is called deficient if the sum of its proper divisors is less than n and it is called abundant if this sum exceeds n.

As 12 is the smallest abundant number, 1 + 2 + 3 + 4 + 6 = 16, the smallest number that can be written as the sum of two abundant numbers is 24. By mathematical analysis, it can be shown that all integers greater than 28123 can be written as the sum of two abundant numbers. However, this upper limit cannot be reduced any further by analysis even though it is known that the greatest number that cannot be expressed as the sum of two abundant numbers is less than this limit.

Find the sum of all the positive integers which cannot be written as the sum of two abundant numbers.

i have been trying to debug this code ever since i wrote it. all i managed to do is speed it up

that is a good thing but getting wrong answer is not. i added a HashSet to eliminate duplicates and it gave a smaller answer, but still much greater than correct answer. i changed the limit to 30000 and i am getting 29999 as a number that cannot be written as the sum of two abundant numbers. so i am definately doing something wrong but i can't see it. my ArrayList of abundant numbers looks ok, as does the listing of all the positive integers which cannot be written as the sum of two abundant numbers(24 is missing as is 30 etc.

A number n is called deficient if the sum of its proper divisors is less than n and it is called abundant if this sum exceeds n.

As 12 is the smallest abundant number, 1 + 2 + 3 + 4 + 6 = 16, the smallest number that can be written as the sum of two abundant numbers is 24. By mathematical analysis, it can be shown that all integers greater than 28123 can be written as the sum of two abundant numbers. However, this upper limit cannot be reduced any further by analysis even though it is known that the greatest number that cannot be expressed as the sum of two abundant numbers is less than this limit.

Find the sum of all the positive integers which cannot be written as the sum of two abundant numbers.

i have been trying to debug this code ever since i wrote it. all i managed to do is speed it up

that is a good thing but getting wrong answer is not. i added a HashSet to eliminate duplicates and it gave a smaller answer, but still much greater than correct answer. i changed the limit to 30000 and i am getting 29999 as a number that cannot be written as the sum of two abundant numbers. so i am definately doing something wrong but i can't see it. my ArrayList of abundant numbers looks ok, as does the listing of all the positive integers which cannot be written as the sum of two abundant numbers(24 is missing as is 30 etc.

SCJP

Visit my download page

posted 3 years ago

Have you considered using an approach like the Sieve of Erastosthenes algorithm instead? Line 61 doesn't look right. You're adding up the index, not the values themselves.

Junilu - [How to Ask Questions] [How to Answer Questions]

posted 3 years ago

i think you found the problem(line 61). so obvious it's no wonder i missed it

SCJP

Visit my download page

posted 3 years ago

well, that line was wrong. i replaced it with this

but it still has the same problem. in fact the answer i get is even bigger now

but it still has the same problem. in fact the answer i get is even bigger now

SCJP

Visit my download page

posted 3 years ago

My first inclination is to refactor for clarity -- make solve() a composed method. That will allow me to attack the problem one step at a time.

However, if you don't want to do that, I would look carefully at Line 31 and the breaks you are doing on Lines 37, 43, and 48. There are good reasons to discourage the use of breaks and I suspect the way you are using them and the bug that you are seeing is one of them.

However, if you don't want to do that, I would look carefully at Line 31 and the breaks you are doing on Lines 37, 43, and 48. There are good reasons to discourage the use of breaks and I suspect the way you are using them and the bug that you are seeing is one of them.

Junilu - [How to Ask Questions] [How to Answer Questions]

posted 3 years ago
There are only two hard things in computer science: cache invalidation, naming things, and off-by-one errors

Can you describe your algorithm? How are you actually trying to solve the problem? I find your code hard to follow, so i'm not sure what you are doing. And since it isn't a complete program, I can't run and debug it.

A couple of suggestions...take out the performance enhancements and try running it. a working slow program is always better than a fast program that doesn't. If it then works, put them back one at a time and see which breaks it - the look carefully at that bit to see why.

System.out.println() is always your best friend. put some in to see what your code is really doing. are you adding up the abundant numbers, or their index? are you adding in a number you think you should skip? Are you generating the correct list of abundant numbers?

FWIW, my entire class for Euler23 is 72 lines. That includes an eight-line method that I wrote for debugging - it's not called in my program to come up with the solution.

A couple of suggestions...take out the performance enhancements and try running it. a working slow program is always better than a fast program that doesn't. If it then works, put them back one at a time and see which breaks it - the look carefully at that bit to see why.

System.out.println() is always your best friend. put some in to see what your code is really doing. are you adding up the abundant numbers, or their index? are you adding in a number you think you should skip? Are you generating the correct list of abundant numbers?

FWIW, my entire class for Euler23 is 72 lines. That includes an eight-line method that I wrote for debugging - it's not called in my program to come up with the solution.

posted 3 years ago

the problem was line 43

break label;

should be

break;

i should only break out of the inner loop at that point.

it is slower now(about 2 minutes on my old laptop) but the answer is correct

break label;

should be

break;

i should only break out of the inner loop at that point.

it is slower now(about 2 minutes on my old laptop) but the answer is correct

SCJP

Visit my download page