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も見ています
-
あなたの「必」の書き順を教えてください
ふだん、どういう書き順で「必」を書いていますか? みなさんの色んな書き順を知りたいです。 画像のA~Eを使って教えてください。
-
人生最悪の忘れ物
今までの人生での「最悪の忘れ物」を教えてください。 私の「最悪の忘れ物」は「財布」です。
-
【お題】逆襲の桃太郎
【大喜利】桃太郎が1回鬼退治に失敗したところから始まる新作昔話「リベンジオブ桃太郎」にはこんなシーンがある
-
この人頭いいなと思ったエピソード
一緒にいたときに「この人頭いいな」と思ったエピソードを教えてください
-
14歳の自分に衝撃の事実を告げてください
タイムマシンで14歳の自分のところに現れた未来のあなた。 衝撃的な事実を告げて自分に驚かせるとしたら何を告げますか?
-
特定のセルが空白だったら、その行を非表示にしたい。。。
Visual Basic(VBA)
-
EXCEL VBA 指定範囲内で特定の列に空白セルを含む行を非表示
Excel(エクセル)
-
エクセルで特定の列が0表示の場合、その行を非表示にしたい
Excel(エクセル)
-
-
4
特定のセルが空白の行を非表示にするマクロについて
Excel(エクセル)
-
5
エクセルで特定の値を含む列のみVBAで非表示にしたい
Excel(エクセル)
-
6
【VBA】全て空白のセルの列幅を指定
Excel(エクセル)
関連するカテゴリからQ&Aを探す
おすすめ情報
- ・「みんな教えて! 選手権!!」開催のお知らせ
- ・漫画をレンタルでお得に読める!
- ・「これいらなくない?」という慣習、教えてください
- ・今から楽しみな予定はありますか?
- ・AIツールの活用方法を教えて
- ・【選手権お題その3】この画像で一言【大喜利】
- ・【お題】逆襲の桃太郎
- ・自分独自の健康法はある?
- ・最強の防寒、あったか術を教えてください!
- ・【大喜利】【投稿~1/9】 忍者がやってるYouTubeが炎上してしまった理由
- ・歳とったな〜〜と思ったことは?
- ・ちょっと先の未来クイズ第6問
- ・モテ期を経験した方いらっしゃいますか?
- ・好きな人を振り向かせるためにしたこと
- ・【選手権お題その2】この漫画の2コマ目を考えてください
- ・【選手権お題その1】これってもしかして自分だけかもしれないな…と思うあるあるを教えてください
- ・スマホに会話を聞かれているな!?と思ったことありますか?
- ・それもChatGPT!?と驚いた使用方法を教えてください
- ・見学に行くとしたら【天国】と【地獄】どっち?
- ・これまでで一番「情けなかったとき」はいつですか?
- ・この人頭いいなと思ったエピソード
- ・あなたの「必」の書き順を教えてください
- ・14歳の自分に衝撃の事実を告げてください
- ・人生最悪の忘れ物
- ・あなたの習慣について教えてください!!
- ・都道府県穴埋めゲーム
このQ&Aを見た人がよく見るQ&A
デイリーランキングこのカテゴリの人気デイリーQ&Aランキング
-
画面を強制的に再描画させる方法
-
Escキーを押すと、中断する時と...
-
DoEventsが必要な理由について
-
VBAでの一時停止と再開の方法
-
ListBox 複数選択 で オートフ...
-
VBA for i=1 to lastrow
-
イベントの発生を待つ
-
乱数の桁数指定、または範囲指定。
-
Do whileでExitせず、ループの...
-
ループフリー
-
UWSCの終了の仕方
-
null 参照の例外が実行時に発生...
-
pythonでファイルのコメント行...
-
sscanfより速い方法について。
-
レインボー色ってどうやって表...
-
GIFアニメをループさせたくない
-
WHILE (CHKIMG(”A.bmp”)=FALSE)...
-
範囲指定したセルを1つずつ飛...
-
.htaccessがループしてる?それ...
-
エディットボックスのテキスト...
マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング
-
画面を強制的に再描画させる方法
-
UWSCの終了の仕方
-
DoEventsが必要な理由について
-
VBAでの一時停止と再開の方法
-
Escキーを押すと、中断する時と...
-
範囲指定したセルを1つずつ飛...
-
DOSコマンドのループ内のTIMEコ...
-
vb.netからエクセル関数書き込み
-
vb.netです。2次元配列の要素を...
-
VBAで3秒だけ時間を止めたい
-
アクティブセルから、A列最終行...
-
VBA for i=1 to lastrow
-
Do whileでExitせず、ループの...
-
乱数の桁数指定、または範囲指定。
-
CSVファイルの特定の行だけを読...
-
VB2010でCSVファイルの読み込み
-
テキストボックスの名前に変数...
-
UWSCに制限時間を付けたいです
-
イベントの発生を待つ
-
ボタンが押された時にループか...
おすすめ情報
ご回答いただき本当にありがとうございます!
こちらの希望が叶いました。
もし、お願いできたら追加で教えてください。
上記のマクロを実行したところ、ファイルによっては無限ループに陥るものがありました。
無限ループ回避のため、
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素人の希望にお答えいただき、
またとても丁寧な説明をいただき本当に感謝しております。
ご教示頂いたことを今後も活かせるよう、
勉強していきたいと思います。
ありがとうこざいました。