C# エクセルファイルの複数の列を持つ２行をListViewで2列に表示したい

Question

【質問】
・指定した行(2行)をListViewで2列に分けたい
・1行目は1列目、2行目は2列目をListViewで表示したい

【目的】
・エクセルファイルの行を指定し、繰り返し文で列をすべて取得。その後、行ごとに1列ずつ表示したい(※今回は2行取得したので、1行目は1列目、2行目は2列目にしたい。)
・エクセルで指定した行の列は400列ある。
・2行の列は同じ数。

【相談前実装】
・1行に対し、ループで回し。List<string[]>に格納をし、[0]と[1]のforeach文でまわし、listView1.Items.Addに追加した。が、かさなってしまったため。subItems.addも用いましたが、行並びになってしまった。

【環境】
・C# .net framework4.7 visual studio2022 windowsform

よろしくお願いいたします。

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using NPOI.SS.UserModel;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace test
{
    public partial class Form4 : Form
    {
        List<string[]> datalists = new List<string[]>();
        public Form4()
        {
            InitializeComponent();

            datalists.Clear();

            //エクセルファイルを開いた
            IWorkbook workbook = WorkbookFactory.Create(@"エクセルファイル");
            //シート2を参照
            ISheet worksheet = workbook.GetSheetAt(1);
            //シートの行をworksheet.LastRowNum; で最終行まで参照
            int lastRow = worksheet.LastRowNum;
            //int i = 8 パラメータ値の行からアドレスの行まで
            //8行目から9行目までの2行を取得
            for (int i = 8; i <= 9; i++)
            {
                //i番目の行を取得
                IRow row = worksheet.GetRow(i);
                //回して参照した値を格納
                List<string> sv = new List<string>();
                //セルを4番目から(左から右に)ある数だけ回す。
                //空白を避けるため。
                for (int j = 4; j < row.Cells.Count; j++)
                {
                    ICell cell = row?.GetCell(j);

                    //セルの型に応じたプロパティを参照する

                    switch (cell.CellType)
                    {
                        //セルの型がstringのとき
                        case CellType.String:
                            //List<string>のsvに値を格納
                            sv.Add(cell.StringCellValue);
                            //switch文から抜ける
                            break;
                        //セルの型が数値のとき
                        case CellType.Numeric:
                            //数値をstringに変更し格納
                            sv.Add(Convert.ToString(cell.NumericCellValue));
                            //switch文から抜ける
                            break;
                        //セルの中身が空
                        case CellType.Blank:
                            //空を代入
                            sv.Add("");
                            //switch文から抜ける
                            break;
                        //case以外のとき(elseの役割)
                        default:
                            //空を代入
                            sv.Add("");
                            //switch文から抜ける
                            break;
                    }
                }

                //1行のセルをすべて取り終わったあとにstringの値をstring[]に変換し、格納
                datalists.Add(sv.ToArray());
            }
            //シート2を閉じる
            workbook.CloneSheet(1);
            workbook.Close();

           

            foreach (string list in datalists[0])
            {
              
                listView1.Items.Add(list);
            }
            foreach (string lists in datalists[1])
            {
                listView1.Items.Add(lists);
            }
         
        }
    }
}

Comments

[退会済みユーザー]

https://teratail.com/questions/5dk3mck2b6jptz　←コメントで聞かれたことをほっといて、ここもまた同じことになるのかな？

[sueteacher]

teratailは「プログラミングに関する問題解決を行う場」です
プログラミングに関して本当に困っている人をサポートすること、また、価値のあるコンテンツを世の中に残し届けることを目的としています。

[退会済みユーザー]

その前にマナーというものがあるのですよ。質問 https://teratail.com/questions/5dk3mck2b6jptz に対してコメントを受けてるのに何日も無言で放置しておく人が言えるセリフではありません。

Answer 1

誰も回答しないようなので小言とフォローを

１．ListViewに複数列を表示

subItems.addも使用したとなってますが使い方を間違ってませんか？
・ListViewの表示形式を詳細モードにしましたか？
・ListViewの指定行のSubItemsにちゃんと設定しましたか？

２．nullの扱い

ICell cell = row?.GetCell(j);

としてますがその直前のコードで

