0

我试图简化我大量的 else if 语句。有没有更好的方法来获得更短的代码？大多数条件都使用不同的 css 属性和插入/附加到元素的方式..

谢谢您的帮助！

var img = $("<img src='images//anim.gif class='Img'>");

            var cssProp = {'position':'absolute',
                       'z-index':99,
                       'top': topPos,
                       'left': leftPos
            }

//$element var represent all kinds of elements I have in my DOM           

            if($element.attr('id')=='count' || $element.attr('id')=='assign' ){
                img.prependTo($element)          
            }else if($element.is('#title')){
                cssProp.top=+20;
                img.insertBefore($element);

            }else if($element.is('#tab')){
                detail_img = $("<img src='images/anim.gif' id='tab1' class='Img'>");
                detail_img.insertBefore($element.parent());
                detail_img.css(cssProp);

            }else if($element.is('#tab2')){ 
                detail_img = $("<img src='images/anim.gif' id='tab2' class='Img'>");
                detail_img.insertBefore($element.parent());
                detail_img.css(cssProp);         

            }else if ($element.is('#e_btn')){
                cssProp.left=+20;
                cssProp.top=+3;
                img.insertBefore($element);
                img.css(cssProp);

            }else if($element.is('#date_set')){
                img.insertBefore($element);

            }else if($element.is('td')){
                img.prependTo($element).css(cssProp);

            }else if($element.is('#l_title')){
                cssProp.top+=10;
                img.insertBefore($element);
                img.css(cssProp);

            }else if($element.is('#status')){
                cssProp.top =+10;
                img.insertBefore($element).css(cssProp);

            }else if($element.is('label')){
                img.appendTo($element)

            }else if($element.closest('#divTab').length>0){
                ($element.attr('id')=='work')? cssProp.left+=10 : '';
                cssProp.top+=5;
                img.insertBefore($element);
                img.css(cssProp)
           }else{      
                img.insertBefore($element);
                img.css(cssProp)
            }
4

1 回答 1

1

要减少行数：

  • 将每个子句中的表达式if{}减少到一行 - 这是相当微不足道的。必要时，使用$.extend()来操纵cssProp.
  • 有些表达式是相同的，所以它们的if条件可以用||.
  • 删除空行。

这样做，我将 if/else 链从 49 行减少到 21 行。

要提高运行时效率，请从以下语句开始：

var id = $element.attr('id');
var tag = $element.get(0).tagName;

好的，这增加了 2 行

然后，在条件中，对而不是使用 jQuery 表达式if执行字符串比较。唯一的例外是，它不遵循相同的模式。idtagif($element.closest('#divTab').length>0)

于 2012-09-07T02:35:56.943 回答