C# Npoiを使用してエクセルファイルの1行の全取得ができない。途中で読み込みが止まっていた。

Question

【質問】
1.Npoiを使用して1行すべての取得ができない。(Npoiの仕様？？)
2.別で取得したList<string[]>の値を、リストビューの別の列に表示したい。(追加したい。)
3.エクセルファイルを1度で読み込みたいがエラーになる(先頭に空があるため)
【環境】
C# .netframework4.7 visual studio2022 Windows Form

【仮実装】
・Npoiでエクセルファイルを読み込み、変数に格納し、その後リストビューに表示
・1行がすごく長い列のため、ループ文を使用

【目的】
・エクセルファイル1行ずつ全取得後、使用するところだけ変数や配列に格納したい。(【例】1行取得したらG列目以降を格納または表示)
・検索後、検索した1行の{"文字列,文字列"}のところをリストビューに列に追加して別の1列としてにまとめたい。

【参考】
https://teratail.com/questions/8rmkh3qufecejj

【例】
・1行の列を700用意した所、1行の列の全取得数が695と合わなかった。
・列を500にしたとき、1行の列の全取得が496となってしまっている。

C#初心者学習中。
手厳しくお願い致します。

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

namespace WindowsFormsApp6
{
    public partial class Form1 : Form
    {
        List<string[]> list = new List<string[]>();
        List<string[]> reslist = new List<string[]>();


        public Form1()
        {
            InitializeComponent();
            listView1.Columns.Add("No");
            listView1.Columns.Add("在庫場所");
            listView1.Columns.Add("取引先");
            listView1.Columns.Add("文字列,文字列");
            listView1.View = View.Details;

            SetItems();

        }

