如何简单的if/else条件代码

Question

我想简化下面的脚本。条件陈述似乎是多余的有没有人在这里帮助我？非常感谢。

function createHelpImage(element, elementType){ 

    $element = $(element); 
    if(elementType=='img'){ 

    $(document.createElement('img')) 
    .attr({src:'inc/images/help_bubble.png', title:'help Image', 'class': 'helpImg'}) 
    .insertBefore($element) 
    .css({'position':'absolute', 
     'z-index':999 
    }) 
    .position({ 
     my: 'left bottom', 
     at: 'left center', 
     of: $element 

    }); 
    }else{ 
    $(document.createElement('img')) 
    .attr({src:'inc/images/help_bubble.png', title:'help Image', 'class': 'helpImg'}) 
    .prependTo($element) 
    .css({'position':'absolute', 
     'z-index':999 
    }) 
    .position({ 
     my: 'left bottom', 
     at: 'left center', 
     of: $element[0] 

    }); 
    } 
} 

Answer 1

刚从删除冗余代码的if/else，使用一个变量来保存价值of再经过

$element = $(element); 
var of; 
var img = $(document.createElement('img')) 
    .attr({src:'inc/images/help_bubble.png', title:'help Image', 'class': 'helpImg'})  
    .css({'position':'absolute', 
     'z-index':999 
    }) 
if(elementType=='img'){ 
    img.insertBefore($element); 
      of = $element; 
} 
else{ 
    img.prependTo($element); 
      of = $element[0]; 
} 
img.position({ 
    my: 'left bottom', 
    at: 'left center', 
    of: of 
}); 

Answer 2

设置它只是采取差异，并把if语句周围的人。有一个原则叫做不要重复自己（简称DRY），这在这里适用。

Answer 3

尝试在最后一行用ternary operator：

of: (elementType=='img') ? $elemen : $elemen[0] 

这是假设，只有最后一行是在你的if/else代码块不同。你的代码如下所示：

function createHelpImage(element, elementType){ 
    $(document.createElement('img')) 
    .attr({src:'inc/images/help_bubble.png', title:'help Image', 'class': 'helpImg'}) 
    .insertBefore($element) 
    .css({'position':'absolute', 
     'z-index':999 
    }) 
    .position({ 
     my: 'left bottom', 
     at: 'left center', 
     of: (elementType=='img') ? $element : $element[0] 
    }); 
}