CSVファイル出力でデータを列単位で追加したい

Question

CSV出力で、1データを列方向に追加するロジックを作成しています。
例えば、Data1=1,2,3,4,5 Data2=A,B,C,D,Eの場合、この2つを1つのCSVファイルに出力する場合に、
1,A
2,B
3,C
4,D
5,E
と出力させようとしています。
そこで以下のように実装してみました。

メンバ変数

List<StringBuilder> WriteData = new List<StringBuilder>();

コンストラクタ内

for(int i=0;i<100000;i++)
{
     WriteData.Add(new StringBuilder().Append(string.Empty));
}

これはCSVファイルの行数が99999行までと言う仕様があるため、一旦すべてstring.Emptyを入れてます。

あとデータとしてstring[]型のデータ dataがあるとします。

メインのロジック

int i=0;
foreach(var sb in WriteData)
{
    if(sb.Length > 0)
    {
        sb.Append(",");  //何かデータが存在する場合は先にカンマを付加
    }
    if(i >= data.Length)
    {
        sb.Append("-");  //データ配列の数を超えた場合はハイフンを補う
    }
    else
    {
        sb.Append(data[i]);  //それ以外は実データをAppend
    }
    i += 1;
}

このメインのロジックをメソッドとし、string[]データ配列を取得後、このメソッドに渡す感じにしています。

動かしてみた感じでは、良さそうでしたが、何となく分かり難いような気もしています。
他に効率の良い方法がありましたら、アドバイス頂ければと思います。
また上記コードの問題点もありましたらお願いします。

Answer 1

• data1とdata2のサイズ（要素数）が異なる場合がある
  ⇒その場合 "-" で補う

• データは最大1000行まで

というのが条件でしたら、こんなのはどうでしょうか?

//結合元データ
var data1 = new int[] { 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12 };
var data2 = new string[] { "A", "B", "C", "D", "E" };

//結合するデータのサイズを設定
var padSize = Math.Abs(data1.Length - data2.Length);
//必ず1000行出力する必要があるならこれ
//padSize = 1000;
            
//必要なサイズ分データがない場合は "-" で補う
var padding1 = data1.Select(v => v.ToString()).Concat(Enumerable.Repeat("-", padSize));
var padding2 = data2.Select(v => v.ToString()).Concat(Enumerable.Repeat("-", padSize));

//サイズを揃えたものを結合
var ziped = padding1.Zip(padding2, (v1, v2) => string.Join(",", v1, v2));
        
//ファイル出力(最大1000行出力)
System.IO.File.WriteAllLines(@"test.csv", ziped.Take(1000));

Comments

[say5140]

ebiryoさん、ありがとうございます。データサイズが異なる場合については仕様がまだ決まっておらず、とりあえずハイフンで補うことにしてコーディングしていました。データのサイズ分ハイフンを補うあたりがとても参考になりました。

Answer 2

LINQ使ってみました｡
こんなのはいかがでしょう?

提示されたコードの問題点として､行毎にStringBuilderを構築するのは割とコストが高いので避けた方が無難だと思います｡
また､行毎のListにするなら､CSVファイル生成時に一発でString又はTextWriterに出力した方が望ましいかも知れません｡

using System;
using System.IO;
using System.Linq;
using System.Text;

namespace ConsoleApp4
{
    class Program
    {
        static void Main(string[] args)
        {
            var num = Enumerable.Range(0, 26).ToArray();
            var alpha = Enumerable.Range(0, 26).Select(i => (char) (i + 'A')).ToArray();

            var seq = num.Zip(alpha, (i, c) => string.Join(",", i, c));

            //CSVファイルの変わり
            var str = new StringWriter();

            using (str)
            {
                foreach (var element in seq)
                {
                    str.WriteLine(element);
                }
            }
            Console.WriteLine(str.ToString());

            Console.WriteLine();
            Console.WriteLine(OneLine());
        }

