リストアップしたファイルのコピー

Question

下記のように、フォルダからテキストファイル抽出し、抽出したファイルの内容を１つのテキストファイルに読み書きしようとしたのですが、上手くいきません。
テキストファイルの抽出は、コンソール上に記すことができました。
しかし、抽出したファイルの読み書きにおいて、別のメソッド(readFileメソッド)で処理しようとしましたが、
読み込みさえも行われないという状況です。オブジェクトの生成で、java.io.FileNotFoundException: C:\Users\nakamura\Documents\テキスト (アクセスが拒否されました。)が出て、はじかれてしまいました。
また、引数に関して、fromPathの引数をC:\Users\nakamura\Documents\テキストというフォルダ名とし、 toPathの引数をC:\Users\nakamura\Documents\テキスト\sample.txtとあらかじめ設定しました。
プログラムの内容を詳しく説明いたしますと、C:\Users\nakamura\Documents\テキストにあるテキストファイルをリストアップし、リストアップしたファイルの内容をC:\Users\nakamura\Documents\テキスト\sample.txtにコピーさせるというプログラムを作ろうとしています。
やり方として、恐らく読み込み元の引数をフォルダ名にしているため、上手くいかないのだろうと考えます。
しかし、どう改善すれば出力されるのか分からないので、教えていただけると幸いです。
よろしくお願い致します。

import java.io.BufferedReader;
import java.io.File;
import java.io.FileInputStream;
import java.io.FileOutputStream;
import java.io.FileReader;
import java.io.FileWriter;
import java.io.IOException;
import java.io.InputStreamReader;
import java.io.OutputStreamWriter;
import java.io.PrintWriter;
public class FileSearch2 {

	public static void main(String[] args) {
		// TODO 自動生成されたメソッド・スタブ

		//変数の初期化
		String fromPath = "";
		String toPath = "";

		//コマンドライン引数
		fromPath = args[0];
		toPath = args[1];

		File dir = new File(fromPath);
		File[] files = dir.listFiles();


		for (int i = 0; i < files.length; i++) {
			if(files[i].isFile() && files[i].getName().endsWith(".txt")) {
				System.out.println(files[i]);

			}
		}

		readFile(fromPath, toPath);

	}

	private static void readFile(String fromPath, String toPath) {
		// TODO 自動生成されたメソッド・スタブ
		try {

		File file = new File(toPath);

		//ファイルの存在確認
		if(!file.exists()) {
			System.out.println("ファイルを作成します。");
			return;
		} else {
			System.out.println("ファイルを上書きします。");
		}


		//配列の定義
		int a = 0;
		String[] text = new String[a];
		int h =0;
		String line;

		//読み込みオブジェクトの生成する
		FileReader fr = new FileReader(fromPath);
		BufferedReader br = new BufferedReader(new InputStreamReader(new FileInputStream(fromPath), "Shift-JIS"));

		//ファイルの読み込み
		while ((line = br.readLine()) != null) {
			text[h] = line;
			h++;
		}

		//書き込みオブジェクトの生成
		FileWriter fw = new FileWriter(toPath);
		PrintWriter pw = new PrintWriter(new OutputStreamWriter(new FileOutputStream(toPath), "Shift-JIS"));

		//ファイルの書き込み
		for (int k = 0; k < text.length; k++) {
			pw.write(text[k]);
			pw.write("\r\n");
		}

		//ストリームをフラッシュ
		pw.flush();

		//ファイルクローズ
		pw.close();
		fw.close();
		br.close();
		fr.close();

		System.out.println("コピーが完了しました。");

		} catch (IOException e) {
			System.out.println("読み込みに失敗しました。");
		}

	}
}

Comments

[y_waiwai]

うまく行かないとはどうなるんでしょうか。
そこらへん詳しく質問に追記してください

[kabutohyper]

編集しました。

Answer 1

抽出したファイルの読み書きにおいて、別のメソッド(readFileメソッド)で処理しようとしました

  File dir = new File(fromPath);
  File[] files = dir.listFiles();

  for (int i = 0; i < files.length; i++) {
    if(files[i].isFile() && files[i].getName().endsWith(".txt")) {
      System.out.println(files[i]);
    }
  }

  readFile(fromPath, toPath);

表示ではファイルを「抽出」していますが, readFile には渡っていませんし, readFile 内でも「抽出」していません. ですので, ご想像の動作をするようなコードにはなっていません.

  //ファイルの存在確認
  if(!file.exists()) {
    System.out.println("ファイルを作成します。");
    return;
  } else {
    System.out.println("ファイルを上書きします。");
  }

ファイルが無かった場合「ファイルを作成します。」と表示して return し, 何の処理も行いません. これは想定した動作とは思えません.

  //配列の定義
  int a = 0;
  String[] text = new String[a];
    :
  //ファイルの読み込み
  while ((line = br.readLine()) != null) {
    text[h] = line;
    h++;
  }

text 配列にファイルを全行読み込むにも関わらず, 配列のサイズは 0 です. これでは読み込みは不可能です.

  //読み込みオブジェクトの生成する
  FileReader fr = new FileReader(fromPath);
    :
  //書き込みオブジェクトの生成
  FileWriter fw = new FileWriter(toPath);

どちらも使用されていません.

  BufferedReader br = new BufferedReader(new InputStreamReader(new FileInputStream(fromPath), "Shift-JIS"));

フォルダそのものを読もうとしています.

  } catch (IOException e) {
    System.out.println("読み込みに失敗しました。");
  }

IOException は読み込み以外でも発生しえます.

Comments

[kabutohyper]

回答ありがとうございます。
解決しました。

Answer 2

ファイルが見つからない、という例外なんで、
readFileの引数の fromPath, toPath をプリントアウトするなりして、どんなパスが入ってるのか確認してみればどうでしょう

Comments

[kabutohyper]

質問内容に不備があり、修正いたしました。