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

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

ただいまの
回答率

90.51%

  • C#

    9018questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

  • Visual Studio 2013

    319questions

    Microsoft Visual Studio 2013は、Microsoftによる統合開発環境(IDE)であり、多種多様なプログラミング言語に対応しています。 Visual Studio 2012の次のバージョンです

  • MVC

    257questions

    MVC(Model View Controller)は、オブジェクト指向プログラミングにおけるモデル・ビュー・コントローラーの総称であり、ソフトフェア開発で使われている構築パターンとしても呼ばれます。

  • ASP.NET MVC Framework

    100questions

    ASP.NET MVC Frameworkは、MVCパターンをベースとした、マイクロソフトのウェブアプリケーション開発用のフレームワークです。

  • checkbox

    50questions

    checkboxは、GUIのエレメントです。また、HTML<input>タグのtype属性で扱われる値を指します。

checkboxの値を取得するにはどうすればよいでしょうか

解決済

回答 2

投稿 編集

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

DinKa

score 31

初めて質問させていただきます。

チェックボックスの値を取得し、trueフラグによってExcel帳票に出力する判定を行う処理を作成しています。
ところが、cshtml(view)からチェックボックスの判定を取得しよとしたところ、index(Model)では取得できてますが、出力用のプログラム(Generator)で必ずfalseになってしまいます。
おそらくプロパティの取得の仕方に問題があるのではないかと思うのですが、いろんな方法を試してみましたが、勉強不足のためかうまくいきませんでした。

言語:C#(ASP.NET Framework4.5 MVC)
開発環境:Visual Studio 2013 Pro
質問者:C#は独学で学習時間は半年程度

関係がある部分を抜き出して記述してみます(usingの整理が上手くできていなくてすみません)。

(index.cshtml)
@model A.Models.index
~
<div>
    <select id="select_ABC" name="select_ABC" size="5" multiple="multiple"></select>
</div>

<div>
    @Html.EditorFor(model => model.checkbox)
    @Html.LabelFor(model => model.checkbox)
</div>
~
@section scripts //ABC選択フォームを選択するときに?
<script type="text/javascript">
    $(function () {
    //サブミット時(ダウンロード)
    $("#form1").submit(function (e) {
           util.selectboxAllSelect("#select_ABC");
           return true;
    });
})
</script>

(index.cs)
using System;
using Collections.Generic;
using System.Linq;
using System.Web;
using A.Generator.Report;

namespace A.Models
{
    public class index
    {
        [DisplayName("選択フォーム")]
        public string[] select_ABC { get; set; }
        [DisplayName("チェックボックス")]
        public bool checkbox { get; set; }
    }
}

(Control.cs)
using System;
using OfficeOpenXml;
using Collections.Generic;
using System.Data.Linq.SqlClient;
using System.IO;
using System.Text;
using System.Web;
using A.Generator.Report;

namespace A.Controllers
{
    public class Control : Controller
    {
        ///<param name="postdata"></param>
        ///<returns></returns>
        ///<remarks>
        ///Excel出力の実行
        ///</remarks>
        public AcctionResult Report(index postdata)
        {
            var generator.Generate();

            //コレクションの生成(checkboxとは関係ないコレクションです)
            postdata.select_ABC = postdata.select_ABC ?? (new string[] { }); //ABCを選択したコレクション
            foreach (var item in postdata.select_ABC)
            {
                generator.abccodes.Add(new AbcOutputCode(item));
            }//チェックボックス以外にpostdataの内容です
            //印字用条件のセット
            generator.SetFromSession(); //印字用検索条件の設定(画面から出力、セッション)
            //帳票の生成
            generator.Generate();
            //帳票を出力しレスポンスとして返す
            return generator.Response(); //生成したEXCELをWebのレスポンスとして返す
        }
    }
}



(Report.cs)
using System;
using System.Collections.Generic;
using System.Data.Linq.SqlClient;
using System.IO;
using System.Linq;
using System.Text;
using System.Text.RegularExpressions;
using System.Threading.Tasks;
using System.Web;
using System.Web.Mvc;
using A.Models;

