[JAVA] path.listFiles().length がnullになる原因が知りたい。

Question

提示コードのFileControl.javaの GetFilePath_Directory関数の File file[] = new File[path.listFiles().length];で以下のエラーが発生するのですがこれはなぜでしょうか？原因がわかりません。ファイルパスは読み込まれています。

Exception in thread "main" java.lang.NullPointerException
	at FileManager.GetFilePath_Directory(FileManager.java:23)
	at Main.main(Main.java:29)

Main.java

import java.io.IOException;
import java.io.File;

import javax.imageio.ImageIO;

import java.awt.image.BufferedImage;

import java.util.concurrent.Callable;
import java.util.Scanner;

import java.util.ArrayList;
import java.util.Collection;
import java.util.List;


import org.apache.pdfbox.pdmodel.PDDocument;
import org.apache.pdfbox.pdmodel.PDPage;
import org.apache.pdfbox.pdmodel.PDPageContentStream;
import org.apache.pdfbox.pdmodel.common.PDRectangle;
import org.apache.pdfbox.pdmodel.graphics.image.PDImageXObject;

public class Main
{
    public static void main(String args[])
    {
        Scanner scanner = new Scanner(System.in);
        String fileName = scanner.nextLine();
			
        List<Image> image = FileManager.GetFilePath_Directory(new File(fileName));
        Generater gen = new Generater();
        gen.setFile(image);

        gen.run();


        scanner.close();
    }    
}

FileControl.java

import java.io.File;
import java.io.File;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.Collection;

import java.util.List;

public class FileManager
{
    //ファイルタイプ
	public enum type
	{
		Directory,	//ディレクトリ
		File,		//単一の画像ファイル
		Invalid		//対応していないファイル
	}

	static List<Image> GetFilePath_Directory(File path)
	{
		System.out.println(path.toPath());

		File file[] = new File[path.listFiles().length];
		file = path.listFiles();
		List<String> fileList = new ArrayList<String>();


		for(int i = 0; i < file.length; i++)
		{
			fileList.add(file[i].toString());
		}		

		Arrays.sort(fileList.toArray());

		List<Image> image = new ArrayList<Image>();


		for(int i = 0; i < fileList.size(); i++)
		{
			System.out.println(fileList.get(i));
			image.add(Image.GetImage(new File(fileList.get(i)).toPath()));
		} 

		
		return image;

	}
    


	//ファイルかディレクトリかを選別
    public static type GetFileType(File file)
	{
		if(file.exists() == true)
		{				
			if(file.isFile() == true)
			{
				String extension = file.getName().substring(file.getName().lastIndexOf("."));
				
				if(extension.equals(".png") || extension.equals(".jpg") || extension.equals(".jpeg"))
				{					
					return type.File;
				}	
			}
			else
			{	
				return type.Directory;
			}
		}
		else
		{
			System.out.println("対応形式ではりません: " + file.getName());
			return type.Invalid;
		}	
		
		return type.Invalid;
	}
}

Comments

[m.ts10806]

>ファイルパスは読み込まれています

第三者が判断できる確固たる根拠の提示を

Answer 1

「どこがNullか」はデバッグモードでステップ実行で追えばすぐわかるでしょう（タイトルが何を根拠にされたのか言及されてないのでわかりませんが）

そもそも基礎である変数定義の仕方がおかしい。
File file[]

こうでは？

File[] file = new File[path.listFiles().length];

むしろ変数定義して、また格納して、と二度手間されてるようなので直でいい気がします。

File[] file = path.listFiles();

Comments

[dodox86]

> そもそも基礎である変数定義の仕方がおかしい。
> File file[]

Javaでは一般的、多くの場合で「型[] 変数名;」のように配列を宣言しますが、変数名の後で[]をつけるのも文法的には有りですね。

https://docs.oracle.com/javase/specs/jls/se7/html/jls-10.html

> The [] may appear as part of the type at the beginning of the declaration, or as part of the declarator for a particular variable, or both.

「一般的ではない」と言う意のご指摘では同意します。

[m.ts10806]

あぁ、原因だけ知りたいのか。
やりすぎた。

[m.ts10806]

補足ありがとうございます。
この質問者のこれまでからあえてそういう変数名にしているとは到底思えないので、指摘しました。
new File[]としてるので変数の型は明らかに間違いですね。

Answer 2

resource/kairakuten ファイルパスをその.jarファイルからの相対パスに直接記述したから動きました。
結論
ファイルパスに` が入力されていた。

Comments

[m.ts10806]

自身の思い込みを正とする思考をなんとかしないといけないのでは？
初めからそうですよね。
「〜は問題ない」が本当に問題なかった試しがない。まずは自分を疑う癖をつけてください。

[dodox86]

「resource/kairakuten」なんてPATH名は提示のコード、文章中にまったく出てこないので、関連も不明で回答自体ほとんど無意味です。

Answer 3

File#listFiles() は null を返すことがあります。リファレンスを読んで対処しましょう。

Comments

[退会済みユーザー]

リファレンスを見たのですが`この抽象パス名が示すディレクトリ内のファイルおよびディレクトリを示す抽象パス名の配列。配列は、ディレクトリが空の場合は空になる。この抽象パス名がディレクトリを示さない場合、または入出力エラーが発生した場合は null を返す。`とありますがディレクトリの中には確かにファイルが存在しているのですがこれはなぜでしょうか？

[m.ts10806]

自身の思い込みを排除しないと永遠に理解はできないかと。

[int32_t]

この path が本当に読めるディレクトリを示しているかどうか質問者さん以外には判断できません。自分で調べるか他人が判断できるような情報を出しましょう。

情報を出すとしたら、
* エラーが出る直前の System.out.println(path.toPath()); の出力
* その出力のパスをコマンドラインで 「ls -ld パス」をした結果
など。