Not passing the 5th condition.

The code now takes a file name from the user, but it seems to have made things worse. Now only the first requirement is passing. I notice that the user can choose a file, but it has to be an existing one. None of the other requirements are passing, but from what I've checked, everything seems to fit. public class Solution { public static void main(String[] args) { // write your code here try { // Get the file path from the user BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); String fileName = reader.readLine(); FileInputStream fileStream = new FileInputStream(fileName); //******************************************************************************** // Writes data to a file BufferedReader reader2 = new BufferedReader(new InputStreamReader(System.in)); BufferedWriter writer = new BufferedWriter(new FileWriter(fileName)); for(;;){ String text = reader2.readLine(); if(!(text.equals("exit"))){ writer.write(text + "\n"); // The + \n is required so that the strings get sent to the file on separate lines } else{ writer.write(text + "\n"); // This is so that "exit" also gets printed. break; } } writer.close(); } catch (IOException e) { e.printStackTrace(); } } }

You should tidy up your code a little bit and wipe out things you do not need anymore. Best would be to just take your first version as that is less work. Change line 18 change to

BufferedWriter writer = new BufferedWriter(new FileWriter(reader.readLine()));

and it asks for a file name from the keyboard instead of using the hardcoded one. That should do the trick. In your second code it may be that the validator got confused with the FileInputStream. The validator expects a Writer and is presented as first option a FileInputStream. Some more notes to the for loop and the if-else construct you're using: Inside of the if-else code blocks in each you have the same code parts. In such a case you can move that code outside the if - else structure, like

writer.write(text + "\n"); // The + \n is required so that the strings get sent to the file on separate lines
if (!(text.equals("exit"))) {
}
else {
    break;
}

now you may see that this can be written a little bit easier

writer.write(text + "\n");
if (text.equals("exit"))
    break;

And this case here is one of the rare opportunities where you could think about using a do-while loop (of course that's not really necessary).

String input;
do {
    input = reader.readLine();
    writer.write(input);
    writer.newLine();

} while (!input.equals("exit"));

You also can see here, that I didn't use the second BufferedReader you instanciated but resued the first one. For adding a new line Writer functionality is used. Another possibility (despite the \n) is to use System.lineSeparator() or System.getProperty(“line.separator”). Then the code is platform independent (linux, macos and windows use differnt combinations of \r and \n - for more info just google that).