Question

如何缩小此代码？也许是一个切换，但人们说这很容易在jQuery中完成。但问题是我不喜欢在我的代码中只使用jQuery。

function open() {
document.getElementById('message').style.display='block';
document.getElementById('fade').style.display='block';
}

function close() {
document.getElementById('message').style.display='none';
document.getElementById('fade').style.display='none';
}

Answer 1

var message = document.getElementById('message'),
    fade = document.getElementById('fade');

function open() {
    message.style.display = fade.style.display = 'block';
}

function close() {
    message.style.display = fade.style.display = 'none';
}

或者：

function toggle() {
    var message = document.getElementById('message'),
        fade = document.getElementById('fade'),
        currentdisplay = getComputedStyle(message, null)['display'];

    if(currentdisplay == 'block' || currentdisplay == 'inline')
        message.style.display = fade.style.display = 'none';
    else
        message.style.display = fade.style.display = 'block'; /* or inline */
}

或者：

function toggle() {
    var currentdisplay = getComputedStyle(arguments[1], null)['display'],
        i,
        newdisplay;

    if(currentdisplay == 'block' || currentdisplay == 'inline')
        newdisplay = 'none';
    else
        newdisplay = 'block';

    for(i = 0; i < arguments.length; i++)
        arguments[i].style.display = newdisplay;
}

var message = document.getElementById('message'),
    fade = document.getElementById('fade');

toggle(message, fade); /* hide */
toggle(message, fade); /* show */

document.body.onclick = function(){
    toggle(message, fade);
}

切换示例：

http://jsfiddle.net/djHTq/

Answer 2

DRY它。

var b='block',h='none',m='message',f='fade';
function s(i,d){document.getElementById(i).style.display=d}
function open(){s(m,b);s(f,b)}
function close(){s(m,h);s(f,h)}

使用空格和适当的变量名称（传递给minifier），这看起来像：

var show = 'block', hide = 'none', message = 'message', fade = 'fade';

function setStyle(id, display) {
  document.getElementById(id).style.display=display;
}
function open() {
  setStyle(message, show);
  setStyle(fade, show);
}
function close() {
  setStyle(message, hide);
  setStyle(fade, hide);
}

有些最佳做法与问题无关，但如果您的项目超出这个微不足道的情况，则值得考虑：

使用缩小器。我最喜欢的是uglifyjs。这允许您在未分类的代码中使用有意义的变量名称（如第二个示例）。 minifier将输出更像第一个例子的代码（但可能甚至更好）。即使使用minifier，也要考虑它能做什么和不能做什么 - 如果经常使用API，创建一个像document.getElementById这样的长公共API的私有快捷方式可以帮助缩小。查看缩小的代码，确保没有可以做的事情来优化它。
将您的javascript分离为与页面分开加载的.js模块，如果可能的话，将其分开。
管理所有静态资产（例如.js模块），以便它们具有较长的缓存超时 - 使用Expires: http标头。然后在实际更改时更改其URL。这样，客户端可以无限期地缓存它们，直到您更改它们为止。然后客户端将立即获取新版本。
将离散模块放在函数包装器中，这样您的变量就不会与其他代码片段冲突 - 无论是您自己的还是第三方模块。如果您想公开变量，请明确地执行：window.pubvar =

Answer 3

var toggle = function(doc){
    var $ = doc.getElementById, message = $('message'), fade = $('fade'), open = true;
    return function(){
        var display = open ? 'none' : 'block';
        message.style.display = display;
        fade.style.display = display;
        open = !open;
    } 
}(document);

toggle(); // Hide both elements
toggle(); // Show both elements. Rinse and repeat.

Answer 4

避免污染全球范围：

(function() {
    var msgstl = document.getElementById('message').style,
        fdestl = document.getElementById('fade').style;
    window.open = function() {msgstl.display = fdestl.display = "block";};
    window.close = function() {msgstl.display = fdestl.display = "none";};
})();

Answer 5

有一件事是你可以创建一个辅助函数来设置元素的样式。这在您需要设置许多不同元素的情况下非常有用。

function setStyle(element, style, value) {
    document.getElementById(element).style[style] = value;
}
function open() {
    setStyle('message', 'display', 'block');
    setStyle('fade', 'display', 'block');
}
function close() {
    setStyle('message', 'display', 'none');
    setStyle('fade', 'display', 'none');
}

如果您经常使用这些元素，您可能还希望将元素设置为变量。

var message = document.getElementById('message'), fade = ...

使节目/隐藏更小

5 个答案: