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

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

ただいまの
回答率

90.53%

  • Java

    13740questions

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

  • Eclipse

    1655questions

    Eclipseは、IBM社で開発された統合開発環境のひとつです。2001年11月にオープンソース化されました。 たくさんのプラグインがあり自由に機能を追加をすることができるため、開発ツールにおける共通プラットフォームとして位置づけられています。 Eclipse自体は、Javaで実装されています。

  • CSV

    631questions

    CSV(Comma-Separated Values)はコンマで区切られた明白なテキスト値のリストです。もしくは、そのフォーマットでひとつ以上のリストを含むファイルを指します。

  • ファイル

    170questions

    ファイルとは、文字列に基づいた名前又はパスからアクセスすることができる、任意の情報のブロック又は情報を格納するためのリソースです。

Javaのオブジェクト指向でコンストラクタを用いてcsvファイルを読み書きすると共に辞書を使う方法

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 313

tenjin

score 212

 前提・実現したいこと

ここに質問の内容を詳しく書いてください。
以下の参考資料に記載されているコードを元に
Javaのオブジェクト指向で書かれたプログラムで
csvファイルを読み込み形式を変更した上で新たなcsvファイルを書き出そうとしています。

以下excelですが、csv形式にしたファイルを読み書きします。
イメージ説明

参考資料

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

現状のコード(CarクラスとCSVInandOutクラス)では、csvファイルの読み書きは問題なく行われます。
これに加え、車種の辞書型データ(JavaではHashMap)に関する実装をCSVInandOutクラスに加えようとしています。
csvで読み込んだ車種(catname)が辞書になければ、カテゴリーのインスタンスが作成され、
辞書内に同じ車種(Key)があればそのKey値である個数(num)を1増やすということを
CSVInandOutクラスの中で行おうとしています。

これらを実現するために、現状のコードをどのように組み合わせたら良いか
教えていただきたいです。

 該当のソースコード

Carクラス

public class Car {
    String name; //車名
    String company; //メーカー
    String catname; //車種

  //Carクラスに追加したコンストラクタ
    Car(String name, String company, String catname){
        System.out.println("Carクラスのコンストラクタが呼びだされました");
        this.name = name;
        this.company = company;
        this.catname = catname;
    }


}

CSVInandOutクラス(csvの入出力を行う)

import java.io.BufferedReader;
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.StringTokenizer;

public class CSVInandOut {
    public static void main(String[] args) {
        List<Car> carList = new ArrayList<Car>(); //データ格納用リスト
        String input = "carIn.csv";
        String output = "carOut.csv";

        try {
            BufferedReader InputLine = new BufferedReader(new FileReader(input));
            PrintWriter outputLine = new PrintWriter(new FileWriter(output));
            String line;

            line = InputLine.readLine(); //ヘッダの読み書き
            outputLine.println(line);

            //1行ずつ読み込み
            while ((line = InputLine.readLine()) != null) { //String変数lineに格納
                StringTokenizer st = new StringTokenizer(line); //lineを分割の対象にする

                //フィールドの設定を行う
                String name = st.nextToken(",");
                String company = st.nextToken(",");
                String catname = st.nextToken(",");
                carList.add(new Car(name, company, catname)); //リストに追加する
            }

            //carListの中身を順に出力する
            for (Car car : carList) {
                outputLine.print(car.name + ",");
                outputLine.print(car.company+ ",");
                outputLine.print(car.catname);
                outputLine.println();
            }

            //開いていた出入力ファイルを閉じる
            InputLine.close();
            outputLine.close();

        }
        catch (IOException e) {
            System.out.println("ファイルが開けません");
        }
    }
}

Categoryクラス

import java.util.HashMap;

public class Category {
    String catname; //カテゴリー名
    int num;
    HashMap<String, Integer> car_map = new HashMap<String, Integer>();

    //Categoryクラスに追加したコンストラクタ
    Category(String catname, int num) {
        System.out.println("Categoryクラスのコンストラクタが呼びだされました");
        this.catname = catname;
        this.num = num;
    }

    //更新・追加
    void add(String catname) {
        //すでにカテゴリー名が存在すれば個数を1増やす
        if (car_map.containsKey(catname)) {
            increaseNum(catname, 1);
        } else {
            //新たなカテゴリーならkeyと値を追加
            car_map.put(catname, 1);
        }
        System.out.println("[キー:トラック]:" + car_map.get(catname));
    }

    void increaseNum(String car, int num) {
        int new_num = this.car_map.get(car) + num;
        this.car_map.put(car, num); //値を更新したものに置換
    }
}


