Running 2 IF functions in a single command button

smilelover

New Member
Joined
Jul 24, 2011
Messages
32
I want to run 2 different IF functions in a single command button
the first one ran, but the 2nd one is not giving any output
how to change it?

Code:
Private Sub CommandButton1_Click()
For i = 1 To 100
    For j = 1 To 8
   If (IsError(Sheets("4. Freight Per Unit").Cells(8 * i + j - 5, 43))) Then
   Sheets("5. SUPPLIER Freight INC").Cells(2 * i, j + 2) = 0
   Else
   Sheets("5. SUPPLIER Freight INC").Cells(2 * i, j + 2) = Sheets("3. SUPPLIER DUTIES Repriced ").Cells(i + 1, j + 2).Value + Sheets("4. Freight Per Unit").Cells(8 * i + j - 5, 43).Value
   End If
    Next j
Next i

For k = 1 To 100
    For l = 1 To 8
   If (Sheets("4. Freight Per Unit").Cells(8 * k + j - 5, 62).Value) < 0.02 Then
   Sheets("5. SUPPLIER Freight INC").Cells(2 * i + 1, j + 2) = 0
   Else
   Sheets("5. SUPPLIER Freight INC").Cells(2 * i + 1, j + 2) = Sheets("3. SUPPLIER DUTIES Repriced ").Cells(i + 1, j + 2).Value + Sheets("4. Freight Per Unit").Cells(8 * i + j - 5, 62).Value
   End If
   Next l
   Next k
  
End Sub

Thanks guys
 

Excel Facts

Copy a format multiple times
Select a formatted range. Double-click the Format Painter (left side of Home tab). You can paste formatting multiple times. Esc to stop
Structurally Ok;
But your second For/Next segment uses variable j [ + j - 5 ] which is not part of either For/Next loop, so it is essentially a constant 8 from the first group of For/Next loops; is that intended?

[same with "i" [2 * i + 1] ]

( "l" looks too much like "1", imo)
 
Last edited:
Upvote 0

Forum statistics

Threads
1,224,602
Messages
6,179,844
Members
452,948
Latest member
UsmanAli786

We've detected that you are using an adblocker.

We have a great community of people providing Excel help here, but the hosting costs are enormous. You can help keep this site running by allowing ads on MrExcel.com.
Allow Ads at MrExcel

Which adblocker are you using?

Disable AdBlock

Follow these easy steps to disable AdBlock

1)Click on the icon in the browser’s toolbar.
2)Click on the icon in the browser’s toolbar.
2)Click on the "Pause on this site" option.
Go back

Disable AdBlock Plus

Follow these easy steps to disable AdBlock Plus

1)Click on the icon in the browser’s toolbar.
2)Click on the toggle to disable it for "mrexcel.com".
Go back

Disable uBlock Origin

Follow these easy steps to disable uBlock Origin

1)Click on the icon in the browser’s toolbar.
2)Click on the "Power" button.
3)Click on the "Refresh" button.
Go back

Disable uBlock

Follow these easy steps to disable uBlock

1)Click on the icon in the browser’s toolbar.
2)Click on the "Power" button.
3)Click on the "Refresh" button.
Go back
Back
Top