weird error in a jar file when i trie to copy and create a file

[Cyan Rick]

Hi everyone
this question is also in the I/O and stream and im sorry i do not really know hos to delet post and i moved it here because i dont think it belong to this section anymore if you know how to delet post hope you can guide me
i have a class GUI that edit multiple png and show it in the window if i execute this code from eclipse it work perfectly but when i want to export to a jar file and launch it (in windows) some weird error message are poping and i do not really understand it

 java.nio.file.NoSuchFileException: src\morpion\tictactoe_sav.png
        at sun.nio.fs.WindowsException.translateToIOException(Unknown Source)
        at sun.nio.fs.WindowsException.rethrowAsIOException(Unknown Source)
        at sun.nio.fs.WindowsException.rethrowAsIOException(Unknown Source)
        at sun.nio.fs.WindowsFileCopy.copy(Unknown Source)
        at sun.nio.fs.WindowsFileSystemProvider.copy(Unknown Source)
        at java.nio.file.Files.copy(Unknown Source)
        at morpion.GUI.copyFile(GUI.java:27)
        at morpion.GUI.temp(GUI.java:46)
        at morpion.MemoryEdit.modif(MemoryEdit.java:42)
        at morpion.WindowSetting.play_pve(WindowSetting.java:152)
        at morpion.WindowSetting$2.run(WindowSetting.java:203)
        at java.lang.Thread.run(Unknown Source)

here is my class that edit and read my png

package morpion;

import java.awt.Graphics;
import java.awt.image.BufferedImage;
import java.io.File;
import java.io.IOException;
import java.nio.file.Files;

import javax.imageio.ImageIO;

public class GUI { 
	// tictactoe_sav.png will always be clean he is to make sure that we can set a clean board if we want or need it and combined is always the actual board
	private static int test;
	// path to the folder that contain the png
	private static String path = "src//morpion//";
	private static BufferedImage combined = null;
	static BufferedImage board = null;
	BufferedImage x = null;
	BufferedImage o = null;
	BufferedImage cSymbol = null;
 
	// this method will make a copy of the save to create the board so after each game the board.png is deleted and a new is created
	private static void copyFile() throws IOException 
	{
		File source = new File(path,"tictactoe_sav.png");
		File dest = new File(path,"tictactoe.png");
	    Files.copy(source.toPath(), dest.toPath());
	}
	  
	//yeah bad name for this but its ok this method will make the fusion of the actual board and the new token into a new board and delet the previous
    public void temp(int symbol,int coo_x,int coo_y) throws Exception
    {
    	//this will delet the board if he is not clean after the game start
    	File b = new File(path,"tictactoe.png");
    			
    			if(test != 1)
    			{
    				test ++;
    				try
    				{
    					copyFile();
    				}
    				catch(Exception e)
    				{
    					b.delete();
    					copyFile();
    				}
    			}
    			// load source images
    			board = ImageIO.read(b);
    			x = ImageIO.read(getClass().getResource("X.png"));
    			o = ImageIO.read(getClass().getResource("O.png"));
    			
    			// chek to know what token we will use
    			if(symbol == 1)
    			{
    				cSymbol = x;
    			}
    			else
    			{
    				cSymbol = o;
    			} 
    			 
    			// create the new image, canvas size is the max. of both image sizes
    			int w = Math.max(board.getWidth(), cSymbol.getWidth());
    			int h = Math.max(board.getHeight(), cSymbol.getHeight());
    			combined = new BufferedImage(w, h, BufferedImage.TYPE_INT_ARGB);

    			// paint both images, preserving the alpha channels 
    			Graphics g = combined.getGraphics();
    			g.drawImage(board, 0, 0, null);
    			g.drawImage(cSymbol, coo_x, coo_y, null);

    			g.dispose();

    			// Save as new image
    			ImageIO.write(combined, "PNG", new File(path,"tictactoe.png"));
    }
    
    //this will return the buffer of the image if the buffer of combined is null then return a clean one else return the combined buffer image (so the actual board)
    public static BufferedImage get_buff() throws IOException
    {
    	if(combined == null)
    	{
    		BufferedImage tmp = ImageIO.read(GUI.class.getResource("tictactoe_sav.png"));
    		return tmp;
    	}
    	return combined;
    }
    
