使用递归JavaScript函数导致无限循环

3

我正在为一位摄影师开发后台,其中照片将会被归类到相册和子分类中。
在相册创建页面中,我使用JavaScript来添加一个新的.photo块(包含其HTML内容)。
问题是,在albums.addPhoto函数中似乎出现了无限循环。下面是我的JS代码(albums.addPhotoonclick属性调用并调用albums.createBlock):

    var albums =  {
    createBlock: function (tag, attributes, text, elemID, elemCLASS) {
        var block = document.createElement(tag);
        switch (tag) {
            case "label":
            block.setAttribute('for', attributes[0]);
            break;
            case "input":
            block.setAttribute('type', attributes[0]);
            block.setAttribute('name', attributes[1]);
            if (attributes[0] == "text") {
                block.setAttribute('placeholder', attributes[2]);
            } else {
                block.setAttribute('value', attributes[2]);
            }
            break;
            default:

        }

        if (text) {
            block.textContent = text;
        }
        if (elemID) {
            block.id = elemID;
        }
        if (elemCLASS) {
            block.classList.add(elemCLASS);
        }

        return block;
    },
    addPhoto: function (currentSsCat, requiredPhotoID) {
        var elem = albums.createBlock("div", null, false, ("photo_" + currentSsCat + "_" + requiredPhotoID), "photo");
        document.querySelector('#ss_cat_1').appendChild(elem);

        elem.appendChild(albums.createBlock("label", ["file_" + currentSsCat + "_" + requiredPhotoID], "Importez le fichier photo", false, false));
        elem.appendChild(albums.createBlock("input", ["file", ("file_" + currentSsCat + "_" + requiredPhotoID), "Votre photo"], false, false, false));
        elem.appendChild(albums.createBlock("label", ["ref_" + currentSsCat + "_" + requiredPhotoID], "Référence de la photo", false, false));
        elem.appendChild(albums.createBlock("input", ["text", ("ref_" + currentSsCat + "_" + requiredPhotoID), "Exemple: 20160811_CLS_0005"], false, false, false));
        elem.appendChild(albums.createBlock("label", ["desc_" + currentSsCat + "_" + requiredPhotoID], "Description de la photo", false, false));
        elem.appendChild(albums.createBlock("input", ["text", ("desc_" + currentSsCat + "_" + requiredPhotoID), "Exemple: Paolo FABBRI et Lenas Drawin His Gun (ITA)."], false, false, false));

        document.querySelector("#add_photo_" + currentSsCat).onclick = albums.addPhoto(currentSsCat, (requiredPhotoID + 1));
    }
};

然后你有HTML部分(我只给你form部分,我可以发送整个页面,但我认为这没用):
<form action="index.html?page=aa" method="post">
<!-- SOUS CATEGORIE -->
        <div class="ss_cat" id="ss_cat_1">
            <label for="ss_cat_1_titre">Titre :</label>
            <input type="text" name="ss_cat_1_titre" placeholder="Titre de la sous-catégorie...">

            <!-- PHOTOS DE LA CATEGORIE -->
            <div class="photo" id="photo_1_1">
                <label for="file_1_1">Importez le fichier photo :</label>
                <input type="file" name="file_1_1" value="Votre photo">
                <label for="ref_1_1">Référence de la photo :</label>
                <input type="text" name="ref_1_1" placeholder="Exemple: 20160811_CLS_0005">
                <label for="desc_1_1">Description de la photo :</label>
                <input type="text" name="desc_1_1" placeholder="Exemple: Paolo FABBRI et Lenas Drawin His Gun (ITA).">
            </div>
        </div>


        <!-- This is the link calling the function albums.addPhoto -->
        <a href="#" class="btn" id="add_photo_1" onclick="albums.addPhoto(1, 2)">Ajouter une photo</a>
    </div>
    <a href="#" class="btn" id="add_ss_cat">Ajouter un sous-catégorie</a>
    <input type="submit" name="submit" value="Ajouter le nouvel album">
</form>

测试:该函数完成了其工作,但会产生无限循环(浏览器崩溃或询问是否停止脚本,这表明已创建了大量.photo块)。
Chrome中,我会收到消息“最大调用堆栈大小超出”,而在Firefox中,我会收到“递归太深”的消息。

备注:这是我创建后台的第一次尝试,如果您有任何替代方法的建议,请随时提出。感谢您的阅读和帮助。

1个回答

4
这是您的问题:
 document.querySelector("#add_photo_" + currentSsCat).onclick = albums.addPhoto(currentSsCat, (requiredPhotoID + 1));

你在 albums.addPhoto 中无条件调用了 albums.addPhoto,导致出现了无限递归。你可能想要使用下面这个版本代替:
document.querySelector("#add_photo_" + currentSsCat).onclick = function () { albums.addPhoto(currentSsCat, (requiredPhotoID + 1)); };

这样,您将onclick处理程序设置为函数,而不是函数的结果(该结果未定义并且无意义),仅当单击事件发生时才运行。


谢谢,它完美地运行了!我在这个项目中遇到了很多问题,不知道问题的根源在哪里。 - AymDev
@QuangdaoNguyen 不好意思,我不知道那个。已经完成了,谢谢 :) - AymDev

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