遍历数组并汇编字符串

Question

public ActionResult Create(RecurringTask recurringTask, FormCollection collection, ICollection<string> dayOfTheWeek)

我试图遍历dayOfTheWeek（这是一组复选框），并且试图找出哪一个是正确的，因此我可以用它来组装一个字符串，例如：星期一，星期二等。

我只是很难找到一种遍历我的收藏集的方法。 我一直无法将==应用于string类型的布尔错误。

var days = dayOfTheWeek.ToString();
                    foreach (string day in dayOfTheWeek)
                    {
                        if(day == true)
                        {

                        }
                    }
                    recurringTask.DaysOfTheWeek = days;

这就是我正在思考的方法。 但是我想象外面有人比我有更好的主意。 day == true给我带来了字符串错误的错误，并且很明显地说明了为什么发生这种错误，我只是不知道该如何解决。

我的看法是这样的：

<input type="checkbox" name="dayOfTheWeek" value="Monday" />
<input type="checkbox" name="dayOfTheWeek" value="Tuesday" />
<input type="checkbox" name="dayOfTheWeek" value="Wednesday" />
<input type="checkbox" name="dayOfTheWeek" value="Thursday" />
<input type="checkbox" name="dayOfTheWeek" value="Friday" />
<input type="checkbox" name="dayOfTheWeek" value="Saturday" />
<input type="checkbox" name="dayOfTheWeek" value="Sunday" />

Answer 1

错误消息正在显示。 您正在尝试将字符串day与true或false进行比较。 string为true或false是什么意思？

您确实要查看是否已选中复选框。 您声称dayOfTheWeek是一组CheckBoxes ，但是您正在对其调用ToString() ，这会将其转换为字符串列表。 var days = dayOfTheWeek.ToString();之前的代码是什么？ 您在哪里宣布dayOfTheWeek ？

真正拥有List<Checkbox> ，您真的想要遍历该列表以查看是否选中了每个元素。

foreach(CheckBox cb in dayOfTheWeek)
{
    if(cb.Checked)
    {
      // Logic
    }
}

Answer 2

如果将复选框命名为“ dayOfTheWeek”，则ICollection<string> dayOfTheWeek仅包含选定的复选框。

因此，您可以将此列表视为“结果”，但我建议您使用有效值列表过滤天数列表，以防止恶意攻击。

Answer 3

公共ActionResult创建（RecurringTask recurringTask，FormCollection集合，ICollection dayOfTheWeek）

抱歉地说，但这可能是我见过的最糟糕的动作签名之一。 域模型，FormCollection和一些ICollection<string> 。

使用视图模型，强类型视图和编辑器模板（这可能是我在StackOverflow上写这句话以回应asp.net-mvc标记中的问题的第10次第4次）！ 它们会让您的生活变得更加轻松。 因此，列出了日期和相应的布尔属性，以指示是否选择了这一天：

public class MyViewModel
{
    public IEnumerable<DayOfWeekViewModel> DaysOfWeek { get; set; }

    ... put any other properties that you consider useful for this view
}

public class DayOfWeekViewModel
{
    public string DayOfWeek { get; set; }
    public bool IsSelected { get; set; }
}

然后是一个控制器：

public class HomeController : Controller
{
    public ActionResult Index()
    {
        var model = new MyViewModel
        {
            DaysOfWeek = CultureInfo
                .CurrentCulture
                .DateTimeFormat
                .DayNames
                .Select(x => new DayOfWeekViewModel
                {
                    DayOfWeek = x,
                })
        };
        return View(model);
    }

    [HttpPost]
    public ActionResult Index(MyViewModel model)
    {
        // model.DaysOfWeek will contain all you need here
        // TODO: do some processing
        // here you can loop through model.DaysOfWeek to identify which 
        // days have been selected and take respective actions
        // ...

        // once you have finished processing you could redirect
        return RedirectToAction("success");
    }
}

然后是一个对应的视图：

@model MyViewModel

@using (Html.BeginForm())
{
    ... you could put any other fields from your view model that
        will be used by this form here

    @Html.EditorFor(x => x.DaysOfWeek)
    <input type="submit" value="OK" />
}

以及相应的编辑器模板（ ~/Views/Home/EditorTemplates/DayOfWeekViewModel.cshtml ）：

@model DayOfWeekViewModel

<div>
    @Html.CheckBoxFor(x => x.IsSelected) @Html.DisplayFor(x => x.DayOfWeek)
    @Html.HiddenFor(x => x.DayOfWeek)
</div>