Someone can help me plz

Question about the task

Let's make the code do something useful!

Java Syntax, Level 9, Lesson 11

Under discussion

its work in my lab but still give me the issue

Task: The program reads in two file names. It copies the first file to the location specified by the second file name.
New task: The program reads in two file names. It copies the first file to the location specified by the second file name.
If the specified file (to be copied) does not exist, then the program should display "File does not exist." and read a file name one more time and only then read the destination file name.

Requirements:

The program must read file names.
The main method must handle exceptions thrown by the getInputStream method. If an exception occurs, you should display "File does not exist.".
The program must copy the contents of the first file to the second one.
The main method should call the getInputStream method.
Don't change the getInputStream method.
The main method should call the getOutputStream method.
Don't change the getOutputStream method.

package com.codegym.task.task09.task0929; import java.io.*; /* Let's make the code do something useful! */ public class Solution { public static void main(String[] args) throws IOException { BufferedReader reader = new BufferedReader( new InputStreamReader( System.in ) ); String sourceFileName; String destinationFileName; InputStream fileInputStream; OutputStream fileOutputStream; sourceFileName = reader.readLine(); destinationFileName = reader.readLine(); try { fileInputStream = getInputStream( sourceFileName ); fileOutputStream = getOutputStream( destinationFileName ); while (fileInputStream.available() > 0) { int data = fileInputStream.read(); fileOutputStream.write( data ); } fileInputStream.close(); fileOutputStream.close(); } catch (FileNotFoundException e) { System.out.println( "File does not exist." ); } finally { sourceFileName = reader.readLine(); fileInputStream = getInputStream( sourceFileName ); fileOutputStream = getOutputStream( destinationFileName ); while (fileInputStream.available() > 0) { int data = fileInputStream.read(); fileOutputStream.write( data ); } fileInputStream.close(); fileOutputStream.close(); } } public static InputStream getInputStream(String fileName) throws IOException { return new FileInputStream( fileName ); } public static OutputStream getOutputStream(String fileName) throws IOException { return new FileOutputStream( fileName ); } }

Comments (4)

Popular
New
Old

You must be signed in to leave a comment

Deepak Joshi

Level 17 , Nainital, India

3 May 2019, 16:09

1. Code written in finally block will definitely execute. So let say your try block runs perfectly and raise no exception, now look what is happening again in finally block you are reading a new file, and this is not task requirement , 2. Only read destinationFileName when you will read a valid sourceFileName, as you are reading in line number 21. Well, maybe you had solved this task your post is too old, but if not maybe this will help you. package com.codegym.task.task09.task0929; import java.io.*; /* Let's make the code do something useful! */ public class Solution { public static void main(String[] args) throws IOException { BufferedReader reader = new BufferedReader( new InputStreamReader( System.in ) ); String sourceFileName; String destinationFileName; InputStream fileInputStream; OutputStream fileOutputStream; sourceFileName = reader.readLine(); try { fileInputStream = getInputStream( sourceFileName ); } catch (FileNotFoundException e) { System.out.println( "File does not exist." ); sourceFileName = reader.readLine(); } finally { fileInputStream = getInputStream( sourceFileName ); destinationFileName = reader.readLine(); fileOutputStream = getOutputStream( destinationFileName ); while (fileInputStream.available() > 0) { int data = fileInputStream.read(); fileOutputStream.write( data ); } fileInputStream.close(); fileOutputStream.close(); } } public static InputStream getInputStream(String fileName) throws IOException { return new FileInputStream( fileName ); } public static OutputStream getOutputStream(String fileName) throws IOException { return new FileOutputStream( fileName ); } }

Level 41 , Wilson, United States

10 August 2020, 00:44

Why do you re-type this line? fileInputStream = getInputStream( sourceFileName ); It was already included in the try block, why is this needed to repeat in the finally block? Wouldn't this cause the line to be ran twice?

Guadalupe Gagnon

Level 37 , Tampa, United States

10 January 2019, 14:40

You are suppose to read the destination file name only after the source file name is confirmed. Your program reads one right after the other, which is incorrect. You need to fix this.

Mohammed Amine EL AMRANI

Level 11 , Rabat, Morocco

10 January 2019, 19:00

package com.codegym.task.task09.task0929;

import java.io.*;

/*
Let's make the code do something useful!

*/

public class Solution {
    public static void main(String[] args) throws IOException {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));

        String sourceFileName = reader.readLine();

        File fileChecker = new File( sourceFileName );

        if (!fileChecker.exists()) {
            try{
                sourceFileName = reader.readLine();
            }catch (FileNotFoundException e) {
                System.out.println( "File does not exist." );
            }
        }


        try{
            InputStream fileInputStream = getInputStream(sourceFileName);
            String destinationFileName = reader.readLine();
            OutputStream fileOutputStream = getOutputStream(destinationFileName);

            while (fileInputStream.available() > 0) {
                int data = fileInputStream.read();
                fileOutputStream.write(data);
            }

            fileInputStream.close();
            fileOutputStream.close();

        }catch (FileNotFoundException e) {
            System.out.println( "File does not exist." );
        }



    }

    public static InputStream getInputStream(String fileName) throws IOException {
        return new FileInputStream(fileName);
    }

    public static OutputStream getOutputStream(String fileName) throws IOException {
        return new FileOutputStream(fileName);
    }
}

i did what y said but still have the issue