        private void SetItems()
        {

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

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

            var items = new List<ListViewItem>();

            //7行目、8行目
            var row1 = worksheet.GetRow(5);
            var row2 = worksheet.GetRow(6);
            var row3 = worksheet.GetRow(7);

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

            //左から6列目のセルから最後まで
            for (var i = 6; i < cellCount; i++)
            {

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

                // 1行目と2行目のセルの値を格納
                for (var j = 0; j < 3; 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();

        }

        private void ExRead1()
        {
            //エクセルファイルを開いた
            IWorkbook workbook = WorkbookFactory.Create(@"test.xlsx");
            //index1のシートを参照
            ISheet worksheet = workbook.GetSheetAt(0);
            //シートの行をworksheet.LastRowNum; で最終行まで参照
            int lastRow = worksheet.LastRowNum;
            //int i = 8 最初の行　から最後の行まで
            for (int i = 8; i <= lastRow; i++)
            {
                //i番目の行を取得
                IRow row = worksheet.GetRow(i);
                //回して参照した値を格納
                List<string> sv = new List<string>();
                //セルを0番目から(左から右に)ある数だけ回す。
                for (int j = 0; j < row.Cells.Count; j++)
                {
                    ICell cell = row?.GetCell(j);

                    //セルの型に応じたプロパティを参照する
                    if (cell == null)
                    {
                        sv.Add("");
                    }
                    else
                    {
                        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[]に変換し、格納
                list.Add(sv.ToArray());
            }
            workbook.Close();
        }
        private void Seachitem()
        {
            //検索のまわす。
            foreach (string[] v in list)
            {
                //入力された文字とおなじのとき
                if (textBox1.Text == v[1])
                {
                    label1.Text = v[1];
                    label2.Text = v[2];
                    reslist.Add(v);
                }
               
            }
        }

        private void button1_Click(object sender, EventArgs e)
        {
            ExRead1();
            Seachitem();

            List<string> lnsv = new List<string>();
            var lnitems = new List<ListViewItem>();
           
            //foreachで回す。
            int j = 0;
            foreach (string[] l in reslist)
            {
                lnitems.Add(new ListViewItem(l));
                for(int i = 0; i < l.Length; i++)
                {
                    ListViewItem lvi = new ListViewItem(l[i]);
                    listView1.Items.Add(lvi);
                }
              
            }
           

        }
    }

    
}

Comments

[YAmaGNZ]

空のセルが含まれてないのでは？

[KOZ6.0]

[NPOI C#] シート使用されている最終行・最終列を取得する
https://hensa40.cutegirl.jp/archives/4122
Cells.Count でなく、LastCellNum を使えば解消しないでしょうか？

[sueteacher]

返事が遅れてしまいすみません。
LastCellNumを使用したら最後まで読み取りができましたが、row1?の後ろではなく、for文の中での使用になるのでしょうか。
------------修正してみた。　動いた。
// 1行目のセルの数を列数として扱う //LastCellNumにすることで最後まで取得できました。
var cellCount = row1?.LastCellNum ?? 0;
-------------
-------------こちらだとエラーが発生
//左から6列目のセルから最後まで
for (var i = 0; i < cellCount; i++)
-------------

[KOZ6.0]

row1?.LastCellNum と記述すると short? です。
row1 が null でないことが保証されているなら、row1.LastCellNum で良いでしょう。
for は、LastCellNum まで回さなければならないので
for (var i = 0; i <= row1.LastCellNum; i++)
となります。

Answer 1

ちょっと答えになってしまいますがやりたいことは
・textbox1に商品名を入力して対象行の情報をlistview1に表示
・label1には商品名を表示
・label2には値段を表示
だと予想します。
プログラムは同じような処理がまとまってないので自分なりに修正してみました。
掲示してあるプログラムを大きくは以下のように修正しました。
・セルの値を文字列として取得する関数を追加
・画面を開いた時の処理は不要と考え、リストビューのみの初期化
・データの保持は行わず、button1の押下で検索＆画面に結果表示
・try～finalyでbookを閉じる
という感じです。

示したコードで想定通り動いてしまうと勉強にはならないかもしれませんが、その場合は何とか自分のコードになるように努力してみてください。

※前回、回答してしまったのでフォローします。

    public partial class Form1 : Form {

        public Form1() {
            InitializeComponent();

            listView1.Columns.Add("No");
            listView1.Columns.Add("在庫場所");
            listView1.Columns.Add("取引先");
            listView1.Columns.Add("文字列,文字列");
            listView1.View = View.Details;
        }

        private void button1_Click(object sender, EventArgs e) {

            listView1.SuspendLayout();
            SetItems();
            listView1.ResumeLayout();

        }

        private void SetItems() {

            listView1.Items.Clear();

            label1.Text = "";
            label2.Text = "";

            var targetItemName = textBox1.Text;

            if (string.IsNullOrEmpty(targetItemName)) {
                return;
            }

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

            try {

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

                var lastRowNum = worksheet.LastRowNum;
                IRow dataRow = null;

                for (var i = 8; i <= lastRowNum; i++) {

                    var row = worksheet.GetRow(i);

                    if (row == null) {
                        continue;
                    }

                    // 商品名取得
                    if (GetCellValue(row.GetCell(1)) != targetItemName) {
                        continue;
                    }

                    dataRow = row;
                    break;
                }

                // 商品名に一致した行が見つからない
                if (dataRow == null) {
                    return;
                }

                var items = new List<ListViewItem>();

                // 商品名と金額を設定
                label1.Text = GetCellValue(dataRow.GetCell(1));
                label2.Text = GetCellValue(dataRow.GetCell(2));

                // №(列)のセルの数を列数として扱う
                var lastCellNum = dataRow.LastCellNum;

                for (var i = 6; i <= lastCellNum; i++) {

                    var number = GetCellValue(worksheet.GetRow(5)?.GetCell(i));
                    var zaiko = GetCellValue(worksheet.GetRow(6)?.GetCell(i));
                    var torihikisaki = GetCellValue(worksheet.GetRow(7)?.GetCell(i));
                    var data = GetCellValue(dataRow.GetCell(i));

                    // すべて空のデータは対象外
                    if (string.IsNullOrEmpty(number)
                        && string.IsNullOrEmpty(zaiko) 
                        && string.IsNullOrEmpty(torihikisaki) 
                        && string.IsNullOrEmpty(data)) {
                        continue;
                    }

                    items.Add(new ListViewItem(new[] { number, zaiko, torihikisaki, data }));
                }

                if (items.Count == 0) {
                    return;
                }

                listView1.Items.AddRange(items.ToArray());

            } finally {

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

            }

        }

        /// <summary>
        /// セルの値を文字列で取得
        /// </summary>
        /// <param name="cell">対象セル</param>
        /// <returns>対象セルが存在し、セルの値が文字列か数値の場合はセルの値を示す文字列。それ以外は空文字。</returns>
        private string GetCellValue(ICell cell) {

            if (cell == null) {
                return "";
            }

            //セルの型に応じたプロパティを参照する
            switch (cell.CellType) {
                //セルの型がstringのとき
                case CellType.String:
                    return cell.StringCellValue;
                //セルの型が数値のとき
                case CellType.Numeric:
                    return Convert.ToString(cell.NumericCellValue);
            }

            return "";
        }

    }