Guidance

This is hw, so im only looking to be guided in the right direction i have to find and fix the bug. it is supposed to display a cross.

Code:

---

// Question 2-13-a

import java.awt.*;
import javax.swing.*;

/**
 *  This program displays a red cross on a white
 *  background.
 */

public class RedCross extends JPanel
{
  public void paintComponent(Graphics g)
  {
    super.paintComponent(g);  // Call JPanel's paintComponent method
                              //  to paint the background
    int xCenter = getWidth() / 2;
    int yCenter = getHeight() / 2;
    g.setColor(Color.RED);
    g.fillRect(xCenter, yCenter, 10, 50);
    g.fillRect(xCenter, yCenter, 100, 10);
   
  }

  public static void main(String[] args)
  {
    JFrame window = new JFrame("Red Cross");
    window.setBounds(300, 300, 200, 200);
    window.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    RedCross panel = new RedCross();
    panel.setBackground(Color.WHITE);
    Container c = window.getContentPane();
    c.add(panel);
    window.setVisible(true);
  }
}

---

Code:

---

g.fillRect(xCenter, yCenter, 10, 50);
    g.fillRect(xCenter, yCenter, 100, 10);

---

This won't display a cross because you start them at the same place (xCenter, yCenter). You should make vars with the width/height/etc of the cross so that you can calculate what x and y to use, and what w and h, instead of just random numbers (10, 50; 100, 10)

Got my point?

I know what you mean about starting at the same point. But I do not even know where to start about fixing it

Thanks

g.fillRect(xCenter, yCenter-45, 10, 100);
g.fillRect(xCenter-45, yCenter, 100, 10);

For example, yes.

Code:

---

final int length = 100; // The length of both rects
final int width = 10;    // The width of the rects

g.fillRect(xCenter-length/2, yCenter-width/2, length, width);
g.fillRect(xCenter-width/2, yCenter-length/2, width, length);

---

It could be something like that. You understand that, I suppose?

thanks!!!!!