Quote Originariamente inviata da Frenza Visualizza il messaggio
non è giusto usare gli stream di caratteri?
Per copiare file pari pari, senza alcuna nozione sul contenuto, sarebbero meglio gli Stream (orientati ai byte) ... non i Reader/Writer (orientati ai caratteri).

Comunque il tuo codice è sbagliato e per svariati motivi!
- Se proprio si volesse usare readLine(), si dovrebbe testare il suo valore di ritorno. L'idioma tipico d'uso del readLine() è dentro un ciclo while nella cui espressione di test si assegna ad una variabile ed allo stesso tempo si testa per diverso da null.
- Se anche usi readLine(), dovresti scrivere esplicitamente un newline (nel tuo codice i newline li perdi!)
- quel br.read()!= -1 non ha senso messo lì così
- cosa importantissima, mancano i close, che vanno sempre fatti (anche quando una lettura/scrittura lancia una eccezione).