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

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

ただいまの
回答率

90.85%

  • VBA

    1467questions

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

  • Excel

    1268questions

    Excelは、マイクロソフト社が開発しているデータ集計や分析を行う表計算ソフトの一つです。文書作成や表計算、資料作成などの多彩な機能を備えており、統合パッケージであるMicrosoft Officeに含まれています。

Excel VBAのsubまたfunctionが定義されませんのエラー

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 119

rura

score 52

 前提・実現したいこと

こんちわ。Excel VBA 初心者です。
会員登録画面を作成しています。
下記エラーバグの削除方法がよくわかりません。
どなたかお願いします。
■■な機能を実装中に以下のエラーメッセージが発生しました。

 発生している問題・エラーメッセージ

引数の判定するコードで
コンパイルエラー
subまたfunctionが定義されませんのエラー

 該当のソースコード

Option Explicit
Public Sub LS_BookReturn_Click()

    Dim TextCamnameS As String
    Dim Textdescrip As String
    Dim TextCskiDay As String
    Dim Textemployees As String
    Dim TextFounding As String '
    Dim Textoname As String
    Dim Textzipcode As String
    Dim ComBprefectures As String
    Dim Texminicic As String
    Dim Textaddress As String '
    Dim Textworkplice As String

    TextCamnameS = clientForm.TextCamnameS.Text
    Textdescrip = clientForm.Textdescrip.Text
    TextCskiDay = clientForm.TextCski.Text
    Textemployees = clientForm.Textemployees.Text
    TextFounding = clientForm.TextFounding.Text
    Textoname = clientForm.Textoname.Text
    Textzipcode = clientForm.Textzipcode.Text
    'ComBprefectures = clientForm.ComBprefectures.Text
    Texminicic = clientForm.Texminicic.Text
    Textaddress = clientForm.Textaddress.Text
    Textworkplice = clientForm.Textworkplice.Text


    If LF_user(TextCamnameS, Textdescrip, TextCskiDay, Textemployees, Textoname, Textzipcode, Texminicic, Textaddress, Textworkplice) = 0 Then

        '会員No、氏名が未入力でなければ、情報を登録しましたの表示'
        MsgBox "「返却情報を登録しました」"
    End If

End Sub


Public Function LF_BookReturnRegistration(ByVal Textdescrip As String, ByVal TextCamnameS As String, ByVal Textemployees As String, _
ByVal TextCskiDay As String, ByVal Textemployees As String, ByVal TextFounding As String, ByVal Textoname As String, _
ByVal Textzipcode As String, ByVal Texminicic As String, ByVal Textaddress As String, _
ByVal Textworkplice As String) As Integer



    If TextCamnameS = "" Or Textdescrip = "" Or Textemployees = "" Then
        MsgBox "「入力がエラーです。」", vbOKOnly + vbExclamation, "エラーメッセージ"

         LF_user = -1

        Exit Function

    End If



            'もし、該当物ならば罫線と中央配置を設定
            With Sheet2.Cells(D4)
                 .Borders(xlEdgeTop).LineStyle = xlContinuous
                 .Borders(xlEdgeBottom).LineStyle = xlContinuous
                 .Borders(xlEdgeLeft).LineStyle = xlContinuous
                 .Borders(xlEdgeRight).LineStyle = xlContinuous
                 .HorizontalAlignment = xlCenter
                 '(返却日)4列目にTextemployeesを入れる
                 Sheet3.Cells(BNumC, 4).Value = Textemployees
            End With


    LF_user = 0

End Function

 補足情報(FW/ツールのバージョンなど)

Windows10 、Excel 2010

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • m6u

    2018/04/17 20:19

    重複質問:https://teratail.com/questions/122296

    キャンセル

  • rura

    2018/04/17 20:24

    大変、申し訳ございません。誤って、2回投稿しました。こちらで質問を受け付けします。

    キャンセル

  • Zuishin

    2018/04/17 20:52

    もう一つの方に回答がついてるのでそちらで受け付けした方がいいと思います。

    キャンセル

回答 2

checkベストアンサー

0

VBE画面のツールバーの「デバッグ(D)」→ 「コンパイル」で確認しました?

関数名が「LF_BookReturnRegistration」なのに、
「LF_user = -1」だの、「LF_user = 0」だの、
宣言してる関数名と、返そうとしてる関数名が違う。。。

まず、IsUser とか他の関数名にした方がいいでしょうね。

で、判定なら、Boolean型にすべきでは?
複数の判定になるなら、Enum を使用すべきでしょうね。

VBAは基本 Byref なので、わざわざ全部 ByVal にするのは苦労するでしょうね。

テキストボックスから値を取得するからって、
変数名の頭に Text ばっかり付けて、
余計に見にくいし、分かりにくいし、意味あるの?
※「Texminicic」は「Textminicic」で[t]が抜けてる。
VBA初心者というか、プログラマ初心者だと思われますよ~

これだけ変数を使用して、かつ、ひとまとまりの情報群なのであれば、
Type を使った方がコーディングしやすいと思うな~

まぁ、「なんで VBA なんか。。。」なんて嫌々やってると思うけど、
まぁ、がんばって~~~

推薦本:ExcelVBAを実務で使い倒す技術
※回しものではないですが、技術者目線で読める良書かと

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

LF_user という関数を作ったのに、関数名をLF_BookReturnRegistration に変更してしまい、ついでに引数の順序も変えてしまった。
ということでは...

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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

関連した質問

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

  • VBA

    1467questions

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

  • Excel

    1268questions

    Excelは、マイクロソフト社が開発しているデータ集計や分析を行う表計算ソフトの一つです。文書作成や表計算、資料作成などの多彩な機能を備えており、統合パッケージであるMicrosoft Officeに含まれています。