
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で質問しましょう!
似たような質問が見つかりました
- Visual Basic(VBA) 【Excel VBA】条件に合った行の表示・非表示を行う方法 3 2023/03/18 12:31
- Visual Basic(VBA) VBAで列の再表示設定 1 2023/04/25 10:19
- Visual Basic(VBA) 【VBA】データを入力後に,同一シート内に履歴として転記するVBAコードを教えていただきたいです。 3 2022/11/16 01:37
- Excel(エクセル) 【マクロ】マクロが保存されているエクセルとは、別のエクセルブックの全シートの非表示列を再表示したい 1 2022/12/24 20:48
- Excel(エクセル) エクセルでセルの日付を和暦表示設定にしたらおかしなことに? 3 2022/05/25 11:47
- Visual Basic(VBA) 3つのプロシージャをまとめたら実行時エラー発生で対応不能 6 2022/05/17 01:47
- Visual Basic(VBA) ファイル全てを .xlsm に変更したところ、プログラムが途中で落ちてしまっています 17 2022/12/07 12:03
- Visual Basic(VBA) エクセルVBAについて 2 2023/01/31 16:21
- Excel(エクセル) Excelでnullになるような式のセルをマクロで空白行と認識させるにはどうすればいいですか? 3 2023/03/13 13:42
- Excel(エクセル) フォルダ内のエクセルファイルを開かずにデータ採取する関数式 2 2022/12/22 22:15
このQ&Aを見た人はこんなQ&Aも見ています
-
特定のセルが空白だったら、その行を非表示にしたい。。。
Visual Basic(VBA)
-
EXCEL VBA 指定範囲内で特定の列に空白セルを含む行を非表示
Excel(エクセル)
-
EXCELマクロで、ある列の値が0の場合、非表示/表示ボタンで該当列を非表示切り替えしたいんです。
Excel(エクセル)
-
-
4
Excel VBA 空白セルがあったらメッセージボックスで知らせたい
Visual Basic(VBA)
-
5
Enterキーでマクロを起動さす。
その他(ソフトウェア)
-
6
エクセルVBAでセルに入力したパスでブックを開く
Excel(エクセル)
-
7
エクセルで特定の列が0表示の場合、その行を非表示にしたい
Excel(エクセル)
-
8
エクセルで特定の値を含む列のみVBAで非表示にしたい
Excel(エクセル)
関連するカテゴリからQ&Aを探す
おすすめ情報
このQ&Aを見た人がよく見るQ&A
デイリーランキングこのカテゴリの人気デイリーQ&Aランキング
-
UWSCの終了の仕方
-
画面を強制的に再描画させる方法
-
VBAでの一時停止と再開の方法
-
範囲指定したセルを1つずつ飛...
-
Escキーを押すと、中断する時と...
-
スプレッドシート GASの繰り返...
-
なぜⅰなのか?
-
excel2000のVBAでループ内で...
-
VBAでln関数の計算
-
GIFアニメをループさせたくない
-
乱数の桁数指定、または範囲指定。
-
VBA for i=1 to lastrow
-
別窓を開くと親窓のGIFアニ...
-
csh foreachで「*」でエラ...
-
Javaでゲーム
-
ListBox 複数選択 で オートフ...
-
CSVファイルの特定の行だけを読...
-
vb.netです。2次元配列の要素を...
-
for (i = N - 1; i >= 0; i--) ...
-
ループ利尿薬について
マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング
-
UWSCの終了の仕方
-
画面を強制的に再描画させる方法
-
Escキーを押すと、中断する時と...
-
範囲指定したセルを1つずつ飛...
-
vb.netです。2次元配列の要素を...
-
VBAで3秒だけ時間を止めたい
-
エクセルの当番表を作っていま...
-
UWSCに制限時間を付けたいです
-
DOSコマンドのループ内のTIMEコ...
-
VBAでの一時停止と再開の方法
-
CSVファイルの特定の行だけを読...
-
DoEventsが必要な理由について
-
GIFアニメをループさせたくない
-
VBA for i=1 to lastrow
-
Do whileでExitせず、ループの...
-
VBA Boxが空白の場合のメッセー...
-
vb.netからエクセル関数書き込み
-
イベントの発生を待つ
-
乱数の桁数指定、または範囲指定。
-
エクセル関数で1〜12の数字がル...
おすすめ情報
ご回答いただき本当にありがとうございます!
こちらの希望が叶いました。
もし、お願いできたら追加で教えてください。
上記のマクロを実行したところ、ファイルによっては無限ループに陥るものがありました。
無限ループ回避のため、
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素人の希望にお答えいただき、
またとても丁寧な説明をいただき本当に感謝しております。
ご教示頂いたことを今後も活かせるよう、
勉強していきたいと思います。
ありがとうこざいました。