コードの添削

Question

Java開発初心者です。
※質問内容を編集しました

以下のコードについて質問です。
質問①なぜ配列が必要なのでしょうか。
質問②while文はなにをしているのでしょうか。

ーーーーーーーーーーーーーーーーーーーーーーーーーーーーー
import java.io.FileInputStream;
import java.io.FileOutputStream;
import java.io.IOException;

public class Copy {

public static void main(String[] args)throws IOException {
	// TODO Auto-generated method stub
	 try{
	 FileInputStream input=new FileInputStream("コピーするファイルの所在");
	 FileOutputStream output=new FileOutputStream("コピー先のファイルの所在");
	 
     byte buf[] = new byte[256];


int len;
while ((len = input.read(buf)) != -1) {
output.write(buf,0,len);
//output.flush();
//↑表示
output.close();
input.close();
}
}catch(IOException e){
e.printStackTrace();
}
}
}

Answer 1

修正したコードと実行例を示します。
Copy.java

import java.io.FileInputStream;
import java.io.FileOutputStream;
import java.io.IOException;

public class Copy {

    public static void main(String[] args) throws IOException {
        // TODO Auto-generated method stub
        try {
            FileInputStream input = new FileInputStream("input.txt");
            FileOutputStream output = new FileOutputStream("output.txt");

            byte buf[] = new byte[256];
            int len;
            while ((len = input.read(buf)) != -1) {
                output.write(buf, 0, len);
                output.flush();
            }
            output.close();
            input.close();
        } catch (IOException e) {
            e.printStackTrace();
        }
    }
}

実行例

Comments

[koto12]

ご回答いただきありがとうございます。
正常に実行することができました。

[katoy]

https://techstars.jp/blog/java-nyumon-362/
で紹介されている入門ページを試してみては如何でしょう。
あるいは、入門書籍を購入して読むことをお勧めします。

Answer 2

質問①この行は何をするための行なのでしょうか。

buf = input;
//↑コンパイルエラー発生

自分で追加したとのことですが、何をするのか判らないコードを書いてはいけません。
その行は不要です。byte配列型の変数にFileInputStream型のオブジェクトを代入することはできません。その手前でbufに作成したインスタンスを設定しているのに、別のものを入れようとすることは無意味です。

質問②このエラーを解消するにはどのようにコードを書き換えるとよいでしょうか。

エラーの原因はたぶんこれです。

//表示
output.close();
input.close();

closeメソッドは何かを表示するためのメソッドではありません。それはストリームをクローズするメソッドです。ストリームをクローズすると、それ以降ファイルの読み書きはできなくなります。それをループの中でやってしまっているので、ループ2週目にinput.read(buf)を実行しようとしたときに、すでにクローズされているため読み込みができないというエラーになります。

closeメソッドの呼び出しはwhileループを抜けてから行ってください。

それと、エラーではありませんが、output.flush();は、それが必要だと確信できないなら不要です。むやみに使うと書き込み速度が低下します。

Answer 3

参考資料に書いてあるんですか？その1行は不要だと思います。

ファイルがあるかどうかはどうやって判断しているんですか？Fileクラスを使って確認していますよね？

また、コードはコード専用の記述方法があるのでそちらに修正してください。

Comments

[koto12]

コンパイルエラーが起こっている一文は自分で追加しました。
理由はinputファイルの中身を配列に入れて、順番に読み込んで表示していくのだと考えたからです。
この一文が必要ないのであれば、この配列は何のためにあるのでしょうか。
（配列は参考資料に書いてあったので、必要なのだろうと思って書きましたが、
意味を理解していません）

ファイルクラスを確認というのは、FileInputStreamクラスのことでしょうか。
自分はこのクラスはファイルの所在を確認し開くクラスだと認識しています。
ファイルがあるかを判断というのは、そのためのコードが抜けているということですか？

自分は勉強をはじめたばかりの初心者であり、
コードは自力で書けず調べながら、引用をしつつ書いています。
そのため、このコードのことを全て理解はできていません。

また、teratailを使い始めたばかりでまだ使い方を把握できていません。
コードの専用の書き方があるとのことですが、
申し訳ないですが分からないので今回はこのままとさせてください。

[yona]

■bufについて
自分でコードを書いておいて、「何をする行ですか」という質問はだめですよ。
while文の中でinput.read(buf)でbufの中にファイルの文字列が入ります。

■ファイルの存在確認
FileクラスはFileクラスですよ。Fileクラスの詳細については調べてください。
Fileクラスのexistメソッドを使うことでファイルクラスがあるかないかを判断できます。
FileInputStreamを使うと存在しないファイルにアクセスすると例外が発生するので先に調べるべきだと思います。
今回指定しているパスが正しいかを確認してください。