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

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

ただいまの
回答率

90.33%

  • C#

    7710questions

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

  • ASP.NET MVC Framework

    69questions

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

MVC モデルのバリデーションについて

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 3,671

ShouOkada

score 26

モデルでアノテーション(Required)を指定しているプロパティが4つあります。

 [Required]
       public string A { get; set; }
  [Required]
       public string B { get; set; }
  [Required]
       public string C { get; set; }
  [Required]
       public string D { get; set; }

 
条件によってこのアノテーションを無効にしたり有効にしたりする方法はありますでしょうか?(Dだけは必ず有効)

FLGというプロパティがあるとして、その値が1の時はRequiredを有効0の時は無効みたいな事が出来ればいいと思っています。

カスタムバリデーションなどで下記のようなことも考えましたが、「m.A」の所をABCすべてに対応させる方法がわかりませんでした。

  public static ValidationResult CheckRequired(Test m)
    {
        if (m.FLG=="1" && m.A == null)
        {
            return new ValidationResult("エラー");
        }
        return ValidationResult.Success;
    }
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • SurferOnWww

    2017/06/14 22:17

    ASP.NET MVC の話ですよね? であれば、ASP.NET とか ASP.NET MVC Framework というタグがありますのでそちらを使っていただければと思います。

    キャンセル

  • SurferOnWww

    2017/06/14 22:20

    「条件によって」とは具体的にどういうことでしょう? また、できれば全体的なやりたいことのシナリオ・ストーリーも書いていただけませんか。局所的な部分を切り出して質問すると、真にやりたいことの解決にはなかなかつながりにくいケースが多いので。また、こうした方が良いという代案が出てくるかもしれませんし。

    キャンセル

  • ShouOkada

    2017/06/15 07:15 編集

    タグの編集しました。A,B,C,D4つのテキストボックスとFLGというチェックボックスがあるときに、FLGのチェックボックスがONのときのみテキストボックスA、B、Cの必須チェックを行いたいと思っています。(チェックボックスがOFFの時はABCは非活性でDだけは常に入力チェックを行う)

    キャンセル

  • SurferOnWww

    2017/06/15 12:19

    クライアント側での検証は行わず、サーバー側だけで検証するなら比較的簡単にできそうな気がしますが(未検証なので今のところ気がするだけです)、それで OK ですか?

    キャンセル

回答 2

checkベストアンサー

0

私のコメント(サーバー側だけの検証で OK か?)に返事がないですが、それで OK と勝手に理解して回答します。

質問者さんが考えたように CustomValidationAttribute を使うのが正解だと思います。

FLG プロパティも Model 定義に含めてテキストボックスの文字列と一緒に View から POST するようにすれば、検証メソッドの引数に FLG およびテキストボックス A ~ D のユーザー入力すべてがモデルバインドされますので、そこで検証できます。

具体的には以下のような感じです。

Controller と Model

Controller と Model を一緒にしたのは単に分けるのが面倒だったから。CustomValidation 属性は各プロパティではなくクラスに設定。

using System;
using System.Collections.Generic;
using System.Linq;
using System.Web;
using System.Web.Mvc;
using AdventureWorksLT;
using Mvc5App.Extensions;
using System.ComponentModel.DataAnnotations;

namespace Mvc5App.Controllers
{
    [CustomValidation(typeof(SampleModel), "ValidateSampleModel")]
    public class SampleModel
    {
        public bool Flg { get; set; }
        public string A { get; set; }
        public string B { get; set; }
        public string C { get; set; }
        public string D { get; set; }

        public static ValidationResult ValidateSampleModel(SampleModel model)
        {
            if (model == null)
            {
                throw new NullReferenceException();
            }

            if (model.Flg)
            {
                // 事前に Trim した方がよさそうだが省略
                if (String.IsNullOrEmpty(model.A) || String.IsNullOrEmpty(model.B) ||
                    String.IsNullOrEmpty(model.C) || String.IsNullOrEmpty(model.D))
                {
                    return new ValidationResult("Flg = 1 and one or more of TextBox(es) A, B, C & D empty");
                }
            }
            else
            {
                if (String.IsNullOrEmpty(model.D))
                {
                    return new ValidationResult("Flg = 0 and TextBox D empty");
                }
            }

            return ValidationResult.Success;
        }
    }


    public class HomeController : Controller
    {        
        public ActionResult Create()
        {
            return View();
        }

        [HttpPost]
        public ActionResult Create(SampleModel sample)
        {
            if (!ModelState.IsValid)
            {
                return View(sample);
            }

            return RedirectToAction("Index", "Home");
        }
    }
}


View

エラーメッセージは ValidationSummary を利用して表示。

@model Mvc5App.Controllers.SampleModel

@{
    ViewBag.Title = "Create";
    Layout = "~/Views/Shared/_Layout.cshtml";
}

<h2>Create</h2>

