Javascript函數格式

Question

如果條件格式不正確，我很擔心。
這看起來合適嗎？

function validateCoupon(form){ 

if (form.textCoupon.value.length){ 

    if (form.textCoupon.value.toLowerCase() == "Item01") { 
    _gaq.push(['_trackEvent', 'Coupon', 'Activated', 'Item01']); 
    } 

    if (form.textCoupon.value.toLowerCase() == "Item02") { 
    _gaq.push(['_trackEvent', 'Coupon', 'Activated', 'Item02']); 
    } 

$.get("/include/checkCoupon.php", { coupon: form.textCoupon.value }, validateShipping); 
} 
    else { 
    form.textCoupon.style.border = ''; 
    validateShipping("yes"); 
    } 
return false; 
} 

Answer 1

如果這是我的代碼，這將是我怎麼寫的：

var validateCoupon = function (form) { 
    var textCoupon = form.textCoupon, 
     value = textCoupon.value, 
     track = function (value) { 
      _gaq.push(['_trackEvent', 'Coupon', 'Activated', value]); 
     }; 

    if (value.length) { 
     if (value === 'Item01' || value === 'Item02') { 
      track(value); 
     } 

     $.get('/include/checkCoupon.php', { coupon: value }, validateShipping); 
    } else { 
     textCoupon.style.border = ''; 
     validateShipping('yes'); 
    } 

    return false; 
}; 

Answer 2

的源代碼的格式是無關的JavaScript運行。這只是個人品味的問題。

Answer 3

好了，東西似乎是一個冗餘：form.textCoupon.value可能是Item01或Item02。如果它是一個它不可能是另一個，所以我建議你開關聲明。

的另一個問題是，如果你調用.toLowerCase()這絕不會返回Item01但item01，和字符串平等是區分大小寫的。要麼將這個函數調用到條件的兩個部分，要麼就是不要使用它。

Answer 4

避免在縮進中使用製表符：它們在所有平臺上的渲染效果不盡相同。