VBA - Remove all dashes in cells when dash does not have a number before and after it.

I have this code where i replace all dashes with an empty space.

Columns("C").Replace " - ", " ", xlPart, xlByRows, True

Open in new window


Now, I would like to replace dashes when the character before AND after are not numbers.
(Format NUMBER & DASH & NUMBER)

Ex:
If I have in the cell:
St-Mary  1234-56

Open in new window


I see 2 dashes in the same cell. But one of them as a number before and after the dash. So the macro would only replace the St-Mary to St Mary.

If for example i have in the cell:
10234 Saint-John auto parc-0234

Open in new window


In this cell, Saint-John dash would be removed as i see a letter before and after the dash, and parc-0234 dash also would be removed as i don't have a combination of a Number before and after the dash.
So it would return to: 10234 Saint John auto parc 0234.

If i have in the cell:
1234 -123456

Open in new window


Since before the dash, there is an empty space, it would remove the dash as only format Number & dash & number needs to stay the same. So the result would be: 1234 123456

I need to specify that the text in each cells of column C may have different characters length.

How can i do that.

Thanks again.
LVL 11
Wilder1626Asked:
Who is Participating?
I wear a lot of hats...

"The solutions and answers provided on Experts Exchange have been extremely helpful to me over the last few years. I wear a lot of hats - Developer, Database Administrator, Help Desk, etc., so I know a lot of things but not a lot about one thing. Experts Exchange gives me answers from people who do know a lot about one thing, in a easy to use platform." -Todd S.

Michael FowlerSolutions ConsultantCommented:
Add VBA reference to "Microsoft VBScript Regular Expressions 5.5" and then use the following function

Function RegexReplace(str As String) As String

    Dim regEx As New RegExp
    Dim matchPattern As String, replacePattern As String
    Dim match As Boolean
    
    matchPattern = "(\D+)(-)(\D+)?"
    replacePattern = "$1 $3"
    
    With regEx
        .Global = True
        .MultiLine = True
        .IgnoreCase = False
        .Pattern = matchPattern
    End With
        
    match = regEx.test(str)
       
    While match
        str = regEx.Replace(str, replacePattern)
        match = regEx.test(str)
    Wend
    
    RegexReplace = str

End Function

Open in new window

Replace-Example.xlsm
0
QlemoBatchelor, Developer and EE Topic AdvisorCommented:
I would not define the dash as an own capture group, as we don't want to keep or work with it. Further the regex shown will work only if there is no digit on either side. I.e.:
    matchPattern = "(\D+)-(.*)|(.*)-(\D+)"
    replacePattern = "$1 $3"

Open in new window

0
Wilder1626Author Commented:
It looks like it work as a formula, but i will just need to transfer that into a macro.

I need to be able to replace actual cell, and not use another cell to give me the results needed.
0
Cloud Class® Course: Microsoft Exchange Server

The MCTS: Microsoft Exchange Server 2010 certification validates your skills in supporting the maintenance and administration of the Exchange servers in an enterprise environment. Learn everything you need to know with this course.

Michael FowlerSolutions ConsultantCommented:
@Wilder1626 the function provided will happily replace the cell contents I just implemented it this way to give a demo.

eg

Range("A1").Text = RegexReplace(Range("A1").Text)

Open in new window

0
Wilder1626Author Commented:
That was one of the thing i jtested using your file, but i was getting an runtime error 424: Object required.

Range("A1").Text = RegexReplace(Range("A1").Text)

Open in new window

0
Michael FowlerSolutions ConsultantCommented:
Sorry use

Range("A2").Value = RegexReplace(Range("A2").Text)

Open in new window

0

Experts Exchange Solution brought to you by

Your issues matter to us.

Facing a tech roadblock? Get the help and guidance you need from experienced professionals who care. Ask your question anytime, anywhere, with no hassle.

Start your 7-day free trial
Wilder1626Author Commented:
Thanks for your help

I was able to tweak it to:

Dim i As Integer
Dim LastRow As Integer

LastRow = [A65536].End(xlUp).Row
 For i = LastRow To 2 Step -1

If Cells(i, 1) <> "" Then
 Cells(i, 1).Value = RegexReplace(Cells(i, 1).Text)
End If

Next i
0
Michael FowlerSolutions ConsultantCommented:
Cool. Just a hint

The following will find the last row and work on all versions of excel. Just replace the "A" for the column you want to use.

lastrow = Range("A" & rows.count).end(xlup).row

Open in new window

0
QlemoBatchelor, Developer and EE Topic AdvisorCommented:
Using which regex?
0
It's more than this solution.Get answers and train to solve all your tech problems - anytime, anywhere.Try it for free Edge Out The Competitionfor your dream job with proven skills and certifications.Get started today Stand Outas the employee with proven skills.Start learning today for free Move Your Career Forwardwith certification training in the latest technologies.Start your trial today
Microsoft Excel

From novice to tech pro — start learning today.

Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.