Create rays with fillArc

[Start4me]

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;
            repaint();
        }
    }

The following code executes:

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

                    g2d.setColor(Color.BLACK);
                    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.

[rproffitt]

Code incomplete? I may be missing it but where is arcStart defined, initialized?

Also, can you put this up in an online code play ground? Such as https://www.tutorialspoint.com/compile_java8_online.php

[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)

[Start4me]

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.

[JamesCherrill]

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.

[JamesCherrill]

OK!

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.