編集履歴

回答編集履歴

文章の修正

2020/06/01 16:20

投稿

スコア2663

test CHANGED Viewed

@@ -142,13 +142,13 @@
 今回の場合では、文字列を切ったり繋いだりするだけでも頑張れば実現できるかもしれません。
-しかし、[コレクション](https://docs.microsoft.com/ja-jp/dotnet/csharp/programming-guide/concepts/collections)を覚えて使うことで大分楽に作れます。
+しかし、[コレクション](https://docs.microsoft.com/ja-jp/dotnet/csharp/programming-guide/concepts/collections)を覚えて使うことで文字列を気合で処理するよりかは非常に楽に作れます。
 詳しくはリンク先のページを見たり、「C# コレクション」と検索してください。
-さらに詳しく書くと、日付とユーザー名をキーにしてカウントができる`Dictionary`クラスを使う方法が1番簡単だと思います。
+さらに書くと、KeyとValueにひも付けて保持するコレクションである`Dictionary`クラスを使う方法が1番簡単だと思います。

文章の修正

2020/06/01 16:20

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -86,6 +86,10 @@
+もっと問題を小さくするならば、`2019-10-18 user:Me`の入力したとき`2019-10-18,Me,1`と出力すれば要件を満たしているのではないでしょうか。
 ここまではコーディング一切無しで問題解決に必要な情報だけを考えてきました。
 インプットとアウトプットが固まってきたら、それを実現する手段である C# と絡めていきます。
@@ -100,7 +104,13 @@
-インプットの型は文字列なのでC#では `string` です。…と書きたいところですが、数行のテキストなので、後述するコレクションで管理するのが良いです。アウトプットもインプットと同様です。
+インプットの型は文字列なので1行ずつ処理する場合は `string` です。数行の文字列を処理する場合は`string`ではなく`string`を要素に持つコレクションが良いでしょう。
+アウトプットは複数行でかつ、日付とユーザ、アクセス回数を識別する必要があるため、オブジェクト型のコレクションもしくはKeyが日付とユーザ名、Valueがアクセス回数の`Dictionary`が良いでしょう。
+※コレクション、`Dictionary`については後述
@@ -152,7 +162,7 @@
-メソッドを定義して、下記実装を行えば良いのです。
+メソッドを定義して下記実装を行えば良いのです。
@@ -160,12 +170,16 @@
 0. テキストの1行から「日付」と「ユーザー」を取得
-0. `Dictionary`に「日付」と「ユーザー」に対応する要素が存在するならば、そのValueをインクリメントする
+0. `Dictionary`に「日付」と「ユーザー」に対応するKeyが存在するならば、対応するValueをインクリメントする
 0. `Dictionary`の各要素をループで取得し、各要素のKeyとValueを表示
+`Dictionary`を使う方法が恐らく1番簡単なので、その方向性で解説しました。
 ### 最後に
@@ -176,7 +190,7 @@
-正直な話、C#を始めて1週間のkanayawa1さんにハードルが高いことは承知の上ですが、そういう問題に手をつけているという事は理解してください。
+正直な話、上記の内容がC#を始めて1週間のkanayawa1さんにとってハードルが高いであろうことは承知の上ですが、そういう問題に手をつけているという事は理解してください。

文章の修正

2020/06/01 16:16

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -152,6 +152,10 @@
+メソッドを定義して、下記実装を行えば良いのです。
 0. 取得した日付とユーザーをKeyに、アクセス回数をValueに指定した`Dictionary`オブジェクトを生成
 0. テキストの1行から「日付」と「ユーザー」を取得

文章の修正

2020/06/01 15:54

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -172,11 +172,13 @@
-正直な話、1週間のkanayawa1さんにハードルが高いことは承知の上ですが、そういう問題に手をつけているという事は理解してください。
+正直な話、C#を始めて1週間のkanayawa1さんにハードルが高いことは承知の上ですが、そういう問題に手をつけているという事は理解してください。
 難しいようであれば、一度この問題を解くのはやめて、もっと簡単な物から作ることをお勧めします。
+業務であれば、上司に「難易度が高い」とアラートを投げてください。

文章の修正

2020/06/01 15:52

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -134,6 +134,8 @@
 しかし、[コレクション](https://docs.microsoft.com/ja-jp/dotnet/csharp/programming-guide/concepts/collections)を覚えて使うことで大分楽に作れます。
+詳しくはリンク先のページを見たり、「C# コレクション」と検索してください。
 さらに詳しく書くと、日付とユーザー名をキーにしてカウントができる`Dictionary`クラスを使う方法が1番簡単だと思います。

Dictionaryを使う方法に回答内容を変更

2020/06/01 15:48

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -136,13 +136,7 @@
-コレクションの各要素に下記を保持するオブジェクトを保持し、見つかった日付＆ユーザー数だけカウントをしていく方法です。`Dictionary`クラスなどキーを持つものなら尚簡単に作れるかと思います。
+さらに詳しく書くと、日付とユーザー名をキーにしてカウントができる`Dictionary`クラスを使う方法が1番簡単だと思います。
-- 日付
-- ユーザー
-- アクセス回数
@@ -150,27 +144,19 @@
-テキストの1行から「日付」と「ユーザー」を取得し、取得した日付とユーザーを元にオブジェクトを作成し、作成したオブジェクトをコレクションで管理し、コレクションの各要素をループで取得してオブジェクトが持つ日付とユーザーを元にアクセス回数を計算し、同じ日付とユーザーが何回出たかを表示すれば良いのです。
+> ログからC#で日付ごとの各ユーザーのアクセス回数を集計しようとしています。
+しかし、重複しているuserを数えるコードの作成ができません。
-> 取得した日付とユーザーを元にオブジェクトを作成し
+0. 取得した日付とユーザーをKeyに、アクセス回数をValueに指定した`Dictionary`オブジェクトを生成
+0. テキストの1行から「日付」と「ユーザー」を取得
+0. `Dictionary`に「日付」と「ユーザー」に対応する要素が存在するならば、そのValueをインクリメントする
-例えば、日付とユーザーをメンバーに持った`UserLog`クラスを定義して、そのオブジェクトを生成するようなイメージです。
-```C#
-public class UserLog{
-    public DateTime Date { get; set; }
+0. `Dictionary`の各要素をループで取得し、各要素のKeyとValueを表示
-    public string Name { get; set; }
-}
-```

文章の修正

2020/06/01 15:47

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -100,9 +100,7 @@
-インプットの型は文字列なのでC#では `string` です。後述するコレクションでもありです。
+インプットの型は文字列なのでC#では `string` です。…と書きたいところですが、数行のテキストなので、後述するコレクションで管理するのが良いです。アウトプットもインプットと同様です。
-アウトプットもインプットと同様です。
@@ -110,11 +108,13 @@
-C#でコーディングする際はまずクラスを定義して、メソッドを定義します。
+C#でコーディングする際はまずクラスを定義してからメソッドを定義します。
-メソッドに実処理を記述することができます。この内、
+そして、メソッドに実処理を記述することができます。
-C#における インプット とは **メソッドの引数** (もしくはクラス内のメンバー)のことで
+この内、C#における インプット とは **メソッドの引数** (もしくはクラス内のメンバー)のことで
 C#における アウトプット とは **メソッドの戻り値** (もしくはクラス内のメンバー)のことです。
@@ -146,7 +146,7 @@
-### ここまで前提の説明をして、C#に関係のある回答
+### ここまで前提の説明をして、ようやくC#に関係のある回答

文章の修正

2020/06/01 15:36

投稿

BluOxy

スコア2663

test CHANGED Viewed

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

回答がなかったので追加

2020/06/01 15:31

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -38,15 +38,11 @@
-#### そもそも「プログラム」とは
+そもそも「プログラム」とはなんでしょうか。
 プログラムには何事も **インプット(入力)** があって、 **アウトプット(出力)** があります。
 そして、**インプットからアウトプットに変換する処理** があります。
-#### インプット と アウトプット が何かを考える
@@ -96,7 +92,7 @@
-#### インプット と アウトプット が何かを C# ベースで考える
+#### C# ベースでインプット と アウトプットを考える
@@ -104,21 +100,13 @@
-インプットの型は文字列なのでC#では `string` です。…と書きたいところですが、後述するオブジェクト型のコレクションがオススメです。
+インプットの型は文字列なのでC#では `string` です。後述するコレクションでもありです。
+アウトプットもインプットと同様です。
-具体的な型で表記すると`List<UserLog>`、`Dictionary<UserLog,int>`、`IEnumerable<UserLog>`などです。
-※`UserLog`クラスは日付とユーザーとアクセス回数をメンバーにしたものを想定しています。
-アウトプットもインプットと同様の型ですね。
-#### インプット と アウトプット をどこに実装するか
+では、C#においてインプットとアウトプットはどこに実装するのでしょうか。
@@ -158,6 +146,34 @@
+### ここまで前提の説明をして、ようやく回答
+テキストの1行から「日付」と「ユーザー」を取得し、取得した日付とユーザーを元にオブジェクトを作成し、作成したオブジェクトをコレクションで管理し、コレクションの各要素をループで取得してオブジェクトが持つ日付とユーザーを元にアクセス回数を計算し、同じ日付とユーザーが何回出たかを表示すれば良いのです。
+> 取得した日付とユーザーを元にオブジェクトを作成し
+例えば、日付とユーザーをメンバーに持った`UserLog`クラスを定義して、そのオブジェクトを生成するようなイメージです。
+```C#
+public class UserLog{
+    public DateTime Date { get; set; }
+    public string Name { get; set; }
+}
+```
 ### 最後に

文章の修正

2020/06/01 15:30

投稿

BluOxy

スコア2663

test CHANGED Viewed

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

文章の修正

2020/06/01 15:09

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -92,7 +92,7 @@
 ここまではプログラミング抜きで問題解決に必要な情報を考えてきました。
-インプットとアウトプットが固まってきたら、それを実現する手段である `C#` と絡めていきます。
+インプットとアウトプットが固まってきたら、それを実現する手段である C# と絡めていきます。

文章の修正

2020/06/01 15:06

投稿

BluOxy

スコア2663

test CHANGED Viewed

@@ -20,11 +20,11 @@
 ### 回答
-実現したいことの難易度やkanayawa1さんのスキルを踏まえると、問題解決の方法を先に学んだ方が良いです。
+実現したいことの難易度やkanayawa1さんのスキルを踏まえると、プログラムの書き方を先に学んだ方が良いです。
-#### 問題解決の方法
+#### プログラムの書き方