JPanel Multipe Images are cutoff (layout problem I think)

Question

I am adding images (custom JPanel: Entity class) to a JPanel but the images are getting cut off. I was originally trying BorderLayout but that only worked for one image and adding others added image cut-off the first or completely removed the original image all together. So I switched to other layouts and the closest I could get was BoxLayout however that adds a very large cut-off which is not acceptable either.
So basically; How can I add images (from a custom JComponent) to a custom JPanel without bad effects such as the one present in the code.
How do I go about adding images from Entity class to a JPanel without them getting cutoff or not showing at all?
Note: the images need to animate.
Entity Class:
import java.awt.Dimension;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.image.BufferedImage;
import javax.imageio.ImageIO;
import javax.swing.JComponent;
public class Entity extends JComponent implements Runnable {
private BufferedImage bImg;
private int x = 20;
private int y = 20;
private int entityWidth, entityHeight;
public void run() {
bImg = loadBImage("test.png", bImg);
entityWidth = bImg.getWidth();
entityHeight = bImg.getHeight();
setPreferredSize(new Dimension(entityWidth, entityHeight));
@Override
public void paintComponent(Graphics g) {
super.paintComponent(g);
Graphics2D g2d = (Graphics2D) g.create();
g2d.drawImage(bImg, x, y, null);
g2d.dispose();
public BufferedImage loadBImage(String filename, BufferedImage tmpBI) {
try {
tmpBI = ImageIO.read(getClass().getResource(filename));
} catch (Exception e) { }
return tmpBI;
public int getEntityWidth() { return entityWidth; }
public int getEntityHeight() { return entityHeight; }
public int getX() { return x; }
public int getY() { return y; }
public void setX(int x) { this.x = x; }
public void setY(int y) { this.y = y; }
}Main:
import java.awt.EventQueue;
import javax.swing.JFrame;
public class MainWindow
public static void main(String[] args) {
new MainWindow();
private JFrame frame;
private GraphicsPanel gp = new GraphicsPanel();
MainWindow() {
EventQueue.invokeLater(new Runnable() {
public void run() {
frame = new JFrame("Graphics Practice");
frame.setSize(680, 420);
frame.setVisible(true);
frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
frame.add(gp);
}Graphics Panel: This is where the problem arrises.
import java.awt.BorderLayout;
import java.awt.Color;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import javax.swing.JPanel;
import javax.swing.Timer;
public class GraphicsPanel extends JPanel implements MouseListener {
private Entity entity;
private Entity ent2;
private Timer timer;
private long startTime = 0;
private int numFrames = 0;
private float fps = 0.0f;
GraphicsPanel() {
entity = new Entity();
entity.setOpaque(false);
Thread t1 =  new Thread(entity);
t1.start();
ent2 = new Entity();
entity.setOpaque(false);
Thread t2 = new Thread(ent2);
ent2.setX(100);
ent2.setY(100);
t2.start();
//ESSENTIAL
setLayout(new BorderLayout());
//COMMENT OUT add(ent2) to see the problem. Any image will do.
add(entity);
add(ent2);
//GAMELOOP
timer = new Timer(30, new Gameloop(this));
timer.start();
addMouseListener(this);
@Override
public void paintComponent(Graphics g)
super.paintComponent(g);
Graphics2D g2 = (Graphics2D) g.create();
g2.setClip(0, 0, getWidth(), getHeight());
g2.setColor(Color.BLACK);
g2.drawString("FPS: " + fps, 1, 15);
public void getFPS()
++numFrames;
if (startTime == 0) {
startTime = System.currentTimeMillis();
} else {
long currentTime = System.currentTimeMillis();
long delta = (currentTime - startTime);
if (delta > 1000) {
fps = (numFrames * 1000) / delta;
numFrames = 0;
startTime = currentTime;             
public void mouseClicked(MouseEvent e) {}
public void mousePressed(MouseEvent e) {}
public void mouseReleased(MouseEvent e) {}
public void mouseEntered(MouseEvent e) { }
public void mouseExited(MouseEvent e) { }
class Gameloop implements ActionListener
private GraphicsPanel gp;
Gameloop(GraphicsPanel gp) {
this.gp = gp;
public void actionPerformed(ActionEvent e) {
try {
gp.getFPS();
gp.repaint();
} catch (Exception ez) { }

Answer 1

You got a bit of a concurrency time bomb on your hands with those Entity threads of yours. Your Entity class also inadvertently overrides Component#getX() and Component#getY() to return where the image is painted within the component. That can't be good. I'm also not sure how your Entity class is functionally different from a simple JLabel with an ImageIcon and a (20,20,20,20) empty border.
But as for your immediate problem: 1) Use FlowLayout for your Graphics panel, 2) Include a revalidate() call after you set the preferred size in the run() method of your Entity class 3) Don't override Component#getX() and Component#getY() in your Entity class and 4) Add the GraphicsPanel to the frame before, not after, you make the frame visible.