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

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

ただいまの
回答率

88.62%

VBA 修飾子不明、うまく実行しない

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,775

OOOOO

score 11

初めまして、観覧ありがとうございます。
VBAについて、2点ほどうまくいかず、質問させていただきました。

1、sheet1で実行した際、sheet2の内容をクリアにすることができない
2、修飾子不明のエラーが出るが、原因が見つからない

勉強不足で多雨異変申し訳ありませんが、自力では解決できずに時間だけがすきていきます。
どうか、お力をお貸ししていただえたら幸いです。
よろしくお願い致します。

Option Explicit

VBA


Dim name As String, FM As String, adress1 As String, adress2 As String, Row As String
Dim data As Long, i As Long

'Sheet2の内容を消しておく ★解決
Sheet2.Range("A2:H100").ClearContents

'検索結果に記入されたデータを格納
name = Range("M2").Value
FM = Range("P2").Value
adress1 = Range("M3").Value
adress2 = Range("M4").Value

'cellの値とそれぞれが一致したら、その配列をコピーペースト
For i = 2 To Rows.Count

If Cells(i, 2).Value = "" Then
GoTo continue1
Else
If Cells(i, 2).Value <> name Then
GoTo continue2
Else

continue1:
Row = Range(Cells(i, 1), Cells(i, 8)).Value
Row.Copy Sheets("Sheet2").Cells(Rows.Count, 1).End(xlUp).Offset(1, 0)  ★Rowの修飾子不明

修飾子が不正です

continue2:
Next i

End If
End If

End Sub

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • m.ts10806

    2018/05/08 13:30

    プログラムコード(およびエラーメッセージ)は```で囲ってください。(わからなければ質問編集画面でコード部分を選択し<code>ボタンを押してください)正しく反映されているかどうかは質問編集画面のプレビューを見ながら編集していってください。

    キャンセル

  • m.ts10806

    2018/05/08 13:32

    「多雨異変」とは?おそらく「大変」の打ち間違えかと思いますが、少し落ち着いて文章を見直して投稿してください。

    キャンセル

回答 3

checkベストアンサー

0

Sheet2が正しく参照できていないのでは。

Worksheets("Sheet2").Range("A2:H100").ClearContents


2番目は.CopyはRangeオブジェクトに対するメソッドですので、単なるStringとしてしか定義されていないRowにはそんなメソッドはありませんというエラーです。
コードの記述としての是非はありますが、エラーとならない意図するコードとしては、以下のような記述です。

Range(Cells(i, 1), Cells(i, 8)).Copy Sheets("Sheet2").Cells(Rows.Count,1).End(xlUp).Offset(1, 0)


全体的に作りかけのコードですね。色々とありそうですが頑張ってください。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/05/08 13:22

    迅速な回答ありがとうございます!
    1つ目の問題は無事解決できました。とても助かりました。

    キャンセル

0

Sheet2ってなに?

Dim Sheet2 As Worksheet


とか定義があってのSheet2なのであれば、

Set Sheet2 = ActiveWorkbook.Worksheets("シートの名前")


とかすれば良さそう。

もしくは、Activebook.Sheets("Sheet2")ってことなら、

Dim Sheet2 As Worksheet
Set Sheet2 = ActiveWorkbook.Worksheets("Sheet2")

もう一点、
Dim (略), Row As Stringなのに、
Row.Copyってのはそりゃエラーになる。
何がしたいのか不明。
Copyメソッドを使うにはRangeオブジェクト。
コピー元Range().Copy Destination:=コピー先Range()
みたいな。
それとも、セル内の値だけコピーしたいなら、
セルのcolumn用のカウンター変数を用意して
Range().ValueもしくはCells().Valueでアクセスすれば良さげ。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/05/08 13:29

    sheet2についての動作は、
    'Sheet2の内容を消しておく ★うまく機能しない
    の1行のみで使いました。どうやら問題はsaziさんの回答によって問題は解決できたようです。

    Row = Range(Cells(i, 1), Cells(i, 8)).Value
    はsheet1のある行iの1列目から8列目までの値をとりたいと思い書きました。
    配列の指定がわからないためこのようにしています。

    キャンセル

0

「Row = Range(Cells(i, 1), Cells(i, 8)).Value」で、
Row は String型 の変数ですよね?
Row には何の値が入るんでしょうか?
(個人的には、値が入ることの方が不思議に思います)

で、Row.Copy って何がしたいんでしょうか?
残念ながらVBAでは、String型の変数に「Copy」等、
プロパティもメソッドも用意されてませんので。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

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

  • ただいまの回答率 88.62%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る