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

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

ただいまの
回答率

87.49%

半角英数字のチェック方法がうまくいきません

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 1
  • VIEW 4,477

score 50

先日質問がうまくできていなかったので再度投稿します。
何度も申し訳ございません。
やりたいことはExcelからデータを読み取り、SQLへ流し込むためのCSVを作成するにあたり
いったんExcelに読込エラーチェックをします。
先日も書きましたが、半角英数字のチェックは職員コードになります。
その際に半角英数字以外の文字が入っていたら、セルに色を付けるという風にしたいです。
どうも、半角英数字のチェックがうまくいってないようです。
郵便番号のチェックはできてます。もできてませんでした。
データが文字列だからなのかと思っていたのですが、そうではないようでした。
意図的に文字列から数値に変換してもすべてセルの色が赤になってしまいました。

ご教示よろしくお願いします。

Private Sub ErrCheck()

    Dim i As Long
    Dim r As Long
    Dim c As Long
    Dim Err As Boolean
    Dim reg As New RegExp
    Dim reg2 As New RegExp

    r = Me.Cells(Me.Rows.Count, 3).End(xlUp).Row
    Err = False

'    reg.Pattern = "^[a-zA-Z0-9!-/:-@\[-`{-~]+$" '半角英数字のチェック用
    reg.Pattern = "^[0-9A-Za-z]+$"
    reg.Global = True
    reg.Pattern = "^[0-9]{3}-[0-9]{4}$" '郵便番号チェック
    reg2.Global = True

     Select Case Cells(8, 3).Value
        Case "職員マスタ"


            For i = 14 To r
                '必須項目チェック
                For c = 3 To 5
                    If Me.Cells(i, c) = "" Then
                        Me.Cells(i, c).Interior.Color = RGB(255, 0, 0)
                        Err = True
                    End If
                Next

                If Len(Me.Cells(i, 4).Value) > 16 Then
                    Me.Cells(i, 4).Interior.Color = RGB(255, 0, 0)
                ElseIf Len(Me.Cells(i, 5).Value) > 16 Then
                    Me.Cells(i, 5).Value.Interior.Color = RGB(255, 0, 0)
                ElseIf Len(Me.Cells(i, 9).Value) > 16 Then
                    Me.Cells(i, 9).Value.Interior.Color = RGB(255, 0, 0)
                ElseIf Len(Me.Cells(i, 10).Value) > 16 Then
                    Me.Cells(i, 10).Value.Interior.Color = RGB(255, 0, 0)
                ElseIf Len(Me.Cells(i, 11).Value) > 16 Then
                    Me.Cells(i, 11).Value.Interior.Color = RGB(255, 0, 0)
                End If

                '重複チェック
                If WorksheetFunction.CountIf(Range(Cells(14, 3), Cells(r, 3)), Cells(i, 3)) > 1 Then
                    Me.Cells(i, 3).Interior.Color = RGB(255, 0, 0)
                    Err = True
                End If

                '半角英数字チェック
                If reg.test(Cells(i, 3).Value) = False Then
                    Cells(i, 3).Interior.Color = RGB(255, 0, 0)
                    Err = True
                ElseIf Len(Cells(i, 3)) > 10 Then
                    Cells(i, 3).Interior.Color = RGB(255, 0, 0)
                    Err = True
                 End If
                 '郵便番号チェック
                 If Cells(i, 8) <> "" Then
                    If reg2.test(Cells(i, 8).Value) = False Then
                        Cells(i, 8).Interior.Color = RGB(255, 0, 0)
                        Err = True
                    End If
                   End If
            Next


        Case "受講ファイル(本部一括)"

            For i = 14 To r
                For c = 3 To 5
                    If Me.Cells(i, c) = "" Then
                        Me.Cells(i, c).Interior.Color = RGB(255, 0, 0)
                        Err = True
                    End If
                Next
                If Me.Cells(i, 3) = 16 And Me.Cells(i, 5) = 1 Then
                    Me.Cells(i, 5).Interior.Color = RGB(255, 0, 0)
                    Err = True
                ElseIf Me.Cells(i, 3) = 27 And Me.Cells(i, 5) = 2 Then
                    Me.Cells(i, 5).Interior.Color = RGB(255, 0, 0)
                    Err = True
                ElseIf Me.Cells(i, 3) = 28 And Me.Cells(i, 5) = 2 Then
                    Me.Cells(i, 5).Interior.Color = RGB(255, 0, 0)
                    Err = True
                End If

            Next
        Case "受講ファイル(個別一括)"
            For i = 14 To r
                For c = 3 To 7
                    If Me.Cells(i, c) = "" Then
                        Me.Cells(i, c).Interior.Color = RGB(255, 0, 0)
                        Err = True
                    End If
                Next

                If Me.Cells(i, 5) = 16 And Me.Cells(i, 6) <> 2 Then
                    Me.Cells(i, 3).Interior.Color = RGB(255, 0, 0)
                    Err = True
                ElseIf Me.Cells(i, 3) = 27 And Me.Cells(i, 4) <> 2 Then
                    Me.Cells(i, 3).Interior.Color = RGB(255, 0, 0)
                    Err = True
                End If

                '送付先が1でも2でもない場合
                If Me.Cells(i, 7).Value <> 1 And Me.Cells(i, 7).Value <> 2 Then
                   Me.Cells(i, 7).Interior.Color = RGB(255, 0, 0)
                End If


                '半角英数字チェック
                If reg.test(Cells(i, 3).Value) = False Then
                    Cells(i, 3).Interior.Color = RGB(255, 0, 0)
                    Err = True
                ElseIf Len(Cells(i, 3)) > 8 Then
                    Cells(i, 3).Interior.Color = RGB(255, 0, 0)
                    Err = True
                ElseIf reg.test(Cells(i, 4).Value) = False Then
                    Cells(i, 3).Interior.Color = RGB(255, 0, 0)
                    Err = True
                ElseIf Len(Cells(i, 3)) > 10 Then
                    Cells(i, 3).Interior.Color = RGB(255, 0, 0)
                    Err = True
                 End If

            Next
        Case "受講ファイル(金融法務講座)"
            For i = 14 To r
                If Me.Cells(i, 3) = "" Then
                    Me.Cells(i, 3).Interior.Color = RGB(255, 0, 0)
                    Err = True
                End If

                '重複チェック
                If WorksheetFunction.CountIf(Range(Cells(14, 3), Cells(r, 3)), Cells(i, 3)) > 1 Then
                    Me.Cells(i, 3).Interior.Color = RGB(255, 0, 0)
                    Err = True
                End If
                '半角英数字チェック
                If reg.test(Cells(i, 3).Value) = False Then
                    Cells(i, 3).Interior.Color = RGB(255, 0, 0)
                    Err = True
                ElseIf Len(Cells(i, 3)) > 10 Then
                    Cells(i, 3).Interior.Color = RGB(255, 0, 0)
                    Err = True
                 End If
            Next
    End Select

    If Err = True Then
        MsgBox "エラーがあります。" & vbCrLf & "必須項目または赤いセルを確認して下さい"
    End If

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • Zuishin

    2019/09/13 19:15

    こっちの質問もうまくできていないようですが。

    キャンセル

  • Naoko_Coco

    2019/09/18 14:51

    日本語が下手なんですね。。。すいません。

    キャンセル

