質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.48%

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

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 7,127

score 155

[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");
    }    
}
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • twck

    2016/09/01 22:38

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

    キャンセル

  • shikasama

    2016/09/02 17:21

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

    キャンセル

  • twck

    2016/09/02 17:44

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

    キャンセル

  • shikasama

    2016/09/05 11:50

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

    キャンセル

回答 1

checkベストアンサー

0

どうやら 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 を読み込んで、編集して、送信する、ということができそうですがどうでしょうか?

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/09/07 15:29

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

    キャンセル

  • 2016/09/07 16:12

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

    キャンセル

  • 2016/09/07 16:50

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

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

    キャンセル

  • 2016/09/07 17:02

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.48%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る