1

既にアルファベット順になっている 2 つのファイルから名前と ID 番号を読み取り、それらを 3 つ目のファイルに完全にアルファベット順で結合するプログラムを作成する必要があります..ファイルは次のように編成されています...

姓、名 ID (例: rbb091020) 姓、名 ID

何らかの理由で、私のファイルは 2 つのファイルを読み取り、3 番目のファイルを作成しますが、3 番目のファイルには何も書き込みません。

これが私のコードです...

import java.io.*;
import java.util.Scanner;

public class Combine_Files
{
public static void main(String[]args) throws IOException
{
    String filename1 = "";
    String filename2 = "";
    String combinedFileName = "";
    String response = "";
    String nextName1 = "";
    String nextName2 = "";
    String nextIDNumber1 = "";
    String nextIDNumber2 = "";
    boolean okToOverwrite = false;
    Scanner keyboard = new Scanner(System.in);

    System.out.println("What is the name of the first file?");
    filename1 = keyboard.nextLine();

    File file1 = new File(filename1);

    if (!file1.exists())
    {
        System.out.println(file1 + " does not exist.\nTerminating Program.");
        System.exit(0);
    }

    System.out.println("What is the name of the second file?");
    filename2 = keyboard.nextLine();

    File file2 = new File (filename2);
    if (!file2.exists())
    {
        System.out.println(file2 + " does not exist.\nTerminating Program.");
        System.exit(0);
    }   

    System.out.println("What is the name of the file that you wish to create?");
    combinedFileName = keyboard.nextLine();
    File combinedFile = new File (combinedFileName);

    while (combinedFile.exists() && !okToOverwrite)
    {
        System.out.println("\nError, a file named " +
            combinedFileName + " already exists." +
            "\nWould you like to overwrite this file?" +  
            "\nEnter Y for Yes or N for No.");
        response = keyboard.nextLine();

        // Add input validation on response

        if (response.equalsIgnoreCase("Y"))
        {
            okToOverwrite = true;
        }
        else 
        {
            System.out.println("Enter a new filename.");
            combinedFileName = keyboard.nextLine();
            combinedFile = new File(combinedFileName);
        }
    }

    if (file1.exists() && file2.exists())
    {

        Scanner list1 = new Scanner(file1);
        Scanner list2 = new Scanner(file2);
        PrintWriter outputFile = new PrintWriter(combinedFile);

            while (list1.hasNext() && list2.hasNext())
            {
                nextName1 = list1.nextLine();
                nextName2 = list2.nextLine();

                if(nextName1.compareToIgnoreCase(nextName2) <0)
                {
                    outputFile.print(nextName1);
                    nextName1 = list1.nextLine();
                    outputFile.print(nextName1);
                }
                else if(nextName1.compareToIgnoreCase(nextName2) >0)
                {
                    outputFile.println(nextName2);
                    nextName2 = list2.nextLine();
                    outputFile.println(nextName2);
                }
                else
                {
                    outputFile.println(nextName1);
                    nextName1 = list1.nextLine();
                    outputFile.println(nextName1);
                    outputFile.println(nextName2);
                    nextName2 = list2.nextLine();
                    outputFile.println(nextName2);
                }       
            }

            while (list1.hasNext() && !list2.hasNext())
            {
                outputFile.println(nextName1);
            }

            while (list2.hasNext() && !list1.hasNext())
            {
                outputFile.println(nextName2);
            }       
    }
}   
}
4

2 に答える 2

1

ようなライター クラスを使用する場合、何かを STDOUT またはファイルに出力するたびPrintWriterにメソッドを呼び出すようにする必要があります。flush()したがって、基本的PrintWriterに、内部バッファーに書き込む印刷メソッドを呼び出すと、呼び出しflush()によってバッファーが適切な出力ストリームに送信されます。

        while (list1.hasNext() && list2.hasNext())
        {
            nextName1 = list1.nextLine();
            nextName2 = list2.nextLine();

            if(nextName1.compareToIgnoreCase(nextName2) <0)
            {
                outputFile.print(nextName1);
                nextName1 = list1.nextLine();
                outputFile.print(nextName1);
            }
            else if(nextName1.compareToIgnoreCase(nextName2) >0)
            {
                outputFile.println(nextName2);
                nextName2 = list2.nextLine();
                outputFile.println(nextName2);
            }
            else
            {
                outputFile.println(nextName1);
                nextName1 = list1.nextLine();
                outputFile.println(nextName1);
                outputFile.println(nextName2);
                nextName2 = list2.nextLine();
                outputFile.println(nextName2);
            }
            outputFile.flush(); // <--- flush added here       
        }

        while (list1.hasNext() && !list2.hasNext())
        {
            outputFile.println(nextName1);
            outputFile.flush(); // <--- flush added here 
        }

        while (list2.hasNext() && !list1.hasNext())
        {
            outputFile.println(nextName2);
            outputFile.flush(); // <--- flush added here 
        }

これらの呼び出しはflush()、ファイルに書き込む必要があります。

于 2012-04-07T17:16:26.527 に答える
0

このプログラムには論理エラーがあります。不均一な長さのファイルでテストすると、最後の 2 つの while ループの 1 つで無限ループに陥り、プログラムが強制終了されるまで実際に出力ファイルに書き込みます。ファイルが同じ長さの場合、スキャナは if/else 句のいずれかで行を読み取ることができません。

于 2012-04-07T17:21:32.663 に答える