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

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

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

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

Q&A

解決済

2回答

14864閲覧

VBAでアクティブじゃないシートから値を取得する方法

tarotarosu

総合スコア114

VBA

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

0グッド

0クリップ

投稿2017/08/30 10:25

###前提・実現したいこと
VBAでアクティブじゃないシートから値を取得するのに苦戦しています。

今、「sheet1」と「sheet2」という二つのシートがあり、「sheet1」をアクティブにした状態で「sheet2」のセルの値を取得し配列に格納するというようなことをしようと考えています。
ちなみに「sheet2」には下記のような情報がA1から入っています。

name
taro
hanako

これらのセルの内容を、下記関数を使用して取得します。

###セルの値を配列として返す関数

vba

1Function getNameList() 2 Dim namesSheet As Worksheet 3 Set namesSheet = Worksheets("names") 4 5 Dim names() As String: ReDim names(0) ' Index 0で初期化 6 7 Dim i 8 For i = 2 To namesSheet.Cells(Rows.count, 1).End(xlUp).Row 9 names(UBound(names)) = Cells(i, 1) 10 ReDim Preserve names(UBound(names) + 1) ' 拡張 11 Next 12 13 ReDim Preserve names(UBound(names) - 1) 14 15 getNamesList = names 16End Function

「sheet1」をアクティブにした状態で下記のように実行すると、値が取得できず何も表示されません。

配列の中身を取得して表示

vba

1Sub main() 2 ' 名前一覧を取得 3 Dim names As Variant 4 Dim name As Variant 5 names = getNamesList.getNamesList 6 7 ' 名前の表示 8 For Each name In names 9 Debug.Print (name) 10 Next name 11End Sub

「sheet1」をアクティブにしたまま「sheet2」の値を配列として取得するにはどのような方法があるのでしょうか?
ご回答を頂けると非常に助かります_(..)

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

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

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

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

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

guest

回答2

0

ベストアンサー

私の場合、namesSheet.Cells(i.1)ではなく、nameSheet.Cells(i.1).valueにした方が、セルの値を明示的に指定するのでこの方が良いと思います。

ちなみに、
Dim i
For i = 2 To namesSheet.Cells(Rows.count, 1).End(xlUp).Row
names(UBound(names)) = Cells(i, 1)
ReDim Preserve names(UBound(names) + 1) ' 拡張
Next
とありますが、

Dim names As Variant()
names = namesSheet.Range("A2:A" & namesSheet.Cells(Rows.count, 1).End(xlUp).Row)

だけで2次元の動的配列に一発で格納することができます。
これだと、ループもReDimも不要になる上、高速の処理が可能です。

詳しくは、高速化テクニック〜配列とセルの相互転記
をご覧ください。

投稿2017/08/30 14:10

diracpaul

総合スコア157

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

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

tarotarosu

2017/08/31 01:15

ありがとうございます_(._.)_ ご指摘いただいたように修正したところうまくいきました! 元々の書き方では冗長だと感じていたので助かりました! ありがとうございました_(._.)_
guest

0

FunctionのFor~Next内

names(UBound(names)) = Cells(i, 1)

これを

names(UBound(names)) = namesSheet.Cells(i, 1)

に変更してみたらどうでしょう?
元のコードではブックやシートの指定を省略しているのでアクティブシート(sheet1)を参照しているはずです。

投稿2017/08/30 11:12

moh1ee

総合スコア73

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

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

tarotarosu

2017/08/31 01:15

ありがとうございます_(._.)_ ご指摘いただいたように修正したところうまくいきました! ありがとうございました_(._.)_
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問