0

私はこの機能を持っています:

//add links to called classes
function addLinks () {
    var classElements = [];
    var idElements = [];
    var finalRender;
    var expand = document.createTextNode("+");
    var contract = document.createTextNode("-");
    var elementsList = [];
    var count = 0;

    //create the dom nodes
    renderPElements = document.createElement ("p");

        renderAElements = document.createElement ("a");
        renderAElements.setAttribute("href", "#");
        renderAElements.setAttribute("class", "expander");
        renderAElements.appendChild(expand);
        finalRender = renderPElements.appendChild(renderAElements);


    //get arrays of elements with class or id set to provided string
    for (var i = 0; i< show_hide_class_selectors.length; i++) {
        classElements[i] = document.getElementsByClassName(show_hide_class_selectors[i]);

        //if prevents null value appearing in array and freezing script
        if (document.getElementById(show_hide_class_selectors[i])) {
        idElements[i] = document.getElementById (show_hide_class_selectors[i]);
        }
    }

    //loop though selected id's / classes and generate a single array of selected elements
    for (var i = 0; i< idElements.length; i++) {
        elementsList[count] = idElements[i];
        count = count +1;
    }

    //must loop twice as variable is 2 dimensional i=class name y=elements of that class
    for (var i = 0; i< classElements.length; i++) {
        for (var y = 0; y< classElements[i].length; y++) {
        elementsList[count] = classElements[i][y];
        count = count +1;
        }
    }

    //render
    for (var i = 0; i< elementsList.length; i++) {
        alert ("render");
        elementsList[i].parentNode.firstChild.appendChild(finalRender);

        alert (elementsList[i]);
    }
}

これは、グローバル変数として提供されるクラス/ID の配列を取得し、要求されたすべての要素を含む配列を生成することを意味します。次に、appendchild を使用して生成された配列をループすることにより、子ノード (この場合はリンク) を兄弟ノードに追加することになっています。

ただし、余分なリンクがたくさんあるページになるのではなく、最終ループは代わりに子を追加してすぐに削除し、生成されたリンクを保持することが許可されている最終要素までループを処理します。

この動作や同様の問題を抱えている人については、説明がありません。

4

1 に答える 1

2

要素を DOM の複数の場所に追加することはできません。ループのたびに(および子孫) の別のコピーを追加する必要があります。finalRender

elementsList[i].parentNode.firstChild.appendChild(finalRender.cloneNode(true));

ブラウザによって、イベントがバインドされたクローン要素の処理が異なることに注意してください。元のイベントと同じイベントがバインドされたクローンになるものもあれば、そうでないものもあります。

また、これらのループは簡潔にするために叫んでいます。真ん中の 2 つは次のように完全に置き換えることができます。

elementList = idElements;
for (var i=0; i<classElements.length; ++i)
    elementList = elementList.concat(classElements[i]);

または、さらに良いelementListことに、最初のループ内で直接構築するだけで、後で実際に使用しない一連の配列を作成しないか、または最初のループで追加を行うだけです。

于 2011-07-01T14:18:26.377 に答える