質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.62%

  • Java

    13525questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

Java:入力した数値(1-8)に対応する行を削除する(ファイル操作)

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 157
退会済みユーザー

退会済みユーザー

 前提・実現したいこと

こんばんは、Javaについての質問です。
とあるJavaの問題を学習中に途中で分からなくなってしまい、困っています。
お手数おかけしますが、お力お貸し頂けますと幸いです。よろしくお願いいたします。
下記が問題文となります。

 問題文

数値(1-8)を入力させ、「sample.txt」の入力された行を削除する。
(1~8)以外が入力された場合はもう一度入力させること。

「sample.txt」の内容

あいうえお
かきくけこ
さしすせそ
おえういあ
12
34
56
あ1

 作成途中のソースコード

import java.io.BufferedReader;
import java.io.BufferedWriter;
import java.io.File;
import java.io.FileReader;
import java.io.FileWriter;
import java.io.IOException;

public class Insert2File {
    public static void main(String[] args) throws IOException {

    //書き込み用
    File file = new File("sample.txt");
    FileWriter filewriter = new FileWriter(file);
    BufferedWriter bw = new BufferedWriter(filewriter);

    //読み込み
    FileReader fr = new FileReader("sample.txt");
    BufferedReader br = new BufferedReader(fr);

    //一行ずつ読み込んでいく
    String line;

    System.out.println("削除する行を入力してください");


    try {

        for(int i = 1; (line = br.readLine()) != null; i++) {

        switch (i) { 
                    case 0: // あいうえお 
            bw.write(line);
            bw.newLine();
                        break; 

                    case 1: // かきくけこ 
            bw.write(line);
            bw.newLine();
                        break; 

                    case 2: // さしすせそ 
            bw.write(line);
            bw.newLine();
                        break;

           case 3: // おえういあ 
            bw.write(line);
            bw.newLine();
                        break;

           case 4: // 12
            bw.write(line);
            bw.newLine();
                        break;

           case 5: // 34 
            bw.write(line);
            bw.newLine();
                        break;

           case 6: // 56
            bw.write(line);
            bw.newLine();
                        break;

           case 7: // あ1 
            bw.write(line);
            bw.newLine();
                        break;


        }

        bw.flush();
        bw.close();

        br.close();

        }

    } catch (IOException e) {
        System.out.println(e);
}
}
}

 実行したいこと等

現状のプログラムだとエラーは起こらずコンパイルは通りますが、テキストファイルには何も効果は効いておらず、何も変化はありませんでした。
問題文のイメージからとりあえず作成しようと試みてここまでは作成できた次第です。
まだ完成までの全体のイメージは掴みかねていますが、まずはここからswich文をどう効かせれば良いのか、また再入力を求めるにはwhile文を使用するとは思うのですがどこに使用すれば良いのかが分かりません。
ご教授くださいますと大変助かります。何卒よろしくお願いいたします。

 補足情報(FW/ツールのバージョンなど)

Java
java version "1.8.0_161"

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 2

+2

テキストファイルには何も効果は効いておらず、何も変化はありませんでした。

そうでしょうか?ご質問のコードをそのまま動かすと元のテキストファイルの内容如何によらず必ず内容が全部消えてしまうと思います。

原因はプログラム開始直後にnew FileWriter(file);としているためです。これやると大抵のOSで(Linux, Macintosh, Windowsともに)元のファイルの中身は捨てられ0バイトのファイルになってしまうと思います。その同じファイルをnew FileReader(file)で開いてもそこからはもはや1行も入力できないと思います。

まずは方針を考え直したほうがよさそうです。ファイルを編集する平易な方法は

(1) 編集対象のファイルを入力用のストリーム(FileReaderなど)で開く。
(2) 編集結果を格納するファイルは別のファイル(一次ファイル)にする。
(3) 全ての編集結果を出力し終わったらファイルをクローズして元のファイルを削除し、一時ファイルを元のファイル名へrenameする。

他の方法も有り得ますが処理が難しくなります。まずは上記の平易な方法をトライしてみることをお奨めします。


なお・・・他にもおかしなところがあるので一応指摘しておきます。ただ、本題に直接関係ない(C)は本筋を解決してから取り組んだ方がよいかも知れません。一度にあれこれ考え始めると混乱するかもしれませんので。

(A) 削除対象行の入力処理漏れ
どの行を削除すべきか入力するコードが欠落しています。

(B)switch文
単純な繰り返し制御では処理しきれないような変則パターンを処理するときにこそswitch文を使うべきであって、規則的な処理の場合は使うべきでないと考えてください。Nが与えられていてN行目を削除するという仕様の場合は充分に規則的であり、switch文の出番はないと考えるべきです。
N行目を削除するには入力ファイルから全ての行を読み込み、N行目を除く全ての行を出力ファイルへコピーすればよいだけです。