        //旨さ極めれば一筆書き
        static string OneLine()
        {
            var num = Enumerable.Range(0, 26).ToArray();
            var alpha = Enumerable.Range(0, 26).Select(i => (char)(i + 'A')).ToArray();

            return num.Zip(alpha, (i, c) => string.Join(",", i, c))
                .Aggregate(new StringBuilder(), (bld, s) => bld.Append($"{s}\n")).ToString();
        }

        //速さが足りないならこっち(マネージヒープをなるたけ汚さない=ガベコレが動かない=少しだけ速い)
        static string Performance()
        {
            var num = Enumerable.Range(0, 26).ToArray();
            var alpha = Enumerable.Range(0, 26).Select(i => (char)(i + 'A')).ToArray();

            var bld = new StringBuilder();

            for (int i = 0; i < 26; i++)
            {
                bld.Append(num).Append(',').Append(alpha).Append('\n');
            }
            return bld.ToString();
        }
    }
}

Comments

[say5140]

Tokeiya3さん、ありがとうございます。データを50000件にして簡単にですが処理時間を計測してみましたが、「旨さ極めれば一筆書き」の方は確かに早かったです（もちろんperformanceの方も）。私もLINQをうまく使えないかなと思っていたのでとても参考になります。まだ仕様的に未確定の部分がありますが、参考にさせて頂きます。

[Tokeiya3]

：）

Answer 3

お求めはこれかも

class Program
{
    static void Main(string[] args)
    {
        var data1 = new [] { "1", "2", "3", "4" };
        var data2 = new [] { "A", "B", "C", "D", "E", "F" };
        var data3 = new [] { "a", "b", "c" };
        var data4 = new [] { "い", "ろ", "は", "に", "ほ", "へ", "と", "ち", "り" };


        var myData = new MyData(data1);
        myData.AddData(data2);
        myData.AddData(data3);
        myData.AddData(data4);

        Console.WriteLine(myData.ToString(99999));
    }
        
}

public class MyData: List<List<string>>
{
    public MyData(string[] data)
    {
        foreach(var d in data)
        {
            this.Add(new List<string> { d });
        }
    }

    public void AddData(string[] data)
    {
        for(int i = 0; i < new[] { this.Count, data.Length}.Max(); i++)
        {
            if(i >= this.Count)
            {
                var bars = Enumerable.Range(0, this.First().Count - 1)
                    .Select(c => "-").ToArray();
                this.Add(new List<string>(bars));
                this[i].Add(data[i]);
            }
            else if(i >= data.Length)
            {
                this[i].Add("-");
            }
            else
            {
                this[i].Add(data[i]);
            }
        }
    }

    public override string ToString()
    {
        return ToString(this.Count);
    }

    public string ToString(int maxCount)
    {
        return this
            .Take(maxCount)
            .Select(l => l.Aggregate((a, b) => a + "," + b))
            .Aggregate((a, b) => a + "\r\n" + b);
    }
}

Comments

[say5140]

hihijijiさん、ありがとうございます。確かに求めているものがありそうです。仕様がまだ未確定の部分がありますので、ここで教えて頂いた方法を試してみて仕様に合わせてより良い方法を考えたいと思います。

Answer 4

CSVデータの行数が同じならこんな感じ？でどうでしょう？
仕様が不明な部分がありますので、とりあえずシンプルに書いてみました。
実際は＋αが必要だと思います。

var data1 = new int[] { 1, 2, 3, 4, 5 };
var data2 = new string[] { "A","B","C","D","E" };
var sb = new StringBuilder();
foreach (var element in data1.Select((value,index) => new {value,index})) {
    sb.AppendLine(element.value + "," + data2[element.index]);
}
System.Console.WriteLine(sb);

Comments

[say5140]

suzu_catさん、ありがとうございます。実はまだ仕様が未確定の部分があり、データの行数についてもまだ決まっていないため、念のため行数が不定の場合で考えておりました。
ここで教えて頂いた方法を試してみたいと思います。