Question

我正在制作一个多选列表，您可以在其中选择项目，然后单击“向上”或“向下”按钮，该按钮将允许您对列表中的这些项目重新排序。

我有一个简单的自包含示例：

<html>
    <head>
        <title>Example</title>
    <script src="https://www.google.com/jsapi"></script>
    <script>
        google.load('jquery', '1.4.1');
    </script>
    </head>
    <body>
        <select id="selectedDataPoints" multiple="multiple">
            <option>Pig</option>
            <option>Duck</option>
            <option>Dog</option>
            <option>Zebra</option>
            <option>Snake</option>
            <option>Giraffe</option>
            <option>Cow</option>
        </select>
    <input type="button" id="btnReorderUp"  value="Up" />
    <input type="button" id="btnReorderDown" value="Down" />
    </body>
</html>

<script type="text/javascript">
    var DataPointSelector = (function() {

    var $selectedList = $('#selectedDataPoints');

        $('#btnReorderUp').click(function(e) {
            moveUp();
            e.preventDefault();
        });

        $('#btnReorderDown').click(function(e) {
            moveDown();
            e.preventDefault();
        });

    function moveUp() {
            var select = $selectedList[0];
            for (var i = 1, n = select.options.length; i < n; i++)
                if (select.options[i].selected && !select.options[i - 1].selected)
                select.insertBefore(select.options[i], select.options[i - 1]);
        }

        function moveDown() {
            var select = $selectedList[0];
            for (var i = select.options.length - 1; i > 0; i--)
                if (select.options[i].selected && !select.options[i + 1].selected)
                    select.insertBefore(select.options[i + 1], select.options[i]);           
        }

    } ());
</script>

但是，向上/向下按钮需要3-8秒才能在IE7 / 8中生效。第一次点击有时会很快但在此之后，选择列表需要很长时间才能使用新位置进行更新（特别是如果您选择了几个项目）。

Chrome / FF中不存在此问题。我不会想到这个简单的重新排序会如此密集！

有没有人有任何可用于帮助我加快moveUp / moveDown功能的信息？！

Answer 1

不是移动现有的DOM元素，而是重新创建整个<select>元素，包括IE的所有子元素。

Answer 2

IE缓慢，缓慢，缓慢。所以你应该检查你的代码，以尽量减少不必要的操作。

@Arron Digulla的想法很好。但您也可以通过以下更改来最小化处理器时间：

function moveUp() {
  var select = $selectedList[0];
  for (var i = 1, n = select.options.length; i < n; i++)
    var selected = select.options[i],
        selectedPrior = select.options[i - 1]; 
        // Don't locate obj/array element more than once
    if (selected.selected && !selectedPrior.selected) {
      select.insertBefore(selected, selectedPrior);
      break; // Since we have found the right elements, don't do 
    }        // any further checking
  }

补充：嗯，我现在看到可以选择多个项目。所以你需要移动多个项目，是吗？在那种情况下，break语句是不对的。

IE UI图层是否会陷阱？ - 在JS线程结束之前，IE往往不会更新屏幕的外观。解决方法是以0秒运行计时器。这将产生IE渲染引擎并使屏幕更新。

为什么这个javascript在IE中运行得如此之慢，只要它修改一个选择列表？

2 个答案: