質問

典型的な作成方法は、 CSV 文字列 (擬似コード):

  1. CSV コンテナ オブジェクト (C# の StringBuilder など) を作成します。
  2. 追加する文字列をループし、各文字列の後にカンマを追加します。
  3. ループの後で、最後の余分なカンマを削除します。

コードサンプル:

public string ReturnAsCSV(ContactList contactList)
{
    StringBuilder sb = new StringBuilder();
    foreach (Contact c in contactList)
    {
        sb.Append(c.Name + ",");
    }

    sb.Remove(sb.Length - 1, 1);
    //sb.Replace(",", "", sb.Length - 1, 1)

    return sb.ToString();
}

コンテナが空かどうかをチェックしてカンマを追加するというアイデアは気に入っていますが、出現するたびに文字列の長さをチェックする必要があるため、処理が増えることを意味しませんか?

最後のコンマを削除する、より簡単、よりクリーン、より効率的な方法があるべきだと思います。何か案は?

役に立ちましたか?

解決

使用できます LINQ to オブジェクト:

string [] strings = contactList.Select(c => c.Name).ToArray();
string csv = string.Join(",", strings);

もちろん、これらすべてを 1 行で行うこともできますが、2 行にするともう少し明確になります。

他のヒント

あなたのコードは実際には準拠していません 完全な CSV 形式. 。カンマ、先頭/末尾のスペース、タブ、改行、引用符のないデータから CSV を生成するだけであれば、問題ありません。ただし、実際のデータ交換シナリオのほとんどでは、完全な実装が必要です。

適切な CSV を生成するには、これを使用できます。

public static String EncodeCsvLine(params String[] fields)
{
    StringBuilder line = new StringBuilder();

    for (int i = 0; i < fields.Length; i++)
    {
        if (i > 0)
        {
            line.Append(DelimiterChar);
        }

        String csvField = EncodeCsvField(fields[i]);
        line.Append(csvField);
    }

    return line.ToString();
}

static String EncodeCsvField(String field)
{
    StringBuilder sb = new StringBuilder();
    sb.Append(field);

    // Some fields with special characters must be embedded in double quotes
    bool embedInQuotes = false;

    // Embed in quotes to preserve leading/tralining whitespace
    if (sb.Length > 0 && 
        (sb[0] == ' ' || 
         sb[0] == '\t' ||
         sb[sb.Length-1] == ' ' || 
         sb[sb.Length-1] == '\t' ))
    {
        embedInQuotes = true;
    }

    for (int i = 0; i < sb.Length; i++)
    {
        // Embed in quotes to preserve: commas, line-breaks etc.
        if (sb[i] == DelimiterChar || 
            sb[i]=='\r' || 
            sb[i]=='\n' || 
            sb[i] == '"') 
        { 
            embedInQuotes = true;
            break;
        }
    }

    // If the field itself has quotes, they must each be represented 
    // by a pair of consecutive quotes.
    sb.Replace("\"", "\"\"");

    String rv = sb.ToString();

    if (embedInQuotes)
    {
        rv = "\"" + rv + "\"";
    }

    return rv;
}

世界で最も効率的なコードではないかもしれませんが、テストされています。簡単なサンプルコードに比べて、現実世界は最悪です:)

オープンソースの CSV ライブラリを使用してみてはいかがでしょうか?

非常に単純に見えるのにやりすぎのように聞こえるかもしれませんが、コメントとコード スニペットからわかるように、見た目以上のものがあることがわかります。完全な CSV 準拠の処理に加えて、最終的には CSV の読み取りと書き込みの両方を処理する必要があるでしょう...ファイル操作が必要になる場合もあります。

使ったことがある CSVを開く 以前に私のプロジェクトの 1 つでした (ただし、他にもたくさんの選択肢があります)。それは確かに私の人生を楽にしてくれました。;)

私たちの旧友「for」を忘れないでください。foreach ほど見た目は良くありませんが、2 番目の要素から開始できるという利点があります。

public string ReturnAsCSV(ContactList contactList)
{
    if (contactList == null || contactList.Count == 0)
        return string.Empty;

    StringBuilder sb = new StringBuilder(contactList[0].Name);

    for (int i = 1; i < contactList.Count; i++)
    {
        sb.Append(",");
        sb.Append(contactList[i].Name);
    }

    return sb.ToString();
}

また、2 番目の Append を「if」で囲んで、Name プロパティに二重引用符またはコンマが含まれているかどうかをテストし、含まれている場合はそれらを適切にエスケープすることもできます。

代わりに、foreach 内の最初にカンマを追加することもできます。

if (sb.Length > 0) sb.Append(",");

の配列を作成することもできます c.名前 データと使用 文字列.結合 ラインを作成する方法。

public string ReturnAsCSV(ContactList contactList)
{
    List<String> tmpList = new List<string>();

    foreach (Contact c in contactList)
    {
        tmpList.Add(c.Name);
    }

    return String.Join(",", tmpList.ToArray());
}

これは、 文字列ビルダー アプローチですが、見た目は明らかにすっきりしています。

