I'm trying to replicate the ray-looking design, just like in the first image.

Here's my code:

When the user clicks on the button:

private class drawInnerShape implements ActionListener{
        public void actionPerformed( ActionEvent e){
            paintInnerDesign = true;

The following code executes:

class DrawArtwork extends JPanel {
        public void paintComponent(Graphics g) 
            Graphics2D g2d = (Graphics2D) g;
                Color nextColor = new Color(255, 216, 0);
                int arcEnd = 2;
                for(int i = 0; i <= 89; i++)
                    g2d.fillArc((int) ((canvasWidth / 4) + 35), (int) (canvasHeight / 12) + 100, (int)(canvasHeight * .75), (int)(canvasHeight * .75), arcStart, arcEnd);

                    g2d.fillArc((int) ((canvasWidth / 4) + 35), (int) (canvasHeight / 12) + 100, (int)(canvasHeight * .75), (int)(canvasHeight * .75), arcEnd, arcEnd + 2);
                    arcStart = arcStart + 4;
                    arcEnd = arcEnd + 4;

However, I'm getting the results in image two. And if I change i value from 89 to 79, I get the results in image three.

I can't figure out why it won't paint the object just like in the first image.

Attachments Sacred_geometry_illusion.jpg 93.52 KB Error.png 23.65 KB Error2.png 95.42 KB
3 Months
Discussion Span
Last Post by JamesCherrill

The last parameter for fillArc is the size of the arc in degrees, not the angle at which it finishes. So it should always be 2 or 4 (sorry, don't have the time to work out which)


rproffitt, forgot to include arcStart, but it's initialized at 0. Here's the link to the code paly ground: https://goo.gl/m0a3tZ

JamesCherrill, I know that the size is in degrees. I'm iterating through, and for degrees 0 to 2, the arc is filled with gold, from 2 to 4, its black, and so on until 356 degrees.


I'm looking at line 14 where the last argument is arcEnd (also line 17) which should be 2 but is actually increasing by 4 on each pass of the loop.
It looks like you think the last two parameters are the starting and ending angle for the arc, but they are not. To quote the api doc "The resulting arc begins at startAngle and extends for arcAngle degrees".
Your last parameter should be 2, and you do not need endArc at all.

Votes + Comments
That makes sense. I must've overlooked the fact that the second parameter is the degrees to which it needs to draw for. Thank you so much. Now it work


A couple of small suggestions:

you could start by drawing a single complete black circle, then you only need to do the gold arcs in the loop. Depending on the application context, drawing 90 separate black arcs could be a performance hit.

If you compute int left,top,width,height once at the start then the code will be much easier to read, and possibly more efficient than having all those calculations repeated inside the loop.

Edited by JamesCherrill

This question has already been answered. Start a new discussion instead.
Have something to contribute to this discussion? Please be thoughtful, detailed and courteous, and be sure to adhere to our posting rules.