我有一个可以编辑的表单视图。这个观点相当复杂。表格是合同。合同名称可以在顶部进行编辑。合同包含许多广告,并且可以修改每个广告的详细信息。表单的长度取决于合同的广告数量。我在视图中有一个for循环,显示每个广告的所有表单字段。
此外,模型中有一些字段用于确定某些广告详细信息,但是它们本身不应保存到数据库中(但它们必须显示为视图中的下拉列表)。
此外,有些模型字段不需要出现在视图中,但在POST控制器方法中是需要的,以便正确保存数据库(我手动保存它),所以他们必须以某种方式通过视图并传递回控制器。这是我的模特:
public class ModContract
{
public int contract_id; // pk; needed
public string contract_name { get; set; } // save
public List<ModAds> ads { get; set; }
}
public class ModAds
{
public int contr_ad_id; // pk; needed
public string name { get; set; } // save
public string print_product_id { get; set; } // used for product_name
public string product_name { get; set; } // appear
public string print_ad_option_id { get; set; } // save
public string adv_product { get; set; } // appear
public List<string> editions { get; set; } // save
public double freq_disc { get; set; } // save
public double other_dis_dol { get; set; } // save
public double? other_dis_per { get; set; } // save
public string non_cash_note { get; set; } // save
public double non_cash_cons { get; set; } // save
}
评论为&#34;保存&#34;的字段之后应将它们保存到数据库中。出现&#34;出现&#34;必须在视图中,但在POST控制器方法中不需要。需要&#34;需要&#34;在POST控制器方法中需要但不是视图。
顺便说一句,这些模型类没有与数据库表一对一映射,它们是由几个不同的表组成的。
据我所知,我不知道如何使用模型绑定来解决这个问题。例如,传递回控制器的模型对于ModContract的ads字段具有空值。我想知道如何在不依赖模型绑定的情况下手动将我想要的视图传递回控制器。
我知道我可能会将@ Html.HiddenFor用于POST方法所需的ID而不是视图,但出于某种原因,当我用contract_id和_name尝试它时,它没有用。
<li data-role="fieldcontain">
<label>Contract Name</label>
@Html.HiddenFor(m => m.contract_id)
@Html.EditorFor(m => m.contract_name)
</li>
POST方法返回的contract_id为0。
让我知道,如果我的问题听起来很愚蠢,因为这是不可能的,或者因为我过于复杂,我对这一切都很陌生。
提前致谢!
修改
这是观点。我不能让它缩进,但希望它是可读的。
@model oulookmediaweb.Models.ModContract
@{
ViewBag.Title = "ModifyContract";
}
@{
var num = 1;
}
<h2>Modify Contract</h2>
@using (Html.BeginForm("ModifyContract", "Contract", FormMethod.Post, htmlAttributes: new { data_ajax = "false" }))
{
@Html.ValidationSummary();
<ul data-role="listview" data-inset="true">
<li data-role="fieldcontain">
<label>Contract Name</label>
@Html.HiddenFor(m => m.contract_id)
@Html.EditorFor(m => m.contract_name)
</li>
</ul>
<div>
@foreach (var ad in Model.ads)
{
<div id="@num">
<ul data-role="listview" data-inset="true">
<li data-role="list-divider">@ad.name</li>
<li data-role="fieldcontain">
<label><strong>Product Name</strong></label>
<div>@ad.product_name</div>
<div id="@num-drdn0" hidden="true">@Html.DropDownListFor(m => ad.print_product_id, ViewData["products_list"] as SelectList)</div>
<input id="@num-editbutton" type="button" onclick="edit(@num)" value="Edit" />
</li>
<li data-role="fieldcontain">
<label><strong>Advertising Product</strong></label>
<div>@ad.adv_product</div>
<div id="@num-drdn1" class="hid">
<select></select>
</div>
</li>
<li data-role="fieldcontain">
<label><strong>Editions to run in:</strong></label>
@foreach (var ed in ad.editions)
{
<div>@ed</div>
}
<div id="@num-drdn2" class="hid">
<select multiple="multiple" data-native-menu="false"></select>
</div>
</li>
<li data-role="fieldcontain">
<label><strong>Frequency Discount (%)</strong></label>
<div>@Html.EditorFor(m => ad.freq_disc)</div>
</li>
<li data-role="fieldcontain">
<label><strong>Other Discount ($)</strong></label>
<div>@Html.EditorFor(m => ad.other_dis_dol)</div>
</li>
<li data-role="fieldcontain">
<label><strong>Other Discount (%)</strong></label>
<div>@Html.EditorFor(m => ad.other_dis_per)</div>
</li>
<li data-role="fieldcontain">
<label><strong>Non Cash Note</strong></label>
<div>@Html.EditorFor(m => ad.non_cash_note)</div>
</li>
<li data-role="fieldcontain">
<label><strong>Non Cash Consideration</strong></label>
<div>@Html.EditorFor(m => ad.non_cash_cons)</div>
</li>
</ul>
@{num++;}
</div>
}
</div>
<ul data-role="listview" data-inset="true">
<li data-role="fieldcontain">
<input type="submit" data-theme="e" value="Submit" />
</li>
</ul>
}
<script type="text/javascript">
var nu;
window.onload = function () {
// hide adv prods and editions select for now
$(".hid select").closest('.ui-select').hide();
}
// called when the edit button for product is clicked
function edit(num) {
nu = num;
$("#" + nu + "-drdn0").show(); // show dropdown
$("#" + nu + "-editbutton").closest('.ui-btn').hide(); // hide edit button; '.ui-btn'? WTF?
// remove current product selection div
// remove adv product selection div
// remove editions div
$("#" + nu + "-drdn0 select").change(prodChange); // on select change
$("#" + nu + "-drdn0 select").trigger("change"); // trigger for default; happens twice; WHY?
}
// called when a magazine is selected
function prodChange() {
// ajax
var url = '@Url.Action("GetAdvProdList", "Contract")' + '?prod_id=' + this.value;
$.getJSON(url, null, function (list) {
// for adv list dropdown
$("#" + nu + "-drdn1 select").empty(); // remove old stuff
$("#" + nu + "-drdn1 select").closest('.ui-select').show(); // show dropdown
var arr = list.advlist; // get the array from object
$.each(arr, function (ind, val) {
// add each item in list as an option of select
$("#" + nu + "-drdn1 select").append('<option value=' + val.Value + '>' + val.Text + '</option>');
});
$("#" + nu + "-drdn1 select").selectmenu('refresh', true); // refresh menu
// for ed list
$("#" + nu + "-drdn2 select").empty(); // remove old stuff
$("#" + nu + "-drdn2 select").closest('.ui-select').show(); // show list
var lis = list.edlist; // get the array from object
$.each(lis, function (ind, val) {
// add each item to list
$("#" + nu + "-drdn2 select").append('<option value=' + val.Value + '>' + val.Text + '</option>');
});
$("#" + nu + "-drdn2 select").selectmenu('refresh', true); // refresh menu
});
}
答案 0 :(得分:3)
你有很多问题,如果你有一个复杂的表格,这是可以理解的。因此,让我尝试总结一下您的问题,以便提供最佳解决方案。
模型中有一些字段......(那)本身不应该 保存到数据库(但它们必须显示为,例如,下拉列表 视图)
您将这些字段放在模型中并将其视为只读。我稍后会说明如何。
有些模型字段不需要出现在视图中,但是 在POST控制器方法中需要保存数据库 正确(我手动保存)
你不需要这样做。您正在传递不必要的数据,并且您正在介绍过帐。此外,您不允许用户编辑它们,而是将其传递给客户端,接收它并将其保存回数据库。
它甚至暴露了系统中的安全漏洞。例如,您将product_name
传递给视图,不希望它被编辑,但随后将其保存到另一个表中。有人可以很容易地破解这个价值,你可能会得到一个你从未想过的product_name
。
应将保存后注释为“save”的字段保存到 数据库。
接受这些作为表单上的输入。代码将在稍后显示。
具有“出现”的那些必须在视图中但不需要在视图中 POST控制器方法。
正如我所提到的,这些应该是viewmodel上的只读字段,稍后会显示代码。
POST控制器方法中需要“需要”的那些但是 不是观点。
您无需传递此信息,也无需立即查询。在保存时间到来之后,从数据库中获取它。稍后会显示伪代码。
在这一点上,我假设您通过回答问题的某些方面来理解我上面提到的内容。以下是这些解释的相关代码。
public class ContractViewModel {
// start:read-only fields.
// These are the fields that will be used as read-only
public string product_name { get; set; }
public string adv_product { get; set; }
public IEnumerable<SelectListItem> PrintAdOptions { get; set; }
// end:read-only fields.
public InputModel Contract {get;set;}
}
// you use this class to capture "all your inputs"
public class InputModel {
public int contract_id {get;set;}
public string contract_name { get; set; }
public IEnumerable<InputSubModel> ads {get;set;}
}
public class InputSubModel {
public int contr_ad_id {get;set;}
public string print_ad_option_id {get;set;} // is this really a string?
public string name {get;set;}
}
您将创建一个名为ContractViewModel
的“主”视图模型,并在您的视图中使用它(当然您可以拥有任何您想要的名称)。它包含显示信息的字段和捕获所有输入的“输入”字段。它是所有实体的展平表示。名为InputModel
的“输入”字段是所有实体的另一个展平表示。该视图模型的输入将被分散并保持正确。
// method for creating a contract
public ActionResult Create() {
// implement the ff methods, whatever method you're using ORM, plain ADO, etc.
var product = getEntityFromYourDb();
// you may eagerly load this (ORM) or query it separately, it's up to you
// AdOption is a simple entity with structure: Id, Name
IEnumerable<AdOption> adOptions = getAdOptionsFromDb();
// map entity to model
// I am guessing here that "product" has the following fields.
// Just replace this accordingly based on you entities
var model = new ContractViewModel{
product_name = product.Name,
adv_product = product.AdvProduct,
PrintAdOptions = aOptions.Select(x=>
new SelectListItem{ Value = x.Id, Text = x.Name })
Contract = getEntitiesFromDbAndMapToModel();
};
// at this point you have everything you need
// (1) a field that you can just show (and not save)
// (2) a list of fields that you can use on a drop-down
// (3) input fields, the ONLY ones that you will POST
return View(model);
}
这里给出的例子也解决了这个问题:
据我所知,我没有看到如何使用模型绑定 这个。传递回控制器的模型具有空值 对于ModContract的广告字段
永远不要在集合上使用foreach
,因为剃刀引擎无法以可以保存的方式正确编写它。这样做的原因是所写的元素具有相同的名称,并且它们无法正确绑定(或绑定,我认为是英语中的正确单词 - 在编程中绑定:)。始终使用for-loop
,请参阅以下代码。
@model ContractViewModel
<h2>Modify Contract</h2>
<p>@Model.product_name</p>
<p>@Model.adv_product</p>
@Html.HiddenFor(m=>m.Contract.contract_id) // your reference Id for saving
@Html.TextBoxFor(m=>m.Contract.contract_name)
@for(var i=0;i<Model.Contract.ads.Count();i++){
@Html.HiddenFor(m=>m.Contract.ads[i].contr_ad_id) // your reference Id for saving
@Html.TextBoxFor(m=>m.Contract.ads[i].name)
@Html.DropdownListFor(m=>m.Contract.ads[i].print_ad_option_id, m.PrintAdOptions)
}
在下面给出的示例中,您会注意到我们使用了Bind
属性。这告诉控制器只绑定并获取我们的“输入”模型,即InputModel
。无需回发保存或查询等所需的所有其他值。请注意,如果您需要字段,请说出id
(例如产品ID)稍后用于从产品表中查询产品因为,您需要将其名称保存到ModAds
表,然后将其包含在InputModel
中。这样你就可以InputModel.ProductId
。
[HttpPost]
public ActionResult Create([Bind(Prefix = "Contract")]InputModel model) {
// map back the values from the flatten model into your entities
// it could be a single entity or a complex one
// but I'm sure you can easily determine how to do it
// as an example:
var modContract = getContractFromDb(model.contract_id);
modContract.contract_name = model.contract_name;
// do the rest of the mapping here
}
这是我能根据你的模特和你想要做的事情给你的最接近的例子。如果您仔细阅读该示例,我相信它会解决您的问题。