Slice text and add trails with jQuery.

jQueryで文字列を特定の文字数でカットして末尾に「…」などを追加する方法 | BlackFlag

このエントリーのコード、もう少し可読性を上げた方が良いと思ったので、書き直してみた。

jQueryって、

$foo.html(something).css(something).animate(something)... ;

とできるので、処理分けしているところで同じ関数を書いたりしてると、コードが見難くなる。

最終的に同じ後処理をするなら、if文を抜けてから書くべきと思ってる。

// css() が2つある
if(cutFigure < textLength) {
    $(this).html(textTrim + afterTxt).css({visibility:'visible'});
} else if(cutFigure >= textLength) {
    $(this).css({visibility:'visible'});
}

// こうするべき
if(cutFigure < textLength) {
    $(this).html(textTrim + afterTxt);
}
$(this).css({visibility:'visible'});

あと、ダミーテキストはいいのですが、もうちょい３０文字で切れることを確認できるものがよいかと思われます。