// 糟糕
h = $('#element').height();
$('#element').css('height',h-20);
// 建议
$element = $('#element');
h = $element.height();
$element.css('height',h-20);

// 糟糕
$element = $('#element');
h = $element.height();
$element.css('height',h-20);
// 建议
var $element = $('#element');
var h = $element.height();
$element.css('height',h-20);

// 糟糕
var first = $('#first');
var second = $('#second');
var value = $first.val();
// 建议 - 在jQuery对象前加$前缀
var $first = $('#first');
var $second = $('#second'),
var value = $first.val();

var
 $first = $('#first'),
 $second = $('#second'),
 value = $first.val(),
 k = 3,
 cookiestring = 'SOMECOOKIESPLEASE',
 i,
 j,
 myArray = {};

// 糟糕
$first.click(function(){
 $first.css('border','1px solid red');
 $first.css('color','blue');
});
$first.hover(function(){
 $first.css('border','1px solid red');
})
// 建议
$first.on('click',function(){
 $first.css('border','1px solid red');
 $first.css('color','blue');
})
$first.on('hover',function(){
 $first.css('border','1px solid red');
})

// 糟糕
$first.click(function(){
 $first.css('border','1px solid red');
 $first.css('color','blue');
});
// 建议
$first.on('click',function(){
 $first.css({
 'border':'1px solid red',
 'color':'blue'
 });
});

// 糟糕
$second.html(value);
$second.on('click',function(){
 alert('hello everybody');
});
$second.fadeIn('slow');
$second.animate({height:'120px'},500);
// 建议
$second.html(value);
$second.on('click',function(){
 alert('hello everybody');
}).fadeIn('slow').animate({height:'120px'},500);

// 糟糕
$second.html(value);
$second.on('click',function(){
 alert('hello everybody');
}).fadeIn('slow').animate({height:'120px'},500);
// 建议
$second.html(value);
$second
 .on('click',function(){ alert('hello everybody');})
 .fadeIn('slow')
 .animate({height:'120px'},500);

// 糟糕
function initVar($myVar) {
 if(!$myVar) {
 $myVar = $('#selector');
 }
}
// 建议
function initVar($myVar) {
 $myVar = $myVar || $('#selector');
}

// 糟糕
if(collection.length > 0){..}
// 建议
if(collection.length){..}

// 糟糕
var
 $container = $("#container"),
 $containerLi = $("#container li"),
 $element = null;
$element = $containerLi.first();
//... 许多复杂的操作
// better
var
 $container = $("#container"),
 $containerLi = $container.find("li"),
 $element = null;
$element = $containerLi.first().detach();
//... 许多复杂的操作
$container.append($element);

// 糟糕
$('#id').data(key,value);
// 建议 (高效)
$.data('#id',key,value);

// 糟糕
var
 $container = $('#container'),
 $containerLi = $('#container li'),
 $containerLiSpan = $('#container li span');
// 建议 (高效)
var
 $container = $('#container '),
 $containerLi = $container.find('li'),
 $containerLiSpan= $containerLi.find('span');

// 糟糕
$('.container > *');
// 建议
$('.container').children();

// 糟糕
$('.someclass :radio');
// 建议
$('.someclass input:radio');

// 糟糕
$('div#myid');
$('div#footer a.myLink');
// 建议
$('#myid');
$('#footer .myLink');

// 糟糕
$('#outer #inner');
// 建议
$('#inner');

// 糟糕 - live 已经废弃
$('#stuff').live('click', function() {
 console.log('hooray');
});
// 建议
$('#stuff').on('click', function() {
 console.log('hooray');
});
// 注：此处可能不当，应为live能实现实时绑定，delegate或许更合适