@using (Html.BeginForm())
{
    @Html.ValidationSummary(true)

    <fieldset>
        <legend>SampleModel</legend>

        <div class="editor-label">
            @Html.LabelFor(model => model.Flg)
        </div>
        <div class="editor-field">
            @Html.EditorFor(model => model.Flg)
        </div>

        <div class="editor-label">
            @Html.LabelFor(model => model.A)
        </div>
        <div class="editor-field">
            @Html.EditorFor(model => model.A)
        </div>

        <div class="editor-label">
            @Html.LabelFor(model => model.B)
        </div>
        <div class="editor-field">
            @Html.EditorFor(model => model.B)
        </div>

        <div class="editor-label">
            @Html.LabelFor(model => model.C)
        </div>
        <div class="editor-field">
            @Html.EditorFor(model => model.C)
        </div>

        <div class="editor-label">
            @Html.LabelFor(model => model.D)
        </div>
        <div class="editor-field">
            @Html.EditorFor(model => model.D)
        </div>

        <p>
            <input type="submit" value="Create" />
        </p>
    </fieldset>
}

<div>
    @Html.ActionLink("Back to List", "Index")
</div>

@section Scripts {
    @Scripts.Render("~/bundles/jqueryval", "~/bundles/inarray")
}

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/06/15 20:41

    只今帰宅しました。
    返事が遅くなり申し訳ありません。
    サーバー側だけの検証で大丈夫です。
    回答ありがとうございました。上記の方法を試してみたいと思いますm(__)m

    キャンセル

0

条件付きの必須ということは、その条件のロジックを実装する必要があります。
つまりアノテーション1行では表現できないはずです。
従って、モデルにIValidatableObjectインターフェイスを実装してValidateメソッドで行うのが普通ではないでしょうか。

質問者様の追記を見て下記仕様と想定し、ソースコードを記載します。

  • FLGのチェックボックスがONのときのみテキストボックスA、B、Cの必須チェックを行う
  • チェックボックスがOFFの時はABCは非活性
  • Dだけは常に必須チェックを行う
[Serializable]
public class TestModel : IValidatableObject
{
    public string Flg { get; set; }
    public string A { get; set; }
    public string B { get; set; }
    public string C { get; set; }
    [Required]
    public string D { get; set; }

    public IEnumerable<ValidationResult> Validate(ValidationContext validationContext)
    {
        if (this.Flg == "1")
        {
            if (string.IsNullOrEmpty(this.A))
            {
                yield return new ValidationResult("Aは入力が必須です。", new string[] { "A" });
            }
            if (string.IsNullOrEmpty(this.B))
            {
                yield return new ValidationResult("Bは入力が必須です。", new string[] { "B" });
            }
            if (string.IsNullOrEmpty(this.C))
            {
                yield return new ValidationResult("Cは入力が必須です。", new string[] { "C" });
            }
        }
    }
}

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/06/15 18:08

    > アノテーション1行では表現できないはずです

    今回の質問者さんのケースでは CustomValidation 属性一行を付与すれば検証できます。詳細は私の回答を見てください。

    キャンセル

  • 2017/06/15 19:20

    アノテーションを有効無効に切り替えるという方向で行こうとすると、どうしても条件分岐等を記述する必要がある、という意味で書きました。
    IValidatableObjectインターフェースにしろCustomValidation属性にしろ、結局は切替えるロジックをどこかに書く必要がありますよね。
    ちなみにCustomValidation属性はValidationResultが1つしか返せないので私はいつもIValidatableObjectインターフェースを使用しています・・・。

    キャンセル

  • 2017/06/15 19:36

    ソースコードを追記しました。

    キャンセル

  • 2017/06/15 20:31 編集

    マイクロソフト公式解説書「プログラミング ASP.NET MVC」に書いてあったことですが、そのまま抜粋すると・・・

    "IValidatableObject インターフェイスを実装すると、クラスレベルで CustomValidation 属性を使用するのと機能的には同じになります。"

    ・・・中略・・・

    "クラスレベルの検証を実装するにあたって、クラスレベルで IValidatableObject インターフェイスと CustomrValitdation 属性のどちらを使用するかは、完全にあなた次第です。"

    ・・・だそうです。

    自分的に気になるのは、DB First で既存の SQL Server データベースから Entity Data Model (EDM) を生成して ASP.NET MVC アプリで使用する場合にどうするかです。

    ユーザー入力を検証するためのデータアノテーション属性を付与するのは、以下の記事の「Entity Framework でのデータ アノテーション検証コントロールの使用」のセクションに書いてあるようにすれば可能ですが、IValidatableObject を使った場合はどうするのでしょう?

    [C#] #21. データ アノテーション検証コントロールでの検証
    https://technet.microsoft.com/ja-jp/subscriptions/ff642519.aspx

    あと、IValidatableObject インターフェイスする場合、Validate メソッドが呼び出されない可能性を排除するため、データアノテーションを完全に削除することがお勧めだそうです。

    それは普通のケースではちょっと大変ですよね。なので、自分はデータアノテーション派です。

    キャンセル

  • 2017/06/15 20:41

    わかりました、ありがとうございますm(__)m

    キャンセル

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

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

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

  • C#

    7710questions

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

  • ASP.NET MVC Framework

    69questions

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