Your code works fine, just put :
If Asc(UCase(Right(c.Value, 1))) <= 64 Or Asc(UCase(Right(c.Value, 1))) >= 91 Then Instead of : If Asc UCase(Right(c.Value,1)) <= 64 OR Asc UCase(Right(c.Value,1)) >=91 Then Regards. Daniel From: excel-macros@googlegroups.com [mailto:excel-mac...@googlegroups.com] On Behalf Of ashish koul Sent: lundi 7 septembre 2009 15:32 To: excel-macros@googlegroups.com Subject: $$Excel-Macros$$ hi daniel i was trying this code also but it is showing some errors can u check plzzzzzzzz Sub test() Dim c As Range Dim x As Integer For Each c In Selection If c.Value <> "" Then c.Value = Application.Clean(c.Value) c.Value = Application.Trim(c.Value) c.Value = Application.Substitute(c.Value, " ", "") c.Value = LCase(c.Value) For x = 1 To Len(c.Value) If Asc UCase(Right(c.Value,1)) <= 64 OR Asc UCase(Right(c.Value,1)) >=91 Then c.Value = Left(c.Value, Len(c.Value) - 1) End If Next x End If Next c End Sub --~--~---------~--~----~------------~-------~--~----~ ---------------------------------------------------------------------------------- Some important links for excel users: 1. Excel and VBA Tutorials(Video and Text), Free add-ins downloads at http://www.excelitems.com 2. Excel tutorials at http://www.excel-macros.blogspot.com 3. Learn VBA Macros at http://www.vbamacros.blogspot.com 4. Excel Tips and Tricks at http://exceldailytip.blogspot.com To post to this group, send email to excel-macros@googlegroups.com If you find any spam message in the group, please send an email to: Ayush Jain @ jainayus...@gmail.com or Ashish Jain @ 26may.1...@gmail.com <><><><><><><><><><><><><><><><><><><><><><> HELP US GROW !! We reach over 5,200 subscribers worldwide and receive many nice notes about the learning and support from the group. Our goal is to have 10,000 subscribers by the end of 2009. Let friends and co-workers know they can subscribe to group at http://groups.google.com/group/excel-macros/subscribe -~----------~----~----~----~------~----~------~--~---
