同じkeyの数を数えるプログラムの作成

Question

前提・実現したいこと

日付ごとのユーザーのアクセス回数を集計する。

2019-10-18 user:Me
2019-10-18 user:Mother
2019-10-18 user:Me
2019-10-18 user:Father
2019-10-18 user:Father

上記のようなログからC#で日付ごとの各ユーザーのアクセス回数を集計しようとしています。
しかし、重複しているuserを数えるコードの作成ができません。
最終的に下記のような形にしたいです。
2019-10-18,Me,2
2019-10-18,Mother,1
2019-10-18,Father,2

該当のソースコード

static void main(){
   System.IO.StreamReader file =new System.IO.StreamReader(@"c:\test.txt"); //ログを読み込む
while((line = file.ReadLine()) != null)  
{  
     string date = line.Substring(0,10);  //日付を抜き出す
     string user = line.Substring(16,18);  //userを抜き出す
　　　string key = date + ',' + user; //日付とuserを一つにする
　　　
}  
}

試したこと

言い訳になってしまいますが、
C#を始めて一週間であるためここからどのようにすれば良のかが分かりません。
どなたかお力を貸していただけないでしょうか？

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

Visual Stadioを使用しています。

Comments

[BluOxy]

他の言語は触ったことがあるでしょうか。C#が始めてなら、その内容はハードルが高すぎると思います。

[kanayawa1]

ほとんどプログラミングを行ったことはありません。しかし、業務で必要であるため、C#の習得が必要となり、勉強をしています。よろしければ、回答お願いいたします。

Answer 1

BluOxyさんのおっしゃる通りですが、コード書いてみました。

    class Program {
        static void Main(string[] args) {
            //入力
            Dictionary<Log, int> Logs = new Dictionary<Log, int>();//ログを入力する入れ物
            using (System.IO.StreamReader file = new System.IO.StreamReader(@"test.txt")) {
                string line = null;
                while ((line = file.ReadLine()) != null) {
                    Log l = new Log(line);
                    if (Logs.ContainsKey(l)) {//日付、誰のパターンがすでにあるか確認
                        Logs[l]++;//すでにある場合は増やす
                    } else {
                        Logs.Add(l, 1);//ない場合は新たに追加する
                    }
                }
            }

            //出力
            string output = null;
            foreach (Log l in Logs.Keys) {
                output += l.Date + "," + l.Who + "," + Logs[l] + "\r\n";
            }
            Console.WriteLine(output.Substring(0, output.Length - 2));//最後の改行を消して出力
            Console.Read();
        }
    }
    public class Log {
        public string Date;//日付が入る
        public string Who;//Me,Mother,Fatherが入る
        public Log(string line) {
            string[] parts = line.Split(' ');
            string[] user = parts[1].Split(':');
            Date = parts[0];
            Who = user[1];
        }
        public override bool Equals(object obj) {//ContainsKeyに必要
            if (obj is Log) {
                var l = (Log)obj;
                return this.Date==l.Date && this.Who==l.Who;
            } else {
                return false;
            }
        }
        public override int GetHashCode() {//ContainsKeyに必要
            return this.GetType().ToString().Length;
        }
    }

Answer 2

前提

言い訳になってしまいますが、

C#を始めて一週間であるためここからどのようにすれば良のかが分かりません。
どなたかお力を貸していただけないでしょうか？

仰る通り、言い訳です。
C#を始めて一週間であるからといって、考えるのを止めて良い理由にはなりません。
そこを直さないと初心者からは抜けられないことを認識してください。

回答

実現したいことの難易度やkanayawa1さんのスキルを踏まえると、プログラムの書き方を先に学んだ方が良いです。

プログラムの書き方

どのようにすれば良のかが分かりません。

kanayawa1さんの実現したいことをそのままプログラムにすれば良いのです。
この質問の根本問題はその方法が分からないことです。

そもそも「プログラム」とはなんでしょうか。
プログラムには何事も インプット(入力) があって、 アウトプット(出力) があります。
そして、インプットからアウトプットに変換する処理 があります。

