teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

3

追記

2019/02/11 01:19

投稿

退会済みユーザー
answer CHANGED
@@ -30,7 +30,7 @@
30
30
 
31
31
  先に自分が自分の環境で検証したコードに加えて、入力できる金額の範囲などを検証するデータアノテーション属性を追加し、質問者さんが質問にアップされたスクリプトと、shinji709 さんがアップされたコードの this を elm に変更したスクリプトを追加したものです。
32
32
 
33
- 結果、独自に追加したスクリプトと ASP.NET MVC の検証用スクリプトがバッティングすることなくクライアント側での検証は期待通り動きました。(さらなる検証は必要だと思いますが基本的な動きは OK でした)
33
+ 結果、独自に追加したスクリプトと ASP.NET MVC の検証用スクリプトがバッティングすることなくクライアント側での検証は期待通り動きました。(**さらなる検証は必要**だと思いますが基本的な動きは OK でした ← 下の 2019/02/11 10:15 の私のコメントを見てください
34
34
 
35
35
  変な先入観を持たず、諦めない心で何でもやってみるというのは大切なのだと今さらながら思い出し、自分としても勉強になりました。
36
36
 

2

追記

2019/02/11 01:19

投稿

退会済みユーザー
answer CHANGED
@@ -22,4 +22,250 @@
22
22
 
23
23
  (2) Model のプロパティを decimal 型ではなく string 型とし、検証はデータアノテーション属性を付与して正規表現で行なう。(DB のフィールドの型が money 型のようですので、サーバー側での取り扱いを考えると難しいかもしれませんが)
24
24
 
25
- 自分が考えるとすると (2) かなぁ・・・という感じです。
25
+ 自分が考えるとすると (2) かなぁ・・・という感じです。
26
+
27
+ **【追記】**
28
+
29
+ 上の質問に対する 2019/02/10 19:37 の私のコメントで「ご参考に検証に使ったコードを回答欄にアップしておきます」書きましたが、それを以下に書いておきます。
30
+
31
+ 先に自分が自分の環境で検証したコードに加えて、入力できる金額の範囲などを検証するデータアノテーション属性を追加し、質問者さんが質問にアップされたスクリプトと、shinji709 さんがアップされたコードの this を elm に変更したスクリプトを追加したものです。
32
+
33
+ 結果、独自に追加したスクリプトと ASP.NET MVC の検証用スクリプトがバッティングすることなくクライアント側での検証は期待通り動きました。(さらなる検証は必要だと思いますが基本的な動きは OK でした)
34
+
35
+ 変な先入観を持たず、諦めない心で何でもやってみるというのは大切なのだと今さらながら思い出し、自分としても勉強になりました。
36
+
37
+ 不明点があれば質問してください。
38
+
39
+ **Model**
40
+
41
+ ```
42
+ using System;
43
+ using System.Collections.Generic;
44
+ using System.Linq;
45
+ using System.Web;
46
+ using System.ComponentModel.DataAnnotations;
47
+ using System.ComponentModel.DataAnnotations.Schema;
48
+ using System.Globalization;
49
+
50
+ namespace Mvc5App2.Models
51
+ {
52
+ public class PurchaseRecord
53
+ {
54
+ [Display(Name = "ID")]
55
+ [Required(ErrorMessage = "{0} は必須")]
56
+ [RegularExpression(@"^\d{1,5}$", ErrorMessage = "数字 1 ~ 5 文字")]
57
+ public int ID { get; set; }
58
+
59
+ [Display(Name = "契約日")]
60
+ [Required(ErrorMessage = "{0} は必須")]
61
+ [RegularExpression(@"^\d{4}/\d{2}/\d{2}( \d{1,2}:\d{2}:\d{2})?$",
62
+ ErrorMessage = "yyyy/MM/dd 形式")]
63
+ [DisplayFormat(DataFormatString = "{0:yyyy年M月d日}")]
64
+ public DateTime ContractDate { get; set; }
65
+
66
+ [Display(Name = "価格")]
67
+ [Required(ErrorMessage = "{0} は必須")]
68
+ // RegularExpression がないと 1x というような文字列を入力すると "The field 価格 must be a number."
69
+ // という data-val-number 属性に設定されたエラーメッセージが出る。
70
+ [RegularExpression(@"^\d{1,6}$", ErrorMessage = "数字 1 ~ 6 文字")]
71
+ [Range(100, 10000, ErrorMessage = "{0}は{1}~{2}の間で入力してください。")]
72
+ [DisplayFormat(DataFormatString = "{0:N0}", ApplyFormatInEditMode = true)]
73
+ public decimal Price { get; set; }
74
+ }
75
+ }
76
+ ```
77
+
78
+
79
+ **View**
80
+
81
+ ```
82
+ @model Mvc5App2.Models.PurchaseRecord
83
+
84
+ @{
85
+ ViewBag.Title = "Edit";
86
+ }
87
+
88
+ <h2>Edit</h2>
89
+
90
+
91
+ @using (Html.BeginForm())
92
+ {
93
+ @Html.AntiForgeryToken()
94
+
95
+ <div class="form-horizontal">
96
+ <h4>PurchaseRecord</h4>
97
+ <hr />
98
+ @Html.ValidationSummary(true, "", new { @class = "text-danger" })
99
+
100
+ <div class="form-group">
101
+ @Html.LabelFor(model => model.ID, htmlAttributes: new { @class = "control-label col-md-2" })
102
+ <div class="col-md-10">
103
+ @Html.EditorFor(model => model.ID, new { htmlAttributes = new { @class = "form-control" } })
104
+ @Html.ValidationMessageFor(model => model.ID, "", new { @class = "text-danger" })
105
+ </div>
106
+ </div>
107
+
108
+ <div class="form-group">
109
+ @Html.LabelFor(model => model.ContractDate, htmlAttributes: new { @class = "control-label col-md-2" })
110
+ <div class="col-md-10">
111
+ @*@Html.EditorFor(model => model.ContractDate, new { htmlAttributes = new { @class = "form-control" } })*@
112
+ @Html.TextBoxFor(model => model.ContractDate, "{0:yyyy/MM/dd}", new { @class = "form-control" })
113
+ @Html.ValidationMessageFor(model => model.ContractDate, "", new { @class = "text-danger" })
114
+ </div>
115
+ </div>
116
+
117
+ <div class="form-group">
118
+ @Html.LabelFor(model => model.Price, htmlAttributes: new { @class = "control-label col-md-2" })
119
+ <div class="col-md-10">
120
+ @Html.EditorFor(model => model.Price, new { htmlAttributes = new { @class = "form-control" } })
121
+ @Html.ValidationMessageFor(model => model.Price, "", new { @class = "text-danger" })
122
+ </div>
123
+ </div>
124
+
125
+ <div class="form-group">
126
+ <div class="col-md-offset-2 col-md-10">
127
+ <input type="submit" value="Save" class="btn btn-default" />
128
+ </div>
129
+ </div>
130
+ </div>
131
+ }
132
+
133
+ <div>
134
+ @Html.ActionLink("Back to List", "Index")
135
+ </div>
136
+
137
+ @section Scripts {
138
+ @Scripts.Render("~/bundles/jqueryval")
139
+
140
+ <script type="text/javascript">
141
+ //<![CDATA[
142
+
143
+ // これは shinji709 さんのコードの this を elm に変更
144
+ var form = document.getElementsByTagName('form')[0]
145
+ form.addEventListener('submit',
146
+ function () {
147
+ elm.value = delFigure(elm.value);
148
+ }, false);
149
+
150
+
151
+ // ここから下は質問者さんのコードをそのままコピペ
152
+
153
+ //出典: https://webllica.com/add-comma-as-thousands-separator/
154
+ //**** 画面表示完了後に、契約金額の表示をコンマ区切りとする ****
155
+ window.onload = function () {
156
+ elm.value = addFigure(document.getElementById("Price").value);
157
+ };
158
+
159
+ // * 数値の3桁カンマ区切り
160
+ // * 入力値をカンマ区切りにして返却
161
+ // * [引数] numVal: 入力数値
162
+ // * [返却値] String(): カンマ区切りされた文字列
163
+ // */
164
+ function addFigure(numVal) {
165
+ // 空の場合そのまま返却
166
+ if (numVal === '') {
167
+ return '';
168
+ }
169
+ // 全角から半角へ変換し、既にカンマが入力されていたら事前に削除
170
+ numVal = toHalfWidth(numVal).replace(/,/g, "").trim();
171
+ // 数値でなければそのまま返却
172
+ if (!/^[+|-]?(\d*)(.\d+)?$/.test(numVal)) {
173
+ return numVal;
174
+ }
175
+ // 整数部分と小数部分に分割
176
+ var numData = numVal.toString().split('.');
177
+ // 整数部分を3桁カンマ区切りへ
178
+ numData[0] = Number(numData[0]).toString().replace(/\B(?=(\d{3})+(?!\d))/g, ',');
179
+ // 小数部分と結合して返却
180
+ return numData.join('.');
181
+ }
182
+
183
+ ///**
184
+ // * カンマ外し
185
+ // * 入力値のカンマを取り除いて返却
186
+ // * [引数] strVal: 半角でカンマ区切りされた数値
187
+ // * [返却値] String(): カンマを削除した数値
188
+ // */
189
+ function delFigure(strVal) {
190
+ return strVal.replace(/,/g, "");
191
+ }
192
+
193
+ ///**
194
+ // * 全角から半角への変換関数
195
+ // * 入力値の英数記号を半角変換して返却
196
+ // * [引数] strVal: 入力値
197
+ // * [返却値] String(): 半角変換された文字列
198
+ // */
199
+ function toHalfWidth(strVal) {
200
+ // 半角変換
201
+ var halfVal = strVal.replace(/[!-~]/g,
202
+ function (tmpStr) {
203
+ // 文字コードをシフト
204
+ return String.fromCharCode(tmpStr.charCodeAt(0) - 0xFEE0);
205
+ });
206
+ return halfVal;
207
+ }
208
+
209
+ ///**
210
+ // * 処理を適用するテキストボックスへのイベント設定
211
+ // * Blur : カンマ区切り処理実施
212
+ // * Focus : カンマ削除処理実施
213
+ // */
214
+ var elm = document.getElementById('Price');
215
+
216
+ elm.addEventListener('blur',
217
+ function () {
218
+ this.value = addFigure(this.value);
219
+ }, false);
220
+
221
+ elm.addEventListener('focus',
222
+ function () {
223
+ this.value = delFigure(this.value);
224
+ }, false);
225
+ //]]>
226
+ </script>
227
+ }
228
+ ```
229
+
230
+
231
+ **Controller**
232
+
233
+ ```
234
+ using System;
235
+ using System.Collections.Generic;
236
+ using System.Linq;
237
+ using System.Web;
238
+ using System.Web.Mvc;
239
+ using Mvc5App2.Models;
240
+
241
+ namespace Mvc5App2.Controllers
242
+ {
243
+ public class HomeController : Controller
244
+ {
245
+ public ActionResult Edit()
246
+ {
247
+ PurchaseRecord model = new PurchaseRecord
248
+ {
249
+ ID = 1,
250
+ ContractDate = DateTime.Now,
251
+ Price = 1234m
252
+ };
253
+
254
+ return View(model);
255
+ }
256
+
257
+ [HttpPost]
258
+ [ValidateAntiForgeryToken]
259
+ public ActionResult Edit(PurchaseRecord model)
260
+ {
261
+ if (ModelState.IsValid)
262
+ {
263
+ // DB の編集処理
264
+ return RedirectToAction("Index");
265
+ }
266
+
267
+ return View(model);
268
+ }
269
+ }
270
+ }
271
+ ```

1

訂正

2019/02/10 10:48

投稿

退会済みユーザー
answer CHANGED
@@ -12,7 +12,7 @@
12
12
 
13
13
  質問者さんが諦めない心でとにかく何でもトライしてみるのは良いとは思いますが、そのあたりの事情は回答者・閲覧者の方も知っておいた方がよさそうです。
14
14
 
15
- 回避できない場合は、\1,000 から 1000 に書き換えのために追加したスクリプトの動作と、ASP.NET MVC の検証用のスクリプトの動作に整合が取れるように、どちらかもしくは両方を書き直して対応できないかを検討するという話になると思います。
15
+ 回避できない場合は、~~\1,000~~ **1000** から ~~1000~~ **\1,000** に書き換えのために追加したスクリプトの動作と(**1000 と \1,000 が逆でした。訂正します**)、ASP.NET MVC の検証用のスクリプトの動作に整合が取れるように、どちらかもしくは両方を書き直して対応できないかを検討するという話になると思います。
16
16
 
17
17
  でも、書き直すことで対処できるかどうかは分かりません。そもそも、検討するにしても、1,600 行を超える ASP.NET MVC のスクリプトを調べて、好ましからざる副作用が出ないよう完璧に書き直すのは現実的ではないと思われます。時間の無駄に終わるかもしれませんし。
18
18