ファイルからデータを表示して、そのデータを別のファイルに書き込みたい

Question

前提

質問するのはもちろんのことプログラム自体初心者ですので、おかしな点が多々あると思いますがよろしくお願いします。

実現したいこと

商品コードを入力させる
個数を入力させる
入力された商品コードの商品と個数分の金額を”McdDate”と”McdDate４”から表示する
表示したものを"McdDate２に書き込む"

発生している問題・エラーメッセージ

	個数Exception in thread "main" java.lang.NumberFormatException: For input string: "[2, 4]"
	at java.base/java.lang.NumberFormatException.forInputString(NumberFormatException.java:67)
	at java.base/java.lang.Integer.parseInt(Integer.java:668)
	at java.base/java.lang.Integer.parseInt(Integer.java:786)
	at McdSaleInput.main(McdSaleInput.java:78)
実行するとこういったエラーメッセージが表示されました

該当のソースコード

import java.io.BufferedReader;
import java.io.File;
import java.io.FileReader;
import java.io.FileWriter;
import java.io.IOException;
import java.io.PrintWriter;
import java.util.ArrayList;
import java.util.List;
import java.util.Scanner;

public class McdSaleInput {
  public static void main(String[] args) {
    String inFilename = "McdDate.txt";
    String r = "McdDate4.txt";
    String outFilename = "McdDate2.txt";
    
    System.out.println("-------------------------------");
    System.out.println("　　　　　商品コード　　　　　");
    System.out.println("-------------------------------");
    try {
        
        File file = new File("McdDate3.txt");
     
        FileReader fileReader = new FileReader(file);
        int data;
        while ((data = fileReader.read()) != -1) {
            System.out.print((char) data );
        }
     
        fileReader.close();
     
    } catch (IOException e) {
        e.printStackTrace();
    }
    System.out.println("-----------------------------------------");
    System.out.println("【販売する商品コードを入力してください】");
    System.out.println("【終了する場合は0を押してください】");
    List<Integer> nums = new ArrayList<>();
    List<Integer> abc=new ArrayList<>();
    try(Scanner scanner = new Scanner(System.in);) {
      int v;
      int y;
      while(true) {
         v = scanner.nextInt();
         System.out.print("	個数");
         if(v <= 0) break;
         y= scanner.nextInt();
        
        
        nums.add(v);
     abc.add(y);
     
 
      }
    }
    
    if(nums.size() == 0) {
      System.out.print("エラーです");
      System.exit(0);
    }

    
    
            
     
    try(BufferedReader br = new BufferedReader(new FileReader(inFilename));
    	BufferedReader m = new BufferedReader(new FileReader(r));
        PrintWriter u = new PrintWriter(new FileWriter(outFilename));
    	PrintWriter pw = new PrintWriter(new FileWriter(outFilename));) {
    	 String line;
    	 String d;
    	
         for(int i=1; (line=br.readLine()) != null; i++)
        	 for(int j=1; (d=m.readLine()) != null; j++){
           if(nums.contains(i))
        	   if(nums.contains(j)){
        	int num = Integer.parseInt(d);
        	int w = Integer.parseInt(abc.toString());
        	int nh = num * w;
             System.out.println(line+nh );
             
             pw.println(line);
            
        }
      }
         
      System.out.print("を販売しました");
    } catch(IOException e) {
      e.printStackTrace();
      System.exit(1);
      
    }
  }
}

試したこと

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

【McdDate.txt】ビックマック
てりやきマックバーガー
チキンフィレオ
エビフィレオ
月見バーガー(期間限定)
チキンマックナゲット5個
ハッシュポテト
マックフライポテト(S)
マックフライポテト(M)
マックフライポテト(L)
コカ・コーラ(S)
コカ・コーラ(M)
コカ・コーラ(L)
ファンタメロン(M)
Qooすっきり白ブドウ(M)
アイスティ(M)
プレミアムローストコーヒー(S)
【McdDate４.txt】
410
370
380
400
360
200
140
160
290
240
100
220
250
220
220
220
100

Comments

[jimbe]

エラーメッセージに、問題の内容、発生個所等が書かれています。

>商品コードを入力させる
>個数を入力させる

お作りになったプログラムがどんな入力に対応しているのかは、作った本人が一番分かっていなければなりません。
scanner.nextInt() は、 "[2, 4]" という入力は処理できません。

[jimbe]

ご質問で言及されていない McdDate3.txt をいうファイルをコードが扱っていますが、これは何でしょうか。