また、次の使用を検討することもできます。 .CurrentCulture.TextInfo.ListSeparator ハードコーディングされたカンマの代わりに -- 出力を他のアプリケーションにインポートする場合、問題が発生する可能性があります。ListSeparator は文化が異なれば異なる場合がありますが、少なくとも MS Excel はこの設定を尊重します。それで:

return String.Join(
    System.Globalization.CultureInfo.CurrentCulture.TextInfo.ListSeparator,
    tmpList.ToArray());

コンテナが空かどうかをチェックしてカンマを追加するというアイデアは気に入っていますが、出現するたびに文字列の長さをチェックする必要があるため、処理が増えることを意味しませんか?

時期尚早に最適化を行っているため、パフォーマンスへの影響は無視できます。

ちょっと考えてみましたが、フィールド値のカンマと引用符 (") を処理することを忘れないでください。そうしないと、CSV ファイルがコンシューマー リーダーを破損する可能性があります。

他の誰かが役に立つと思った場合に備えて、これ用の小さなクラスを書きました...

public class clsCSVBuilder
{
    protected int _CurrentIndex = -1;
    protected List<string> _Headers = new List<string>();
    protected List<List<string>> _Records = new List<List<string>>();
    protected const string SEPERATOR = ",";

    public clsCSVBuilder() { }

    public void CreateRow()
    {
        _Records.Add(new List<string>());
        _CurrentIndex++;
    }

    protected string _EscapeString(string str)
    {
        return string.Format("\"{0}\"", str.Replace("\"", "\"\"")
                                            .Replace("\r\n", " ")
                                            .Replace("\n", " ")
                                            .Replace("\r", " "));
    }

    protected void _AddRawString(string item)
    {
        _Records[_CurrentIndex].Add(item);
    }

    public void AddHeader(string name)
    {
        _Headers.Add(_EscapeString(name));
    }

    public void AddRowItem(string item)
    {
        _AddRawString(_EscapeString(item));
    }

    public void AddRowItem(int item)
    {
        _AddRawString(item.ToString());
    }

    public void AddRowItem(double item)
    {
        _AddRawString(item.ToString());
    }

    public void AddRowItem(DateTime date)
    {
        AddRowItem(date.ToShortDateString());
    }

    public static string GenerateTempCSVPath()
    {
        return Path.Combine(Path.GetTempPath(), Guid.NewGuid().ToString().ToLower().Replace("-", "") + ".csv");
    }

    protected string _GenerateCSV()
    {
        StringBuilder sb = new StringBuilder();

        if (_Headers.Count > 0)
        {
            sb.AppendLine(string.Join(SEPERATOR, _Headers.ToArray()));
        }

        foreach (List<string> row in _Records)
        {
            sb.AppendLine(string.Join(SEPERATOR, row.ToArray()));
        }

        return sb.ToString();
    }

    public void SaveAs(string path)
    {
        using (StreamWriter sw = new StreamWriter(path))
        {
            sw.Write(_GenerateCSV());
        }
    }
}

私は以前にこの方法を使用したことがあります。StringBuilder の Length プロパティは読み取り専用ではないため、1 減算すると最後の文字が切り捨てられることになります。ただし、長さをゼロ未満に設定するとエラーになるため、最初から長さがゼロではないことを確認する必要があります (リストが空の場合はこれが起こります)。

public string ReturnAsCSV(ContactList contactList)
{
    StringBuilder sb = new StringBuilder();

    foreach (Contact c in contactList)       
    { 
        sb.Append(c.Name + ",");       
    }

    if (sb.Length > 0)  
        sb.Length -= 1;

    return sb.ToString();  
}

私が使う CSVヘルパー - これは、準拠した CSV ストリームを一度に 1 要素ずつ生成したり、クラスをカスタム マップしたりできる優れたオープンソース ライブラリです。

public string ReturnAsCSV(ContactList contactList)
{
    StringBuilder sb = new StringBuilder();
    using (StringWriter stringWriter = new StringWriter(sb))
    {
        using (var csvWriter = new CsvHelper.CsvWriter(stringWriter))
        {
            csvWriter.Configuration.HasHeaderRecord = false;
            foreach (Contact c in contactList)
            {
                csvWriter.WriteField(c.Name);
            }
        }
    }
    return sb.ToString();
}

または、マップする場合は次のようになります。 csvWriter.WriteRecords<ContactList>(contactList);

トリミングなどはいかがでしょうか?

public string ReturnAsCSV(ContactList contactList)
{
    StringBuilder sb = new StringBuilder();

    foreach (Contact c in contactList)
    {
        sb.Append(c.Name + ",");
    }

    return sb.ToString().Trim(',');
}

最初の項目を表示しているかどうかを追跡し、カンマのみを追加してみてはいかがでしょうか 前に 最初のアイテムでない場合は、そのアイテム。

public string ReturnAsCSV(ContactList contactList)
{
    StringBuilder sb = new StringBuilder();
    bool isFirst = true;

    foreach (Contact c in contactList) {
        if (!isFirst) { 
          // Only add comma before item if it is not the first item
          sb.Append(","); 
        } else {
          isFirst = false;
        }

        sb.Append(c.Name);
    }

    return sb.ToString();
}
ライセンス: CC-BY-SA帰属
所属していません StackOverflow
scroll top