Bouton marche pas clique java

Pour casser les imbrications, j'ai fais:

chp.addActionListener(new ActionListener() {
public void actionPerformed(ActionEvent e)
{
parrain pa=new parrain();
Graphics g;
pa.paintComponent(g, "C:/Users/Keurdange/Pictures/ucad.jpg");
}
});

public class parrain extends JPanel
{
public void paintComponent(Graphics g, String s)
{
File fichier= new File(s);
try {
final Image moi1 = ImageIO.read(fichier);
g.drawImage(moi1, 0, 0, this.getWidth(), this.getHeight(), this);
} catch (IOException e1) {
// TODO Auto-generated catch block
e1.printStackTrace();
}
}
}

mais le probleme, c'est que je dois initialiser la valeur de g dans mon ActionListener et je ne peux l'initialiser à null sinon je risque de générer une exception du genre NullPointerException. J'ai lu sur le net qu'on pouvait le faire avec this.getGraphics mais hélas non, ca marche pas avec moi.

Tu ne peux pas rajouter un paramètre String à paintComponent, sinon ce n'est plus une redéfinition de la méthode de JPanel mais une nouvelle méthode qui ne pourra pas être appelé par Swing.

J'ai pas testé, il y a peut-être des erreurs :

class parrain extends JPanel 
{
	private final Image img;

// C'est ici que tu dois passer les paramètres, mais autant mettre directement l'image déjà chargée en mémoire plutôt que relire le fichier, ça ira plus vite
	public parrain(Image image)
	{
		img = image;
	}

// En mettant Override, ton compilateur te dis tout de suite lorsque tu crées une méthode qui ne correspond pas à une redéfinition (avec ton String s par exemple)
	@Override
	public void paintComponent(Graphics g) 
	{
		super.paintComponent(g);
		g.drawImage(img, 0, 0, this.getWidth(), this.getHeight(), this);
	}
}

public class Toto extends JFrame
{
// mettre moi1 dans le corps de la fenêtre évite de relire le fichier et recréer une nouvelle image à chaque fois que tu en as besoin.
	private final Image moi1;
	
	public Toto(String imageFileName) throws IOException
	{
		moi1 = ImageIO.read(new File(imageFileName));
		
		chp.addActionListener(new ActionListener()
		{
// mettre pa dans le corps de l'ActionListener évite de recréer un objet parrain à chaque nouvel appel de actionPerformed, tu en as un pour le bouton ça suffit
			private final parrain pa = new parrain(moi1);
			
			@Override
			public void actionPerformed(ActionEvent e) 
			{
				// tu n'as pas à manipuler de Graphics g ici
				pa.repaint(); 
			}
		});
	}
	
	public static void main(String...args)
	{
		new Toto("C:/Users/Keurdange/Pictures/ucad.jpg");
	}
}

ok, je vais m'inspirer de l'idée pour me débrouiller. merci