[abc0507]

【McdDate3.txt】の中身は
ビックマック,410円は1
てりやきマックバーガー,370円は2
チキンフィレオ,380円は3
エビフィレオ,400円は4
月見バーガー(期間限定),360円は5
チキンマックナゲット5個,200円は6
ハッシュポテト,140は7
マックフライポテト(S),160円は8
マックフライポテト(M),290円は9
マックフライポテト(L),240円は10
コカ・コーラ(S),100円は11
コカ・コーラ(M),220円は12
コカ・コーラ(L),250円は13
ファンタメロン(M),220円は14
Qooすっきり白ブドウ(M),220円は15
アイスティ(M),220円は16
プレミアムローストコーヒー(S),100円は17
プレミアムローストコーヒー(M),150円は18
で商品コードの入力の際にわかりやすいようにと思い作成したものです。

[abc0507]

自分なりのエラーメッセージの解釈なのですが、scanner.nextInt() は"[2, 4]" を処理できないため
67行目のBufferedReader m = new BufferedReader(new FileReader(r));と
78行目のint w = Integer.parseInt(abc.toString());にエラーが生じた。
scanner.nextInt() を"[2, 4]" に対応して処理できる形に修正すれば実行できますでしょうか。

[jimbe]

>【McdDate3.txt】の中身は

商品名は McdDate.txt に入っていますし、 単価は McdDate4.txt に入っているのですよね。
その上で McdDate3.txt にまた商品名や単価を書いているのですか。
それでは商品名が変わったり新製品の追加、既存商品の廃止、価格改定の際は全てのファイルを同じに修正しなければなりません。
しかしそれでは間違いが入る可能性が増えます。現に "プレミアムローストコーヒー(M)" は McdDate.txt にありませんね?
(計算用の単価ファイルの書き換えで1行位置を間違えただけで後で大変なことになります。)
そのような場合はコード・商品名・単価を CSV 等の一行として1つのファイルにし、表示にも計算等にもそれのデータを(必要に応じてフォーマットして)用いるのが常です。

なぜ "[2, 4]" と入力したのでしょう。
仕様として "[2, 4]" と入力しなければならないと決まっているのでしょうか。

[abc0507]

なるほど、こんなにファイルを作成する必要はないんですね。了解しました。

このプログラムを実行して個数を入力する時になぜ[2,4]と入力したかと言うことですか。

[episteme]

> 【McdDate3.txt】の中身は
ビックマック,410円は1
てりやきマックバーガー,370円は2
チキンフィレオ,380円は3
エビフィレオ,400円は4

たとえば チキンフィレオ を4つ欲しいなら
3 4 と入力するん?　
[3,4] と入力するん?
それともほかのなにか?

なんにせよ、その入力から 二つのint値: 3 と 4 を取り出さにゃならんけど、
その部分はどこ? それは正しい?

[jimbe]

>なぜ[2,4]と入力したか

scanner.nextInt は、スペースやタブを区切りとする数字列の入力を数値に変換します。従って、まず数字で無ければ例外を発しますし、"," 区切りのつもりとして入力しても同様です。
とこかの他のシステムで「2つの値を入力するときは "[2,4]" のようにする」という経験があって、java でも同じだろうとされたのであれば、残念ながらそうではありません、ということになります。
逆にこのプログラムはコード・個数の入力は「"[2,4]"のように"[]"で囲み","で区切って入力することに決まっている」ということであれば、 scanner.nextInt では無い他の方法で入力を解析しなければなりません。

Answer 1

McdDate.txt/McdDate3.txt/McdDate4.txt に散らばるデータを McdMaster.csv に纏めました。

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

public class McdSaleInput {
  public static void main(String[] args) throws IOException {
    Map<Integer,Master> masterMap = readMaster("McdMaster.csv");

    printProduct(masterMap);

    List<Input> inputList = 入力();
    if(inputList.size() == 0) {
      System.out.print("エラーです");
      System.exit(0);
    }

    出力(masterMap, inputList, "McdData2.txt");
  }

  private static void printProduct(Map<Integer,Master> masterMap) {
    System.out.println("-------------------------------");
    System.out.println("　　　　　商品コード　　　　　");
    System.out.println("-------------------------------");
    for(Map.Entry<Integer,Master> entry : masterMap.entrySet()) {
      System.out.println(entry.getValue().toString());
    }
  }