    //this will clean the board by setting the buffer combined with the clean board and write it as the actual board and return it
    public static BufferedImage clean() throws IOException
    {
    		combined = ImageIO.read(GUI.class.getResource("tictactoe_sav.png"));
    		ImageIO.write(combined, "PNG", new File(path,"tictactoe.png"));
    		return combined;
    }
    public GUI() throws Exception
    {
    	 
    }
}

i hope you can explain why is it working in eclipse ide but not in a jar file

[Campbell Ritchie]

Please check the location of the XXX.png file. Eclipse uses paths relative to the root of the project (I think) but if you run the program from the command line, paths may be relative to the current directory.

[Cyan Rick]

i dont think its this because it work perfectly in eclipse and here my architecture

->morpion
|
|->JRE system Lib
|->src
 |
 |->morpion
   |
   |->all my class file
   |-> all my png file

so my path is actually src//morpion//
and im runing my code from eclipse but it show the error code in my post only when i use the jar file

[Mike Simmons]

I suggest adding a line to your method:

    private static void copyFile() throws IOException
    {

        File source = new File(path,"tictactoe_sav.png");
        File dest = new File(path,"tictactoe.png");
        System.out.println("Attempting to copy from " + source.getAbsolutePath() + " to " + dest.getAbsolutePath());
        Files.copy(source.toPath(), dest.toPath());
    }

I think you will find that the absolute path to the files you're copying is not what you think it is.  Most likely, you are starting from a different home directory.  The simplest way to fix this would be to put an absolute path in your "path" constant, meaning one that begins with / or (for Windows) a drive, like "C:".  Something like this:

    private static String path = "/users/cryan/complete/path/to/your/project/src//morpion//";

[Cyan Rick]

hi mike it seem that you are ritght about my absolute path but if i make a full path then my jar file will not work because it might be in another device or folder no ?

Attempting to copy from
C:\Users\user\eclipse-workspace\morpion\src\morpion\tictactoe_sav.png
to
C:\Users\user\eclipse-workspace\morpion\src\morpion\tictactoe.png

what i want with the information you give me is the absolute path to my png in my jar file so even if i share my jar its still work

[Norm Radder]

path to my png in my jar file

When files are placed in jar files they become resources.  You don't normally use File objects to access resources.
Look at the Class class's getResource and getResourceAsStream methods.

[Stephan van Hulst]

Norm is correct. Resources that are part of the project sources are packaged into the JAR and need to accessed through Class.getResourceAsStream(). The path to use is relative to the path of the class that you called the method on. The easiest thing to do is put the resource in the same package as the source file you will be using it in. Otherwise, use an absolute path. Absolute paths are relative to the root package (the source folder):

GUI.class.getResourceAsStream("/morpion/tictactoe.png");

[Cyan Rick]

hi thanks you Stephan and Norm for your reply and explain that was a hudge help i have edited my code according to your suggestion i can load at least my first png but he show me error telling me that NoSuchFileException because he cant find the file i guess and i dont really understand why when i launch my jar file he load the first png (tictactoe_sav) but cant load the next and i cant make a getRessourcesAsstream to my file b because i want to delet it (replace it)after and cant do that if i use getressources

java.nio.file.NoSuchFileException: src\morpion\tictactoe.png
        at sun.nio.fs.WindowsException.translateToIOException(Unknown Source)
        at sun.nio.fs.WindowsException.rethrowAsIOException(Unknown Source)
        at sun.nio.fs.WindowsException.rethrowAsIOException(Unknown Source)
        at sun.nio.fs.WindowsFileSystemProvider.newByteChannel(Unknown Source)
        at java.nio.file.spi.FileSystemProvider.newOutputStream(Unknown Source)
        at java.nio.file.Files.newOutputStream(Unknown Source)
        at java.nio.file.Files.copy(Unknown Source)
        at morpion.GUI.copyFile(GUI.java:30)
        at morpion.GUI.temp(GUI.java:49)
        at morpion.MemoryEdit.modif(MemoryEdit.java:42)
        at morpion.WindowSetting.play_pve(WindowSetting.java:152)
        at morpion.WindowSetting$2.run(WindowSetting.java:203)
        at java.lang.Thread.run(Unknown Source)

