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

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

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

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

Q&A

解決済

2回答

3477閲覧

FindメソッドをForで繰り返し行う場合に出るエラーの解消法

Apoll047

総合スコア12

VBA

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

0グッド

0クリップ

投稿2020/06/23 13:22

編集2020/06/24 08:11

たびたびお世話になります。
前回Findメソッドについて質問をさせていただきましたが今回はその拡張版です。
やりたいこと
Excel2016の使用するファイルはExcelファイルを2つの1つにはSheet1とSheetシート2があり、もう一つのファイルはSheet1のみです。
これらを用いて

①Sheet2の1列目に別のワークシートのF列にある番号をコピペします。
②Sheet2の1列目に張り付けられた番号をSheet1の1列目からFindメソッドにて探し出します。
③番号の一致したセルの1列目から8列目までの値を参照させます。
④同様の作業を10行目間まで繰り返します。

以上のことを行いたいです。

問題が出るのはFor内のFindメソッドでオブジェクトエラーが起こってしまいます。
かなり雑なコードですがよろしくお願いします。

ExcelVBA

1Sub Test() 2Call main //標準モジュール内でSheet1をWs1、同様にWs2と定義し、それを呼び出す。 3//①コピぺのコード 4On Error Goto myError 5 Windows("ExCExcelOut.csv").Activate 6 Range("F1:F10").Select 7 Selection.Copy 8 Window("Test.xlsm").Activate 9 Ws2.Range("A1").Select 10 Selection.PasteSpecial Paste:=xlPasteFormulas 11 12myError: 13//もしコピペするものがない場合は途中で止める 14If Ws2.Range("A2").Value = "" Then 15 Exit Sub 16End If 17 18//ここからが問題のFindメソッドとForの合わせ技です。 19Dim myrng As Range 20 Set myrnge = Ws1.Range("A:A") 21Dim i As Long 22For i = 1 to 10 23 //問題の箇所↓ 24 If Not myrng.Find(Ws2.Cells(i,1).Value,LookAt := xlPart) Is Nothing Then 25 Dim j As Long 26 For j = 1 to 8 27 Ws2.Cells(i,j).Value = Ws1.Cells(myrnd.Find(Ws2.Cells(i,j).Value, LookAt 28 := xlPart).Row,j).Value 29 Next j 30 End If 31Next i 32 33End Sub() 34 35

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

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

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

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

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

guest

回答2

0

ベストアンサー

ExcelVBA

1Option Explicit 2 3Sub testMain() 4 Dim rngKeys As Range 5 Dim rngList As Range 6 Dim rngTarget As Range 7 Dim rngResults As Range 8 Dim c As Range 9 Dim ix As Range 10 11 If GetRange(rngKeys, rngList, rngResults) Then 12 For Each c In rngkyes 13 Set rngTarget = GetData(c.Text, rngList) 14 If Not rngTarget Is Nothing Then 15 ix = ix + 1 16 rngResults.Rows(ix).Value = rngTarget.Value 17 End If 18 Next 19 End If 20End Sub 21 22Function GetRange(ByRef rngKeys As Range, _ 23 ByRef rngList As Range, _ 24 ByRef rngResults As Range) 25 On Error GoTo WayOut 26 Set rngKeys = Workbooks("ExCExcelOut.csv").Sheets(1).Range("F1:F10") 27 Set rngList = ThisWorkbook.Sheets(1).UsedRange 28 Set rngResults = ThisWorkbook.Sheets(2).Range("A2:H2") 29 30 GetRange = True 31Exit Function 32 33WayOut: 34End Function 35 36Function GetData(ByVal sKey As String, _ 37 ByRef Rng As Range) As Range 38 Dim c As Range 39 40 With Rng.Columns(1) 41 Set c = .Find(what:=sKey, _ 42 after:=.Cells(.Cells.Count), _ 43 LookIn:=xlValues, _ 44 LookAt:=xlWhole) 45 If c Is Nothing Then Exit Sub 46 End With 47 48 Set GetData = Intersect(Rng, c.EntireRow) 49End Function

こういうことがやりたいんでしょうか?

あと、
同じセルを指定するのにFindメソッドを何回も実行するのは無駄です。
変数をうまく使いましょう。

投稿2020/06/26 00:23

mattuwan

総合スコア2163

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

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

0

とりあえず、

VBA

1 //問題の箇所↓ 2 If Not myrng.Find(Ws2.Cells(i,1).Value,LookAt := xlPart) Is Nothing Then 3 Dim j As Long 4 For j = 1 to 8 5 Ws2.Cells(i,j).Value = Ws1.Cells(myrnd.Find(Ws2.Cells(i,j).Value, LookAt := xlPart).Row,j).Value 6 Next j 7 End If

xlPart (部分一致)を指定するのはLookInではなくLookAtでは? 2つめはLoolInとタイプミスもしているし。

投稿2020/06/23 13:44

編集2020/06/23 15:06
Daregada

総合スコア11990

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

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

Apoll047

2020/06/23 13:47

おっしゃる通りでした。 タイプミスしていました。失礼いたしました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.35%

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

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

質問する

関連した質問