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 回答