日付ごとのユーザーのアクセス回数を集計する

kanayawa1さんがそれを作りたいというのであれば、まずは インプット と アウトプット が何かを洗い出してください。

今回の場合は下記がインプットで

2019-10-18 user:Me
2019-10-18 user:Mother
2019-10-18 user:Me
2019-10-18 user:Father
2019-10-18 user:Father

下記がアウトプットですね。

2019-10-18,Me,2
2019-10-18,Mother,1
2019-10-18,Father,2

もっと問題を小さくするならば、2019-10-18 user:Meの入力したとき2019-10-18,Me,1と出力すれば要件を満たしているのではないでしょうか。

ここまではコーディング一切無しで問題解決に必要な情報だけを考えてきました。
インプットとアウトプットが固まってきたら、それを実現する手段である C# と絡めていきます。

C# ベースでインプット と アウトプットを考える

C# では型の概念が存在します。C# で処理を作りたければ、インプットが何の型でアウトプットが何の型かを常に考える必要があります。

インプットの型は文字列なので1行ずつ処理する場合は string です。数行の文字列を処理する場合はstringではなくstringを要素に持つコレクションが良いでしょう。

アウトプットは複数行でかつ、日付とユーザ、アクセス回数を識別する必要があるため、オブジェクト型のコレクションもしくはKeyが日付とユーザ名、Valueがアクセス回数のDictionaryが良いでしょう。
※コレクション、Dictionaryについては後述

では、C#においてインプットとアウトプットはどこに実装するのでしょうか。

C#でコーディングする際はまずクラスを定義してからメソッドを定義します。
そして、メソッドに実処理を記述することができます。

この内、C#における インプット とは メソッドの引数 (もしくはクラス内のメンバー)のことで
C#における アウトプット とは メソッドの戻り値 (もしくはクラス内のメンバー)のことです。

なぜクラスをまたいでメソッドを定義しなければならないのか。それは C# がオブジェクト指向言語だからです。
ここは、話がそれるので詳しい説明は割愛します。

コレクションって？

今回の場合では、文字列を切ったり繋いだりするだけでも頑張れば実現できるかもしれません。
しかし、コレクションを覚えて使うことで文字列を気合で処理するよりかは非常に楽に作れます。
詳しくはリンク先のページを見たり、「C# コレクション」と検索してください。

さらに書くと、KeyとValueにひも付けて保持するコレクションであるDictionaryクラスを使う方法が1番簡単だと思います。

ここまで前提の説明をして、ようやくC#に関係のある回答

ログからC#で日付ごとの各ユーザーのアクセス回数を集計しようとしています。

しかし、重複しているuserを数えるコードの作成ができません。

メソッドを定義して下記実装を行えば良いのです。

1. 取得した日付とユーザーをKeyに、アクセス回数をValueに指定したDictionaryオブジェクトを生成
2. テキストの1行から「日付」と「ユーザー」を取得
3. Dictionaryに「日付」と「ユーザー」に対応するKeyが存在するならば、対応するValueをインクリメントする
4. Dictionaryの各要素をループで取得し、各要素のKeyとValueを表示

Dictionaryを使う方法が恐らく1番簡単なので、その方向性で解説しました。

最後に

コードの書き方とC#の基礎について、最低限のことを書きました。
あとは内容さえ理解できれば実践できるはずです。

正直な話、上記の内容がC#を始めて1週間のkanayawa1さんにとってハードルが高いであろうことは承知の上ですが、そういう問題に手をつけているという事は理解してください。

難しいようであれば、一度この問題を解くのはやめて、もっと簡単な物から作ることをお勧めします。
業務であれば、上司に「難易度が高い」とアラートを投げてください。

余談

LINQを覚えることでさらにスマートに処理が実現できます。（今回の場合だとGroupByメソッドが使える）
しかし、コレクションを理解した前提の機能なので、学習コストは高いです。紹介だけに留めます。

Comments

[kanayawa1]

無事、完成させることができました。ありがとうございます。少しずつでも、C#の勉強を進めていきたいと思います。