Besoin d'aide pour merge deux macros pour Excel VBA

Ces deux macros sont des macros que j'ai trouvées en ligne et adaptées à mon utilisation. J'utilise ce code et ça marche bien pour séparer des données spécifiques en nouvelles feuilles:

Sub Copy_To_Worksheets() 'Note: This macro use the function LastRow Dim My_Range As Range Dim FieldNum As Long Dim CalcMode As Long Dim ViewMode As Long Dim ws2 As Worksheet Dim Lrow As Long Dim cell As Range Dim CCount As Long Dim WSNew As Worksheet Dim ErrNum As Long 'Set filter range on ActiveSheet: A1 is the top left cell of your filter range 'and the header of the first column, D is the last column in the filter range. 'You can also add the sheet name to the code like this : 'Worksheets("Sheet1").Range("A1:D" & LastRow(Worksheets("Sheet1"))) 'No need that the sheet is active then when you run the macro when you use this. Set My_Range = Range("A1:Z" & Range("A" & Rows.Count).End(xlUp).Row) My_Range.Parent.Select If ActiveWorkbook.ProtectStructure = True Or _ My_Range.Parent.ProtectContents = True Then MsgBox "Sorry, not working when the workbook or worksheet is protected", _ vbOKOnly, "Copy to new worksheet" Exit Sub End If 'This example filters on the first column in the range(change the field if needed) 'In this case the range starts in A so Field:=1 is column A, 2 = column B, ...... FieldNum = 5 'I changed this to 3 for column C 'Turn off AutoFilter My_Range.Parent.AutoFilterMode = False 'Change ScreenUpdating, Calculation, EnableEvents, .... With Application CalcMode = .Calculation .Calculation = xlCalculationManual .ScreenUpdating = False .EnableEvents = False End With ViewMode = ActiveWindow.View ActiveWindow.View = xlNormalView ActiveSheet.DisplayPageBreaks = False 'Add a worksheet to copy the a unique list and add the CriteriaRange Set ws2 = Worksheets.Add With ws2 'first we copy the Unique data from the filter field to ws2 My_Range.Columns(FieldNum).AdvancedFilter _ Action:=xlFilterCopy, _ CopyToRange:=.Range("A1"), Unique:=True 'loop through the unique list in ws2 and filter/copy to a new sheet Lrow = .Cells(Rows.Count, "A").End(xlUp).Row For Each cell In .Range("A2:A" & Lrow) 'Filter the range My_Range.AutoFilter Field:=FieldNum, Criteria1:="=" & _ Replace(Replace(Replace(cell.Value, "~", "~~"), "*", "~*"), "?", "~?") 'Check if there are no more then 8192 areas(limit of areas) CCount = 0 On Error Resume Next CCount = My_Range.Columns(1).SpecialCells(xlCellTypeVisible) _ .Areas(1).Cells.Count On Error GoTo 0 If CCount = 0 Then MsgBox "There are more than 8192 areas for the value : " & cell.Value _ & vbNewLine & "It is not possible to copy the visible data." _ & vbNewLine & "Tip: Sort your data before you use this macro.", _ vbOKOnly, "Split in worksheets" Else 'Add a new worksheet Set WSNew = Worksheets.Add(After:=Sheets(Sheets.Count)) On Error Resume Next WSNew.Name = cell.Value If Err.Number > 0 Then ErrNum = ErrNum + 1 WSNew.Name = "Error_" & Format(ErrNum, "0000") Err.Clear End If On Error GoTo 0 'Copy the visible data to the new worksheet My_Range.SpecialCells(xlCellTypeVisible).Copy With WSNew.Range("A1") ' Paste:=8 will copy the columnwidth in Excel 2000 and higher ' Remove this line if you use Excel 97 .PasteSpecial Paste:=8 .PasteSpecial xlPasteValues .PasteSpecial xlPasteFormats Application.CutCopyMode = False .Select End With End If 'Show all data in the range My_Range.AutoFilter Field:=FieldNum Next cell 'Delete the ws2 sheet On Error Resume Next Application.DisplayAlerts = False .Delete Application.DisplayAlerts = True On Error GoTo 0 End With 'Turn off AutoFilter My_Range.Parent.AutoFilterMode = False If ErrNum > 0 Then MsgBox "Rename every WorkSheet name that start with ""Error_"" manually" _ & vbNewLine & "There are characters in the name that are not allowed" _ & vbNewLine & "in a sheet name or the worksheet already exist." End If 'Restore ScreenUpdating, Calculation, EnableEvents, .... My_Range.Parent.Select ActiveWindow.View = ViewMode With Application .ScreenUpdating = True .EnableEvents = True .Calculation = CalcMode End With End Sub 

