Jshint.comは、以下のコードに対してこれを提案しています。
Line 150: }, false );
Don't make functions within a loop.
ただし、document.getElementById()を複数回記述する必要はなく、代わりにIDを配列に保存してループすることができます。
より簡潔で保守しやすいコードだと思います。
function styleTwitter( pair_array )
{
var i, input, label;
for ( i = 0; i < pair_array.length; i+=2 )
{
input = document.getElementById( pair_array[ i ] );
label = document.getElementById( pair_array[ i + 1 ] );
input.addEventListener( "focus", function()
{
if( input.value === '' )
{
label.style.opacity = 0;
input.style.border = '1px solid #888888';
}
}, false );
input.addEventListener( "blur", function()
{
if( input.value === '' )
{
label.style.opacity = 1;
new EffectsFont( label ).fade( 'up', 150 );
input.style.border = '1px solid #dddddd';
}
}, false );
}