這是否正確的JavaScript語法？

Question

良好的星期六上午到網上最有幫助的在線社區！ =）

我有幾百行JavaScript我想重新格式化，以方便我閱讀。我想，在我浪費了一天的一半之前，我會先在這裏問我的問題。

我是JavaScript新手，想知道下面的語法是否正確？

我原來的代碼是這樣的：

function formfocus() { 
    if(document.getElementById('inputida')) { 
     document.getElementById('inputida').focus(); 
     document.getElementById('inputida').value = ''; 
    } 
    else if(document.getElementById('inputidb')) { 
     document.getElementById('inputidb').focus(); 
     document.getElementById('inputidb').value = ''; 
    } 
} 

function popitup(url, name, width, height) { 
    newwindow = window.open(url, name, 'width=' + width + ',height=' + height + ',scrollbars=yes'); 

    if(window.focus) { 
     newwindow.focus(); 
    } 
    return false; 
} 

......我想改變的代碼到這個（注意間距）：

function formfocus() 
{ 
    if (document.getElementById('inputida')) 
    { 
     document.getElementById('inputida').focus(); 
     document.getElementById('inputida').value = ''; 
    } 
    else if (document.getElementById('inputidb')) 
    { 
     document.getElementById('inputidb').focus(); 
     document.getElementById('inputidb').value = ''; 
    } 
} 

function popitup(url, name, width, height) 
{ 
    newwindow = window.open(url, name, 'width=' + width + ', height=' + height + ', scrollbars=yes'); 

    if (window.focus) 
    { 
     newwindow.focus(); 
    } 
    return false; 
} 

的區別是：

1. 後面的空格'if'和'else if'陳述
2. 間距圍繞parentheses
3. 換行開curly braces

Answer 1

是的，你的新的語法是有效的，等於把老了。

除了非常隱晦的情況之外，JavaScript代碼中的換行符和空格被忽略，所以您可以根據自己的喜好進行佈局。

但是舊的語法是如何編寫慣用的JavaScript - 一位經驗豐富的JavaScript程序員看着你的新語法會覺得很奇怪。

Answer 2

是它是有效的。

這增加了可讀性。

但是，在您的發佈版本中，最好是縮小您的js文件以減少帶寬使用量。

Jsmin是一個javascript minifer。

JSMin是一個過濾器，其去除JavaScript文件 註釋和不必要的空白 。它通常 減少一半的文件大小，導致 更快的下載。它還鼓勵 更具表現力的編程風格 ，因爲它消除了下載 清潔成本，識字 自我文檔。

Answer 3

儘量避免多次調用的document.getElementById：

var objInputId = document.getElementById('inputid'); 
objInputId.focus(); 
objInputId.value = "val"; 

Answer 4

是，JavaScript並不是基於行的，並且不需要最間距。

你甚至可以這樣寫在riddiculously不可讀的方式代碼：

function formfocus(){if(document.getElementById(
'inputid')){document.getElementById('inputid').focus 
();document.getElementById('inputid').value='';}else 
if(document.getElementById('inputid')){document. 
getElementById('inputid').focus();document.getElementById 
('inputid').value='';}}function popitup(url,name,width, 
height){newwindow=window.open(url,name,'width='+width+', 
height='+height+',scrollbars=yes');if(window.focus 
){newwindow.focus();}return false;} 

或者這樣：

function 
formfocus 
(
) 
{ 
if 
(
document 
. 
getElementById 
(
'inputid' 
) 
) 
{ 
document 
. 
getElementById 
(
'inputid' 
) 
. 
focus 
(
) 
; 
document 
. 
getElementById 
(
'inputid' 
) 
. 
value 
= 
'' 
; 
} 
else 
if 
(
document 
. 
getElementById 
(
'inputid' 
) 
) 
{ 
document 
. 
getElementById 
(
'inputid' 
) 
.focus 
(
) 
; 
document 
. 
getElementById 
(
'inputid' 
) 
. 
value 
= 
'' 
; 
} 
} 
function 
popitup 
(
url 
, 
name 
, 
width 
, 
height 
) 
{ 
newwindow 
= 
window 
. 
open 
(
url 
, 
name 
, 
'width=' 
+ 
width 
+ 
',height=' 
+ 
height 
+ 
',scrollbars=yes' 
) 
; 
if 
(
window 
. 
focus 
) 
{ 
newwindow 
. 
focus 
(
) 
; 
} 
return 
false 
; 
} 

Answer 5

你的變化看起來有效，但還有很多清理的代碼做。

• 正如James Wiseman所說的，您要避免重複調用document.getElementById，它效率低下，難於閱讀。
• 在formfocus，前兩個if語句是相同的;第二個是毫無意義的。
• 在popitup，代碼要創建一個隱式的全局變量，這是一個壞的事情。它需要一個var。

這樣：

function formfocus() 
{ 
    var elm; 

    elm = document.getElementById('inputid'); 
    if (elm) 
    { 
     elm.focus(); 
     elm.value = ''; 
    } 
} 

function popitup(url, name, width, height) 
{ 
    var newwindow = window.open(
     url, 
     name, 
     'width=' + width + ', height=' + height + ', scrollbars=yes' 
    ); 

    if (window.focus) 
    { 
     newwindow.focus(); 
    } 
    return false; 
} 

（有關隱含的全局變量，看blog.niftysnippets.org/2008/03/horror-of-implicit-globals.html;我做了不一個環節，因爲這是我自己的博客 - 不，還有很多有 - 我不希望鏈接的垃圾郵件）