Thread: Filling in alternating colors of a "BullsEye"

I'm back again with another question

My code works...sort of. After I tried to fill in my bullseye with alternating blue and pink colors
I get:
-bullseye is pink
-next circle is pink
-next circle is pink
-next circle is pink
-last circle is blue

I want:
-bullseye to be blue
-next circle to be pink
-next circle to be blue
-next circle to be pink
-next circle to be blue

I've tried deleting some "fillOval" but I can't seem to get what I need.

package mis.hw2;
 
import java.awt.Color;
import java.awt.Graphics;
import javax.swing.JFrame;
 
@SuppressWarnings("serial")
public class P2_BullsEye extends JFrame
{
    //Declare Variables
    int centerX, centerY, outermostRadius, ringWidth;
 
    // Constructor
public P2_BullsEye ()
    {
        super ("BullsEye");
        //Assign Variables
    }
 
public void paint (Graphics g)
{
        //Call the paint method of the JFrame
        super.paint (g);
 
    centerX = 150;
    centerY = 150;
    outermostRadius = 100;
    ringWidth = 22;
 
    //Make color blue
    g.setColor(Color.blue);
    //Draw blue Oval
    g.drawOval(125, 140, 20, 20);	
    g.drawOval(90, 110, 90, 80);	
    g.drawOval(50, 80, 170, 140);	
 
    //Fill blue Oval you just drew
    g.fillOval(125,140, 20, 20);
    g.fillOval(90, 110, 90, 80);
    g.fillOval(50, 80, 170, 140);
 
    //Make color pink
    g.setColor(Color.pink);
 
    //Draw pink Oval
 
    g.drawOval(70, 95, 130, 110);
 
    //Fill pink Oval you just drew
    g.fillOval(110, 125, 50, 50);
    g.fillOval(70, 95, 130, 110);
 
}    
    public static void main(String args []) {
    // Create app object
    P04_BullsEye app = new P04_BullsEye();
    app.setSize(300, 300);
    app.setVisible(true);
    app.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
 
 
    }
 
}

Am I missing a piece of code that makes my circle lines thicker or something?

As always, thanks for any help!

The code you posted is confused - the bullseye class is called P2_Bullseye, but the main method is instantiating class P04_Bullseye. So what you've posted won't compile, and P2_Bullseye probably has nothing to do with your problem - and in any case has a bunch of redundant drawOval and fillOval calls. It only draws a pink oval inside a blue oval because that's all the code there is...

Originally Posted by dlorde

The code you posted is confused - the bullseye class is called P2_Bullseye, but the main method is instantiating class P04_Bullseye. So what you've posted won't compile, and P2_Bullseye probably has nothing to do with your problem - and in any case has a bunch of redundant drawOval and fillOval calls. It only draws a pink oval inside a blue oval because that's all the code there is...

Good eye! But your right, the P2 and P04 mix up is an error of me copying and pasting to here. My mistake, they should all be P2. I get no errors when compiling. I just don't get the correct output

Could you explain the part of your comment that I bolded?

I need to drawOval 5 times to draw 5 circles....dont I? Before I tried to fill the circles in with color, everything worked. I was able to draw 3 blue circles and 2 pink ones that mirrored a "bullsEye" ...just without any fill color.

You also said that "It only draws a pink oval inside a blue oval because that's all the code there is." - I don't get this either... if I need to draw 5 circles, then wouldn't I need to incorporate 5 drawOvals and 5 fillOvals to draw and color in the 5 circles?

Figured it out. I had the order in which I was drawing the circles reversed.