固定の配列に動的に作成した配列を追加したい

Question

こんにちは。

下記の変数名Objの「★固定の配列」に、ループ処理にて「★動的に追加したい値」の内容に
動的Aと動的Bの箇所を動的に変化させた文字列をセットし
変数名Objの「★固定の配列」に追加していきたい。
結果として「★結果」のようにしたいのですが
どのようにすれば良いでしょうか？
ご教授宜しくお願い致します。

★固定の配列
            Object[,] Obj = {
                            { "ＩＤ", "UniqId", 70, false, VerticalAlign.Middle, HorizontalAlign.Center, "LABEL_CHAR" },
                            { "職員名", "ShokuinName", 320, true, VerticalAlign.Middle, HorizontalAlign.Left, "LABEL_CHAR" },                           
                        };

****************************************************************************************************************
★動的に追加したい値
　※動的Aと動的Bはループ処理により動的に命名する
{ 動的A, 動的B, 80, true, VerticalAlign.Middle, HorizontalAlign.Center, "MONEY" },
****************************************************************************************************************

★結果

            Object[,] Obj = {
                            { "ＩＤ", "UniqId", 70, false, VerticalAlign.Middle, HorizontalAlign.Center, "LABEL_CHAR" },
                            { "職員名", "ShokuinName", 320, true, VerticalAlign.Middle, HorizontalAlign.Left, "LABEL_CHAR" }, 
                            { "ほげほげ１", "Hogehoge1", 80, true, VerticalAlign.Middle, HorizontalAlign.Center, "MONEY" },
                            { "ほげほげ２", "Hogehoge2", 80, true, VerticalAlign.Middle, HorizontalAlign.Center, "MONEY" },
                            { "ほげほげ３", "Hogehoge3", 80, true, VerticalAlign.Middle, HorizontalAlign.Center, "MONEY" },
                            { "ほげほげ４", "Hogehoge4", 80, true, VerticalAlign.Middle, HorizontalAlign.Center, "MONEY" },

List形式からObject[,]への変換までを書いてみました。
なんとかできました。ありがとうございました。

            var objList = new List<List<object>>();
            objList.Add(new List<object>() { "UniqId", "UniqId", 70, false, 11111, 22222, "LABEL_CHAR" });
            objList.Add(new List<object>() { "職員名", "ShokuinName", 320, true, 11111, 22222, "LABEL_CHAR" });
            objList.Add(new List<object>() { "科目コード", "AAA", 80, true, 11111, 22222, "MONEY" });

            for (var i = 0; i < 10; i++)
            {
               
                objList.Add(new List<object>() { "ほげほげ" + i, "Hogehoge" + i, 80, true, 11111, 22222, "MONEY" });

            }

            //これだとObject[][]になってしまう
            //var parsedList = objList
            //                .Select(val => val
            //                    .Select(val2 => val2).ToArray()
            //                ).ToArray();

            var rowCount = objList.Count;
            var colCount = 7;

            var r = new Object[objList.Count, 7];

            for (int y = 0; y < rowCount; y++)
            {
                var src = objList[y];
                for (int x = 0; x < colCount; x++)
                {
                    r[y, x] = objList[x];
                }
            }

実装してみました。
うまくいきました。

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;

namespace WindowsFormsApplication1
{
    public class Poco
    {
        public string Name;
        public string Id;
        public int Width;
        public bool IsValid;
        public int Valign;
        public int Halign;
        public string Label;
        public Poco(string name, string id, int width, bool isvalid, int valign, int halign, string label)
        {
            Name = name; Id = id; Width = width; IsValid = isvalid; Valign = valign; Halign = halign; Label = label;
        }


        public static IList<Poco> ToList(object[,] hatchpotch)
        {
            var innerLen = 7;
            var outerLen = hatchpotch.Length / innerLen;
            return Enumerable.Range(0, outerLen).Select(i => new Poco((string)hatchpotch[i, 0], (string)hatchpotch[i, 1], (int)hatchpotch[i, 2], (bool)hatchpotch[i, 3], (int)hatchpotch[i, 4], (int)hatchpotch[i, 5], (string)hatchpotch[i, 6])).ToList();
        }

        public static object[,] ToRect(IList<Poco> data) //理解しやすくするためList<Poco>としてます。実用では副作用に注意しましょう
        {
            var ret = new object[data.Count(), 7];
            for (var i = 0; i < data.Count(); i++)
            {
                ret[i, 0] = (object)data[i].Name;
                ret[i, 1] = (object)data[i].Id;
                ret[i, 2] = (object)data[i].Width;
                ret[i, 3] = (object)data[i].IsValid;
                ret[i, 4] = (object)data[i].Valign;
                ret[i, 5] = (object)data[i].Halign;
                ret[i, 6] = (object)data[i].Label;
            }
            return ret;
        }


    }

}



//実行部
sub main()
{
            Object[,] Obj = {
                            { "UniqId", "UniqId", 70, false, 11111, 22222, "LABEL_CHAR" },
                            { "職員名", "ShokuinName", 320, true, 11111, 22222, "LABEL_CHAR" },
                            { "科目コード", "AAA", 80, true, 11111, 22222, "MONEY" },
                        };
            
            IList<Poco> PocoList = Poco.ToList(Obj);


            for (var i = 0; i < 10; i++)
            {
                Poco _Poco = new Poco("ほげほげ" + i, "Hogehoge" + i, 80, true, 11111, 22222, "MONEY");
                PocoList.Add(_Poco);
            }


            Object[,] Obj2 = Poco.ToRect(PocoList);
}

元々コボラーの独学なので
低レベルなコーディングでお目汚しですが
毎回、案件ごとにSPREAD定義を一から書くのが億劫だったので
SPREAD for WinFormsもASP.NETでも同じように
以下のような関数に[,]を食わせて定義してます。

         /// <summary>
        /// スプレッドシート設定
        /// </summary>
        /// <param name="sheet"></param>
        /// <param name="hd"></param>
        /// <param name="FrozenColumnCount"></param>
        /// <param name="FrozenRowCount"></param>
        /// <param name="rowHeader"></param>
        /// <param name="columnHeader"></param>
        /// <param name="allowGroup"></param>
        /// <param name="columnHeaderSet"></param>
        /// <param name="AlternatingSw"></param>
        public void InitSpreadStyles(FarPoint.Web.Spread.SheetView sheet,
            Object[,] hd = null,
            int FrozenColumnCount = 0,
            int FrozenRowCount = 0,
            Boolean rowHeader = true,
            Boolean columnHeader = true,
            Boolean allowGroup = false,
            Boolean columnHeaderSet = true,
            Boolean AlternatingSw = true)
        {

            FarPoint.Web.Spread.Inset mg = new FarPoint.Web.Spread.Inset(3, 0, 3, 0);

            // 余白の設定をシートに適用
            sheet.DefaultStyle.Margin = mg;

            sheet.GridLines = GridLines.Both;

            sheet.DefaultStyle.Border.BorderColor = Color.LightBlue;
            sheet.DefaultStyle.Border.BorderSize = 1;
            sheet.DefaultStyle.Border.BorderStyle = BorderStyle.Solid;

            //選択行の色
            sheet.SelectionBackColor = System.Drawing.Color.Silver;

            // 縦方向の揃え位置を中央に設定
            sheet.DefaultStyle.VerticalAlign = VerticalAlign.Middle;

            //バインドしたデータでのカラム作成を抑止する
            sheet.AutoGenerateColumns = false;

            //カラム移動を可能にする
            sheet.AllowColumnMove = false;

            if (hd != null)
            {
                //カラムを作成
                sheet.ColumnCount = hd.GetLength(0);
                for (int x = 0; x <= hd.GetLength(0) - 1; ++x)
                {
                    if (columnHeaderSet)
                    {
                        // カラムヘッダを設定します。 
                        sheet.ColumnHeader.Cells[0, x].Text = (String)hd[x, 0];
                    }

                    sheet.Columns[x].DataField = (String)hd[x, 1];

                    sheet.Columns[x].Width = (int)hd[x, 2];

                    sheet.Columns[x].Visible = (Boolean)hd[x, 3];

                    sheet.Columns[x].VerticalAlign = (VerticalAlign)hd[x, 4];

                    sheet.Columns[x].HorizontalAlign = (HorizontalAlign)hd[x, 5];

                    sheet.Columns[x].Font.Name = "Meiryo UI";

                    //セルタイプをセット
                    String cellType = (String)hd[x, 6];

                    if (String.IsNullOrEmpty(cellType))
                    {
                        sheet.Columns[x].CanFocus = false;
                        sheet.Columns[x].Locked = true;
                    }

                    if ("LABEL_NUM".Equals(cellType))
                    {
                        LabelCellType celltype = new LabelCellType();
                        celltype.AllowWrap = false;
                        sheet.Columns[x].CellType = celltype;
                    }
						・
						・
						・いろんなcelltypeの設定を定義してます
						・
						・
						・
						
                }
            }

            // グルーピング設定
            sheet.AllowGroup = allowGroup;
            sheet.GroupBarVisible = allowGroup;

            if (AlternatingSw)
            {
                // １行おきにスタイルを設定します
                sheet.AlternatingRows.Count = 2;
                System.Drawing.Color fc = System.Drawing.Color.FromName("OldLace");
                sheet.AlternatingRows[1].BackColor = fc;
            }

            // 行ヘッダ表示制御
            sheet.RowHeader.Visible = rowHeader;
            // 列ヘッダ表示制御
            sheet.ColumnHeader.Visible = columnHeader;
            // 列の固定
            sheet.FrozenColumnCount = FrozenColumnCount;
            // 行の固定
            sheet.FrozenRowCount = FrozenRowCount;


        }

Comments

[papinianus]

興味本位で申し訳ないのですが、SPREAD for WinFormsって[,]で操作するものなのですか？
SPREADだから、列が変動できる、のだと思いますが。

[cutedog]

質問欄に食わせる関数を追記しました。
[,]では操作しません。関数用です。
SPREAD for WinFormsもASP.NETでも同じように
追記した関数に[,]を食わせて定義してます。
objList.Add(new List<object>() { "UniqId", "UniqId", 70, false, 11111, 22222, "LABEL_CHAR" });が１つの列に相当します。

Answer 1

配列でやりたいということですが、List<T> を使うべきという回答です。

理由 1
配列のサイズは固定されていてループで動的に拡張できません。
Array.Resize() という配列の大きさを拡張するメソッドは確かにありますが、その内部では新しい配列を用意して既存の中身をコピーしています。
10 回拡張した場合は、同じものを 10 回コピーすることになります。

理由 2
それでも配列を拡張する場合には、一つコピーするごとに拡張するのではなく、あらかじめ大きな配列を用意し、それがいっぱいになるまではそれを使うことでパフォーマンスを上げることができます。
しかし、これは自分でやらなくてもすでに標準ライブラリで実装されています。
それが List<T> です。

List<T> のソース

これを見ると、Capacity というプロパティの大きさの配列を用意していることがわかります。

理由 3
List<T> から配列へのコピーは簡単です。
ループ中では List<T> を使い、最終的に配列が必要になった時点でコピーすることができます。

Comments

[cutedog]

ご提案ありがとうございます。
質問欄にLIST<T>をつかうやり方を追記してみましたが
あってますか？
ただ、最終的にListにしたのはいいけど
配列のObject[,] Obj　にコピーするにはどうしたらいいのでしょうか？
Object[,] Obj = objList.ToArray();って書いてもダメみたいです。

[Zuishin]

ざっと見ただけですが、やり方は合っています。
LINQ で二次元配列を一発で作る方法は無いので、List.Count を見て配列を用意して for でセットするのが良いと思います。

また、二次元配列でなくジャグ配列でも良いなら Object[][] Obj = objList.ToArray() でできますが、二次元配列を使う必要はありますか？

[papinianus]

私のほうのコメントに既存コードが四角配列でできあがっているとのこと。

[Zuishin]

そうなんですが、たとえばファイルからの読み込みなど最初に配列の大きさがわかっていないものを動的に作る場合は一度リストを経由するのが良いと思ってこの提案をしました。

[cutedog]

この配列は、SPREADを使用する際の定義情報を書いているので、Zuishinさんのお考えのような行列が動的に変わる仕様を想定していないです。今回動的になるのは行のみの想定です。
皆さまの提案内容を見てList形式をつかってOject[,]への変換を作ってのを
質問欄に追記しました。

[papinianus]

> 配列の大きさがわかっていない
列が固定であっても、行が変動するは、配列の縦方向の大きさが分かっていない、ではないでしょうか。

Answer 2

四角配列じゃなきゃだめですかね？

四角配列つかったことないんですが、4つ増やすとして

var Obj = //もとのやつ
var curLen = Obj.length / 7
var newLen = newLen + 4;
var newObj = new object[newLen,7]();
for(var i = 0; i < curLen; i++)
{
  for(var j = 0; j < 7; j++)
  {
      newObj[i,j] = Obj[i,j];
  }
}
for(var i = curLen; i < newLen; i++)
{
  for(var j = 0; j < 7; j++)
  {
      switch(j)
      {
         case 0: //これを7個定義する
           newObj[i,j] = "ほげほげ1";
           break;
      }
  }
}

とかじゃないでしょうか？

---

ちなみにObjがobject[][]だったら

var newObjs = Obj.Concat(Enumerable.Range(1,4).Select(i => new object[] {$"ほげほげ{i}", $"Hogehoge{i}", 80, true, VerticalAlign.Middle, HorizontalAlign.Center, "MONEY" });

---

class Poco
{
  public string Name;
  public string Id;
  public int Width;
  public bool IsValid;
  public int Valign;
  public int Halign;
  public sring Label;
  public Poco(string name, string id, int width, bool isvalid; int valign, int halign, string label)
  {
    Name = name; Id = id; Width = width; IsValid = isvalid; Valign = valign; Halign = halign; Label = label;
  }
}
public List<Poco> ToList(object[,] hatchpotch)
{
  var inner = 7;
  var outerLen = hatchpotch.Length / innerLen;
  return Enumerable.Range(0, outerLen).Select(i=>new Poco((string)hatchpotch[i,0],(string)hatchpotch[i,1],(int)hatchpotch[i,2],(bool)hatchpotch[i,3],(int)hatchpotch[i,4],(int)hatchpotch[i,5],(string)hatchpotch[i,6])).ToList();
}
public object[,] ToRect(List<Poco> data) //理解しやすくするためList<Poco>としてます。実用では副作用に注意しましょう
{
  var ret = new object[data.Count(), 7];
  for(var i = 0; i < data.Count(); i++)
  {
    ret[i,0] = (object)data[i].Name;
    ret[i,1] = (object)data[i].Id;
    ret[i,2] = (object)data[i].Width;
    ret[i,3] = (object)data[i].IsValid;
    ret[i,4] = (object)data[i].Valign;
    ret[i,5] = (object)data[i].Halign;
    ret[i,6] = (object)data[i].Label;
  }
  return ret;
}

Comments

[papinianus]

四角配列ってlengthとってもforeachしてもシーケンシャルにしか出てこないしLinqとの相性も最悪なんで、私は基本避けてます。
1列のサイズが固定できるメリットに対して、ハンドリングがつらすぎる。

[cutedog]

ご提案ありがとうございます。
Object[,]型を利用したロジックがすでに実装されているので
このObject[,]型でないとちょっと困るんですよね。

[papinianus]

提案以前に水平線より上には、四角配列でやりたいことをコードそのままで書いてます。
原理的にはforでつめていくしかなくて、こういう無駄に面倒なことが必要なんです。

> Object[,]型を利用したロジックがすでに実装されているので
だいたい察していましたが、誠に残念ですとしか言いようがないです。
今回のはそもそもネストした配列の型ではなく、きちんとSomeInfoみたいなstruct/classを作って、SomeInfo[]とすべきでした。

[cutedog]

すいません。なにぶんC#を習いたての頃に書いたコードでして、特に問題なかったのでそのまま使用していまして。。。今にして思えばもっと書きようがあったように思います。

[papinianus]

今からでもいくらでも取り返せます。
リストにするのと、リストに戻すのユーティリティを書いたのでご参考に(型や名前は推測です)。

もう速度の追求とかメモリの追求とかはあきらめていただいて。
一旦、前のデータ(Obj)をToListでList<Poco>にして、そこに追加したいやつをどんどん動的に追加
`list.Add(new Poco("動的1","動的2", 1, true,2,3,"label")`していって、できたものをToRectで戻せば他のメソッドに食わせられるでしょう。

[papinianus]

どっちにコメントするか迷いましたが、こちらで。

追記した方法は意味的には正しいと思いますし、私が書いたのも実体としては同じだと思いますが、関数に抽象化したほうがいいと思います(object[,]で扱いにくい問題がでてきたらまた同じようなことをすることになるから)

あと、型安全の世界に戻ってください。

[cutedog]

ありがとうございます。すごいー。の一言です。
こちらで、貼り付けしたのですがEnumerableで以下のようなエラーになります。
これって何かこちらの環境が悪いんでしょうか？

IEnumerable<out T>を実装するオブジェクトをクエリするための一連のstaticメソッドを提供します。
実行不可能なメンバー'Enumerable'をメソッドのように使用できません。

[papinianus]

Enumerab.Range(0、でした「.Range」が抜けてました。回答を修正してます(VS使わないといけないですね)

[cutedog]

Enumerable.Rangeですかね？
return Enumerable.Range(0, outerLen).Select(i => new Poco((string)hatchpotch[i, 0], (string)hatchpotch[i, 1], (int)hatchpotch[i, 2], (bool)hatchpotch[i, 3], (int)hatchpotch[i, 4], (int)hatchpotch[i, 5], (string)hatchpotch[i, 6])).ToList();

[cutedog]

あっと、素早い回答でありがとうございます。更新してなかったのでわかりませんでした。

[cutedog]

まだ少し文法エラーがありましたが補正できました。。
VS使わずこれが書けるのがすごいです。

[papinianus]

> まだ少し文法エラーがありましたが
おはずかしい限り

[cutedog]

うまく実装できました。
ありがとうございました。
ついでに教えてほしいのですが
コメントで「理解しやすくするためList<Poco>としてます。実用では副作用に注意しましょう」
と書かれていますが、副作用とはどういったものなのかが想像がつかないのですが・・・

[papinianus]

ToRectは本来、かたちをかえるだけで、職員1を「しょくいん1」にすることを想定していません(後から見たときにそういう想像をしません)
しかし、List<T>で渡すと、中身を書きかえることが(ミスかもしれませんし、便利だからついでにやっちゃおうかもしれません)できてしまいます。
この辺、コメントで書けるレベルじゃないので「https://qiita.com/lobin-z0x50/items/248db6d0629c7abe47dd」とかを参考にしてください。

[cutedog]

「https://qiita.com/lobin-z0x50/items/248db6d0629c7abe47dd」読みました。
すごい勉強になりました。

Answer 3

2次元配列の場合、大きい配列を再定義して、中身をコピーする形になるかと思います。
object[][]のようなジャグ配列ですとArray.Resizeを使用することができます。
また、List<T>を使用したほうがすっきりするような気もします。

Comments

[cutedog]

ご提案ありがとうございます。
そうですね。Zuishinさんの提案してくれている
List<T>がいいようにおもったので、
質問欄に追記してみました。
けどObject[,]型にコピーする方法がわからない・・・