namespace A.Generator.Report
{
    ///<summary>
    ///帳票(Excel)出力
    ///</summary>
    public class ReportGenerator
    {
        public List<Info> Infs = new List<Info(); //出力用のIDを選択するリストを呼出
        public bool checkbox { get; set; }

        ///<summary>
        ///コンストラクタ
        ///</summary>
        public Report(DefultContext dbcontext) 
        ///DefaultContextはテーブルのデータを受け渡すクラスで、DbContextを継承しています
        : base(dbcontext) 
      {
      }

        //帳票出力
        public override void Generate()
        {
            //データ行の出力
            foreach(var sellist in Infs)
            {
                excelHelper.CellOutput(sellist.id)
                excelHelper.BeginDataLine();
                foreach(var item in id.abc) //選択されたABCのデータ行
                {
                    excelHelper.Output(item.PrintData());
                }
                if(checkbox)
                {
                    excelHelper.CellOutPut(sellist.check)
                }
                excelHelper.NewRow();
            }

            //見出し行の出力
            excellHelper.HeaderOut("ID")
            foreach (var item in abc)
            {
                OutputHeader(excelHelper, item, "ABC");
            }
            If(checkbox)
            {
                excelHelper.HeaderOut("チェック");
            }
        }
    }
}

    

試した方法
・(Control.cs)のpostdata(内容はAとcheckbox)をウォッチリストに追加して、checkboxの値がtrueになっているか監視しています
→プログラムを修正していたら、勝手にfalseになる場合とならない場合がありました、どちらにしても結果は出力されないになりましたが
・(index.chhtml)@model A.Models.index → @model A.Generator.Report
→にしてみましたが、結果は変わらずでした

つたない文章で申し訳ありませんが、よろしくお願いします。

  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 2

checkベストアンサー

+1

こんばんは
Controllerの
    public AcctionResult Report(index postdata)
部分のpostdataでModelBindingできているということで大丈夫でしょうか。

問題の個所はReportGeneratorクラスで宣言しているcheckboxの値が
常にfalseということなのですが
どこでModelBindingした値をcheckboxにいれているのでしょうか。
代入していない場合、できていない場合のbool型の初期値はfalseです。

関係がある部分を抜き出して記述してみます
とのことなので、見落としているだけでしたら申し訳ないです。