CategoryWorkクラス(Categoryクラスの動作確認のために作成)

public class CategoryWork {

    public static void main(String[] args) {
        Category category = new Category("トラック",1);
        category.add("トラック");
    }
}

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

Eclipse Java EE IDE for Web Developers.
Version: Oxygen.3a Release (4.7.3a)

Microsoft Excel for Mac バージョン16.13

Javaバージョン(java.version):1.8.0_152

オペレーティングシステム名(os.name):Mac OS X

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

質問への追記・修正、ベストアンサー選択の依頼

  • swordone

    2018/05/26 23:03

    肝心のJavaのバージョンは?

    キャンセル

回答 2

checkベストアンサー

+1

Categoryクラスがほとんど意味をなしていないように見えます。「カテゴリ名-数」のつながりを作りたいのであればMapだけで十分で、そもそもCategoryクラスがカテゴリ名とMapを両方持っている意味を感じません。

どうしてもカテゴリ名-数を持つCategoryクラスが必要ならば、Categoryクラスはこうして、

public class Category {
    String catname; //カテゴリー名
    int num = 0;
    // HashMap<String, Integer> car_map = new HashMap<String, Integer>(); 不要

    //Categoryクラスに追加したコンストラクタ(int引数は削除)
    Category(String catname) {
        System.out.println("Categoryクラスのコンストラクタが呼びだされました");
        this.catname = catname;
    }

    /*このメソッドがまるっきり不要
    //更新・追加
    void add(String catname) {
        //すでにカテゴリー名が存在すれば個数を1増やす
        if (car_map.containsKey(catname)) {
            increaseNum(catname, 1);
        } else {
            //新たなカテゴリーならkeyと値を追加
            car_map.put(catname, 1);
        }
        System.out.println("[キー:トラック]:" + car_map.get(catname));
    }


    void increaseNum(String car, int num) {
        int new_num = this.car_map.get(car) + num;
        this.car_map.put(car, num); //値を更新したものに置換
    }
    */

    void increment() {
        num++;
    }
}


こういう使い方になるのでは?

Map<String, Category> map = new HashMap<>();
map.computeIfAbsent(catName, Category::new).increment();

私が作るなら、クラスはCarだけで十分で、まあ実行用クラスとしてCSVInandOutは作るとして、こうなります。

public class Car {
    final String name; //車名
    final String company; //メーカー
    final String catname; //車種

  //Carクラスに追加したコンストラクタ
    Car(String name, String company, String catname){
        System.out.println("Carクラスのコンストラクタが呼びだされました");
        this.name = name;
        this.company = company;
        this.catname = catname;
    }

    @Override
    public String toString() {
        return name + "," + company + "," + catname;
    }
}
// import省略
public class CSVInandOut {
    public static void main(String[] args) {
        List<Car> carList = new ArrayList<Car>(); //データ格納用リスト
        Map<String, Integer> cat_num = new HashMap<>();
        String input = "carIn.csv";
        String output = "carOut.csv";

        try(BufferedReader InputLine = new BufferedReader(new FileReader(input));
            PrintWriter outputLine = new PrintWriter(new FileWriter(output))) {

            String line;

            line = InputLine.readLine(); //ヘッダの読み書き
            outputLine.println(line);

            //1行ずつ読み込み
            while ((line = InputLine.readLine()) != null) { //String変数lineに格納
                String[] st = line.split(","); //lineを分割の対象にする
                Car car = new Car(st[0], st[1], st[2]);
                carList.add(car); //リストに追加する
                cat_num.marge(car.catname, 1, Integer::sum);
                outputLine.println(car);
            }

        }
        catch (IOException e) {
            System.out.println("ファイルが開けません");
        }
    }
}

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/05/27 08:24

    ご回答いただきましてありがとうございます。
    ご回答を受けて伺いたいことがございます。
    Map<String, Category> map = new HashMap<>();
    map.computeIfAbsent(catName, Category::new).increment();
    はカテゴリークラスの
    HashMap<String, Integer> car_map = new HashMap<String, Integer>();
    があった場所に置くという理解で正しいでしょうか。

    キャンセル

  • 2018/05/27 08:51

    いえ、そもそもMapをCategoryクラスのインスタンスが持つ必要がないです。
    カテゴリ名-数は「カテゴリ名」が持つ情報ではないので。
    CategoryクラスやMapをどう使うのかわかりませんが、CSVInandOutクラスでローカル変数として使うか、せめてCategoryクラスのstatic変数でしょうね。

    キャンセル

  • 2018/05/27 13:36

    お返事いただきましてありがとうございます。
    「カテゴリ名-数は「カテゴリ名」が持つ情報ではない」という所が理解できておらず、
    また全てのクラスの構成がどのように変化するのか全体像が掴めておりません。
    可能であれば、Categoryクラス以外のクラスもどのように修正するべきかコードを回答欄に載せていただけますと大変助かります。

    キャンセル

  • 2018/05/27 19:49

    ご丁寧にご説明いただきましてありがとうございました。

    キャンセル

  • 2018/05/27 20:02

    度々失礼しますが、CSVInandOutクラスで
    Map<String, Integer> cat_num = new HashMap<>();
    HashMap<> の型引数を推論できません

    cat_num.marge(car.catname, 1, Integer::sum);
    メソッド marge(String, int, Integer::sum) は型 MappingChange.Map<String,Integer> で未定義
    です

    というエラーメッセージが表示されるのは何が原因なのでしょうか

    キャンセル

  • 2018/05/27 21:25

    Mapのimport宣言を確認してください。
    import java.util.Map;
    が正しいです。

    キャンセル

