Сортируемая, перетаскиваемая или упорядоченная таблица + MVC не обновляет порядок правильно

Question

Я пытаюсь добавить заказываемую таблицу в мое представление.Сортировка работает при выполнении этого в представлении New.cshtml.Он использует jQuerys сортируемый .Если я добавлю некоторые инструкции (в данном случае 1-5), я смогу переместить их, используя сортируемый скрипт.Но когда я пытаюсь добавить новую инструкцию после переупорядочения, порядок изменяется с:

на

Я думаю, это как-то связано со скрытой формой

@Html.HiddenFor(m => m.Recipe.Instructions[i].Number, new { @id = "Number"})

Необходимо обновить его значение, когда я переупорядочиваю поля в сортируемой таблице.

Что мне здесь не хватает?

RecipeController.cs

public ActionResult Create(NewRecipeViewModel viewModel, string command)
{

//...more code...

    var instructionList = new List<Instruction>();

    if (viewModel.Recipe.Instructions != null)
    {
        for (int i = 0; i < viewModel.Recipe.Instructions.Count; i++)
        {
            var instruction = new Instruction
            {
                Id = viewModel.Recipe.Instructions[i].Id,
                Name = viewModel.Recipe.Instructions[i].Name,
                Number = viewModel.Recipe.Instructions[i].Number

            };
            instructionList.Add(instruction);
        }
    }

    if (command.Equals("AddInstruction"))
    {
        var instruction = new Instruction
        {
            Name = viewModel.NewInstruction.Name,
            Number = viewModel.Recipe.Instructions.Count + 1
        };

        recipe.Instructions.Add(instruction);
        viewModel.Recipe = recipe;

        return View("New", viewModel);
    }

//...more code...
}

New.cshtml

<div class="form-group" style="margin-top: 170px;">
    <label class="col-md-2 control-label">
        Lägg till instruktion:
    </label>
    <div class="col-md-10">
        @Html.TextBoxFor(m => m.NewInstruction.Name, new { @class = "form-control" })
    </div>
</div>
<div class="form-group" style="margin-top: 300px;">
    <label class="col-md-2 control-label">
    </label>
    <div class="col-md-10">
        <button type="submit" name="command" value="AddInstruction" class="btn btn-primary">Add instruction</button>
    </div>
</div>
<div class="form-group" style="margin-top: 100px;">

    <label class="col-md-2 control-label">
        Instructions:
    </label>
    <div class="col-md-10">
        <table class="table table-bordered table-hover pagin-table" style="margin-top: 10px">
            <thead>
                <tr bgcolor="#f5f5f5">
                    <th>Order:</th>
                    <th>Instruction:</th>
                </tr>
            </thead>
            <tbody id="sortable">
                @if (Model.Recipe.Instructions != null)
                {
                    for (int i = 0; i < Model.Recipe.Instructions.Count; i++)
                    {
                        <tr>
                            @Html.HiddenFor(m => Model.Recipe.Instructions[i].Id)
                            <td class="order">
                                @Model.Recipe.Instructions[i].Number
                                @Html.HiddenFor(m => m.Recipe.Instructions[i].Number, new { @id = "Number"})
                            </td>
                            <td>
                                @Model.Recipe.Instructions[i].Name
                                @Html.HiddenFor(m => m.Recipe.Instructions[i].Name)
                            </td>
                        </tr>
                    }
                }
            </tbody>
        </table>
    </div>
</div>
 <script type="text/javascript">
     $(document).ready(function() {
             $('#sortable').sortable({
                 update : function(event, ui) { 
                     $('td.order').each(function(index) {
                        var order = index + 1;
                        $(this).find('span').text(order);
                        $(this).find('.Number').val(order);
                     });
                 }
             });
     });
 </script

РЕДАКТИРОВАТЬ: Добавлено

instructionList.Sort((s1, s2) => s1.Number.CompareTo(s2.Number));

в RecipeController.cs.Остальное - заслуга Стивена Мюке.

RecipeController.cs

public ActionResult Create(NewRecipeViewModel viewModel, string command)
{

//...more code...

    var instructionList = new List<Instruction>();

    if (viewModel.Recipe.Instructions != null)
    {
        for (int i = 0; i < viewModel.Recipe.Instructions.Count; i++)
        {
            var instruction = new Instruction
            {
                Id = viewModel.Recipe.Instructions[i].Id,
                Name = viewModel.Recipe.Instructions[i].Name,
                Number = viewModel.Recipe.Instructions[i].Number

            };
            instructionList.Add(instruction);
        }
        instructionList.Sort((s1, s2) => s1.Number.CompareTo(s2.Number));
    }

//...more code...
}

New.cshtml

 @* More code *@
 <td class="order">
      <span>@Model.Recipe.Instructions[i].Number</span>
      @Html.HiddenFor(m => m.Recipe.Instructions[i].Number, new {@class = "Number"})
 </td>

 @* More code *@    

 <script type="text/javascript">
     $(document).ready(function() {
             $('#sortable').sortable({
                 update : function(event, ui) { 
                     $('td.order').each(function(index) {
                        var order = index + 1;
                        $(this).find('span').text(order);
                        $(this).find('.Number').val(order);
                     });
                 }
             });
     });
 </script

Answer 1

Ваш селектор в функции update неверен и будет возвращать undefined (вам не хватает # (селектор идентификатора), и в любом случае $ создает объект jQuery, поэтому он будет .val(...), не value=....

Однако использование $('#Number').val(index + 1) не будет работать корректно, поскольку оно будет только когда-либо обновлять первый элемент с id="Number". Duplicate id`, атрибуты которого недопустимы в формате html.

Использовать классвзамен name и относительных селекторов. Измените HTML на

<td class="order">
    <span>@Model.Recipe.Instructions[i].Number</span>
    @Html.HiddenFor(m => m.Recipe.Instructions[i].Number, new { @class = "Number"})
</td>

, а затем на скрипт на

$('#sortable').sortable({
    update : function(event, ui) { 
        $('td.order').each(function(index) {
            var order = index + 1;
            $(this).find('span').text(order );
            $(this).find('.Number').val(order );
        });
    }
});

Answer 2

В вашей разметке замените Model на спроецированную переменную m в ваших HTML-помощниках Html.HiddenFor ().

<tr>
    @Html.HiddenFor(m => m.Recipe.Instructions[i].Id)
    <td class="order">
        @Model.Recipe.Instructions[i].Number
        @Html.HiddenFor(m => m.Recipe.Instructions[i].Number, new { @id = "Number"})
    </td>
    <td>
        @Model.Recipe.Instructions[i].Name
        @Html.HiddenFor(m => m.Recipe.Instructions[i].Name)
    </td>
</tr>