Can anyone help me to spot a mistake in the method.

[Bobby Marvikuan]

Can anyone help me to spot a mistake in the method. In previous post method now works but the problem with that is it labels a JLabel in JPanel but I want now to place an image in JPanel itself. what I am doing wrong???. Cheers in advance

public void openFavorite(File fileToOpen){
			 try  
	    	  {  
	    		  BufferedImage bi = ImageIO.read(fileToOpen);
	    		  Graphics g = bi.getGraphics();
	    		  Graphics2D g2d =(Graphics2D)g;
	    		  g2d.drawImage(bi, null, 0,0);
// this is a JPanel
	    		  this.repaint();
	    		  this.revalidate();
	    	  }  
	    	  catch(Exception exception)  
	    	  {  
	    	   System.out.println("Problem occour when to open the file");  
	    	  }  
		}

[Greg Brannon]

You should print the exception's stack trace to help pinpoint the problem:

catch(Exception exception)  
{  
    System.out.println("Problem occour when to open the file: " + exception.getMessage() );  
}

[Koen Aerts]

exception.toString() or exception.printStackTrace() is probably more informative.

[Greg Brannon]

Koen Aerts wrote:exception.toString() or exception.printStackTrace() is probably more informative.

Yep, that's what I meant. Sorry.

[Bobby Marvikuan]

Greg Brannon wrote:

Koen Aerts wrote:exception.toString() or exception.printStackTrace() is probably more informative.

Yep, that's what I meant. Sorry.

thanks for the advice but the problem is not in the exception because if there was an exception it would probably print at least "Problem occour when to open the file" but here it works with try block but does not do what I want it to do I mean it does not do anything at all

[Matthew Brown]

Your code appears to be doing the following at the moment:
- you load an image into a BufferedImage
- you get the graphics context of the BufferedImage
- you draw the BufferedImage onto itself
- you don't do anything else with the image

I suspect that's not what you were wanting.

[Bobby Marvikuan]

Matthew Brown wrote:Your code appears to be doing the following at the moment:
- you load an image into a BufferedImage
- you get the graphics context of the BufferedImage
- you draw the BufferedImage onto itself
- you don't do anything else with the image

I suspect that's not what you were wanting.

thanks a lot this makes sense!