Как лучше было сделать? (ASP.NET Core C# + JS)

@EveKS · Регистрация: 19.04.2016

Author24 — интернет-сервис помощи студентам

Как подобное можно было сделать лучше?
У меня есть DropDownList, написанный на HTML:

C#

                        <select id="MarkerGropName" name="MarkerGropName">
                            @foreach (MarkerGroup item in ViewBag.MarkerGroup)
                            {
                                <option value="@item.Id">@item.GroupName</option>
                            }
                        </select>

Далее есть "кусок" JS:

Javascript

                var markerCoordinate;
 
                function CreateMark(data, dataArray, contentType) {
                    $.ajax({
                        type: "POST",
                        url: "Home/Post",
                        dataType: "json",
                        contentType: contentType,
                        data: data,
                        success: function () {
                            var tmp = {};
                            $.each(dataArray, function () {
                                if (tmp[this.name] !== undefined) {
                                    if (!tmp[this.name].push) {
                                        tmp[this.name] = [tmp[this.name]];
                                    }
                                    tmp[this.name].push(this.value || '');
                                } else {
                                    tmp[this.name] = this.value || '';
                                }
                            });
 
                            var title = tmp.MarkerTitle;
                            var description = tmp.MarkerDescription;
                            var groupName = tmp.MarkerGropName;
 
                            var latitude = markerCoordinate.lat();
                            var longitude = markerCoordinate.lng();
 
                            addMarker(latitude, longitude, title, description, groupName, map);
                        }
                    });
                }
 
                $("#submit").click(function (e) {
                    e.preventDefault();
                    var $form = $(this).closest('form');
                    var data = $form.serialize();
                    var dataArray = $form.serializeArray();
 
                    var contentType = "application/x-www-form-urlencoded; charset=utf-8";
 
                    reset();
                    CreateMark(data, dataArray, contentType);
                });

как подобное можно было сделать проще?
Здесь groupName временно, и не стоит обращать внимание на то что там Id.
Так же, возможно кто подскажет как в DropDownList можно было задать Id и Name или как написать что-то подобное:

C#

1
2
3

var groupName = (ViewBag.MarkerGroup as List<MarkerGroup >)
    .FirstOrDefault(mes => mes.Id == SameID)
    .GroupName;

Т.е. как в JS подобное реализавать - передав туда ViewBag.MarkerGroup и вытащить оттуда GroupName по индексу. Идекс передается из формы name DropDownList.

Добавлено через 47 минут
Малость упростил )) - там всё равно пост запрос вернет что-то, сделал так чтоб возвращал мою модель MarkerMessage в JSON

C#

1	return Json(markerMessage);

Этим самым я избавился от всех этих var markerCoordinate; dataArray и тд.
Изменив success на

Javascript

success: function (result) {
 
    var title = result.title;
    var description = result.description;
    var id = result.markerGroupId;
    var latitude = result.latitude;
    var longitude = result.longitude;
 
    addMarker(latitude, longitude, title, description, id, map);
}

	14.04.2017, 12:01