for (int j = 4; j < row.Cells.Count; j++) {

のようにrow.Cellsを使用してますので本当にrowがnullの場合、row?.GetCellを使用する前のrow.CellsでNullReferenceExceptionが発生するはずです
なのでrow?.GetCellの「?」は意味がありません。

３．cellの値取得

CellType.StringとCellType.Numericの時しか値を使用しないのであればCellType.Blankのケースはdefault側で処理したほうがシンプルだと思います。

４．datalistsの型

//1行のセルをすべて取り終わったあとにstringの値をstring[]に変換し、格納
datalists.Add(sv.ToArray());

としてますが掲示してあるソースから配列とコレクションの使い分けが出来てるとは思えません。
わざわざ配列に変換しなくてもList<string>のまま入れてしまったほうが簡単です。
なのでdatalistsの型は

List<List<string>> datalists = new List<List<string>>();

でいいと思います。
※この点は本来のソースの状況次第で考えは変わると思います。

５．シート2を閉じる/ブックを閉じる

//シート2を閉じる
workbook.CloneSheet(1);
workbook.Close();

「シート2を閉じる」となってますが前者のメソッド(CloneSheet)はクローン(複製)処理ですよ？

６．変数名

もう少し.NETの名前付けのガイドラインを読まれたほうがいいと思います。
名前付けのガイドライン

７．シートの取り扱い

workbook.GetSheetAt(1)

workbook.CloneSheet(1);

のようにシート位置で参照してますがシート名でアクセスするほうがいいのではと思ってます。
シートの位置よりシート名のほうが変更されにくいのかなとは思いますがシート名が不明ということであればシート位置で参照するというのは仕方がないと思います。

その他

それ以外にSurferOnWwwさんの言われるように質問に対して反応してくださった方がおられたらちゃんと返信すべきだと思います。
今回の内容であればかなり簡単な内容なので回答は付きやすいはずですが今までの経緯等で回答が付きにくくなってると思いますよ。

また、今回の質問であればSubItemsを使用した場合のほうや対象としてるエクセルのサンプルイメージを掲示してくれてたらもう少し違ったと思います。

さらに、
C# NPOIを使用時、エクセルファイルの値が空のときのエラーについて
でepistemeさんが解決策をくれてるのに今回のソースでそれが生かされていません。

ということで小言はこのぐらいに私なら下記のような感じでコーディングします。

※前提条件として取得する行は8行目と9行目のみを取得として処理してます。ほかの行も同様に取得するのであれば、修正が必要です
また、元となるエクセルやリストビューにどのように表示したいかが言葉だけで説明されてるので勘違いはあるかもしれません。
(イメージとしてはエクセルのペースト機能の一部の「行/列の入れ替え」をしたいのかなと予想してます)

public Form1() {
    InitializeComponent();

    listView1.Columns.Add("項目1");
    listView1.Columns.Add("項目2");
    listView1.View = View.Details;

    SetItems();
}

private void SetItems() {

    // エクセルファイルを開く
    var workbook = WorkbookFactory.Create(@"test.xlsx");

    // 対象シートを取得
    var worksheet = workbook.GetSheet("シート2");

    var items = new List<ListViewItem>();

    var row1 = worksheet.GetRow(8);
    var row2 = worksheet.GetRow(9);

    // 1行目のセルの数を列数として扱う
    var cellCount = row1?.Cells.Count ?? 0;

    for (var i = 4; i < cellCount; i++) {

        var cells = new[] { row1.GetCell(i), row2.GetCell(i) };
        var values = new string[] { "", "" };

        // 1行目と2行目のセルの値を格納
        for (var j = 0; j < 2; j++) {

            var cell = cells[j];

            //セルの型に応じたプロパティを参照する
            switch (cell?.CellType ?? CellType.Unknown) {
                //セルの型がstringのとき
                case CellType.String:
                    values[j] = cell.StringCellValue;
                    break;
                //セルの型が数値のとき
                case CellType.Numeric:
                    values[j] = Convert.ToString(cell.NumericCellValue);
                    break;
                default:
                    //空を代入
                    values[j] = "";
                    break;
            }

        }

        // リストビューアイテムを作成
        items.Add(new ListViewItem(values));
    }

    // ブックを閉じる
    workbook.Close();

    // リストビューをクリアして項目を一括追加
    listView1.SuspendLayout();
    listView1.Items.Clear();
    listView1.Items.AddRange(items.ToArray());
    listView1.ResumeLayout();

}

Comments

[sueteacher]

イメージ通りで動きました。ありがとうございます。