コンストラクタにコストがかかる場合は、つい面倒で

{
　　Foo foo;

　　for(...)
　　{
　　　　foo = …
　　}
}

ってやってしまう。
関数に切り分けるのが一番スマートなのは分かってるつもりなんだけども。