0

我已经开始编写这个脚本，它允许你一次上传多个文件。所以我还没有进入上传部分，只是文件存储等。

到目前为止，我已经完成了文件添加，添加文件后，它会创建一个 div 以便用户可以看到他将要上传的文件。

工作得很好。

现在我为每个 div 添加了一个 id，例如：<div id="file_filename" class="files">Filename</div> 现在我想在你点击文件名时将其删除。

所以我添加了一个文件删除功能，如下所示：

（我从 document.body 加载，因为它不是自然 div）

$(document.body).on("click", ".files", function(e) {
    console.log("Sent id: " + e.target.id);
    removeFile(e.target.id);
});

function removeFile(file) {
    var split = file.split("file_");
    console.log("Now going to loop for file: " + file);
    console.log("Splitted version: " + split[1]);
    for (i = 0; i < images.length; i++) {
        if (images[i].name != split[1]) {
            continue;
        }
        console.log("removed " + split[1] + " with index: " + i + " and file name: " + images[i].name);
        images.pop(i + 1);
    }

    updateDivs();

    console.log("files left: ");
    for (i = 0; i < images.length; i++) {
        console.log(images[i].name);
    }
}

基本上，它获取点击类的 ID，并将 id 发送到 removeFile 函数。removeFile 函数将拆分文本，以便在之后获取文件名file_

然后我遍历图像数组并检查名称是否相等，如果是，则删除该文件并重新加载 div。

重新加载 div 函数：

function updateDivs() {
    var divs;
    for (i = 0; i < images.length; i++) {
        divs = (divs != null) ? divs + "<div id='file_" + images[i].name + "'class='files'>" + images[i].name + "</div>" : "<div id='file_" + images[i].name + "'class='files'>" + images[i].name + "</div>";
    }
    $("#myImages").html(divs);
}

如您所见，我有很多日志正在更新，因此删除后，我得到了以下日志：

Sent id: file_images.jpg upload.js:82
Now going to loop for file: file_images.jpg upload.js:63
Splitted version: images.jpg upload.js:64
removed images.jpg with index: 0 and file name: images.jpg upload.js:69
files left:  upload.js:75
images.jpg

问题：

如果你注意到了，我点击了 images.jpg，然后它说它已删除 images.jpg，但随后它说 files left: images.jpg。

它没有删除 images.jpg，而是删除了我上传的另一个文件。

我已经尝试了我所知道的一切，我知道最好自己解决这些问题，但是这个问题对我来说毫无意义。

我在那里做错了什么？

编辑：顺便说一下，images[0] = images.jpg，我已经先上传了。

4

1 回答 1

0

不知道您的图像阵列是什么样子，这可能不是 100% 正确，但我怀疑您的问题与线路有关images.pop(i + 1);

.pop()不接受任何参数，只从数组中删除最后一项。如果要按索引从数组中删除元素，请splice()像这样使用：

images.splice(i-1, 1);（这将在你的 for 循环的当前索引中删除 1 项）

一个例子：

<script type="text/javascript">

var images = ["banana", "orange", "apple"];
images.splice(1,1);     // Will remove 1 item at position 1 (orange)
alert(images);          // Should read: "banana, apple"

</script>

希望这可以帮助。

于 2013-10-08T14:01:56.850 回答