poiを使用してExcelの値を表示したい

Question

前提・実現したいこと

Excelの「投手」シートの選手の背番号、名前を順に表示していき、最後の行まで読み込み終わると「外野手」シートで同じ処理を行いたい。

発生している問題・エラーメッセージ

Excelの行を一行ずつ見ていき、nullが出るまで処理を続けたいが、値が入っていないセルを見ると次の処理へ移らずNullPointerExceptionが発生する。
![投手シート
外野手シート

Cell cell2 = row2.getCell(0);
この行でエラーが出ているようです。

Workbook workbook = WorkbookFactory
					.create(new FileInputStream(
							"C:Sample.xlsx"));
			Sheet sheet = workbook.getSheet("投手");
			int a = 1;

			Row row = sheet.getRow(a);
			Cell cell = row.getCell(0);
			while (cell != null) {
				Row row2 = sheet.getRow(a);
				Cell cell2 = row2.getCell(0);
				Row row3 = sheet.getRow(a);
				Cell cell3 = row3.getCell(1);
				switch (cell2.getCellType()) {
				case NUMERIC:
					System.out.print("背番号" + (int)cell2.getNumericCellValue()+",");
				default:
					break;
				}
				switch (cell3.getCellType()) {
				case STRING:
					System.out.println("選手名 " + cell3.getStringCellValue());
				a++;
				default:
					break;
				}
			}
			Workbook workbook2 = WorkbookFactory
					.create(new FileInputStream(
							"C:Sample.xlsx"));
			Sheet sheet2 = workbook2.getSheet("外野手");
			int b = 1;
			Row row4 = sheet2.getRow(b);
			Cell cell4 = row4.getCell(0);
			while (cell4 != null) {
				Row row5 = sheet2.getRow(b);
				Cell cell5 = row5.getCell(0);
				Row row6 = sheet2.getRow(b);
				Cell cell6 = row6.getCell(1);
				switch (cell5.getCellType()) {
				case NUMERIC:
					System.out.print("背番号" + (int)cell5.getNumericCellValue()+",");
				default:
					break;
				}
				switch (cell6.getCellType()) {
				case STRING:
					System.out.println("選手名 " + cell6.getStringCellValue());
					b++;
				default:
					break;
				}
			}

試したこと

なかなかpoiについて詳しく書いている記事がなかったので「poi Excel　繰り返し」「java Excel 値取得　繰り返し」などで検索しても有用な記事を見つけることができませんでした。

while文ではなくfor文を使い、値が入っている部分を条件式として記述すると正常に動作したのですが、それだとExcelに値を追加した際コードもいちいち書き換える必要があるため、while文を使い「値があれば処理を行う」というようにすべきだと思いました。
自分で調べたり試していては埒が明かないと感じたので力をお借りしたいです。

Comments

[jimbe]

> NullPointerExceptionが発生する
それはどの行で発生しているのでしょうか。
表示された中に行番号も書かれていたはずです。そして、その行での該当変数には null が入る可能性があり、またその変数が null になることが"何か"(例えば空行であったりデータの終わりであったり)を表すヒントとなる場合もあります。
「なんだ例外か」で終わるのではなく、そのメッセージから情報(「この変数に値を入れているメソッドの戻り値は null もありなのか」)を得た上で調査し、ついでにその現象(「 null になるのはデータがもうないのか」)を利用(「データが無いならループは終わっちゃおう」)することまで思い付くと、イロイロ面白くなってくるかと思います。

[退会済みユーザー]

＞それはどの行で発生しているのでしょうか。
ありがとうございます。書き足しました。
＞「データが無いならループは終わっちゃおう」
このようにループを抜けることがしたいのですがそのためのコードが思いつきませんでした。

[jimbe]

> このようにループを抜けることがしたいのですが
while や for は(switch の中で無ければ) break で抜けられます。

Answer 1

参考までに、貴方なら「空白まで繰り返す」という処理はどのように書きますか？

import org.apache.poi.ss.usermodel.*;

import java.nio.file.Path;
import java.nio.file.Paths;

public class xxHogeHawks {

  public static void main(final String[] args) throws Exception {
    final Path path = Paths.get("./Sample.xlsx");
    final Workbook book = WorkbookFactory.create(path.toFile());

    for (int sheetIndex = 0; sheetIndex < book.getNumberOfSheets(); sheetIndex++) {
      final Sheet sheet = book.getSheetAt(sheetIndex);
      if (sheet.getSheetName().equals("投手") || sheet.getSheetName().equals("外野手"))
        ;
      else
        continue;

      //
      for (int rowIndex = sheet.getFirstRowNum(); rowIndex < sheet.getLastRowNum(); rowIndex++) {
        final Row row = sheet.getRow(rowIndex);
        if (row == null || row.getRowNum() == 0)
          continue;

        final Cell a = row.getCell(0);
        final Cell b = row.getCell(1);
        if (a == null || b == null)
          continue;

        System.out.println(String.format("背番号 %s, 選手名 %s", cellToString(a), cellToString(b)));
      }
    }
  }

  private static String cellToString(final Cell cell) {
    assert (cell != null);

    switch (cell.getCellType()) {
    case NUMERIC:
      return String.valueOf((int) cell.getNumericCellValue());
    case STRING:
      return cell.getStringCellValue();
    default:
      return cell.toString();
    }
  }
}

or

  public static void main(final String[] args) throws Exception {
    final Path path = Paths.get("./Sample.xlsx");
    final Workbook book = WorkbookFactory.create(path.toFile());

    for (Sheet sheet : (Iterable<Sheet>) () -> book.sheetIterator()) {
      if (sheet.getSheetName().equals("投手") || sheet.getSheetName().equals("外野手"))
        ;
      else
        continue;

      for (Row row : (Iterable<Row>) () -> sheet.rowIterator()) {
        if (row.getRowNum() == 0)
          continue;

        final Cell a = row.getCell(0);
        final Cell b = row.getCell(1);
        if (a == null || b == null)
          continue;

        System.out.println(String.format("背番号 %s, 選手名 %s", cellToString(a), cellToString(b)));
      }
    }
  }

Comments

[退会済みユーザー]

上のコードを使用させていただき上手くいきました。
丁寧に回答していただきありがとうございます。

Answer 2

値が入っていないセルを見ると次の処理へ移らずNullPointerExceptionが発生する

「値が入っていないセルを見」なければいい。

• getRow()

Returns: Row representing the row-number or null if its not defined on the sheet

• getCell()

Returns: Cell representing that column or null if undefined.

getRow()もgetCell()もnullを返す可能性があるのだから、返された値のメソッドを呼び出す前にnullかどうかをチェックすればいいだけ。

POIも何も関係ありません。

---

garyageというアカウント名に期待を込めて追記。

• sheetIterator
• rowIterator
• cellIterator

これらのドキュメントをみると、nullを返すとは書いていない。NullPointerExceptionを心配せずに値を取得できる。かもしれない。

Comments

[退会済みユーザー]

返された値のメソッドを呼び出す前にnullかどうかをチェックすればいいだけ。＜＜
if(cell2==nul){nullであることを表示}else{switch分}というようなコードを挿入してみたのですが上手くいきませんでした。
やはりIteratorを使った方がいいですかね。

[shiketa]

> というようなコードを挿入してみたのですが上手くいきませんでした。
cell2だけチェックしたのですか。 rowもcellもrow2もチェックしていますか。ソースが全部ぜんぶ提示されているわけではないのでわかりませんが、そのほかの場所はどうなのでしょうか。ドキュメントにnullを返すとは書いていなくても、それはただのドキュメントなので、実際にnullを返してくるものはありふれています。
> やはりIteratorを使った方がいいですかね。
用途に応じてご随意に。

[shiketa]

おまけ。Excelファイルの仕様はご存じですよね。
https://support.microsoft.com/ja-jp/office/excel-%E3%81%AE%E4%BB%95%E6%A7%98%E3%81%A8%E5%88%B6%E9%99%90-1672b34d-7043-467e-8e27-269d656771c3
> ワークシートの行数と列数 1,048,576 行、16,384 列
XFD1048576のセルに値があったときをかんがえて、ためして、効率がよさげな方法をわたしは採ります。

[退会済みユーザー]

＞rowもcellもrow2もチェックしていますか。
cellをチェックしていなかったのはミスでした。ありがとうございます。
row,row2はチェックする必要がないのではないでしょうか。cell,cell2と違いrow,row2にはセルの値がはいるわけではないので。

参考までに、貴方なら「空白まで繰り返す」という処理はどのように書きますか？
＜「値が入っていないセルを見」なければいい。
と回答していただけましたが、自分ではこの処理を行うためのコードが思いつかないのでもう少し具体的に教えていただけると本当に助かります。

[shiketa]

> 貴方なら..
提示されているソースだけではナニをしたいのかがわたしにはわかりません。実行可能なソースを提示していただけたなら、考えます。質問欄に追記してください。

[退会済みユーザー]

ありがとうございます。追記します。

[shiketa]

お待ちします。

> row,row2はチェックする必要がないのではないでしょうか。cell,cell2と違いrow,row2にはセルの値がはいるわけではないので
ここだけ。
「セルの値がはいる」かどうかは関係ありません。rowがnullのとき`row.getCell(0)`としたらどうなりますか。row2がnullのとき`row2.getCell(0)`としたらどうなりますか。そいうこうことです。

[退会済みユーザー]

やりたいことを具体的にしてみました。
＞「セルの値がはいる」かどうかは関係ありません。
ありがとうございます。それも試してみようと思います。