|
▼北風 さん:
とくにコード自体で大きな問題があるとは思えません。
まぁ、処理に必要な時間なんじゃないですか。
最初の Columns("A:B").Select は意味がないから不要とか
With ActiveSheet.PageSetup と End With のブロックが2回あるので
まとめて一度の With で済むとか、
With ActiveSheet.PageSetup としてセットしている項目が多いのに、
ぽつんと、ActiveSheet.PageSetup.PrintArea = "" と単独でやっているとか
Columns("F:F").ColumnWidth = 9.5 といったコードを書いているのに
Columns("C:E").Select
Selection.ColumnWidth = 5.5
こんなように、Select/Selectionベースで無駄な2行にしているところがあるとか
そういうところはありますが、微々たるものですから。
もちろん、PageSetup 領域には様々な設定項目があるわけで、
マクロ記録をすると、規定値も含めてすべての項目の設定コードが生成されますが
規定値については、設定コードをなくし、とくに指定したい項目のみに絞るというのは
やってみられたらいいと思いますが。
ところで、最後の
If fs = False Then End
もちろん、これでもいいのですが、 End ステートメントはできるだけ避けたほうがいいですよ。
ふつうに
If fs = False Then Exit Sub
にしておかれたらいかがですか?
|
|