Let's make the code do something useful!

New task: The program reads in two file names. Requirements: 1. The program must read file names. Don't look at those sentences as step-by-step directions. They're talking about the program as a whole. Your task is to: 1. Read the source file name 2. 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.

How I solved this task. Read first file name. Declare fileInputStream. In try block initialize fileInputStream. In catch block print message, read first file name again and initialize fileInputStream again. Then the rest of the code. It took me 25 attempts to solve it ))

I have been trying really hard to be DRY but I am not able to see how that could be with out creating another class...

I solved this by using a try-catch nested into another try-catch. It really helps if you make test file in the CodeGymTasks folder and mess around with actually reading and copying it. Otherwise it's REALLY hard to tell if your code is working at all.

I'm not trying to complain, but WHAT HAPPENED? I understood task after task, Learned a lot, but all of a sudden I don't have a clue what to do on these tasks. Am I the minority? Or is it designed like this? What am I missing?

Lots of people tried to clarify this task, here's what I found useful: You ONLY read the output filename AFTER you've checked whether the INPUT one is valid. 1. Enter Input filename. 2. IF it's not valid, read another Input filename. 3. if that's not valid either, program ends with an exception. 4. If that one is valid or the first is valid THEN read the output filename.

It's not unsolvable in the sense that the task is too complicated. I think it has more with learning how the flow of try catch works and how initializations affect the flow below them. Good learning experience.

Can some one help what mistake Im doing here.

There are a lot of overly complicated suggestions here. here's what you need to know: if you simply wrap the code that throws the exceptions: String sourceFileName = reader.readLine(); ---------> IOException InputStream fileInputStream = getInputStream(sourceFileName);---------> FileNotFoundException the fileInputStream instance will not be visible to the code outside the try block. to fix this, set the InputStream fileInputStream to null before making the try-catch wrap. all the best!

There only 2 cases to keep in mind for this. If the inputFileName is valid in try{} block, keep going to read the next outPutFileName, else go to catch block{} and do the same thing.