Macro to Convert all the data in column in Negative Value

[Chandresh]

HI All ,

I have written below code for converting all the numbers in the column to negative value however if some text comes in between my macro is not running can any one help me also the macro should run in all the sheets in that workbook.


Sub Cnegative()
'
' Cnegative Macro
'
For Each cell In Range("e3:e5000")
If cell.Value > 0 Then
cell.Value = cell.Value * -1
End If
Next cell
End Sub

 

[arpd123]

You can add to your if statement the following to get it to ignore non-numerical data like text:

If cell.Value > 0 And IsNumeric(cell.Value) Then

To get it to run through each sheet in the workbook, you can enclose your code in another for loop like so:

For i = 1 to ActiveWorkbook.Sheets.Count
    For Each cell in ActiveWorkboook.Sheets(i).Range("E3:E5000")
.
.
.
    Next cell
Next i

 

[Yongle]

Perhaps

 If cell.Value < 999999999 And cell.Value > 0 Then

or

If IsNumeric(cell.Value) And cell.Value > 0 Then

 

[Yongle]

or using SpecialCells property

Sub Cnegative3()
    On Error Resume Next
    For Each cell In Range("e3:e5000").SpecialCells(xlCellTypeConstants, xlNumbers)
        If Icell.Value > 0 Then cell.Value = -cell.Value
    Next cell
End Sub

 

[Yongle]

perhaps simplest of all...

Sub Cnegative()
    For Each cell In Range("e3:e5000")
    On Error Resume Next
       If cell.Value > 0 Then cell.Value = -cell.Value
    Next cell
    On Error GoTo 0
End Sub

 

[Chandresh]

hi its working fine thanks for your help ...however how to loop this for all sheets in this workbook.

 

[Yongle]

Like this:

Sub Cnegative()
Dim ws As Worksheet, cell as Range
For Each ws In ThisWorkbook.Worksheets
    For Each cell In ws.Range("e3:e5000")
        On Error Resume Next
           If cell.Value > 0 Then cell.Value = -cell.Value
        Next cell
    On Error GoTo 0
Next ws
End Sub

 

[Chandresh]

thanks it will really help me...

 

[Chandresh]

HI Cold you please help me looping below macro in all the worksheet ... am confused looping the same

Sub Loopforall()

On Error Resume Next
For Each cell In Range("C3:C250000,E3:E250000,Q3:Q250000,S3:s250000").SpecialCells(xlCellTypeConstants, xlNumbers)
If Icell.Value > 0 Then cell.Value = -cell.Value
Next cell

Range("G26658,C:C,E:E,Q:Q,S:S").Select
Range("S26636").Activate
Selection.NumberFormat = "0"
Selection.NumberFormat = "0.00"
Range("A1").Activate
End Sub

 

[Yongle]

Using code tags around your code mnakes it easier to read (to do this next time: click on # icon above post and paste your code inside the code tags)

Try this

For Each cell In Union(Range("C3:C250000"), Range("E3:E250000"), Range("Q3:Q250000"), Range("S3:s250000")).SpecialCells(xlCellTypeConstants, xlNumbers)
    If cell.Value > 0 Then cell.Value = -cell.Value
Next cell