0

Проект asp net core. Есть APIController обрабатывающий удаление пользователей:

[Route("api/[controller]")]
    [ApiController]
    public class APIFriendsController : ControllerBase
    {
        private readonly Db_usersContext _context;

        public APIFriendsController(Db_usersContext context)
        {
            _context = context;
        }        
        // DELETE:
        [HttpPost("DeleteFriends")]
        public async Task<ActionResult<List<Friend>>> DeleteFriends([ModelBinder] long [] idFriends)
        {
            List<Friend> friends = new List<Friend>();
            friends = await Task.Run(()=>SearchFriends(idFriends));

            await Task.Run(() => RemoveFriends(friends));

            return friends;
        }

        private List<Friend> SearchFriends(long[] idFriends)
        {
            List<Friend> friends = new List<Friend>();
            foreach (long id in idFriends)
            {
                Friend friend = _context.Friend.Find(id);

                if (friend != null)
                {
                    friends.Add(friend);
                }
                else { friends.Add(null);}
            }
            return friends;
        }

        private void RemoveFriends(List<Friend> friends)
        {
            foreach(Friend friend in friends)
            {
                if (friend != null)
                {
                    _context.Friend.Remove(friend);
                    _context.SaveChangesAsync();
                }
            }
        }
    }

На странице написал JS функцию, которая перебирает все строки body table, выбирает помеченные на удаление и добавляет их в json-массив, после чего отправляет этот json-массив в POST запросе на сервер для удаления:

function deleteSelected(idObject) {

        var table = document.getElementById(idObject);
        var rows = table.rows;
        var jsonMasId = [];

        for (let i = 1, iLen = rows.length - 1; i <= iLen; i++) {
            var checked_ = table.rows[i].cells[21].getElementsByTagName('input')[0].checked;
            if (checked_ == true) {

                var idFriend = table.rows[i].cells[1].innerHTML;
                idFriend = idFriend.trim(); //Удаляем пробелы в начале и в конце                  
                jsonMasId.push(idFriend);
            }
        }

        if (idFriend.length>0) {
            $.ajax({
                url: 'http://localhost:48329/api/APIFriends/DeleteFriends/',
                type: 'POST',
                data: JSON.stringify(jsonMasId),
                contentType: "application/json;charset=utf-8",
                success: function (response) {
                    alert('Пользователи удалены');
                    console.log(response);
                },
                error: function (result, status, er) {
                    alert("error: " + result + " status: " + status + " er:" + er);
                }
            });
        }
    }

Когда создается/отправляется ajax-запрос возникает ошибка:

POST http://localhost:48329/api/APIFriends/DeleteFriends/ 400 (Bad Request) send @ jquery.min.js:2 ajax @ jquery.min.js:2 deleteSelected @ Admin:551 onclick @ Admin:79

В чем причина? Как можно исправить?

1
  • а вы уверены, что jsonMasId имеет правильный контент, т.е. список id, которые содержит только числа и никаких других символов? Commented 20 апр. 2021 в 7:44

1 ответ 1

1

Обратите внимание на следующий нюанс:

var arr = [];
arr.push('123');
arr.push('345');
console.log(JSON.stringify(arr));
arr = [];
arr.push(123);
arr.push(345);
console.log(JSON.stringify(arr));

["123","345"]
[123,345]

То есть вы передаете в контроллер string[], а не long[].

Это можно исправить двумя способами:

  1. Записывать в массив сразу числа, а не строки
jsonMasId.push(Number.parseInt(idFriend));
  1. Принимать в контроллере как массив строк, а не чисел
[HttpPost("DeleteFriends")]
public async Task<ActionResult<List<Friend>>> DeleteFriends([ModelBinder] string[] idFriends)
{
    long[] data = idFriends.Select(long.Parse).ToArray();
    // ...
}

Я склоняюсь к первому варианту, потому что ту работу, которую можно делать на фронте, лучше делать на фронте.

Еще обратите внимание на то что вы стартуете зачем-то 2 разных потока для выполнения операции, которые выполняются последовательно, можно упростить.

[HttpPost("DeleteFriends")]
public async Task<ActionResult<List<Friend>>> DeleteFriends([ModelBinder] long[] idFriends)
{
    return await Task.Run(() => RemoveFriends(SearchFriends(idFriends)));
}
11
  • 1
    Спасибо! Вы правы. Отдельная благодарность за совет по коду контроллера
    – Vlad-i-mir
    Commented 20 апр. 2021 в 9:05
  • А можете ещё подсказать, как сделать, чтобы после удаления страница у пользователя обновилась?
    – Vlad-i-mir
    Commented 20 апр. 2021 в 9:39
  • @Vlad-i-mir success: function (response) внутри хоть обновляйте страницу, хоть что угодно делайте.
    – aepot
    Commented 20 апр. 2021 в 9:48
  • покажите, как внутри неё отправить запрос на обновление? Я в js плохо шарю, если честно
    – Vlad-i-mir
    Commented 20 апр. 2021 в 10:00
  • 1
    @Vlad-i-mir javascript.info/modifying-document
    – aepot
    Commented 20 апр. 2021 в 10:19

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.