  private static class Master {
    final int code;
    final String name;
    final int unitprice;
    Master(int code, String name, int unitprice) {
      this.code = code;
      this.name = name;
      this.unitprice = unitprice;
    }
    @Override
    public String toString() {
      return name + "," + unitprice + "円は" + code;
    }
  }

  private static Map<Integer,Master> readMaster(String filename) throws IOException {
    Map<Integer,Master> masterMap = new TreeMap<Integer,Master>();

    try(BufferedReader reader = new BufferedReader(new FileReader(filename))) {
      for(String line; (line = reader.readLine()) != null; ) {
        String[] tokens = line.split(",");
        if(tokens.length != 3) throw new IOException(filename + " format error: line=" + line);
        try {
          int code = Integer.parseInt(tokens[0]);
          String name = tokens[1];
          int unitprice = Integer.parseInt(tokens[2]);
          masterMap.put(code, new Master(code, name, unitprice));
        } catch(NumberFormatException e) {
          throw new IOException(filename + " format error: line=" + line, e);
        }
      }
    }

    return masterMap;
  }

  private static class Input {
    final int code, number;
    Input(int code, int number) {
      this.code = code;
      this.number = number;
    }
  }

  private static List<Input> 入力() {
    System.out.println("-----------------------------------------");
    System.out.println("【販売する商品コードと個数を入力してください】");
    System.out.println("【終了する場合は0を入力してください】");

    List<Input> inputList = new ArrayList<>();

    try(Scanner scanner = new Scanner(System.in)) {
      while(true) {
        System.out.print("コード: ");
        int code = scanner.nextInt();
        if(code <= 0) break;

        System.out.print("個数: ");
        int number = scanner.nextInt();

        inputList.add(new Input(code, number));
      }
    }

    return inputList;
  }

  private static void 出力(Map<Integer,Master> masterMap, List<Input> inputList, String filename) throws IOException {
    try(PrintWriter writer = new PrintWriter(new FileWriter(filename))) {
      for(Input input : inputList) {
        Master master = masterMap.get(input.code);
        StringBuilder sb = new StringBuilder()
          .append(master.name).append(" ")
          .append(input.number).append("個 ")
          .append(master.unitprice * input.number).append("円");
        System.out.println(sb.toString());
        writer.println(sb.toString());
      }
    }
    System.out.print("を販売しました");
  }
}

McdMaster.csv

1,ビックマック,410
2,てりやきマックバーガー,370
3,チキンフィレオ,380
4,エビフィレオ,400
5,月見バーガー(期間限定),360
6,チキンマックナゲット5個,200
7,ハッシュポテト,140
8,マックフライポテト(S),160
9,マックフライポテト(M),290
10,マックフライポテト(L),240
11,コカ・コーラ(S),100
12,コカ・コーラ(M),220
13,コカ・コーラ(L),250
14,ファンタメロン(M),220
15,Qooすっきり白ブドウ(M),220
16,アイスティ(M),220
17,プレミアムローストコーヒー(S),100
18,プレミアムローストコーヒー(M),150

実行結果

-------------------------------
　　　　　商品コード　　　　　
-------------------------------
ビックマック,410円は1
てりやきマックバーガー,370円は2
チキンフィレオ,380円は3
エビフィレオ,400円は4
月見バーガー(期間限定),360円は5
チキンマックナゲット5個,200円は6
ハッシュポテト,140円は7
マックフライポテト(S),160円は8
マックフライポテト(M),290円は9
マックフライポテト(L),240円は10
コカ・コーラ(S),100円は11
コカ・コーラ(M),220円は12
コカ・コーラ(L),250円は13
ファンタメロン(M),220円は14
Qooすっきり白ブドウ(M),220円は15
アイスティ(M),220円は16
プレミアムローストコーヒー(S),100円は17
プレミアムローストコーヒー(M),150円は18
-----------------------------------------
【販売する商品コードと個数を入力してください】
【終了する場合は0を入力してください】
コード: 2
個数: 2
コード: 8
個数: 2
コード: 16
個数: 1
コード: 18
個数: 1
コード: 0
てりやきマックバーガー 2個 740円
マックフライポテト(S) 2個 320円
アイスティ(M) 1個 220円
プレミアムローストコーヒー(M) 1個 150円
を販売しました

実行後 McdData2.txt

てりやきマックバーガー 2個 740円
マックフライポテト(S) 2個 320円
アイスティ(M) 1個 220円
プレミアムローストコーヒー(M) 1個 150円