Javascript DataTransfer项目在异步调用中无法持久化

13

我正在使用Vuejs与DataTransfer异步上传文件,并且希望允许一次拖放多个文件进行上传。

我可以完成第一次上传,但是到上传完成的时候,JavaScript已经回收了或者修改了DataTransfer项对象。

我该如何重新设计它(或者克隆事件/DataTransfer对象),以便在整个ajax调用过程中仍然可以使用数据?

我已经按照MDN文档使用DataTransfer,但是我很难将其应用于我的具体情况。 我也尝试过复制事件对象,就像您在我的代码中看到的那样,但这显然并不是深度复制,而只是传递引用,这没有帮助。

    methods: {
        dropHandler: function (event) {
            if (event.dataTransfer.items) {
                let i = 0;
                let self = this;
                let ev = event;

                function uploadHandler() {
                    let items = ev.dataTransfer.items;
                    let len = items.length;

                    // len NOW EQUALS 4

                    console.log("LEN: ", len);
                    if (items[i].kind === 'file') {
                        var file = items[i].getAsFile();
                        $('#id_file_name').val(file.name);
                        var file_form = $('#fileform2').get(0);
                        var form_data = new FormData(file_form); 

                        if (form_data) {
                            form_data.append('file', file);
                            form_data.append('type', self.type);
                        }

                        $('#file_progress_' + self.type).show();
                        var post_url = '/blah/blah/add/' + self.object_id + '/'; 
                        $.ajax({
                            url: post_url,
                            type: 'POST',
                            data: form_data,
                            contentType: false,
                            processData: false,
                            xhr: function () {
                                var xhr = $.ajaxSettings.xhr();
                                if (xhr.upload) {
                                    xhr.upload.addEventListener('progress', function (event) {
                                        var percent = 0;
                                        var position = event.loaded || event.position;
                                        var total = event.total;
                                        if (event.lengthComputable) {
                                            percent = Math.ceil(position / total * 100);
                                            $('#file_progress_' + self.type).val(percent);
                                        }
                                    }, true);
                                }
                                return xhr;
                            }
                        }).done((response) => {
                                i++;
                                if (i < len) {

                                    // BY NOW, LEN = 0.  ????

                                    uploadHandler();
                                } else {
                                    self.populate_file_lists();
                                }
                            }
                        );
                    }
                }

                uploadHandler();
            }
        },

2
问题甚至不仅限于Vue.js...这也是纯JS应用程序的问题。我制作了一个更简单的测试用例来重现此问题:https://jsfiddle.net/rjq6b83t/1/。如果您使用浏览器的开发人员工具,您会发现“下一个循环”甚至没有发生,因为DataTransfer实例似乎在那时已经失效。 - Brad
@Brad,将 Promise 推入数组中并稍后使用 Promise.All 处理它们怎么样?https://jsfiddle.net/g5h4ajm8/2/ - Temo Tchanukvadze
@TemoJr。没错,我认为关键是在调用堆栈之前获取“entry”。 - Brad
3个回答

14

一旦您调用 await,您就不再位于函数的原始调用堆栈中。这在事件监听器中尤为重要。

我们可以使用 setTimeout 来复制相同的效果:

dropZone.addEventListener('drop', async (e) => {
  e.preventDefault();
  console.log(e.dataTransfer.items);
  setTimeout(()=> {
    console.log(e.dataTransfer.items);
  })
});

例如,拖动四个文件会输出:

DataTransferItemList {0: DataTransferItem, 1: DataTransferItem, 2: DataTransferItem, 3: DataTransferItem, length: 4}  
DataTransferItemList {length: 0}

事件发生后,状态已更改,物品已丢失

有两种方法来解决这个问题:

  • 复制项目并对其进行迭代
  • 将异步作业(Promises)推入数组中,并稍后使用Promise.all处理它们

第二种解决方案比在循环中使用await更直观。此外,请考虑并行连接受到限制。使用数组,您可以创建块以限制同时上传的数量。

function pointlessDelay() {
  return new Promise((resolve, reject) => {
    setTimeout(resolve, 1000);
  });
}

const dropZone = document.querySelector('.dropZone');

dropZone.addEventListener('dragover', (e) => {
  e.preventDefault();
});

dropZone.addEventListener('drop', async (e) => {
  e.preventDefault();
  console.log(e.dataTransfer.items);
  const queue = [];
  
  for (const item of e.dataTransfer.items) {
    console.log('next loop');
    const entry = item.webkitGetAsEntry();
    console.log({item, entry});
    queue.push(pointlessDelay().then(x=> console.log(`${entry.name} uploaded`)));
  }
  
  await Promise.all(queue);
});
body {
  font-family: sans-serif;
}

.dropZone {
  display: inline-flex;
  background: #3498db;
  color: #ecf0f1;
  border: 0.3em dashed #ecf0f1;
  border-radius: 0.3em;
  padding: 5em;
  font-size: 1.2em;
}
<div class="dropZone">
  Drop Zone
</div>


在发送请求的情况下使用Promise.all可能会产生连接限制问题。例如,如果用户尝试一次上传20个文件,则浏览器会崩溃几个超额请求。但当然从另一方面来看,它比async ... loop更快。 - Alexandr Tovmach
1
很好的观点@AlexandrTovmach。在这种情况下,解决方案是将数组拆分成块并创建队列。 - Temo Tchanukvadze
...或者只需使用async...loop =) - Alexandr Tovmach
@AlexandrTovmach 如果你使用async循环,那么你的请求就不会并行执行。将承诺数组拆分成块并使用Promise.all,您可以获得更快的结果,并且安全地避免任何限制。 - Christos Lytras
是的,我知道并且注意到了之前的评论,但从代码角度来看,将队列分成块有点繁琐。请记住:“你写的代码是给人看的,而不是给机器看的”,在你真正遇到问题之前,你不需要考虑“更快的结果”。 - Alexandr Tovmach
生成器怎么样?如果我对问题领域和这个答案的理解是正确的,可以考虑类似这样的东西... https://medium.com/javascript-scene/the-hidden-power-of-es6-generators-observable-async-flow-control-cfa4c7f31435 - Jacob

