Question

I wrote an implementation of the Mandelbrot set in Java using a JComponent but I’m getting strange results when I render it. Besides that everything compiles right. I’m just not for sure what I’m doing wrong with it. Any code review also would be appreciated.

My source is posted on pastebin since it would take up too much room here:

JMandelbrot.java
Mandelbrat.java

Answer 1

Problem:

• The image is as I expect from the code. The Mandelbrot set has a diameter of 2, so you only see some pixels in the middle.

Solution:

• Change the renderPoint method to accept double arguments and call it as renderPoint((x - h)/100.0, (k - y)/100.0) to see something more interesting.
• Change the iteration count and color coding, because now you are computing 255^3 iterations for each inner pixel. I managed to see something nice by changing the return of renderPoint to return (((r << 16) | (g << 8) | b)<<4) and setting MaxColorBit = 16.

A code review:

• (int)Math.floor(Width / 2) can be replaced by Width / 2, because that is an integer division.
• You should start your attributes Width and Height with small letters (width and height), because that is a Java convention which helps differentiate classes and attributes.
• The iterations attribute is not used.