package morpion;

import java.awt.Graphics;
import java.awt.image.BufferedImage;
import java.io.File;
import java.io.IOException;
import java.io.InputStream;
import java.nio.file.Files;
import java.nio.file.Paths;

import javax.imageio.ImageIO;

public class GUI { 
	// tictactoe_sav.png will always be clean he is to make sure that we can set a clean board if we want or need it and combined is always the actual board
	private static int test;
	// path to the folder that contain the png
	private static String path = "src/morpion/";
	private static BufferedImage combined = null;
	static BufferedImage board = null;
	BufferedImage x = null;
	BufferedImage o = null;
	BufferedImage cSymbol = null;
 
	// this method will make a copy of the save to create the board so after each game the board.png is deleted and a new is created
	private static void copyFile() throws IOException 
	{
		InputStream source = GUI.class.getResourceAsStream("tictactoe_sav.png");
		//InputStream dest = GUI.class.getResourceAsStream("tictactoe.png");
		String path = "src/morpion/tictactoe.png";
		Files.copy(source, Paths.get(path));
	}
	  
	//yeah bad name for this but its ok this method will make the fusion of the actual board and the new token into a new board and delet the previous
    public void temp(int symbol,int coo_x,int coo_y) throws Exception
    {
    	//this will delet the board if he is not clean after the game start
    	File b = new File(path,"tictactoe.png");
    			
    			if(test != 1)
    			{
    				test ++;
    				try
    				{
    					copyFile();
    				}
    				catch(Exception e)
    				{
    					b.delete();
    					copyFile();
    				}
    			}
    			// load source images
    			board = ImageIO.read(b);
    			x = ImageIO.read(GUI.class.getResourceAsStream("X.png"));
    			o = ImageIO.read(GUI.class.getResourceAsStream("O.png"));
    			
    			// chek to know what token we will use
    			if(symbol == 1)
    			{
    				cSymbol = x;
    			}
    			else
    			{
    				cSymbol = o;
    			} 
    			 
    			// create the new image, canvas size is the max. of both image sizes
    			int w = Math.max(board.getWidth(), cSymbol.getWidth());
    			int h = Math.max(board.getHeight(), cSymbol.getHeight());
    			combined = new BufferedImage(w, h, BufferedImage.TYPE_INT_ARGB);

    			// paint both images, preserving the alpha channels 
    			Graphics g = combined.getGraphics();
    			g.drawImage(board, 0, 0, null);
    			g.drawImage(cSymbol, coo_x, coo_y, null);

    			g.dispose();

    			// Save as new image
    			ImageIO.write(combined, "PNG", new File(path,"tictactoe.png"));
    }
    
    //this will return the buffer of the image if the buffer of combined is null then return a clean one else return the combined buffer image (so the actual board)
    public static BufferedImage get_buff() throws IOException
    {
    	if(combined == null)
    	{
    		BufferedImage tmp = ImageIO.read(GUI.class.getResourceAsStream("tictactoe_sav.png"));
    		return tmp;
    	}
    	return combined;
    }
    
    //this will clean the board by setting the buffer combined with the clean board and write it as the actual board and return it
    public static BufferedImage clean() throws IOException
    {
    		combined = ImageIO.read(GUI.class.getResourceAsStream("tictactoe_sav.png"));
    		ImageIO.write(combined, "PNG", new File(path,"tictactoe.png"));
    		return combined;
    }
    public GUI() throws Exception
    {
    	 
    }
}

[Stephan van Hulst]

As Norm already pointed out, you don't perform File operations on resources. They are not files, resources are compressed into your JAR. Resources are only there to be read by your application.

What you're doing doesn't make sense. Why would you want to copy tictactoe_sav.png to tictactoe.png? Just keep the combined image in memory, don't save it to disk. Even if you insist on saving it to disk, don't save it to your project folder. When your application is deployed on a client system, the installation folder will not reflect your project directory layout.

Why are you combining the images into one at all? If you're using a GUI framework such as Swing, just paint the images to a canvas on the fly, instead of combining them into one image first.

[Norm Radder]

java.nio.file.NoSuchFileException: src\morpion

Does that target path exist?