Ce dont j'ai besoin d'aide, c'est d'append un set particulier de formules au bas de chaque feuille créée à partir de la macro ci-dessus. La macro suivante ajoute les formules à toutes les feuilles du classur. J'ai besoin d'append les formules uniquement aux feuilles créées dans la macro ci-dessus. Le nombre de feuilles créées change chaque fois qu'elles sont générées, selon datatables source. Je pensais qu'il serait préférable de merge la macro en bas du fond mais je ne sais pas comment procéder.

 Sub Insert_Formulas() Dim ws As Worksheet For Each ws In ThisWorkbook.Worksheets ws.Activate 'Start NxtRw = Cells(Rows.Count, "B").End(xlUp).Row + 1 With Cells(NxtRw, "B") .Value = "Total Open Cases" End With With Cells(NxtRw, "C") .EntireRow.Insert .Value = "Total Closed Cases" End With 'Next Row below NxtRw = Cells(Rows.Count, "B").End(xlUp).Row + 1 With Cells(NxtRw, "B") .Formula = "=CountIf(B2:B" & NxtRw - 1 & ", ""Open*"")" End With With Cells(NxtRw, "C") .Formula = "=CountIf(B2:B" & NxtRw - 1 & ", ""Closed*"")" End With Next End Sub 

Toute aide serait grandement appréciée.

Merci Ck

Si vous paramétrez la fonction de sorte qu'il faut la feuille de calcul qui nécessite les formules en tant que paramètre

 Sub Insert_Formulas_Into_WorkSheet(ws As Worksheet) ws.Activate 'Start NxtRw = Cells(Rows.Count, "B").End(xlUp).Row + 1 With Cells(NxtRw, "B") .Value = "Total Open Cases" End With With Cells(NxtRw, "C") .EntireRow.Insert .Value = "Total Closed Cases" End With 'Next Row below NxtRw = Cells(Rows.Count, "B").End(xlUp).Row + 1 With Cells(NxtRw, "B") .Formula = "=CountIf(B2:B" & NxtRw - 1 & ", ""Open*"")" End With With Cells(NxtRw, "C") .Formula = "=CountIf(B2:B" & NxtRw - 1 & ", ""Closed*"")" End With End Sub 

Ensuite, vous pouvez append les formules après chaque nouvelle feuille de calcul est créée en appelant

  Insert_Formulas_Into_WorkSheet WSNew 

Je ne combinerais pas les 2 macros, appelez simplement la macro Insert_Formulas partir de la macro Copy_To_Worksheets lorsque cela est nécessaire.

Pour appeler la macro, tout ce dont vous avez besoin, c'est cette ligne:

 Insert_Formulas 

Modifier pour répondre aux commentaires:

Étant donné que vous ne savez pas combien de feuilles sont ajoutées, j'ai une suggestion que vous pouvez essayer.

Niveau élevé, ajoutez du text à une cellule dans chaque feuille pour indiquer s'il est nouveau ou non. Lorsque la nouvelle feuille est créée, la cellule devrait indiquer "nouveau". Quand il n'est pas nouveau, il devrait dire "existant". Ensuite, dans le

Si vous souhaitez essayer, et faites-moi savoir ce qui ne fonctionne pas, je peux vérifier et aider à mettre à jour le code.

  1. Dans la macro Copy_To_Worksheets vous devez append une ligne pour définir toutes les feuilles existantes à "existant"
  2. Dans la macro Copy_To_Worksheets ajoutez une ligne afin que les nouvelles feuilles soient configurées comme "nouvelles"
  3. Dans la macro Insert_Formulas , continuez à parcourir toutes les feuilles, mais vérifiez si la feuille est "nouvelle" et, si c'est le cas, exécutez le code pour append les formules.

Une option plus propre (mais légèrement plus difficile) serait de définir un nom sur chaque feuille (utiliser le même nom pour chaque cas et limiter la scope de chaque feuille individuelle) et l'utiliser à la place d'une cellule sur chaque feuille.