javascript - 避免为每个元素重复使用 javascript

Question

我正在使用 Google dialog-polyfill对于我网站上的模式。我需要在一个页面上有多个可以独立启动的模式。现在，我唯一可以开始工作的方法是重复所有的 javascript，这似乎效率很低:

<dialog id="dialog" name="modal">
    <p>This is the login modal dialog.</p>
    <input type="text"></input>
    <p><button id="close-dialog">Close dialog</button></p>
</dialog>
<p><button id="open-dialog">Open login dialog</button></p>

<dialog id="dialog2" name="modal">
    <p>This is the registration modal dialog.</p>
    <input type="text"></input>
    <p><button id="close-dialog2">Close dialog</button></p>
</dialog>
<p><button id="open-dialog2">Open registration dialog</button></p>

<script>
    var dialog = document.getElementById('dialog');
    var dialog2 = document.getElementById('dialog2');
    dialogPolyfill.registerDialog(dialog);
    dialogPolyfill.registerDialog(dialog2);

    document.getElementById('open-dialog').addEventListener('click', function () {
        dialog.showModal();     });
    document.getElementById('close-dialog').addEventListener('click', function () {
        dialog.close();     });

    document.getElementById('open-dialog2').addEventListener('click', function () {
        dialog2.showModal();     });
    document.getElementById('close-dialog2').addEventListener('click', function () {
        dialog2.close();     });

    dialog.addEventListener('click', function (event) {
        var rect = dialog.getBoundingClientRect();
        var isInDialog = (rect.top <= event.clientY && event.clientY <= rect.top + rect.height && rect.left <= event.clientX && event.clientX <= rect.left + rect.width);
        if (!isInDialog) {
            dialog.close();
        }
    });

    dialog2.addEventListener('click', function (event) {
        var rect = dialog2.getBoundingClientRect();
        var isInDialog = (rect.top <= event.clientY && event.clientY <= rect.top + rect.height && rect.left <= event.clientX && event.clientX <= rect.left + rect.width);
        if (!isInDialog) {
            dialog2.close();
        }
    });
</script>

我已经尝试过这个并添加 name 属性但无济于事:

    var names = document.getElementsByName("modal");
    for (var i = 0; i < names.length; i++) {
        names[i].onchange = function () {
            document.getElementById('open-dialog').addEventListener('click', function () {
                dialog.showModal();
            });
            document.getElementById('close-dialog').addEventListener('click', function () {
                dialog.close();
            });
        }
    }

我不明白类如何解决这个问题。有没有办法压缩代码，这样我就不需要为每个模式对话框重复它？

Answer 1

我编辑了你的 fiddle :

http://jsfiddle.net/178f81qc/2/

你是这个意思吗？

HTML

<div class="dialog-container">
    <dialog>This is dialog one <button class="close">close</button></dialog>

    <button class="open">open dialog one</button>
</div>

<br>

<div class="dialog-container">
    <dialog>This is dialog two <button class="close">close</button></dialog>

    <button class="open">open dialog two</button>
</div>

Javascript

var dialogContainers = document.querySelectorAll('.dialog-container'), i;
for (i=0; i<dialogContainers.length; i++) {

    var dialog = dialogContainers[i].querySelector('dialog'), // the <dialog> element
        btnOpen = dialogContainers[i].querySelector('.open'),  // the open button
        btnClose = dialogContainers[i].querySelector('.close'); // the close button

    // for old browsers
    dialogPolyfill.registerDialog(dialogContainers[i]);

    btnOpen.addEventListener('click', function () {
        this.showModal();
    }.bind(dialog));

    // because you bind 'dialog' to the function, 'this' will be the dialog.
    // (if you don't use bind, 'this' will be btnOpen instead)

    btnClose.addEventListener('click', function () {
        this.close();
    }.bind(dialog));
}