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

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

ただいまの
回答率

90.03%

ASP.NET MVC5 TimeSpan型入力値の検証

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 684

blackdifferent

score 19

こんにちわ!ASP.NET MVC5でWeb勤務表作成しています。
https://teratail.com/questions/164940の続きになりますが、写真上の終業時刻が始業時刻より早まらないようにクライアントサイド検証をかけたいです。
こちらのサイトhttp://kobarin.hateblo.jp/?page=1499151648を参考にしてExpressiveAnnotationsのAssertThatを使用したのですが、何も検証がかかりませんでした(おそらくTimeSpan型がAssertThatで判定出来ないのではないのかと思います)。
何か良い方法がありましたら教えてください。
イメージ説明
モデル:

public class SampleKintai
    {
        public int Id { get; set; }
        public string Emp_num { get; set; }
        public System.DateTime Date { get; set; }

        [DisplayFormat(ApplyFormatInEditMode = true, DataFormatString = "{0:hh\\:mm}"), DataType(System.ComponentModel.DataAnnotations.DataType.Time)]
        [RegularExpression(@"((([0-1][0-9])|(2[0-3]))(:[0-5][0-9])(:[0-5][0-9])?)", ErrorMessage = "00:00~23:59で入力してください")]
        public Nullable<System.TimeSpan> Open { get; set; }

        [DisplayFormat(ApplyFormatInEditMode = true, DataFormatString = "{0:hh\\:mm}"), DataType(System.ComponentModel.DataAnnotations.DataType.Time)]
        [RegularExpression(@"((([0-1][0-9])|(2[0-3]))(:[0-5][0-9])(:[0-5][0-9])?)", ErrorMessage = "00:00~23:59で入力してください")]
        [AssertThat("Open < Close", ErrorMessage = "終業時刻は始業時刻より遅くなければなりません")]
        public Nullable<System.TimeSpan> Close { get; set; }

        [DisplayFormat(ApplyFormatInEditMode = true, DataFormatString = "{0:hh\\:mm}"), DataType(System.ComponentModel.DataAnnotations.DataType.Time)]
        [RegularExpression(@"((([0-1][0-9])|(2[0-3]))(:[0-5][0-9])(:[0-5][0-9])?)", ErrorMessage = "00:00~23:59で入力してください")]
        public Nullable<System.TimeSpan> Rest { get; set; }

        [DisplayFormat(DataFormatString = "{0:hh\\:mm}")]
        public Nullable<System.TimeSpan> Worktime { get; set; }
        [DisplayFormat(DataFormatString = "{0:hh\\:mm}")]
        public Nullable<System.TimeSpan> Overtime { get; set; }      
        public int Situation { get; set; }
        [StringLength(20,ErrorMessage = "※入力は全角20文字以内")]
        public string Remark { get; set; }
    }
  • 気になる質問をクリップする

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • SurferOnWww

    2018/12/28 10:50

    前のスレッド https://teratail.com/questions/164940 のコメントのコードを拝見すると、モデルのプロパティの TimeSpan 型を string 型に変更することに決めたように見えます。でも、このスレッドでは TimeSpan 型を使うという話に戻っていますが、どういうことですか? 話が矛盾しているように思えます。

    キャンセル

  • SurferOnWww

    2018/12/28 10:51

    それから、前のスレッド https://teratail.com/questions/164940 で、TimeSpan 型に代えて string 型を使ってみようという理由は、Close の方に 30:00 までの入力を許容したいが TimeSpan 型ではできない(1.06:00 と入力せざるを得ない)からだったはずです。それは諦めて元の案通り 23:59 までにするのですか? それともユーザーに 1.06:00 と入力させるのですか? 最終的にどのようにすることに決めたのでしょう?

    キャンセル

  • blackdifferent

    2018/12/28 11:00

    そうですね。
    一度30:00までの入力は諦めて、23:59までにしようと思います。
    退社が24時以降の場合は、他の日の勤務時間を調整することで対応しようと思います。

    キャンセル

回答 1

checkベストアンサー

+2

ExpressiveAnnotationsのAssertThatを使用したのですが、何も検証がかかりませんでした

サーバー側での検証はかかるが、クライアント側での検証がかからないと言ってますか?

であれば、参考にされている記事にもそのように書いてありますし、クライアント側での検証は諦める(重要なサーバー側の検証さえかかれば良しとする)ということにせざるを得ないかと思いますが。

単一のプロパティのカスタム検証であれば、以下の記事の「第4回 検証属性の自作とクラス・レベルのモデル検証」に書かれているようにして、サーバー側とクライアント側両方の検証機能を実装できます。

@IT 連載:ASP.NET MVC入門【バージョン3対応】
http://www.atmarkit.co.jp/fdotnet/aspnetmvc3/index/index.html

今回のケースのように 2 つのプロパティにまたがった検証をする場合は無理そうです。

検証スクリプト自体は作れるとしても、それを ASP.NET MVC の検証システムの中に不整合無く取り込む具体例は少なくとも自分は見たことがないです(自分が知らないだけだという可能性は否定しきれませんが)。

:上の記事の「第4回 検証属性の自作とクラス・レベルのモデル検証」の最後のセクション「複数プロパティをまたがった検証を実装する」は IValidatableObject インターフェイスを実装する例です。

他に CustomrValitdation 属性を作って使用する方法もあります。どちらが良いかの議論は以下のスレッドにありますので興味があれば見てください。

MVC モデルのバリデーションについて
https://teratail.com/questions/80391

【追記】

下の 2019/01/08 17:21 の私のコメントで「回答欄に追記します」と書きましたが、それを以下に書きます。

ValidationResultをPostで判定する方法はありますか?

そもそも、サーバー側での検証は、GET 側のアクションメソッドで初期画面が表示された後、それが POST された時でないとかからないです。

基本は以下のコードの通りで、モデルバインディングの際にモデルに付与したデータアノテーション属性によって検証が行われ、検証結果が OK の場合は ModelState.IsValid が true に、NG の場合は false になります。

[HttpPost]
[ValidateAntiForgeryToken]
public ActionResult Create([Bind(Include = "No, Project")] TestViewModel model)
{
    if (ModelState.IsValid)
    {
        // INSERT 処理
        return RedirectToAction("Index");
    }

    return View(model);
}

上のコードのように、ModelState.IsValid が true の場合は DB への登録などの処置を行って別のページにリダイレクト、false の場合は retrun View(model); としユーザーに訂正と再送を促すというのが普通のパターンです。

検証 NG で retrun View(model); とするのはちゃんと理由があって、POST されてきたユーザー入力をそのまま(間違っていても間違ったまま)表示して、間違っている箇所にはエラーメッセ―ジを表示し、訂正を容易にするためです。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/01/08 16:49

    すみません。
    ValidationResultをPostで判定する方法はありますか?

    キャンセル

  • 2019/01/08 17:21

    > ValidationResultをPostで判定する方法はありますか?

    回答欄に追記します。

    キャンセル

  • 2019/01/08 17:51

    ありがとうございました。ご回答いただいた方法で実装できました。
    これで解決とさせていただきます。いろいろと丁寧に教えていただき、本当にありがとうございました。
    また何かと質問させていただくかもしれませんが、宜しくお願いします。

    キャンセル

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

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