またASP.NET MVCで開発しているので、ある程度でしたら相談にのれるかと思います。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/03/02 10:40 編集

    ご回答ありがとうございます。

    Model Bindingについては勉強不足であまり理解できずに使っていました。
    (http://qiita.com/kazuhisam3/items/94542f6d7ccf3acca41c
    を読んでみたのですが、いまひとつわかりませんでした。
    現在動いているselect_ABCのフォームを参考にチェックボックスを作ったので、checkboxをpostdataに組み込むことで無意識にModel Bindingできてたのだと思います。

    ReportGeneratorクラスでは、そこが抜けているのだと思います。
    実際に、Viewの値を取得するのは、
    (view)@Html.EditorFor(model => model.checkbox)
    (generator)public bool checkbox {get; set;}
    だけで取れると思っていましたから。

    実際にReportgeneratorクラスにModelBindingを作るとしたら、
    public void BindCheckbox(index bc)
    {
    var bc = bc.checkbox;
    }

    if(checkbox) → if(bc)

    以上でよいでしょうか?
    初歩的なところなのかもしれませんが、よろしくお願いします。

    キャンセル

  • 2016/03/02 13:13

    ModelBindingははじめは自動でコントロールの値やQueryStringをサーバーサイドで取得可能な状態にするくらいの認識で大丈夫だと思います。

    今回の場合は、コントロールのname要素が、Modelのプロパティの名称と一致していることから、
    ``
    public AcctionResult Report(index postdata)
    ``
    にModelBindingされたのだと思います。
    なので、今回のケースではModelBindingを作成する必要はありません。

    問題はControllerからReportGeneratorクラスへの値の受け渡しにあると考えています。
    クラス間の値の受け渡し(Controller ->ReportGenerator )は通常のC#同様
    インスタンス化する際に値を渡すなどの方法で、受け渡します。
    なので、ReportGeneratorクラスをインスタンス化する際や
    メソッドを呼び出す際にReportGeneratorクラスに値を渡しその値を利用します。

    参考までに
    https://github.com/shika358/teratail28558
    に簡単なサンプルを置いておきます。

    .Net Framework4.5以上であれば動作すると思います。

    不明な点があればお伝えください。

    キャンセル

  • 2016/03/02 13:18

    Qiitaの記事であれば
    http://qiita.com/kazuhisam3/items/f056819172d2b6d36a8c
    ※少し難しいかもしれないです

    そのほかであれば
    http://www.amazon.co.jp/dp/B00XBF8E9K/ref=dp-kindle-redirect?_encoding=UTF8&amp;btkr=1
    この本が基本的な知識を網羅的にわかりやすく書かれていると感じました。





    キャンセル

  • 2016/03/02 14:07

    丁寧な回答ありがとうございます。
    直感的にはまだ理解できていませんが、早速ご指摘いただいた内容をもとに構築して確認してみます。

    キャンセル

  • 2016/03/02 14:37

    GitHubのサンプルを動かしてみて、ご自身のPGに反映していただければ大丈夫かとは思います。
    構築し躓く点があれば教えてくださいね。

    キャンセル

  • 2016/03/02 15:50

    大変申し訳ありませんが、GitHubの使い方を理解するところから始めているので、少し時間がかかりそうです。

    キャンセル

  • 2016/03/02 15:55

    失礼しました、URLを開いて右中ほどあたりに、Download ZIPというボタンはありませんでしょうか?
    そこから、ソリューションがダウンロードできますよ。

    キャンセル

  • 2016/03/02 18:42

    即お返事いただいたのに、確認遅くなりすみません。
    非常に親切に対応していただきありがとうございます。

    サンプルを動作してみまして、おぼろげながら理解できました。
    実際に結果が出るのはもう少し時間がかかりそうですが、まずはお礼まで。

    キャンセル

  • 2016/03/02 19:34

    サンプルでは問題なく動作しましたが、実際動かしている環境に適応したところ、
    Controlクラスのreportが他のファイルで使われているため、
    2つ値を持ってしまうとうまく動作しなくなってしまいました。

    追加したのは、以下の内容です。
    (Control.cs)
    var report = new ReportGenerator(db, postdata);


    (Report.cs)
    ///<summary>
    ///コンストラクタ
    ///</summary>
    private readonly index _postdata;
    public Report(DefultContext dbcontext, index postdata)
    ///DefaultContextはテーブルのデータを受け渡すクラスで、DbContextを継承しています
    : base(dbcontext)
    {
    _postdata = postdata;
    }

    他におかしなところがないかもう少し内容を確認してみます。

    キャンセル

  • 2016/03/02 20:01 編集

    ①もともとのコンストラクタ
    public Report(DefultContext dbcontext) : base(dbcontext) {}

    ②追加するコンストラクタ
    public Report(DefultContext dbcontext, index postdata) : this(dbcontext)
    {
    _postdata = postdata;
    }

    こんな感じで2つコンストラクタを書いてはいかがですか?

    var report = new ReportGenerator(db, postdata);

    で呼ばれるコンストラクタは、②の追加するコンストラクタです。
    : this(dbcontext) で①もともとのコンストラクタを呼んでいます。

    キャンセル

  • 2016/03/03 09:34

    shika358さん、ご回答ありがとうございます。

    コンストラクタを分けて記述しましたら、問題なく出力することができました!

    C#以外もプログラム経験がほとんどない中だったので、1か月以上この機能で悩んでいましたので、本当に助かりました。
    細かい条件などがまだ少し残っていますが、その点は自分で解決してみます。

    大変親切に丁寧に教えていただいて、ありがとうございました。

    キャンセル

  • 2016/03/03 11:49

    解決したようでよかったです。
    ASP.NET MVC FrameworkやC#あたりのタグは確認していますので
    またわからないことがあれば質問してみてくださいね。

    キャンセル

  • 2016/03/04 16:13 編集

    懇切丁寧に教えていただいたので、本当に助かりました。
    C#もままならない中、ASP.NET MVC Frameworkの環境をやり始めたのでわからないことだらけですが、自分で調べても全く進まないときまた質問させていただきます。

    とは言いましたが、実際にもう1件長期にわたってエラーの解決ができな部分があるので、問題が明確になりましたら改めて質問させていただきます。

    本当にありがとうございました、またよろしくお願いいたします。

    キャンセル

+1

監視すべき部分が監視できていないような気がします。

checkboxの値がtrueになっている」=「checkboxにチェックが付いている状態」
ということでしたら、ウォッチリストに追加するのはcheckboxCheckedプロパティあたりにした方がよいのではないでしょうか。

ウォッチリスト

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/02/29 17:19

    ご回答ありがとうございます。

    ウォッチリストにcheckboxのCheckedを追加とのことですが、
    (Report.cs) のReportGeneratorに、
    var check = checkbox.Checked;
    を追記してウォッチリストを追加しようとしたところ、フィールド初期化子は、静的でないフィールド、メソッドまたはプロパティ'A.Generator.Report.ReportGenerator.checkbox.get'を参照できませんと出てしまいます。
    また、var自体もなぜかusingディレクティブまたはアセンブリ参照が不足していますとでてしまいます。
    おそらく使い方が違うのでしょうけれど。

    せっかくご回答いただいたのに、理解できずに申し訳ありません。

    キャンセル

  • 2016/02/29 17:48

    ソースをいじるのではなく、(Report.cs)のGenerate関数内の適当な行にブレークポイントを張ります。

    ```
    if(checkbox) // ←この辺がいいかもしれないです
    {
    excelHelper.CellOutPut(sellist.check);
    }
    excelHelper.NewRow();
    ```

    で、デバッグを開始し、ウォッチリストにcheckbox.Checkedを追加すればよいと思います(元回答の方に画像添付しましたので参考にしてください)。
    プロパティの値を直接ウォッチリストに追加できるので、敢えて変数に代入する必要はないと思います。

    キャンセル

  • 2016/02/29 19:36

    丁寧なご説明、ありがとうございます。
    参考にさせていただいて、明日実施してみます。

    キャンセル

  • 2016/03/01 09:30

    昨日はありがとうございました。
    ブレークポイントを設定し、ウォッチリストにcheckboxを加えてみたところ、常にfalseになっていました。
    また、
    (index.cshtml)@model A.Models.index → (Report.cs)A.Generator.Report.ReportGenerator
    としても、postdataのcheckboxはtrueを返してくれて、Generateのcheckboxはfalseのままでした。
    上記を見る限り、
    postdataのcheckboxとGenerateのcheckboxの値は連動してない?(以前何かで連動したと勘違いしてたのか、実際に連はできるのか)ことがわかりました。また、viewの@model~も関係ない?ですかね。

    となると、ReportGeneratorでの、checkboxの値の取得の仕方に問題があるのだと思いますが、どのようにして取得をすればいいのでしょうか?

    キャンセル

  • 2016/03/01 12:33

    すみませんASP.NET MVCでWEBアプリを作ったことがないので細かいところはわかりませんが、恐らく値の受け渡しがうまくできていないのでしょう。

    postdataはチェックボックスの配列か何か?と推測しますが、これがどのように定義されていてどのようにControllerに渡しているのか、等を詳しく書いてみると回答が付きやすいかと思います。

    キャンセル

  • 2016/03/01 12:55

    ありがとうございます。
    確かにpostdataがいきなりでてきているので、わかりづらくなっている原因ですね。
    元の内容を少し修正してみます。

    キャンセル

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

  • C#

    9018questions

    C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

  • Visual Studio 2013

    319questions

    Microsoft Visual Studio 2013は、Microsoftによる統合開発環境(IDE)であり、多種多様なプログラミング言語に対応しています。 Visual Studio 2012の次のバージョンです

  • MVC

    257questions

    MVC(Model View Controller)は、オブジェクト指向プログラミングにおけるモデル・ビュー・コントローラーの総称であり、ソフトフェア開発で使われている構築パターンとしても呼ばれます。

  • ASP.NET MVC Framework

    100questions

    ASP.NET MVC Frameworkは、MVCパターンをベースとした、マイクロソフトのウェブアプリケーション開発用のフレームワークです。

  • checkbox

    50questions

    checkboxは、GUIのエレメントです。また、HTML<input>タグのtype属性で扱われる値を指します。