質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.46%
VBA

VBAはオブジェクト指向プログラミング言語のひとつで、マクロを作成によりExcelなどのOffice業務を自動化することができます。

Q&A

解決済

3回答

1092閲覧

VBA Withだけが違う同じ処理のまとめ方

D0NKAMA

総合スコア28

VBA

VBAはオブジェクト指向プログラミング言語のひとつで、マクロを作成によりExcelなどのOffice業務を自動化することができます。

0グッド

0クリップ

投稿2021/10/04 07:35

前提・実現したいこと

Withの値は異なりWithの中身は同じプログラムの、省略した書き方をご教授いただきたいです。

該当のソースコード

↓問題のプログラム

VBA

1If selectSeq = 0 Then 2 With ListView.ListItems.Add 3 ' 同じ処理 4 End With 5Else 6 With ListView.ListItems(selectSeq) 7 ' 同じ処理 8 End With 9End If

このプログラムはListViewにデータを反映させる際に、新規データ(If)だと新しい行に、更新データ(Else)だとその行に上書きする処理です。

試したこと

関数にしてまとめようとしたのですが、Withの部分を動的にする方法がわかりませんでした。

もし省略できれば50行ほど縮まるので何卒宜しくお願い致します。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答3

0

ADDする際に、selectSeq の値が決定できれば、以下は同じ処理で出来そうですけど。

VBA

1If selectSeq = 0 Then 2 ListView.ListItems.Add 3 selectSeq = ListView.Items.Count 4End If 5With ListView.ListItems(selectSeq) 6 ' 同じ処理 7End With

投稿2021/10/04 07:44

編集2021/10/04 13:40
sazi

総合スコア25206

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

D0NKAMA

2021/10/04 08:36

確かにそうですね!Elseの判定を省き可読性もあがりますね! 早速プログラムを修正したいと思います。ありがとうございました。
sazi

2021/10/04 13:41

ADDの際に最後の行を選択するようにしてみました。
guest

0

一度変数に入れるというのはどうですか。

VBA

1 Dim myListViewItem 2 If selectSeq = 0 Then 3 Set myListViewItem = ListView.ListItems.Add 4 Else 5 Set myListViewItem = ListView.ListItems(selectSeq) 6 End If 7 8 With myListViewItem 9 ' 同じ処理 10 End With

投稿2021/10/04 08:26

jinoji

総合スコア4585

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

ベストアンサー

関数の引数で渡せばいいのでは。

vba

1 If selectSeq = 0 Then 2 Proc ListView.ListItems.Add 3 Else 4 Proc ListView.ListItems(selectSeq) 5 End If

関数

vba

1Sub Proc(Item As ListItem) 2 With Item 3 ' 同じ処理 4 End With 5End Sub

あるいは、

新規データ(If)だと新しい行に、更新データ(Else)だとその行に上書きする処理です。

これも含めて関数化したほうがいいかも。

関数

vba

1Sub Proc(LV As ListView, selectSeq As Long) 2 Dim Item As ListItem 3 If selectSeq = 0 Then 4 Set Item = LV.ListItems.Add 5 Else 6 Set Item = LV.ListItems(selectSeq) 7 End If 8 With Item 9 ' 同じ処理 10 End With 11End Sub

関数の呼び出しは1行ですみます。

vba

1 Proc ListView, selectSeq

投稿2021/10/04 07:48

編集2021/10/04 08:10
hatena19

総合スコア33795

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

D0NKAMA

2021/10/04 09:29 編集

Withの変数の型をStringやObjectで試しており、ListItemは全くの盲点でした。 プログラムは問題なく動作いたしました! こんな初歩的な問題に迅速回答していただいき、本当にありがとうございました。<(*_ _)>
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.46%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問