ユーザーが特定のフォルダーを選択し、そのフォルダー内のすべてのJavaファイルと、それらのファイル内のコード行を個別にカウントしてコンソールに表示するアプリケーションを開発しましたが、Javaプロジェクトには非常に多くのパッケージがあり、現在は特定のパッケージまでナビゲートするには、ユーザーが特定のプロジェクトを選択したときに、さらにsrcフォルダーのみにナビゲートし、srcフォルダーからJavaファイルを含むすべてのパッケージのコード行がカウントされるようにアプリケーションを変更します。 。
public class abc {
/**
* @param args
* @throws FileNotFoundException
*/
private static int totalLineCount = 0;
private static int totalFileScannedCount = 0;
public static void main(String[] args) throws FileNotFoundException {
JFileChooser chooser = new JFileChooser();
chooser.setCurrentDirectory(new java.io.File("C:" + File.separator));
chooser.setDialogTitle("FILES ALONG WITH LINE NUMBERS");
chooser.setFileSelectionMode(JFileChooser.DIRECTORIES_ONLY);
chooser.setAcceptAllFileFilterUsed(false);
if (chooser.showOpenDialog(null) == JFileChooser.APPROVE_OPTION) {
Map<String, Integer> result = new HashMap<String, Integer>();
File directory = new File(chooser.getSelectedFile().getAbsolutePath());
List<File> files = getFileListing(directory);
//print out all file names, in the the order of File.compareTo()
for (File file : files) {
System.out.println("Directory: "+file);
result = getFileLineCount(file);
totalFileScannedCount += result.size();
}
System.out.println("*****************************************");
System.out.println("FILE NAME FOLLOWED BY LOC");
System.out.println("*****************************************");
for (Map.Entry<String, Integer> entry : result.entrySet()) {
System.out.println(entry.getKey() + " ==> " + entry.getValue());
}
System.out.println("*****************************************");
System.out.println("SUM OF FILES SCANNED ==>" + "\t" + totalFileScannedCount);
System.out.println("SUM OF ALL THE LINES ==>" + "\t" + totalLineCount);
}
}
public static Map<String, Integer> getFileLineCount(File directory) throws FileNotFoundException {
Map<String, Integer> result = new HashMap<String, Integer>();
File[] files = directory.listFiles(new FilenameFilter() {
@Override
public boolean accept(File directory, String name) {
if (name.endsWith(".java")) {
return true;
} else {
return false;
}
}
});
for (File file : files) {
if (file.isFile()) {
Scanner scanner = new Scanner(new FileReader(file));
int lineCount = 0;
try {
for (lineCount = 0; scanner.nextLine() != null; lineCount++);
} catch (NoSuchElementException e) {
result.put(file.getName(), lineCount);
totalLineCount += lineCount;
}
}
}
return result;
}
/**
* Recursively walk a directory tree and return a List of all
* Files found; the List is sorted using File.compareTo().
*
* @param aStartingDir is a valid directory, which can be read.
*/
static public List<File> getFileListing(
File aStartingDir) throws FileNotFoundException {
validateDirectory(aStartingDir);
List<File> result = getFileListingNoSort(aStartingDir);
Collections.sort(result);
return result;
}
// PRIVATE //
static private List<File> getFileListingNoSort(
File aStartingDir) throws FileNotFoundException {
List<File> result = new ArrayList<File>();
File[] filesAndDirs = aStartingDir.listFiles();
List<File> filesDirs = Arrays.asList(filesAndDirs);
for (File file : filesDirs) {
if(file.isDirectory()) {
result.add(file);
}
if (!file.isFile()) {
//must be a directory
//recursive call!
List<File> deeperList = getFileListingNoSort(file);
result.addAll(deeperList);
}
}
return result;
}
/**
* Directory is valid if it exists, does not represent a file, and can be read.
*/
static private void validateDirectory(
File aDirectory) throws FileNotFoundException {
if (aDirectory == null) {
throw new IllegalArgumentException("Directory should not be null.");
}
if (!aDirectory.exists()) {
throw new FileNotFoundException("Directory does not exist: " + aDirectory);
}
if (!aDirectory.isDirectory()) {
throw new IllegalArgumentException("Is not a directory: " + aDirectory);
}
if (!aDirectory.canRead()) {
throw new IllegalArgumentException("Directory cannot be read: " + aDirectory);
}
}
}
ここで問題となるのは、このアプローチの問題です。TESTRESULTという名前のJavaプロジェクトに3つのパッケージがあり、各パッケージに2つのファイルがある場合、コンソールの結果にファイル名の後に最新のパッケージのみのlocが表示されます。 abc、def、tyuという名前のJavaプロジェクト3パッケージ
com.abc package having files --->abc.java
com.def package having files --->abc.java
com.tyu package having files --->FileBrowse.java , FileCountLine.java
the outcome shown in console is...
Directory: C:\Users\vaio\Desktop\Demo\TESTRESULT\.settings
Directory: C:\Users\vaio\Desktop\Demo\TESTRESULT\bin
Directory: C:\Users\vaio\Desktop\Demo\TESTRESULT\src
Directory: C:\Users\vaio\Desktop\Demo\TESTRESULT\src\com
Directory: C:\Users\vaio\Desktop\Demo\TESTRESULT\src\com\abc
Directory: C:\Users\vaio\Desktop\Demo\TESTRESULT\src\com\def
Directory: C:\Users\vaio\Desktop\Demo\TESTRESULT\src\tyu
*****************************************
FILE NAME FOLLOWED BY LOC
*****************************************
FileBrowse.java ==> 95
FileCountLine.java ==> 53
*****************************************
SUM OF FILES SCANNED ==> 4
SUM OF ALL THE LINES ==> 296
これは完璧ではありません。すべてのファイルについて、ファイル名の後にLOCが表示される方法を教えてください
私が思うに、ファイルと行数を読み取る方法では、ディレクトリごとに新しいマップを作成し、それを同じ結果に再割り当てします。マップを再割り当てする前にデータを表示しないため、以前のマップにアクセスできなくなり、表示に到達するまでに、最後のディレクトリのマップしかありません。代わりに、単一のマップを維持し、各ディレクトリから同じマップに新しい値を挿入する必要があります。
public static void main(String[] args) throws FileNotFoundException {
//...
for (File file : files) {
System.out.println("Directory: "+file);
result = getFileLineCount(file);
そこからマップが返されgetFileLineCount()、結果に割り当てられます。これにより、以前に作成された結果マップが破棄され、すでにそこにあったものはすべて失われます。いくつかのオプションがあります。
- 結果マップをメソッドに渡すと、メソッド
getFileLineCount()内のフォルダーごとに新しいマップを作成するのではなく、1つのマップに結果を追加できます。getFileLineCount() - 結果
getLineCount()マップを置き換えるのではなく、結果を取得して結果マップにコピーします - 次のようなマップのような別のコレクションを作成します:(メソッド
Map<String, Map<String, Integer>>によって返された結果getFileLineCount()を、それらの結果が関連するディレクトリの名前にマップします）、または（親にマッピングせずList<Map<String, Integer>>に返された結果の単純なリストになります）getFileLineCount()ディレクトリ）。
私のコードをリファクタリングする方法を教えてください、事前に感謝します