+1

import java.io.BufferedReader;
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.StringTokenizer;

public class CSVInandOut {
    public static void main(String[] args) {
        List<Car> carList = new ArrayList<Car>(); //データ格納用リスト
        String input = "carIn.csv";
        String output = "carOut.csv";
        Category category = new Category(); // この行を追加

        try {
            BufferedReader InputLine = new BufferedReader(new FileReader(input));
            PrintWriter outputLine = new PrintWriter(new FileWriter(output));
            String line;

            line = InputLine.readLine(); //ヘッダの読み書き
            outputLine.println(line);

            //1行ずつ読み込み
            while ((line = InputLine.readLine()) != null) { //String変数lineに格納
                StringTokenizer st = new StringTokenizer(line); //lineを分割の対象にする

                //フィールドの設定を行う
                String name = st.nextToken(",");
                String company = st.nextToken(",");
                String catname = st.nextToken(",");
                carList.add(new Car(name, company, catname)); //リストに追加する
                category.add( catname ); // この行を追加
            }

            //carListの中身を順に出力する
            for (Car car : carList) {
                outputLine.print(car.name + ",");
                outputLine.print(car.company+ ",");
                outputLine.print(car.catname);
                outputLine.println();
            }

            //開いていた出入力ファイルを閉じる
            InputLine.close();
            outputLine.close();

        }
        catch (IOException e) {
            System.out.println("ファイルが開けません");
        }
    }
}

これでどうでしょうか?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/05/26 21:36

    ご回答いただきましてありがとうございます。
    Category category = new Category(); // この行を追加
    この行が追加されることで、CarクラスとCategoryクラスがどのように
    つながるのか理解できていません。

    Carクラスの catnameとCategoryクラスのcatnameが一致しているかどうか
    判断するときにCarクラスの catnameを引数として渡したりする必要はないのでしょうか。

    キャンセル

  • 2018/05/27 13:26

    ご回答いただいたように、
    Category category = new Category();
    を追加すると引数を追加してCategory(String, int)と一致させるという
    警告がeclipseで表示されました。

    キャンセル

  • 2018/05/27 17:46

    > 引数を追加してCategory(String, int)と一致させるという
    あぁ、すいません。Categoryクラスのコンストラクタをちゃんと読んでいませんでした。
    swordoneさんの回答のほうが良さそうなので、そちらを参考にされたほうがいいですね。

    キャンセル

  • 2018/05/27 19:50

    rtr1950xさんのご回答を読んで学ばせていただいたことも多くありました。
    この度はご回答いただきましてありがとうございました。

    キャンセル

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

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

関連した質問

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

  • Java

    13740questions

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

  • Eclipse

    1655questions

    Eclipseは、IBM社で開発された統合開発環境のひとつです。2001年11月にオープンソース化されました。 たくさんのプラグインがあり自由に機能を追加をすることができるため、開発ツールにおける共通プラットフォームとして位置づけられています。 Eclipse自体は、Javaで実装されています。

  • CSV

    631questions

    CSV(Comma-Separated Values)はコンマで区切られた明白なテキスト値のリストです。もしくは、そのフォーマットでひとつ以上のリストを含むファイルを指します。

  • ファイル

    170questions

    ファイルとは、文字列に基づいた名前又はパスからアクセスすることができる、任意の情報のブロック又は情報を格納するためのリソースです。

  • トップ
  • Javaに関する質問
  • Javaのオブジェクト指向でコンストラクタを用いてcsvファイルを読み書きすると共に辞書を使う方法