回答 3

checkベストアンサー

0

いくつか気になる点。

Dim Err As Boolean
Errはそういうオブジェクトがあるので変数名としては適切ではない。別の名前に変更しましょう。
Err オブジェクト | Microsoft Docs


Me.Cells(i, 4).Value と Cells(i, 3).Value というように Me. が付くのと付いてないのが混在しているが、どちらかに統一しましょう。
ただし、省略するのは可読性が悪くなるので、
シートモジュールに記述していてそのシートが対象ならMe.、標準モジュールに記述してアクティブなシートが対象ならActiveSheet.を付けるようにして、対象のシートを明示しましょう。


意図的に文字列から数値に変換してもすべてセルの色が赤になってしまいました。

セルの書式設定のことですか?
半角英数字(数字とアルファベット)を許可するなら、数値にしてはダメでしょう。アルファベットは数値ではないので。
郵便番号もハイフンを含むし、ハイフンを省略する場合も、数値にすると先頭の0が消えるのでダメです。
文字列にすべきです。


半角英数字と郵便番号のチェックなら、RegExpを使うまでもなくLike演算子でも可能です。

    '半角英数字チェック
    If Me.Cells(i, 3).Value Like "*[!0-9A-Za-z]*" Then
        Me.Cells(i, 3).Interior.Color = RGB(255, 0, 0)
    End If


    '郵便番号チェック    
    If Not Me.Cells(i, 8).Value Like "###-####" Then
        Me.Cells(i, 8).Interior.Color = RGB(255, 0, 0)
    End If

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/09/18 14:47

    ありがとうございました。
    また色々とアドバイスいただきありがとうございます。
    Errは変更しました。
    RegExpを使用しなくてもLikeでできたんですね。
    色々とご教示ありがとうございます。
    今後ともよろしくお願いします。
    返事が遅くなりすいません。
    ほかの仕事がきてしまったのでごめんなさい

    キャンセル

0

reg.Pattern = "^[0-9]{3}-[0-9]{4}$" '郵便番号チェック は
reg2.Pattern = "^[0-9]{3}-[0-9]{4}$" '郵便番号チェック が正しいかと。

If reg.test(Cells(i, 3).Value) = False Then・・・にブレイクポイントを設定し、
step実行したとき、falseなので、次の行の
Cells(i, 3).Interior.Color = RGB(255, 0, 0)
にくるということですか。
もし、そうであれば、そのときの、Cells(i, 3).Value の内容はどうなっていますか。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/09/18 14:50

    回答ありがとうございます。
    hatena19さんの方法で解決しました。
    今後ともよろしくお願いします。

    キャンセル

0

ん。テーマが変わってますね。
チェックがどうのではなく、
問答無用で全角を半角に置き換えちゃえばいいような気がしますが。。。
チェックはそのあとデータとして成立しているかかと。。。

ネットで調べてもだれもそんなことしてないけど、
アルファベット大小で、52文字
数字 10文字
後記号数文字
を置換機能でグルグルループして置き換えたらおそいですかねぇ?

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/09/18 14:48

    回答ありがとうございます。
    hatena19さんの方法で解決しました。
    今後ともよろしくお願いします。

    キャンセル

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

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

関連した質問

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