[openXML]ASP.NETでダウンロードさせたExcelが開けない。

Question

###[openXML]ASP.NETでダウンロードさせたExcelが開けない。
Templateに追記したファイルをダウンロードさせるため、
まず編集せずにダウンロードさせようと思いました。
Excelの操作にはopenXMLを使用しています。
ダウンロードはできましたが、肝心のファイルが開けませんでした。ご教示ください。

###発生している問題・エラーメッセージ
ダウンロードしたファイルを開こうとすると以下のエラーが発生します。

Excelでファイル'ダウンロードファイル名.xlsx'を開くことができません。
ファイル形式またはファイル拡張子が正しくありません。
ファイルが破損しておらず、ファイル拡張子とファイル形式が一致していることを確認してください。

###該当のソースコード

public ActionResult DownloadExcel()
{
	// xlsxドキュメントを開く。
	// 第2引数：編集するかどうか。
	using (SpreadsheetDocument document = SpreadsheetDocument.Open(Server.MapPath("~/App_Data/Template.xlsx"), true))
	using(MemoryStream ms = new MemoryStream())
	{
		// Workbookにアクセス。
		WorkbookPart wbPart = document.WorkbookPart;

		// シート情報を取得
		Sheet targetSheet = document.WorkbookPart.Workbook.Sheets.Elements<Sheet>().First();
		if (targetSheet == null)
		{
			return null;
		}

		// シート情報からIDを取得し、WorksheetPartを得る
		WorksheetPart wsPart = (WorksheetPart)(wbPart.GetPartById(targetSheet.Id));

		// ワークブックを保存
		wsPart.Worksheet.Save(ms);
		return File(ms.ToArray(), "application/msexcel", "ダウンロードファイル名.xlsx");
	}
}

###試したこと
wsPart.Worksheet.Save(ms)したあとにmsが閉じてしました。
↓を参考にしました。
http://xptn.dtiblog.com/blog-entry-19.html

wsPart.Worksheet.Save(ms);
ms.Close();
return File(ms.ToArray(), "application/vnd.ms-excel", "ダウンロードファイル名.xlsx");

が、変わらずダウンロードしたファイルは壊れて開けませんでした。

msが閉じていることはあまり関係ないみたいでした。
http://xptn.dtiblog.com/blog-entry-19.html

MemoryStreamを使わずに直接ダウンロードさせたら開けました。
なので、MemoryStreamにドキュメントをセーブした時点で中身が壊れてしまったものだと思われます。

###解決
以下の様にすることでメモリ上でExcelファイルの編集を行い、一時ファイルを作成することなくクライアントにダウンロードさせることができました。

public ActionResult DownloadExcel()
{
	// テンプレートファイルのパス
	var templateName = Server.MapPath("~/App_Data/Template.xlsx");
	
	// ファイルをbyte配列に変換
	var byteArray = System.IO.File.ReadAllBytes(templateName);
	using (MemoryStream ms = new MemoryStream())
	{
		/* 
		 * MemoryStreamにbyte配列を書き込む
		 * MemoryStream上にExcelドキュメントができる
		 */
		ms.Write(byteArray, 0, (int)byteArray.Length);
		// MemoryStream上のExcelドキュメントを開く
		using (SpreadsheetDocument spreadSheet = SpreadsheetDocument.Open(ms, true))
		{
			WorkbookPart wbPart = spreadSheet.WorkbookPart;

			Sheet targetSheet = spreadSheet.WorkbookPart.Workbook.Sheets.Elements<Sheet>().First();
			if (targetSheet == null)
			{
				return null;
			}

			WorksheetPart wsPart = (WorksheetPart)(wbPart.GetPartById(targetSheet.Id));
			
			/*
			 * ここでいろいろExcelファイルを編集する
			 */
			
			wsPart.Worksheet.Save();
		}
		return File(ms.ToArray(), "application/ms-excel", "ダウンロード時のファイル名.xlsx");
	}	
}

Comments

[twck]

ASP.NETではなくopenXMLの問題ではないかと思うので、質問のタイトルやタグに「openXML」を入れた方が見つけてもらいやすくなりますよ。
あと、openXMLの代わりにFileStreamで読み込んでMemoryStreamに書き込む方法を試してみて、その結果を「試したこと」に追記すれば問題がASP.NETとopenXMLのどちらなのかを切り分ける情報になると思います。

[shikasama]

twck様
アドバイスありがとうございました。

[twck]

私が回答までできればよいのですが、私はopenXMLを使ったことがないのですみません。私はopenXMLではなくNPOIを使用してEXCELを編集しているのですが、よくミスってファイルが破損するので、おそらくASP.NETではなくopenXMLの使い方に問題があると思います。

[shikasama]

twck様
お気遣いありがとうございます。
もう少し調べてみようと思います。

Answer 1

どうやら teratail には Open XML SDK の利用者がいないようなので、導入して少し調べてみました。

// ワークブックを保存
wsPart.Worksheet.Save(ms);

質問文に出てきたこの部分はワークブックではなくワークシートの出力を行っているようです。つまり Template.xlsxファイルの中の一部分しか出力されない。じゃあ Workbook.Saveならどうだとやってみましたが、今度はワークブックの基本情報だけ出力されて、ワークシートが出力されない。しばらく調べてみたのですが、どうにも全体をMemoryStreamに書き出す方法が見つからない。

開いたファイルを書き換えるのは普通にできるので、ファイルの代わりにMemoryStreamから開いてみたらどうだろうとやってみたら出来ました。

// テンプレートファイルのパス
var filename = Server.MapPath("~/App_Data/Template.xlsx");

// ファイルをMemoryStreamに読み込み
var bytes = System.IO.File.ReadAllBytes(filename);
using (MemoryStream ms = new MemoryStream(bytes))
{
	// MemoryStream上のExcelドキュメントを開く
	using (SpreadsheetDocument document = SpreadsheetDocument.Open(ms, true))
	{
		// 編集する
	}

	return File(ms.ToArray(), "application/ms-excel", "ダウンロード時のファイル名.xlsx");
}

これなら Template.xlsx を読み込んで、編集して、送信する、ということができそうですがどうでしょうか？

Comments

[twck]

最初は出来ないと書きましたが、出来たっぽいので回答を書き直しました。

[shikasama]

twck様
ありがとうございます。
わざわざ導入していただいて申し訳ありません。
大変助かりました。
twck様の全体を書きだせないという情報を受け、SpreadsheetDocument自体をどうにか別に保存できないかなと調べてみたところ、以下がヒットしました。
http://stackoverflow.com/questions/5203150/openxml-spreadsheat-save-as
MemoryStreamにファイルのバイト配列を書き込んで、そのMemoryStreamを使ってSpreadsheetDocumentをopenするというやり方です。
結果、意図していたことができました。
参考としてできたコードを追記しておきます。
本当にありがとうございました。

[twck]

あれ～？MemoryStreamを使ってOpenしたらCloseするとき(usingが閉じるとき)に例外が発生したからBufferedStreamに変えたのに、リンク先見てMemoryStreamに戻したら正常に動作するよ！何故だ！

とりあえず出来てよかったですね。後で回答をMemoryStreamだけを使ったものに書き直します。

[twck]

回答を変更しました。
BufferedStreamとMemoryStreamを使用した方法から、MemoryStreamだけを使用した方法に変更しました。