(C)本題から外れますが少なくともJava7以降を前提にするならI/O処理ではtry-with-resource文を使うのが常道です。学習中と思いますので急がなくてもいいですが例外について学ぶ際にtry-with-resourceを使うべきという話を思い出していただければと思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/06/04 23:09

    ご回答いただきありがとうございます。返信が遅くなり大変申し訳ございません。
    テキストファイルにつきましては後にコンパイルしたところ、どうやら私の入力ミスだったようで、ご回答者さまのおっしゃる通り、もともと入っていたデータが全て消えてしまっていました。
    キーボードからある一定の数値を用いて入力を求める際には、if~else文かswitch文を使用するものだと考えていたので、ご指摘いただきまた勉強になりました。
    try-with-resource文はまだ例外の学習の途中のため、理解が及んでおりませんが、これからまた勉強していきたいと思います。
    まだまだ未熟なコードに質問でしたが、ご指摘にご回答いただけて大変助かりました。ありがとうございました!

    キャンセル

checkベストアンサー

+1

こんにちは、
読み込んだデータの指定した行を消す方法を以下の方法でやりました。

(i)読み込んだデータをString型の文字列に変換する.
(ii)改行刻みで文字列を分割し[]Stringに格納する。
(iii)消去したい行を入力する。(再入力はdo~while文で処理する。)
(iv)for文で入力した数字以外をループさせ新たなファイルに書き込む

質問欄のテーマには

入力した数値(1-8)に対応する行を削除する

とありますが、配列は0から始まるので0-7となっています。
1-8にしたい場合は適宜ソースコードを書き換えてください。

また、読み込むファイルや新たに作るファイルの場所の指定も重要と思われます。
私の場合はc:\\prog2\\new.txt,"c:\\prog2\\new2.txt"にしていますが、適宜変更してください。

(ちなみにswitch文は使いませんでした.....)

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

public class Main{ 

    public static void main(String[] args) throws IOException {
        String s = fileToString(new File("c:\\prog2\\new.txt"));
        String []resultArray = s.split("\n");

        for(int i = 0; i < 8; i++){
            System.out.println(i + " " + resultArray[i]);
        }
        Scanner sc = new Scanner(System.in);
        System.out.println("削除する行を入力してください");
        int n;
        do{
            n = sc.nextInt();
            if(n < 0 || 7 < n){
                System.out.println("再入力してください。");
            }
        }while(n < 0 || 7 < n );  

        try {
            // FileWriterクラスのオブジェクトを生成する
            FileWriter file = new FileWriter("c:\\prog2\\new2.txt");
            // PrintWriterクラスのオブジェクトを生成する
            PrintWriter pw = new PrintWriter(new BufferedWriter(file));

            //ファイルに書き込む
            for(int i = 0; i < 8; i++){
                if(i != n){
                    pw.println(resultArray[i]);
                }
            }

            //ファイルを閉じる
            pw.close();
        } catch (IOException e) {
            e.printStackTrace();
        }

        System.out.println("第" + n + "行が消去されました。");

    }
    public static String fileToString(File file) throws IOException { //読み込んだデータをString型に変換するメソッド
      BufferedReader br = null;
      try {

          br = new BufferedReader(new InputStreamReader(new FileInputStream(file)));
          StringBuffer sb = new StringBuffer();
          int c;

          while ((c = br.read()) != -1) {
            sb.append((char) c);
          }

          return sb.toString();
      } finally {

        br.close();
      }
   }
} 

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/06/04 23:10

    ご回答いただきありがとうございます。返信が遅くなり大変申し訳ございません。
    お教えいただいたコードを参考に、ファイルの指定や入力値の調整と修正を行い、想定していた通りの動きになりました。
    キーボードからの入力を求める際には、if~else文かswitch文を使用するものだと考えていたので、do~while文を主に使用して記述する方法もあるのだとまたひとつ勉強になりました。
    コードの解説や重要な点へのコメントも大変助かりました。ありがとうございました!

    キャンセル

  • 2018/06/08 11:03

    すみません、今更ですが 行数の8のところをresultArray.lengthにすると行数がいくらになっても対応します。
    int m = resultArray.length;を宣言し、コードの8のところをmにして、
    do~while文の7のところをm - 1にします。
    (最初lengthを使おうと思いましたがその時はlength()にしたためエラーになってしまい
    定数8を書いていました。)

    キャンセル

  • 2018/06/14 01:37

    返信が遅くなり申し訳ございません。また、ご丁寧にありがとうございます!
    ご教授いただき参考にさせていただいたコードの個人的な不明点は、あちこち調べて自己学習に励んでいたのですが、行数がいくらでも対応するようになる点は気がついておりませんでした…!
    ご指摘いただいて試してなるほどと思い、ますます勉強になりました。
    また、こちらのコードに更に要素を加え、文字を入力した場合エラーを表示して再入力を求める時はdo-while文をtry-catchで囲むのでしょうか…?
    こちらに追加でご質問して良いのか迷っていたのですが、もしよろしければお力をお貸し頂けますと幸いです。繰り返し質問してしまい恐縮ですが、何卒よろしくお願いいたします。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.62%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • Java

    13525questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。