
No.1ベストアンサー
- 回答日時:
こんにちは
こんな感じでしょうか?
アクティブなシートを対象にしてあります。
Sub Sample()
Dim ur As Range, tr As Range
Dim rwC As Long, c As Long, flg As Boolean
Set ur = ActiveSheet.UsedRange
Set tr = Intersect(ur, Rows(5).Resize(Rows.Count - 4))
If tr Is Nothing Then Set tr = Rows(5)
rwC = tr.Rows.Count
For c = ur.Column To ur.Column + ur.Columns.Count - 1
flg = WorksheetFunction.CountBlank(Intersect(Columns(c), tr)) = rwC
Columns(c).Hidden = flg And Cells(1, c).Value = ""
Next c
End Sub
No.2
- 回答日時:
No1です
>ファイルによっては無限ループに陥るものがありました。
ループは一か所のみですが、繰り返しの回数はUsedRangeの列数にしていますので「無限ループ」が発生することはないはずと思いますが・・・??
どこか右側の方のセルに何らかの値(空白文字列とか)が間違って入っていて、繰り返し数が大きくなっている可能性は無いとは言えませんが、最大でも16384回(=最大列数)のループかと。
(数がそれなりなので、多少の時間が掛かる可能性はありますが、大したことはないと想像します)
手順として左側から一列ずつ順に処理をしていますので、上記のような場合には、範囲外と思わっている列も順に非表示になっているはずですので、そのあたりを見てみることで実際に起きていることの確認が可能と思います。
>If c = 70 Then ←最大でも70列目までとしました。
>Exit For
>End If
>をNext cの後ろに組み込みましたが、
>合っていますでしょうか。
Nextの後ろでは意味がありません。(…というより間違いになります)
ループ内(For~Nextの間)で判断しないとループを抜けませんし、仮にその条件がTRUEになったとして、Exit Forを実行しようとしても対象のループが存在しない(すでにループ外なので)ため、エラーが発生するのではないかと思います。(←未確認)
また、補足の趣旨から言えば、条件文は
If c >= 70 Then
等としておく方が確実でしょう。(現状のままでも問題はありませんが)
ループ内に記述することで目的は達成できますが、ループの回数分その条件判断を繰り返して処理することになりますので、若干効率が悪いと言えます。(大した影響はないでしょうけれど)
今回の場合であれば、ループ内で判断しなくてもループの前に判断が可能ですので、ループ処理を行う前に、ループの制御値を「70との最小値」に設定するようにした方が効率的と考えられます。
お探しのQ&Aが見つからない時は、教えて!gooで質問しましょう!
このQ&Aを見た人はこんなQ&Aも見ています
-
特定のセルが空白だったら、その行を非表示にしたい。。。
Visual Basic(VBA)
-
EXCELマクロで、ある列の値が0の場合、非表示/表示ボタンで該当列を非表示切り替えしたいんです。
Excel(エクセル)
-
Excel VBA 空白セルがあったらメッセージボックスで知らせたい
Visual Basic(VBA)
-
-
4
EXCEL VBA 指定範囲内で特定の列に空白セルを含む行を非表示
Excel(エクセル)
-
5
Enterキーでマクロを起動さす。
その他(ソフトウェア)
-
6
エクセルVBAでセルに入力したパスでブックを開く
Excel(エクセル)
-
7
エクセルで特定の列が0表示の場合、その行を非表示にしたい
Excel(エクセル)
-
8
【VBA】複数のシートの指定した列を非表示・表示をさせたい
Excel(エクセル)
関連するカテゴリからQ&Aを探す
おすすめ情報
このQ&Aを見た人がよく見るQ&A
デイリーランキングこのカテゴリの人気デイリーQ&Aランキング
-
UWSCの終了の仕方
-
VBAでの一時停止と再開の方法
-
Do whileでExitせず、ループの...
-
テキストボックスの名前に変数...
-
VB.NETでFTPプログラム
-
xmplayの使い方
-
C言語でファクト関数を使わずに...
-
csh foreachで「*」でエラ...
-
ListBox 複数選択 で オートフ...
-
ループフリー
-
UWSCに制限時間を付けたいです
-
プログラミングで難しいと感じ...
-
EXCEL VBA(初心者)印刷ルー...
-
CSVファイルの特定の行だけを読...
-
エクセルの当番表を作っていま...
-
vb.netからエクセル関数書き込み
-
VBAで3秒だけ時間を止めたい
-
Excel VBAでIf 条件式 then か...
-
流れ図(フローチャート)が分か...
-
Dim flag(4) as boolean で配列...
マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング
-
画面を強制的に再描画させる方法
-
UWSCの終了の仕方
-
DOSコマンドのループ内のTIMEコ...
-
範囲指定したセルを1つずつ飛...
-
VBAでの一時停止と再開の方法
-
Escキーを押すと、中断する時と...
-
乱数の桁数指定、または範囲指定。
-
イベントの発生を待つ
-
流れ図(フローチャート)が分か...
-
UWSCに制限時間を付けたいです
-
vb.netです。2次元配列の要素を...
-
VBA for i=1 to lastrow
-
DoEventsが必要な理由について
-
ループフリー
-
VBA Boxが空白の場合のメッセー...
-
VBA ニュースタイトル取得
-
エクセルの当番表を作っていま...
-
Javaの質問です。 この問題の答...
-
テキストボックスの名前に変数...
-
CSVファイルの特定の行だけを読...
おすすめ情報
ご回答いただき本当にありがとうございます!
こちらの希望が叶いました。
もし、お願いできたら追加で教えてください。
上記のマクロを実行したところ、ファイルによっては無限ループに陥るものがありました。
無限ループ回避のため、
If c = 70 Then ←最大でも70列目までとしました。
Exit For
End If
をNext cの後ろに組み込みましたが、
合っていますでしょうか。
ループ回避はしたようですが、
作業時間が長くなったような気がします。
追加のご回答ありがとうございます。
以下について確認しました。
>If c = 70 Then ←最大でも70列目までとしました。
>Exit For
>End If
>をNext cの後ろに組み込みましたが、
>合っていますでしょうか。
Next c の前に入れていました…
投稿前によく確認せず申し訳ありません…
また、
If c >= 70 Then
へのアドバイスもありがとうございます。
この度は VBA素人の希望にお答えいただき、
またとても丁寧な説明をいただき本当に感謝しております。
ご教示頂いたことを今後も活かせるよう、
勉強していきたいと思います。
ありがとうこざいました。