• Post Reply Bookmark Topic Watch Topic
  • New Topic
programming forums Java Mobile Certification Databases Caching Books Engineering Micro Controllers OS Languages Paradigms IDEs Build Tools Frameworks Application Servers Open Source This Site Careers Other Pie Elite all forums
this forum made possible by our volunteer staff, including ...
Marshals:
  • Campbell Ritchie
  • Jeanne Boyarsky
  • Ron McLeod
  • Paul Clapham
  • Liutauras Vilda
Sheriffs:
  • paul wheaton
  • Rob Spoor
  • Devaka Cooray
Saloon Keepers:
  • Stephan van Hulst
  • Tim Holloway
  • Carey Brown
  • Frits Walraven
  • Tim Moores
Bartenders:
  • Mikalai Zaikin

euler 23 revisited

 
Ranch Hand
Posts: 4716
9
Scala Java
  • Mark post as helpful
  • send pies
    Number of slices to send:
    Optional 'thank-you' note:
  • Quote
  • Report post to moderator
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.
 
Sheriff
Posts: 17644
300
Mac Android IntelliJ IDE Eclipse IDE Spring Debian Java Ubuntu Linux
  • Mark post as helpful
  • send pies
    Number of slices to send:
    Optional 'thank-you' note:
  • Quote
  • Report post to moderator
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.
 
Randall Twede
Ranch Hand
Posts: 4716
9
Scala Java
  • Mark post as helpful
  • send pies
    Number of slices to send:
    Optional 'thank-you' note:
  • Quote
  • Report post to moderator
i think you found the problem(line 61). so obvious it's no wonder i missed it
 
Randall Twede
Ranch Hand
Posts: 4716
9
Scala Java
  • Mark post as helpful
  • send pies
    Number of slices to send:
    Optional 'thank-you' note:
  • Quote
  • Report post to moderator
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
 
Junilu Lacar
Sheriff
Posts: 17644
300
Mac Android IntelliJ IDE Eclipse IDE Spring Debian Java Ubuntu Linux
  • Mark post as helpful
  • send pies
    Number of slices to send:
    Optional 'thank-you' note:
  • Quote
  • Report post to moderator
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.

 
Randall Twede
Ranch Hand
Posts: 4716
9
Scala Java
  • Mark post as helpful
  • send pies
    Number of slices to send:
    Optional 'thank-you' note:
  • Quote
  • Report post to moderator
that is my suspicion as well. thanks for the help.
 
lowercase baba
Posts: 13089
67
Chrome Java Linux
  • Mark post as helpful
  • send pies
    Number of slices to send:
    Optional 'thank-you' note:
  • Quote
  • Report post to moderator
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.
 
Randall Twede
Ranch Hand
Posts: 4716
9
Scala Java
  • Mark post as helpful
  • send pies
    Number of slices to send:
    Optional 'thank-you' note:
  • Quote
  • Report post to moderator
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
 
Farmers know to never drive a tractor near a honey locust tree. But a tiny ad is okay:
a bit of art, as a gift, the permaculture playing cards
https://gardener-gift.com
reply
    Bookmark Topic Watch Topic
  • New Topic