C#で100本ノックをやるにあたり

Question

前提・実現したいこと

C#で100本ノックをやってみようということでやっているのですが、
1章04のコードをスッキリさせたいです。
LINQを使って書きたいのですが、条件指定(~番目のとき)という処理が思い浮かびません。

いい方法がございましたらご教示ください。

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

該当のソースコード

var text = "Hi He Lied Because Boron Could Not Oxidize Fluorine. New Nations Might Also Sign Peace Security Clause. Arthur King Can.";
            var line = text.Split(' ');
            var cut = new List<string>();
            var result = new Dictionary<string, int>();

            for (int i = 0; i < line.Length;i++)
            {
                if(i==0 || i==4 || i==5 || i==6 || i==7 || i==8 || i==14 || i==15 || i==18)
                {
                    cut.Add(line[i].Substring(0,1));
                }
                else
                {
                    cut.Add(line[i].Substring(0, 2));
                }
            }

            for (int i = 0; i < cut.Count;i++)
            {
                result.Add(cut[i], i+1);
            }
            foreach(var x in result)
            {
                Console.Write(x);
            }

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

C#7

Comments

[yoorwm]

素朴な疑問なんですが、100本ノックって他の人にやってもらうようなもんなんすか？

[tmtmshohei]

回答自体はできたのですが、無駄が多いコードだと感じたので最適化、レビュー的な意味合いで投稿させて頂きました。

[退会済みユーザー]

余計なおせっかいかもしれませんが、C#の練習でなく、言語処理そのものの手習いなのであれば、流行りの言語を選択したほうがベターですよ。

[tmtmshohei]

ご助言ありがとうございます。今回は言語処理ではなく最近使う機会が増えてきたC#の練習という趣旨でした。

[退会済みユーザー]

なるほど！失礼しました^^

Answer 1

序数を得るには Select<T, U>(Func<T, int, U>) を使います。
HashSet<T> を使えば集合の中から高速に検索できます。
最後に辞書を作れという指示なので、ToDictionary をつけろということでしょう。

using System;
using System.Collections.Generic;
using System.Linq;

namespace ConsoleApp1
{
	class Program
	{
		static void Main(string[] args)
		{
			var orders = new HashSet<int>(new[] { 1, 5, 6, 7, 8, 9, 15, 16, 19 }.Select(a => a - 1));
			var src = "Hi He Lied Because Boron Could Not Oxidize Fluorine. New Nations Might Also Sign Peace Security Clause. Arthur King Can.";
			var result = src
				.Split(' ')
				.Select((a, i) => new { Value = a.Substring(0, orders.Contains(i) ? 1 : 2), Index = i })
				.ToDictionary(a => a.Value, a => a.Index + 1);

			// 結果出力
			Console.WriteLine(src);
			foreach (var item in result)
			{
				Console.WriteLine($"{item.Key}: {item.Value}");
			}
			Console.ReadKey();
		}
	}
}

Comments

[Zuishin]

使用時のシグネチャに合わせて Select<T, U>(Func<T, int, U>) と書きましたが、本来は public static System.Collections.Generic.IEnumerable<TResult> Select<TSource,TResult> (this System.Collections.Generic.IEnumerable<TSource> source, Func<TSource,int,TResult> selector); です。

[Zuishin]

あと、後で気づいたのですが、HashSet<T> を使えば高速になると書きましたが、それは一般論で、int の場合はハッシュと値が一致するので高速にはなりません。
大量の文字列その他のオブジェクトを探す場合に高速化が期待できます。
本当はここでは集合の量も少なく重複もないので List<T> で十分ですが、せっかくなのでそのままにしておきます。

[tmtmshohei]

ありがとうございます、勉強になります。
ここで使っているSelectの引数(a,i)ですが、aには順次srcの要素が入るのはわかるのですが、
iにはイテレータとしてint型の数値が順次はいるのでしょうか？

[Zuishin]

https://referencesource.microsoft.com/#System.Core/System/Linq/Enumerable.cs,47
これがソースです。
SelectIterator() でインデクスとイテレータを設定しています。
これが同時に設定されているということは、イテレータが次のものになった時にインデクスも次の数字になるということになります。

[tmtmshohei]

SelectIterator()内ですでに設定されているんですね。
HashSet<T>もほとんど使ったことがなかったので
文字列の処理の場合、積極的に使ってみます。
ご丁寧にありがとうございます。

[tmtmshohei]

すみません、
Select以下の new { Value = a.Substring(0, orders.Contains(i) ? 1 : 2), Index = i })ですが
new以下で作られているのは匿名配列になるのでしょうか？

[Zuishin]

配列ではありません。匿名クラスのインスタンスです。
本当はタプルを使いたかったんですが、まだ使えない環境も多いので匿名クラスにしました。

[tmtmshohei]

ありがとうございます、LINQ匿名クラスあたりは昨日から触り始めたのでごちゃごちゃしていました。
同じことがSelectManyと匿名配列でもできるんですね。
SelectMany((a, i) => new[] { new{Value=a.Substring(0,orders.Contains(i)?1:2),Index=i}})
勉強になりました。

[Zuishin]

できますが、要素数が一つの配列をわざわざ作ってその要素を取り出しているので、する意味は無いように思います。

[tmtmshohei]

おっしゃるように実際にコードを書く際にはSelect()で処理したいと思います。