Worksheet_Change处理工作表的不同列的不同操作未正确循环

问题描述 投票:0回答:1

在下面的工作表宏中,我试图执行不同的操作,具体取决于所选的列。在2种情况下,执行的操作取决于所选的列和列值。

例如,如果在A列中输入名称,则日期将自动在B列中输入。

[当在L列中输入一个下拉值时,在M列中输入日期。如果L列中的数据=“收到的费用”或“已发布的政策编号”数据被复制到另一个工作表中,并且该日期输入到列中米

所有单个组件都在工作。但是并非一直如此。

我需要宏来标识列并执行正确的操作,这样我才能在列之间移动,而宏可以在后台不断运行并为所有选定的列正确工作。

我非常沮丧,已经用尽了所有在线资源,但只需要VBA优秀的人来帮助我。非常感谢帮助

Private Sub Worksheet_Change(ByVal Target As Range)

'Dim C As Range, V
Dim answer As Integer
Dim LRowCompleted As Integer

Application.EnableEvents = False

MsgBox "Target Column is " & Target.Column
MsgBox "Target Value is " & Target.Value


    If Target.Column = 1 Then
        GoTo AddEntryDate
    End If

    If Target.Column = 12 Then
        GoTo AddWorkStatusDate
    End If

    If (Target.Column = 12 And Target.Value = "Fees Received") Then
        GoTo FeesReceived
    End If

    If (Target.Column = 12 And Target.Value = "Policy No. Issued") Then
        GoTo PolicyNoIssued
    End If



Exit Sub
AddEntryDate:
    'Update on 11/11/2019 -If data changes in column L Activity , insert
    'today's date into column M - Date of Activity

        Dim WorkRng As Range
        Dim rng As Range
        Dim xOffsetColumn As Integer

        Set WorkRng = Intersect(Application.ActiveSheet.Range("A:A"), Target)
        xOffsetColumn = 1

        If Not WorkRng Is Nothing Then
            Application.EnableEvents = False
            For Each rng In WorkRng
                If Not VBA.IsEmpty(rng.Value) Then
                    rng.Offset(0, xOffsetColumn).Value = Now
                    rng.Offset(0, xOffsetColumn).NumberFormat = "dd/mm/yyyy"
                    rng.Offset(3, xOffsetColumn).Select
                    With Selection.Interior
                        .Pattern = xlNone
                        .TintAndShade = 0
                        .PatternTintAndShade = 0
                    End With
                Else
                    rng.Offset(0, xOffsetColumn).ClearContents
                End If
            Next
            Application.EnableEvents = True
        End If

Exit Sub
AddWorkStatusDate:
    'Update on 11/11/2019 -If data changes in column L Activity , insert
    'today's date into column M - Date of Activity

        Dim WorkRng2 As Range
        Dim rng2 As Range
        Dim yOffsetColumn As Integer
        Set WorkRng2 = Intersect(Application.ActiveSheet.Range("L:L"), Target)
        yOffsetColumn = 1
        If Not WorkRng2 Is Nothing Then
            Application.EnableEvents = False
            For Each rng2 In WorkRng2
                If Not VBA.IsEmpty(rng2.Value) Then
                    rng2.Offset(0, yOffsetColumn).Value = Now
                    rng2.Offset(0, yOffsetColumn).NumberFormat = "dd/mm/yyyy"
                Else
                    rng2.Offset(0, yOffsetColumn).ClearContents
                End If
            Next
            Application.EnableEvents = True
        End If

Exit Sub
PolicyNoIssued:
        Sheets("Income").Select
        LRowCompleted = Sheets("Income").Cells(Rows.Count, "A").End(xlUp).Row '

        'Request confirmation from the user, in form of yes or no
        answer = MsgBox("Do you want to copy this client to the Income Worksheet?", vbQuestion + vbYesNo)

        If answer = vbYes Then
            Range("A" & Target.Row & ":A" & Target.Row).Copy
            Sheets("Income").Range("A" & Rows.Count).End(xlUp).Offset(1).Select
            Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
            :=False, Transpose:=False
            Application.EnableEvents = True
        Else
            MsgBox "This client will not be copied to the Income Worksheet"
            Application.EnableEvents = True
        End If


Exit Sub
FeesReceived:
        'Define last row on Income worksheet to know where to place the row of data
        Sheets("Income").Select
        LRowCompleted = Sheets("Income").Cells(Rows.Count, "A").End(xlUp).Row

        'Request confirmation from the user, in form of yes or no
        answer = MsgBox("Do you want to copy this client to the Income Worksheet?", vbQuestion + vbYesNo)

        If answer = vbYes Then
            Range("A" & Target.Row & ":A" & Target.Row).Copy
            Sheets("Income").Range("A" & Rows.Count).End(xlUp).Offset(1).Select
            Selection.PasteSpecial Paste:=xlPasteValues, Operation:=xlNone, SkipBlanks _
            :=False, Transpose:=False
            Application.EnableEvents = True
        Else
            MsgBox "This client will not be copied to the Income Worksheet"
            Application.EnableEvents = True
        End If



    Application.EnableEvents = True
End Sub
excel vba loops goto
1个回答
0
投票

据我所见,您只需要监视2列。您的其余需求只是这些需求的一部分。

您的代码可以按以下方式重写(未测试)让我知道您是否遇到任何错误?另外,由于您正在使用Worksheet_Change,因此您可能需要查看THIS

Option Explicit

Private Sub Worksheet_Change(ByVal Target As Range)
    Dim aCell As Range
    Dim wsInc As Worksheet
    Dim lRow As Long
    Dim ans As Variant

    On Error GoTo Whoa

    Application.EnableEvents = False

    '~~> Check if the change happened in Col A
    If Not Intersect(Target, Columns(1)) Is Nothing Then
        For Each aCell In Target.Cells
            With aCell
                If Len(Trim(.Value)) = 0 Then
                    .Offset(, 1).ClearContents
                Else
                    .Offset(, 1).NumberFormat = "dd/mm/yyyy"
                    .Offset(, 1).Value = Now
                    With .Interior
                        .Pattern = xlNone
                        .TintAndShade = 0
                        .PatternTintAndShade = 0
                    End With
                End If
            End With
        Next
    '~~> Check if the change happened in Col L
    ElseIf Not Intersect(Target, Columns(12)) Is Nothing Then
        Set wsInc = Sheets("Income")
        lRow = wsInc.Range("A" & wsInc.Rows.Count).End(xlUp).Row + 1

        For Each aCell In Target.Cells
            With aCell
                If Len(Trim(.Value)) = 0 Then
                    .Offset(, 1).ClearContents
                Else
                    .Offset(, 1).NumberFormat = "dd/mm/yyyy"
                    .Offset(, 1).Value = Now
                    With .Interior
                        .Pattern = xlNone
                        .TintAndShade = 0
                        .PatternTintAndShade = 0
                    End With

                    '~~> Check of the value is Fees Received, Policy No. Issued
                    If .Value = "Fees Received" Or .Value = "Policy No. Issued" Then
                        ans = MsgBox("Do you want to copy this client to the Income Worksheet?", vbQuestion + vbYesNo)

                        If ans = False Then Exit For

                        wsInc.Range("A" & lRow).Value = Range("A" & aCell.Row).Value
                    End If
                End If
            End With
        Next
    End If

Letscontinue:
    Application.EnableEvents = True
    Exit Sub
Whoa:
    MsgBox Err.Description
    Resume Letscontinue
End Sub
© www.soinside.com 2019 - 2024. All rights reserved.