5

看起来DataTransfer的上下文信息逐渐丢失了。我的解决方案是在信息丢失之前复制所需数据,并在需要时重用:

const files = [...e.dataTransfer.items].map(item => item.getAsFile());

这是我根据jsfiddle上由@Brad提供的代码所做的修改:

const dropZone = document.querySelector(".dropZone");
const sendFile = file => {
  const formData = new FormData();
  for (const name in file) {
    formData.append(name, file[name]);
  }
  /**
   * https://docs.postman-echo.com/ - postman mock server
   * https://cors-anywhere.herokuapp.com/ - CORS proxy server
   **/
  return fetch(
    "https://cors-anywhere.herokuapp.com/https://postman-echo.com/post",
    {
      method: "POST",
      body: formData
    }
  );
};

dropZone.addEventListener("dragover", e => {
  e.preventDefault();
});

dropZone.addEventListener("drop", async e => {
  e.preventDefault();
  const files = [...e.dataTransfer.items].map(item => item.getAsFile());
  const responses = [];

  for (const file of files) {
    const res = await sendFile(file);
    responses.push(res);
  }
  console.log(responses);
});
body {
  font-family: sans-serif;
}

.dropZone {
  display: inline-flex;
  background: #3498db;
  color: #ecf0f1;
  border: 0.3em dashed #ecf0f1;
  border-radius: 0.3em;
  padding: 5em;
  font-size: 1.2em;
}
<div class="dropZone">
  Drop Zone
</div>


1
我遇到了这个问题,想要持久化整个 DataTransfer 对象,而不仅仅是 itemstypes,因为我的异步代码 API 消耗的是 DataTransfer 类型本身。最终我所做的是创建一个 new DataTransfer(),并有效地复制原始属性(除了拖动图像)。

以下是要点(使用 TypeScript): https://gist.github.com/mitchellirvin/261d82bbf09d5fdee41715fa2622d4a6

// https://developer.mozilla.org/en-US/docs/Web/API/DataTransferItem/kind
enum DataTransferItemKind {
  FILE = "file",
  STRING = "string",
}

/**
 * Returns a properly deep-cloned object of type DataTransfer. This is necessary because dataTransfer items are lost
 * in asynchronous calls. See https://dev59.com/oVMI5IYBdhLWcg3wptRR
 * for more details.
 * 
 * @param original the DataTransfer to deep clone
 */
export function cloneDataTransfer(original: DataTransfer): DataTransfer {
  const cloned = new DataTransfer();
  cloned.dropEffect = original.dropEffect;
  cloned.effectAllowed = original.effectAllowed;

  const originalItems = original.items;
  let i = 0;
  let originalItem = originalItems[i];
  while (originalItem != null) {
    switch (originalItem.kind) {
      case DataTransferItemKind.FILE:
        const file = originalItem.getAsFile();
        if (file != null) {
          cloned.items.add(file);
        }
        break;
      case DataTransferItemKind.STRING:
        cloned.setData(originalItem.type, original.getData(originalItem.type));
        break;
      default:
        console.error("Unrecognized DataTransferItem.kind: ", originalItem.kind);
        break;
    }

    i++;
    originalItem = originalItems[i];
  }
  return cloned;
}

你可以这样使用它,并像最初计划使用 evt.dataTransfer 一样使用 clone:
const clone = cloneDataTransfer(evt.dataTransfer);

请注意,如果您这样做并且需要通过webkitGetAsEntry()调用访问“条目”,这个解决方案将不起作用。 - undefined

网页内容由stack overflow 提供, 点击上面的
可以查看英文原文,
原文链接