【C#】 配列に格納された各文字列の一部分を重複チェックと処理について

Question

プログラミング初心者の者です。
自分では実装ができない処理があったため質問させていただきました。

現在C#で、エクセルファイルを読み込ませ、内容をデータベースに登録させるプログラムを書いています。エクセルファイルの中身は、１行ごとに[名前,性別,生年月日]を入力する表になっています。

ファイルを読み込ませるときに、各列の入力値の検証を行っており、もし不正な値が入力されていた場合は、

[【i】行目：【項目名】の値が不正です]

のような行番号を明示したエラーメッセージを配列に格納するようにしています。

最終的にその配列の中身はログとして出力するのですが、以下のように表示される場合があり、改善したいと考えています。

<Error Log> 3行目：名前の値が不正です 3行目：性別の値が不正です 3行目：生年月日が不正です 4行目：性別の値が不正です . . .

上記の場合、以下のように重複する先頭４文字を削除して表示させたいと考えいます。

<Error Log> 3行目：名前の値が不正です 性別の値が不正です 生年月日が不正です 4行目：性別の値が不正です . . .

私が解決できず困っている点は、
①先頭4文字を対象とした重複チェック処理
②重複する値の削除
③その処理を行うタイミングは、エラーメッセージ格納時か？それともログ出力時か？
以上の3点です。

この３つの問題を解決する方法を、
または何かしらのヒントでも構いません。教えていただけないでしょうか？
よろしくお願いいたします。

Answer 1

ご質問の前提をひっくり返すようで申し訳ないのですが、「改善前」のログの方が良いログです。
「改善」したログは

• 一行読んだだけではそのラインの意味を理解できません。前方へ辿って行番号を探す必要があります。
• 行番号を含むラインとそうでないラインで書式が違うわけで、自動的な処理に向きません。

ログというのは一般的にテキストエディタで読むと限られたものではなく、スクリプトで処理したりExcelに読み込んで分析したりするものです。そのとき、上記の2点は大きな足かせになります。

Comments

[Takayukky]

ご回答ありがとうございます。
ログを扱っていくにあたり、yubaさんのおっしゃる2点は考慮しなくてはいけないと気づきました。
このような回答も非常にありがたいです。勉強になります！
ありがとうございます！

Answer 2

ログを出力するプログラムを変更することが可能であれば、重複をあとから削除するのではなく出力時にそれを考慮して出力する方が簡単で確実です。たとえば次のようなクラスを出力する例では LINQ を使って書くことが出来ます。

実行してみた結果も合わせてどうぞ: http://ideone.com/XJh6Qo

using System;
using System.Linq;
using System.Collections.Generic;
 
class ErrorLog
{
    public int Line { get; set; }
    public string Message { get; set; }
 
    public ErrorLog(int line, string message)
    {
    	this.Line = line;
    	this.Message = message;
    }
}
 
class Logger
{
    public static void Write(IEnumerable<ErrorLog> logs)
    {
        string output = string.Join("\n", logs.GroupBy(x => x.Line)
                                              .Select(x => x.Key + "行目：" + string.Join("\n", x.Select(y => y.Message))));
 
        Console.WriteLine(output);
    }
}
 
public class Test
{
	public static void Main()
	{
		var logs = new[]
		{
			new ErrorLog(3, "名前の値が不正です"),
			new ErrorLog(3, "性別の値が不正です"),
			new ErrorLog(3, "生年月日が不正です"),
			new ErrorLog(4, "性別の値が不正です"),
		};
		Logger.Write(logs);
	}
}

Comments

[Takayukky]

ソースと実行結果をあわせてご回答いただきありがとうございます。

出力時に処理したほうが良いのですね。Linqでは単純な操作しかしたことがなかったので、ご提供していただいたソースで勉強させていただきます。
非常に助かりました！ありがとうございます！